virt*d_config resources should notify virt*d services

Bug #1969401 reported by Takashi Kajinami
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
puppet-nova
Fix Released
Undecided
Unassigned

Bug Description

Currently virt*d_config resources (eg. virtlogd_config) does not notify the virt*d services (eg virtlogd), and any change in configuration files are not applied to actual running services.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to puppet-nova (master)
Changed in puppet-nova:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to puppet-nova (master)

Reviewed: https://review.opendev.org/c/openstack/puppet-nova/+/838417
Committed: https://opendev.org/openstack/puppet-nova/commit/170edf57d8798d7b14770a73d93d0c00ee9bbd4a
Submitter: "Zuul (22348)"
Branch: master

commit 170edf57d8798d7b14770a73d93d0c00ee9bbd4a
Author: Takashi Kajinami <email address hidden>
Date: Tue Apr 19 09:02:15 2022 +0900

    Fix missing notifications about virt*d services

    This change fixes the missing notifications between virt*d_config and
    virt*d services, so that individual service is restarted when its
    configuration file is updated.

    This change also refactors how resource dependencies are managed, using
    the existing anchors.

    Closes-Bug: #1969401
    Change-Id: Iff3ecdff9740dc1cc889b93666f089a8b60ef0ca

Changed in puppet-nova:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to puppet-nova (stable/yoga)

Fix proposed to branch: stable/yoga
Review: https://review.opendev.org/c/openstack/puppet-nova/+/848051

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to puppet-nova (stable/yoga)

Reviewed: https://review.opendev.org/c/openstack/puppet-nova/+/848051
Committed: https://opendev.org/openstack/puppet-nova/commit/d3851769a730ba42215a4724c806ed461ee759a6
Submitter: "Zuul (22348)"
Branch: stable/yoga

commit d3851769a730ba42215a4724c806ed461ee759a6
Author: Takashi Kajinami <email address hidden>
Date: Tue Apr 19 09:02:15 2022 +0900

    Fix missing notifications about virt*d services

    This change fixes the missing notifications between virt*d_config and
    virt*d services, so that individual service is restarted when its
    configuration file is updated.

    This change also refactors how resource dependencies are managed, using
    the existing anchors.

    Note:
    This backport includes the follow-up commit[1] which fixed the wrong
    notification added by the original patch.

    [1] 16ebf1bb8b2daedf449e1ed9259e7266709e9cb5

    Closes-Bug: #1969401
    Change-Id: Iff3ecdff9740dc1cc889b93666f089a8b60ef0ca
    (cherry picked from commit 170edf57d8798d7b14770a73d93d0c00ee9bbd4a)

tags: added: in-stable-yoga
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/puppet-nova 21.0.0

This issue was fixed in the openstack/puppet-nova 21.0.0 release.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to puppet-nova (stable/xena)

Fix proposed to branch: stable/xena
Review: https://review.opendev.org/c/openstack/puppet-nova/+/884224

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to puppet-nova (stable/xena)

Reviewed: https://review.opendev.org/c/openstack/puppet-nova/+/884224
Committed: https://opendev.org/openstack/puppet-nova/commit/30ee494b58c9a76cd3accf6822252e42491d31c8
Submitter: "Zuul (22348)"
Branch: stable/xena

commit 30ee494b58c9a76cd3accf6822252e42491d31c8
Author: Takashi Kajinami <email address hidden>
Date: Tue Apr 19 09:02:15 2022 +0900

    Fix missing notifications about virt*d services

    This change fixes the missing notifications between virt*d_config and
    virt*d services, so that individual service is restarted when its
    configuration file is updated.

    This change also refactors how resource dependencies are managed, using
    the existing anchors.

    Note:
    This backport includes the follow-up commit[1] which fixed the wrong
    notification added by the original patch.

    [1] 16ebf1bb8b2daedf449e1ed9259e7266709e9cb5

    Closes-Bug: #1969401
    Change-Id: Iff3ecdff9740dc1cc889b93666f089a8b60ef0ca
    (cherry picked from commit 170edf57d8798d7b14770a73d93d0c00ee9bbd4a)
    (cherry picked from commit d3851769a730ba42215a4724c806ed461ee759a6)

tags: added: in-stable-xena
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/puppet-nova 20.4.0

This issue was fixed in the openstack/puppet-nova 20.4.0 release.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/puppet-nova xena-eom

This issue was fixed in the openstack/puppet-nova xena-eom release.

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.