Hot-pluggable plugins: deployment failed on boot_time calculation

Bug #1560021 reported by Vladimir Khlyunev
22
This bug affects 3 people
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Released
High
Alexander Kislitsky
description: updated
tags: added: swarm-blocker
Dmitry Klenov (dklenov)
tags: added: area-python
Changed in fuel:
assignee: nobody → Fuel Python Team (fuel-python)
importance: Undecided → High
status: New → Confirmed
Dmitry Pyzhov (dpyzhov)
tags: added: feature-plugins
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-main (master)

Fix proposed to branch: master
Review: https://review.openstack.org/295396

Changed in fuel:
assignee: Fuel Python Team (fuel-python) → Matthew Mosesohn (raytrac3r)
status: Confirmed → In Progress
Revision history for this message
Matthew Mosesohn (raytrac3r) wrote : Re: Hot-pluggable plugins: deployment failed with uninformative message

My mistake. This review isn't for that bug.

Changed in fuel:
status: In Progress → Confirmed
assignee: Matthew Mosesohn (raytrac3r) → Fuel Python Team (fuel-python)
Dmitry Pyzhov (dpyzhov)
Changed in fuel:
assignee: Fuel Python Team (fuel-python) → Alexander Kislitsky (akislitsky)
Revision history for this message
Bug Checker Bot (bug-checker) wrote : Autochecker

(This check performed automatically)
Please, make sure that bug description contains the following sections filled in with the appropriate data related to the bug you are describing:

actual result

expected result

For more detailed information on the contents of each of the listed sections see https://wiki.openstack.org/wiki/Fuel/How_to_contribute#Here_is_how_you_file_a_bug

tags: added: need-info
Changed in fuel:
assignee: Alexander Kislitsky (akislitsky) → Vladimir Sharshov (vsharshov)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-astute (master)

Fix proposed to branch: master
Review: https://review.openstack.org/299424

Changed in fuel:
assignee: Vladimir Sharshov (vsharshov) → Alexander Kislitsky (akislitsky)
status: Confirmed → In Progress
summary: - Hot-pluggable plugins: deployment failed with uninformative message
+ Hot-pluggable plugins: deployment failed on boot_time calculation
Changed in fuel:
assignee: Alexander Kislitsky (akislitsky) → Vladimir Sharshov (vsharshov)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-astute (master)

Reviewed: https://review.openstack.org/299424
Committed: https://git.openstack.org/cgit/openstack/fuel-astute/commit/?id=cbd121f2188ddb6fe8ccd643d35d3bc86afddedc
Submitter: Jenkins
Branch: master

commit cbd121f2188ddb6fe8ccd643d35d3bc86afddedc
Author: Alexander Kislitsky <email address hidden>
Date: Wed Mar 30 12:50:40 2016 +0300

    Failures of reboot task fixed

    Checking of MClient results fixed in
    astute.task.run_shell_without_check. In case of empty
    array of results we had "undefined method 'results'
    for nil:NilClass".

    Function boot_time returns 0 in case of error, thus
    status_calculation for reboot taks is fixed to use proper
    conditions for status calculation.

    Parameter node_uid passed to run_shell_without_check
    shouldn't be an Array.

    Change-Id: Ic9311ae4129ad2a788289a1066c344b3aa437bb8
    Closes-Bug: #1560021

Changed in fuel:
status: In Progress → Fix Committed
Revision history for this message
Aleksei Stepanov (penguinolog) wrote :

Now it's fails with:
err: (/Firewall[007 tinyproxy]) Could not evaluate: undefined method `scan' for nil:NilClass

Changed in fuel:
status: Fix Committed → Confirmed
Revision history for this message
Vladimir Sharshov (vsharshov) wrote :

I do not see any connection between theme of this bug and "Could not evaluate: undefined method `scan' for nil:NilClass".

We should create separate bug to find out the reason

Changed in fuel:
status: Confirmed → Fix Committed
Revision history for this message
Aleksei Stepanov (penguinolog) wrote :

New bug created after discussion

Changed in fuel:
assignee: Vladimir Sharshov (vsharshov) → Alexander Kislitsky (akislitsky)
Revision history for this message
Aleksei Stepanov (penguinolog) wrote :
Changed in fuel:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.