[SWARM]Fuel-qa: error in nodes count checker

Bug #1625456 reported by Aleksei Stepanov
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Released
Critical
Aleksei Stepanov
7.0.x
Fix Released
High
Aleksei Stepanov
8.0.x
Fix Released
High
Aleksei Stepanov
Mitaka
Fix Released
Critical
Aleksei Stepanov
Newton
Fix Released
Critical
Aleksei Stepanov

Bug Description

Error in nodes count checker: strict check instead of allowing additional nodes.
As result, a lot of cases failed with error like:
Exception: Nodes in state discover (online: True): 9, while expected: 4
Mistake was made in:
 Change-Id: I58fbf17d3bb1c0661181aa03d7a0431cf976137f

Actual:
if len(fnodes) != expected_amount
Correct:
if len(fnodes) < expected_amount

Was affected: prepare_slaves_9 due to chunk bootstrap of nodes

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-qa (stable/mitaka)

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

description: updated
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-qa (master)

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

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

Fix proposed to branch: stable/8.0
Review: https://review.openstack.org/372925

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-qa (stable/7.0)

Fix proposed to branch: stable/7.0
Review: https://review.openstack.org/372926

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-qa (stable/mitaka)

Reviewed: https://review.openstack.org/372893
Committed: https://git.openstack.org/cgit/openstack/fuel-qa/commit/?id=284da641337eb259ce5705442c25ca91ae684072
Submitter: Jenkins
Branch: stable/mitaka

commit 284da641337eb259ce5705442c25ca91ae684072
Author: Alexey Stepanov <email address hidden>
Date: Tue Sep 20 08:26:35 2016 +0300

    Fix issue, when accessible more nodes, than required to start test

    validate_amount_nodes -> validate_minimal_amount_nodes
    Closes-bug: #1625456

    Change-Id: I4655b980efa6601df15363fd215251a8d63b78fb

tags: added: in-stable-mitaka
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-qa (stable/8.0)

Reviewed: https://review.openstack.org/372925
Committed: https://git.openstack.org/cgit/openstack/fuel-qa/commit/?id=8b2587ee3f92f925d71067f9d4e6533828726b64
Submitter: Jenkins
Branch: stable/8.0

commit 8b2587ee3f92f925d71067f9d4e6533828726b64
Author: Alexey Stepanov <email address hidden>
Date: Tue Sep 20 08:26:35 2016 +0300

    Fix issue, when accessible more nodes, than required to start test

    validate_amount_nodes -> validate_minimal_amount_nodes
    Closes-bug: #1625456

    Change-Id: I4655b980efa6601df15363fd215251a8d63b78fb

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

Reviewed: https://review.openstack.org/372924
Committed: https://git.openstack.org/cgit/openstack/fuel-qa/commit/?id=e6cfa06edd9b136ba407f2c86147f5ec55d69ddb
Submitter: Jenkins
Branch: master

commit e6cfa06edd9b136ba407f2c86147f5ec55d69ddb
Author: Alexey Stepanov <email address hidden>
Date: Tue Sep 20 08:26:35 2016 +0300

    Fix issue, when accessible more nodes, than required to start test

    validate_amount_nodes -> validate_minimal_amount_nodes
    Closes-bug: #1625456

    Change-Id: I4655b980efa6601df15363fd215251a8d63b78fb

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-qa (stable/7.0)

Reviewed: https://review.openstack.org/372926
Committed: https://git.openstack.org/cgit/openstack/fuel-qa/commit/?id=e0f312b39758def4d0c5941a86cc8dc536bd952c
Submitter: Jenkins
Branch: stable/7.0

commit e0f312b39758def4d0c5941a86cc8dc536bd952c
Author: Alexey Stepanov <email address hidden>
Date: Tue Sep 20 08:26:35 2016 +0300

    Fix issue, when accessible more nodes, than required to start test

    validate_amount_nodes -> validate_minimal_amount_nodes
    Closes-bug: #1625456

    Change-Id: I4655b980efa6601df15363fd215251a8d63b78fb

Revision history for this message
Aleksei Stepanov (penguinolog) wrote :

Mitaka: required to align code with master after 9.1 release

tags: added: on-verification
Revision history for this message
Alexandra (aallakhverdieva) wrote :
tags: removed: on-verification
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.