Custom graph execution does not set list of nodes to 'ALL' by default.

Bug #1566384 reported by Ilya Kutukov
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Committed
High
Bulat Gaifullin
Mitaka
Fix Released
High
Volodymyr Shypyguzov

Bug Description

Actual:
Now when you are running /cluster/:cluster_id/deploy handler all nodes should be specified explicitly, if no node id is defined, deploy will be launched on no nodes

Expected:
No nodes specified = Run on all nodes

Ilya Kutukov (ikutukov)
Changed in fuel:
milestone: none → 9.0
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:

version

steps to reproduce

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:
status: New → Confirmed
Changed in fuel:
assignee: Ilya Kutukov (ikutukov) → Bulat Gaifullin (bgaifullin)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-web (master)

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

Changed in fuel:
status: Confirmed → In Progress
Changed in fuel:
milestone: 9.0 → 10.0
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-web (master)

Reviewed: https://review.openstack.org/305259
Committed: https://git.openstack.org/cgit/openstack/fuel-web/commit/?id=ae4f57517303bc075dea8b7c7cbd5e134ec016ad
Submitter: Jenkins
Branch: master

commit ae4f57517303bc075dea8b7c7cbd5e134ec016ad
Author: Bulat Gaifullin <email address hidden>
Date: Tue Apr 12 15:24:52 2016 +0300

    Fixed launch deployment task w/o specifying list of nodes

    We should select all nodes those were provisioned
    for deploy in LCM. The method nodes_to_deploy has overridden for
    clusters that are LCM ready.
    Also changed selecting default nodes to get deployment info.
    now by default will be selected all nodes except nodes, those are marked
    for deletion.

    Closes-Bug: 1566384
    Change-Id: I264bece7da8aebaef78c15bef43c800ca565e583

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

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

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

Reviewed: https://review.openstack.org/309769
Committed: https://git.openstack.org/cgit/openstack/fuel-web/commit/?id=fbb5a91e2d34f35ed1285e744f72a8600c7cf03f
Submitter: Jenkins
Branch: stable/mitaka

commit fbb5a91e2d34f35ed1285e744f72a8600c7cf03f
Author: Bulat Gaifullin <email address hidden>
Date: Tue Apr 12 15:24:52 2016 +0300

    Fixed launch deployment task w/o specifying list of nodes

    We should select all nodes those were provisioned
    for deploy in LCM. The method nodes_to_deploy has overridden for
    clusters that are LCM ready.
    Also changed selecting default nodes to get deployment info.
    now by default will be selected all nodes except nodes, those are marked
    for deletion.

    Closes-Bug: 1566384
    Change-Id: I264bece7da8aebaef78c15bef43c800ca565e583
    (cherry picked from commit ae4f57517303bc075dea8b7c7cbd5e134ec016ad)

tags: added: on-verification
Revision history for this message
Volodymyr Shypyguzov (vshypyguzov) wrote :

Verified on 9.0 iso #458

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.