notifications needed for novajoin not working when telemetry is disabled

Bug #1841100 reported by Ade Lee
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
tripleo
Fix Released
High
Alex Schultz

Bug Description

A recent change disabled notifications when telemetry is disabledby setting the driver to noop.

This breaks novajoin-notifier which listens for notifications from nova and neutron.
Specifically,

https://opendev.org/x/novajoin/src/branch/master/novajoin/notifications.py#L112

We need this fixed or tls-everywhere is broken.

Revision history for this message
Ade Lee (alee-3) wrote :
Changed in tripleo:
milestone: none → train-3
Changed in tripleo:
assignee: nobody → Alex Schultz (alex-schultz)
status: New → In Progress
Revision history for this message
Alex Schultz (alex-schultz) wrote :

We did this change all the way back to Queens so we'll have to have similar logic in instack-undercloud

Changed in tripleo:
importance: Undecided → High
tags: added: stein
tags: added: queens-backport-potential rocky-backport-potential stein-backport-potential
removed: stein
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to instack-undercloud (stable/rocky)

Fix proposed to branch: stable/rocky
Review: https://review.opendev.org/678087

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to instack-undercloud (stable/rocky)

Reviewed: https://review.opendev.org/678087
Committed: https://git.openstack.org/cgit/openstack/instack-undercloud/commit/?id=5f1b6c6e5001ed917d474bed33ca88103031de94
Submitter: Zuul
Branch: stable/rocky

commit 5f1b6c6e5001ed917d474bed33ca88103031de94
Author: Alex Schultz <email address hidden>
Date: Thu Aug 22 12:50:05 2019 -0600

    Explicitly set notification driver for novajoin

    Novajoin is reliant on some notifications from nova. We recently
    merged an update to the undercloud installation that disables
    the notification drvier across the board if telemetry is disabled. This
    change adds logic for nova to force the notification driver to
    messagingv2.

    Change-Id: I75bb75092fa348dc1c2dc465446cb91d4882cf15
    Closes-Bug: #1841100

tags: added: in-stable-rocky
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to instack-undercloud (stable/queens)

Fix proposed to branch: stable/queens
Review: https://review.opendev.org/678236

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to tripleo-heat-templates (stable/stein)

Fix proposed to branch: stable/stein
Review: https://review.opendev.org/678288

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to instack-undercloud (stable/queens)

Reviewed: https://review.opendev.org/678236
Committed: https://git.openstack.org/cgit/openstack/instack-undercloud/commit/?id=97b77d1d13321833c82fded6b817105e2624ab78
Submitter: Zuul
Branch: stable/queens

commit 97b77d1d13321833c82fded6b817105e2624ab78
Author: Alex Schultz <email address hidden>
Date: Thu Aug 22 12:50:05 2019 -0600

    Explicitly set notification driver for novajoin

    Novajoin is reliant on some notifications from nova. We recently
    merged an update to the undercloud installation that disables
    the notification drvier across the board if telemetry is disabled. This
    change adds logic for nova to force the notification driver to
    messagingv2.

    Change-Id: I75bb75092fa348dc1c2dc465446cb91d4882cf15
    Closes-Bug: #1841100
    (cherry picked from commit 5f1b6c6e5001ed917d474bed33ca88103031de94)

tags: added: in-stable-queens
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to tripleo-heat-templates (stable/rocky)

Fix proposed to branch: stable/rocky
Review: https://review.opendev.org/678316

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to tripleo-heat-templates (master)

Reviewed: https://review.opendev.org/678083
Committed: https://git.openstack.org/cgit/openstack/tripleo-heat-templates/commit/?id=7c9c363c789811d54353d769aab86774eb5c96f8
Submitter: Zuul
Branch: master

commit 7c9c363c789811d54353d769aab86774eb5c96f8
Author: Alex Schultz <email address hidden>
Date: Thu Aug 22 12:36:49 2019 -0600

    Explicitly set notification driver for novajoin

    Novajoin is reliant on some notifications from nova. We recently merged
    an update to the undercloud installation that disables the notification
    drvier across the board if telemetry is disabled. This change adds a
    service_config_setting for nova to force the notification driver to
    messagingv2. This should be merged with the service configurations and
    override the value that is normally set via NotificationDriver.

    Change-Id: Ifee8f7f50fc2ef99605cbbcaa89016514b80b561
    Closes-Bug: #1841100

Changed in tripleo:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to tripleo-heat-templates (stable/stein)

Reviewed: https://review.opendev.org/678288
Committed: https://git.openstack.org/cgit/openstack/tripleo-heat-templates/commit/?id=2ed88dbcd5c7ea19c935aa71155c8e7824b59f7f
Submitter: Zuul
Branch: stable/stein

commit 2ed88dbcd5c7ea19c935aa71155c8e7824b59f7f
Author: Alex Schultz <email address hidden>
Date: Thu Aug 22 12:36:49 2019 -0600

    Explicitly set notification driver for novajoin

    Novajoin is reliant on some notifications from nova. We recently merged
    an update to the undercloud installation that disables the notification
    drvier across the board if telemetry is disabled. This change adds a
    service_config_setting for nova to force the notification driver to
    messagingv2. This should be merged with the service configurations and
    override the value that is normally set via NotificationDriver.

    Change-Id: Ifee8f7f50fc2ef99605cbbcaa89016514b80b561
    Closes-Bug: #1841100
    (cherry picked from commit 7c9c363c789811d54353d769aab86774eb5c96f8)

tags: added: in-stable-stein
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to tripleo-heat-templates (stable/rocky)

Reviewed: https://review.opendev.org/678316
Committed: https://git.openstack.org/cgit/openstack/tripleo-heat-templates/commit/?id=973ae3e83cbe80e2e4858903c10eab257bad52a2
Submitter: Zuul
Branch: stable/rocky

commit 973ae3e83cbe80e2e4858903c10eab257bad52a2
Author: Alex Schultz <email address hidden>
Date: Thu Aug 22 12:36:49 2019 -0600

    Explicitly set notification driver for novajoin

    Novajoin is reliant on some notifications from nova. We recently merged
    an update to the undercloud installation that disables the notification
    drvier across the board if telemetry is disabled. This change adds a
    service_config_setting for nova to force the notification driver to
    messagingv2. This should be merged with the service configurations and
    override the value that is normally set via NotificationDriver.

    Change-Id: Ifee8f7f50fc2ef99605cbbcaa89016514b80b561
    Closes-Bug: #1841100
    (cherry picked from commit 7c9c363c789811d54353d769aab86774eb5c96f8)

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/tripleo-heat-templates 10.6.1

This issue was fixed in the openstack/tripleo-heat-templates 10.6.1 release.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/instack-undercloud 9.5.1

This issue was fixed in the openstack/instack-undercloud 9.5.1 release.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/tripleo-heat-templates 9.4.1

This issue was fixed in the openstack/tripleo-heat-templates 9.4.1 release.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/instack-undercloud 8.4.9

This issue was fixed in the openstack/instack-undercloud 8.4.9 release.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/tripleo-heat-templates 11.2.0

This issue was fixed in the openstack/tripleo-heat-templates 11.2.0 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.