From b0e83084c2b67112cd0dc9d56e233c98e8538d15 Mon Sep 17 00:00:00 2001 From: Michael Johnson Date: Wed, 28 Jul 2021 23:43:23 +0000 Subject: [PATCH] Fix grenade upgrade API check URL path Previously, the grenade API started check during the upgrade was checking if the root of the web server was functioning. In this job that is testing the horizon endpoint and not the designate API. This patch fixes this to have the check run against the designate endpoint under apache. This prevents false failures when horizon has an issue. Change-Id: Idbfec1adca2024cd5f352017a7c9319dcec65d42 --- devstack/upgrade/upgrade.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/devstack/upgrade/upgrade.sh b/devstack/upgrade/upgrade.sh index 8a916d74e..f8bdfc565 100755 --- a/devstack/upgrade/upgrade.sh +++ b/devstack/upgrade/upgrade.sh @@ -88,7 +88,7 @@ if is_service_enabled designate-api && is_service_enabled tls-proxy; then start_tls_proxy '*' $DESIGNATE_SERVICE_PORT $DESIGNATE_SERVICE_HOST $DESIGNATE_SERVICE_PORT_INT & fi -if ! timeout $SERVICE_TIMEOUT sh -c "while ! wget --no-proxy -q -O- $DESIGNATE_SERVICE_PROTOCOL://$DESIGNATE_SERVICE_HOST:$DESIGNATE_SERVICE_PORT; do sleep 1; done"; then +if ! timeout $SERVICE_TIMEOUT sh -c "while ! wget --no-proxy -q -O- $DESIGNATE_SERVICE_PROTOCOL://$DESIGNATE_SERVICE_HOST:$DESIGNATE_SERVICE_PORT/dns; do sleep 1; done"; then die $LINENO "Designate did not start" fi