Skip to content

Commit 9cfefa6

Browse files
committed
Address RabbitMQ quorum PR nits
1 parent 5c5afa4 commit 9cfefa6

File tree

3 files changed

+7
-7
lines changed

3 files changed

+7
-7
lines changed

etc/kayobe/environments/ci-aio/kolla/globals.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,6 @@ opensearch_heap_size: 200m
1515
# Increase Grafana timeout
1616
grafana_start_first_node_retries: 20
1717

18-
# Ensure Rabbit is deployed with HA rather than Quorum queues (to test migrations)
18+
# Ensure Rabbit is deployed with HA rather than quorum queues (to test migrations)
1919
om_enable_rabbitmq_high_availability: true
2020
om_enable_rabbitmq_quorum_queues: false
Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
11
---
22
features:
33
- |
4-
Added a script to automate RabbitMQ Quorum queue migrations.
4+
Added a script to automate RabbitMQ quorum queue migrations.

tools/rabbitmq-quorum-migration.sh

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -20,16 +20,16 @@ if [[ ! "$1" = "--skip-checks" ]]; then
2020
exit 1
2121
fi
2222
kayobe overcloud service configuration generate --node-config-dir /tmp/rabbit-migration --kolla-tags none
23-
# Fail if HA is set or Quorum is not
23+
# Fail if HA is set or quorum is not
2424
if ! grep 'om_enable_rabbitmq_quorum_queues: true' $KOLLA_CONFIG_PATH/globals.yml || grep 'om_enable_rabbitmq_high_availability: true' $KOLLA_CONFIG_PATH/globals.yml; then
2525
echo "Failed precheck: om_enable_rabbitmq_quorum_queues must be enabled, om_enable_rabbitmq_high_availability must be disabled"
2626
exit 1
2727
fi
2828
fi
2929

3030
# Generate new config, stop services using rabbit, and reset rabbit state
31-
kayobe overcloud service configuration generate --node-config-dir /etc/kolla --kolla-skip-tags rabbitmq-ha-precheck &&
32-
kayobe kolla ansible run "stop --yes-i-really-really-mean-it" -kt $RABBITMQ_SERVICES_TO_RESTART &&
31+
kayobe overcloud service configuration generate --node-config-dir /etc/kolla --kolla-skip-tags rabbitmq-ha-precheck
32+
kayobe kolla ansible run "stop --yes-i-really-really-mean-it" -kt $RABBITMQ_SERVICES_TO_RESTART
3333
kayobe kolla ansible run rabbitmq-reset-state
3434

3535
if [[ ! "$1" = "--skip-checks" ]]; then
@@ -46,7 +46,7 @@ if [[ ! "$1" = "--skip-checks" ]]; then
4646
fi
4747
fi
4848

49-
# Redeploy with Quorum Queues enabled
49+
# Redeploy with quorum queues enabled
5050
kayobe kolla ansible run deploy-containers -kt $RABBITMQ_SERVICES_TO_RESTART
5151

5252
if [[ ! "$1" = "--skip-checks" ]]; then
@@ -55,6 +55,6 @@ if [[ ! "$1" = "--skip-checks" ]]; then
5555
if kayobe overcloud host command run -l controllers -b --command "docker exec $RABBITMQ_CONTAINER_NAME rabbitmqctl list_queues type | grep quorum"; then
5656
echo "Queues migrated successfully"
5757
else
58-
echo "Failed post-check: A controller does not have any Quorum queues"
58+
echo "Failed post-check: A controller does not have any quorum queues"
5959
fi
6060
fi

0 commit comments

Comments
 (0)