Stack lifecycle scheduler hints missing orchestration UUID

Bug #1476345 reported by Joe D'Andrea
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Heat
Fix Released
Low
Joe D'Andrea
Kilo
Fix Released
Low
Joe D'Andrea

Bug Description

An "orchestration" UUID assigned to each resource, prior to the invocation of stack lifecycle plug points, allows for simple cross-referencing of heat resources before and after they are instantiated. This is most useful, for instance, in the case of holistic resource scheduling.

At one point, there was some discussion about adding these UUIDs to stack lifecycle scheduler hints, but it did not materialize.

With the debut of convergence in heat-kilo, such a UUID now exists and can be exposed through stack lifecycle scheduler hints.

Additionally, these hints can be added to other scheduled resources (e.g., Cinder Volumes).

For additional context, see the lone comment at:

https://review.openstack.org/#/c/96889/24/heat/engine/resources/aws/ec2/instance.py,unified

Revision history for this message
Joe D'Andrea (jdandrea) wrote :

I have a patch prepared for this, and will post it soon. I may need some help working through test cases though. :)

Changed in heat:
assignee: nobody → Joe D'Andrea (joedandrea)
description: updated
description: updated
description: updated
Joe D'Andrea (jdandrea)
description: updated
Joe D'Andrea (jdandrea)
Changed in heat:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to heat (master)

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

Revision history for this message
Angus Salkeld (asalkeld) wrote :
Changed in heat:
importance: Undecided → Low
Revision history for this message
Angus Salkeld (asalkeld) wrote :

Please ignore the comment above, I pasted into the wrong bug.

Changed in heat:
milestone: none → liberty-3
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to heat (master)

Reviewed: https://review.openstack.org/203823
Committed: https://git.openstack.org/cgit/openstack/heat/commit/?id=b31259a9edf324bd3bdd7c0f18f52a7804e61ab1
Submitter: Jenkins
Branch: master

commit b31259a9edf324bd3bdd7c0f18f52a7804e61ab1
Author: Joe D'Andrea <email address hidden>
Date: Mon Jul 20 17:03:16 2015 -0400

    Adds resource uuid, volume support to lifecycle scheduler hints

    Heat resources are now assigned an orchestration id prior to their
    instantiation by nova, cinder, et. al. This id is now added to
    stack lifecycle scheduler hints. In addition to nova receiving
    such hints, cinder is also now supported.

    Change-Id: I5a13feb1bdedfbbe44de15e3d9eae72e56ec8a25
    Closes-Bug: #1476345

Changed in heat:
status: In Progress → Fix Committed
Joe D'Andrea (jdandrea)
tags: added: kilo-backport-potential
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to heat (stable/kilo)

Fix proposed to branch: stable/kilo
Review: https://review.openstack.org/218992

Changed in heat:
status: Fix Committed → Fix Released
Revision history for this message
shankar ganesh p j (pjshankarganesh) wrote :

Thanks for Fixing

Regards,
Shankar Ganesh P J

Angus Salkeld (asalkeld)
tags: removed: kilo-backport-potential
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to heat (stable/kilo)

Reviewed: https://review.openstack.org/218992
Committed: https://git.openstack.org/cgit/openstack/heat/commit/?id=88a216c491aaa680ae009729c1a6441b36e69fef
Submitter: Jenkins
Branch: stable/kilo

commit 88a216c491aaa680ae009729c1a6441b36e69fef
Author: Joe D'Andrea <email address hidden>
Date: Mon Jul 20 17:03:16 2015 -0400

    Adds resource uuid, volume support to lifecycle scheduler hints

    Heat resources are now assigned an orchestration id prior to their
    instantiation by nova, cinder, et. al. This id is now added to
    stack lifecycle scheduler hints. In addition to nova receiving
    such hints, cinder is also now supported.

    Conflicts (all resolved):

      heat/common/config.py
      heat/engine/resources/aws/ec2/instance.py
      heat/engine/resources/openstack/cinder/volume.py
      heat/engine/resources/openstack/nova/server.py
      heat/tests/openstack/test_volume.py
      heat/tests/test_server.py

    Change-Id: I5a13feb1bdedfbbe44de15e3d9eae72e56ec8a25
    Closes-Bug: #1476345
    (cherry picked from commit b31259a9edf324bd3bdd7c0f18f52a7804e61ab1)

Thierry Carrez (ttx)
Changed in heat:
milestone: liberty-3 → 5.0.0
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.