[fuel-bootstra] multipath-reload wrong timeout values

Bug #1572210 reported by Aleksey Zvyagintsev
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Committed
High
Alexander Gordeev
Mitaka
Fix Released
High
Aleksey Zvyagintsev

Bug Description

2016-04-13T16:08:04Z notice multipath-reload[790]: + for i in '`seq 1 ${SETTLE_ATTEMPTS}`'
2016-04-13T16:08:04Z notice multipath-reload[790]: + udevadm settle --timeout=360s
2016-04-13T16:08:04Z notice multipath-reload[790]: calling: settle
2016-04-13T16:08:04Z notice multipath-reload[790]: Invalid timeout value '360s': Invalid argument
2016-04-13T16:08:04Z notice multipath-reload[790]: + sleep 0.1
2016-04-13T16:08:04Z notice multipath-reload[790]: ++ date +%Y-%m-%d-%H-%M-%S
2016-04-13T16:08:04Z notice multipath-reload[790]: + echo '2016-04-13-16-08-04: Multipath reloading is done'

https://github.com/openstack/fuel-agent/blob/master/contrib/fuel_bootstrap/files/trusty/usr/bin/multipath-reload#L4-L5

Affected version : master and stable/mitaka

tags: added: area-python
tags: added: low-hanging-fruit
Changed in fuel:
importance: Undecided → High
milestone: 9.0 → 10.0
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-agent (master)

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

Changed in fuel:
status: Confirmed → In Progress
Revision history for this message
Alexander Gordeev (a-gordeev) wrote :

this one is considered high due to the overall impact on release critical feature (multipath).

this bug hasn't been caught in the wild. Thus, it may seem kinda overrated.

Anyway, without proper sync with udev activites it's definately broken. We should never underestime the significant importance of being in sync with udev. Especially when dealing with multipath/lvm and other tricky stuff.

Changed in fuel:
assignee: Aleksey Zvyagintsev (azvyagintsev) → Alexander Gordeev (a-gordeev)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-agent (stable/mitaka)

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

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

Reviewed: https://review.openstack.org/307897
Committed: https://git.openstack.org/cgit/openstack/fuel-agent/commit/?id=67e5175f4fce84c088255634d0ab40626a685439
Submitter: Jenkins
Branch: master

commit 67e5175f4fce84c088255634d0ab40626a685439
Author: alexz <email address hidden>
Date: Tue Apr 19 19:05:31 2016 +0300

    [ubuntu-bootstrap]Fix multipath-reload timeout values

    Change-Id: Ieca62067bcff4d056d4fbacd058e98e157139314
    Closes-Bug: #1572210

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

Reviewed: https://review.openstack.org/309420
Committed: https://git.openstack.org/cgit/openstack/fuel-agent/commit/?id=806db5debe5b720505834ca17e0d389cc7c1cef6
Submitter: Jenkins
Branch: stable/mitaka

commit 806db5debe5b720505834ca17e0d389cc7c1cef6
Author: alexz <email address hidden>
Date: Tue Apr 19 19:05:31 2016 +0300

    [ubuntu-bootstrap]Fix multipath-reload timeout values

    Change-Id: Ieca62067bcff4d056d4fbacd058e98e157139314
    Closes-Bug: #1572210

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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