Fuel for OpenStack

Override_resources type is not handling notify/subscribe parameters

Bug #1649380 reported by Dmitry Ukov
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Committed
Undecided
Dmitry Ukov

Bug Description

Steps to reproduce
put following data to Hiera
---
configuration:
  service:
    cinder-api:
      ensure: running
  package:
    mc:
      ensure: present
      notify: "Service[cinder-api]"

Execute any task with override_resources e.g.
puppet apply /etc/puppet/modules/osnailyfacter/modular/fuel_pkgs/fuel_pkgs.pp

Expected result:
  Package mc installed cinder-api service restarted
Actual Result
  Package mc installed but cinder-api service is not restarted

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

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

Changed in fuel:
assignee: nobody → Dmitry Ukov (dukov)
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-library (stable/mitaka)

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

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

Fix proposed to branch: stable/newton
Review: https://review.openstack.org/411751

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

Reviewed: https://review.openstack.org/409893
Committed: https://git.openstack.org/cgit/openstack/fuel-library/commit/?id=0e29cd247d46a8886be0b84580011925b1bddf84
Submitter: Jenkins
Branch: master

commit 0e29cd247d46a8886be0b84580011925b1bddf84
Author: Dmitry Ukov <email address hidden>
Date: Mon Dec 12 23:15:48 2016 +0400

    Added support for Notify/Subscribe

    Since eval_generate executed after relationship_graph has been formed we
    need to insert dependencies for new resources

    Change-Id: Ic9ad10cc4eb5ddbcd9ecf673b0bff3728b1d51e3
    Closes-Bug: 1649380
    Closes-Bug: 1644656

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

Reviewed: https://review.openstack.org/411751
Committed: https://git.openstack.org/cgit/openstack/fuel-library/commit/?id=75682bbdd3bafe5f5485b6b4b87d8d62c37c3190
Submitter: Jenkins
Branch: stable/newton

commit 75682bbdd3bafe5f5485b6b4b87d8d62c37c3190
Author: Dmitry Ukov <email address hidden>
Date: Mon Dec 12 23:15:48 2016 +0400

    Added support for Notify/Subscribe

    Since eval_generate executed after relationship_graph has been formed we
    need to insert dependencies for new resources

    Change-Id: Ic9ad10cc4eb5ddbcd9ecf673b0bff3728b1d51e3
    Closes-Bug: 1649380
    Closes-Bug: 1644656

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

Reviewed: https://review.openstack.org/411659
Committed: https://git.openstack.org/cgit/openstack/fuel-library/commit/?id=e3bd10c2365dc52f617013a8360c3a48e797ecff
Submitter: Jenkins
Branch: stable/mitaka

commit e3bd10c2365dc52f617013a8360c3a48e797ecff
Author: Dmitry Ukov <email address hidden>
Date: Mon Dec 12 23:15:48 2016 +0400

    Added support for Notify/Subscribe

    Since eval_generate executed after relationship_graph has been formed we
    need to insert dependencies for new resources

    Change-Id: Ic9ad10cc4eb5ddbcd9ecf673b0bff3728b1d51e3
    Closes-Bug: 1649380
    Closes-Bug: 1644656

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/fuel-library 11.0.0.0rc1

This issue was fixed in the openstack/fuel-library 11.0.0.0rc1 release candidate.

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

Other bug subscribers

Remote bug watches

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