From 9c9f336f1086256452554b236954753f15b9d4a8 Mon Sep 17 00:00:00 2001 From: Matt Riedemann Date: Mon, 24 Jun 2019 13:07:17 -0400 Subject: [PATCH] Configure nova notification_format for grenade Nova changed the default notification_format from "both" to "unversioned" in Train [1]. Without configuring nova in the grenade job we are not testing the nova versioned notification handler code during upgrades. Note that grenade only runs stack.sh on the base (old) side so this change has to depend on a devstack stable/stein change to add the NOVA_NOTIFICATION_FORMAT variable that we override. Closes-Bug: #1831917 Depends-On: Ied9d50b07c368d5c2be658c744f340a8d1ee41e0 [1] https://review.opendev.org/603079/ Change-Id: I94c2d14477da185310e0fec596a1ad6436b802f1 --- devstack/upgrade/settings | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/devstack/upgrade/settings b/devstack/upgrade/settings index e8198be49..28abebd50 100644 --- a/devstack/upgrade/settings +++ b/devstack/upgrade/settings @@ -9,3 +9,10 @@ devstack_localrc target enable_service watcher-api watcher-decision-engine watch BASE_RUN_SMOKE=False TARGET_RUN_SMOKE=False + +# Enable both versioned and unversioned notifications. Watcher only +# uses versioned notifications but ceilometer uses unversioned. We +# can change this to just versioned when ceilometer handles +# versioned notifications from nova: +# https://bugs.launchpad.net/ceilometer/+bug/1665449 +devstack_localrc base NOVA_NOTIFICATION_FORMAT=both