Reboot task doesn't work

Bug #1592460 reported by Volodymyr Shypyguzov
14
This bug affects 3 people
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Committed
High
Bulat Gaifullin
Mitaka
Fix Released
High
Kyrylo Galanov

Bug Description

In our example reboot plugin we try to use reboot astute task, but it doesn't work.
Steps to reproduce: Create reboot task in the deployment graph
Expected result: Task is executed successfully
Actual result: Task failed

In astute logs:

WARNING [1556] 54a5a704-d74e-4a8d-bf30-df83e839809d: Failed to run shell if [ $(hostname) = bootstrap ]; then
         reboot;
      fi;
      while true; do
         t=0;
         if [ -f /run/cloud-init/status.json -o $t -gt 60 ]; then
             reboot;
         else
             sleep 1;
             t=$((t + 1));
         fi;
      done
 on node 8. Error will not raise because shell was run without check

Tags: 9.1-proposed
Changed in fuel:
milestone: none → 9.0
Changed in fuel:
importance: Undecided → High
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/329538

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-astute (master)

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

commit 22d52ee121ea40a18bec0510dcca6111db733f04
Author: Bulat Gaifullin <email address hidden>
Date: Tue Jun 14 18:45:04 2016 +0300

    Fixed wrong timeout calculation for reboot command

    Change-Id: Idbfad828f49511eb787fb7649dea51492037f936
    Closes-Bug: 1592460

Changed in fuel:
status: In Progress → Fix Committed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-astute (stable/mitaka)

Fix proposed to branch: stable/mitaka
Review: https://review.openstack.org/337820

Dmitry Pyzhov (dpyzhov)
no longer affects: fuel/newton
tags: added: 9.1-proposed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-astute (stable/mitaka)

Reviewed: https://review.openstack.org/337820
Committed: https://git.openstack.org/cgit/openstack/fuel-astute/commit/?id=37a70a9cd3e05803e94bd2b66da70e324eab4e6d
Submitter: Jenkins
Branch: stable/mitaka

commit 37a70a9cd3e05803e94bd2b66da70e324eab4e6d
Author: Bulat Gaifullin <email address hidden>
Date: Tue Jun 14 18:45:04 2016 +0300

    Fixed wrong timeout calculation for reboot command

    Change-Id: Idbfad828f49511eb787fb7649dea51492037f936
    Closes-Bug: 1592460
    (cherry picked from commit 22d52ee121ea40a18bec0510dcca6111db733f04)

tags: added: on-verification
Revision history for this message
ElenaRossokhina (esolomina) wrote :

verified on 9.0 + #115
CUSTOM_VERSION=snapshot #115
MAGNET_LINK=magnet:?xt=urn:btih:bfec808dd71ff42c5613a3527733d9012bb1fabc&dn=MirantisOpenStack-9.0.iso&tr=http%3A%2F%2Ftracker01-bud.infra.mirantis.net%3A8080%2Fannounce&tr=http%3A%2F%2Ftracker01-scc.infra.mirantis.net%3A8080%2Fannounce&tr=http%3A%2F%2Ftracker01-msk.infra.mirantis.net%3A8080%2Fannounce&ws=http%3A%2F%2Fvault.infra.mirantis.net%2FMirantisOpenStack-9.0.iso
FUEL_QA_COMMIT=5279ce17271bc0ac6cefc8c0ac4b9482260531ce
UBUNTU_MIRROR_ID=ubuntu-2016-08-03-174238
CENTOS_MIRROR_ID=centos-7.2.1511-2016-05-31-083834
MOS_UBUNTU_MIRROR_ID=9.0-2016-08-09-160321
MOS_CENTOS_OS_MIRROR_ID=os-2016-06-23-135731
MOS_CENTOS_PROPOSED_MIRROR_ID=proposed-2016-08-09-170321
MOS_CENTOS_UPDATES_MIRROR_ID=updates-2016-06-23-135916
MOS_CENTOS_HOLDBACK_MIRROR_ID=holdback-2016-06-23-140047
MOS_CENTOS_HOTFIX_MIRROR_ID=hotfix-2016-07-18-162958
MOS_CENTOS_SECURITY_MIRROR_ID=security-2016-06-23-140002

tags: removed: on-verification
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.