Skip to content

Fix/rabbitmq inventory syntax #893

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
sghosh23 wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Fix/rabbitmq inventory syntax #893

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
8c02e63
fix: correct inventory syntax errors in 99-static for rabbitmq and re…
sghosh23 Apr 2, 2026
e9d92be
add doc why to use short hostname for rabbitmq
sghosh23 Apr 2, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
20 changes: 5 additions & 15 deletions ansible/inventory/offline/99-static
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -83,19 +83,7 @@

[postgresql:vars]
postgresql_network_interface = enp1s0
repmgr_node_config:
postgresql1: # Maps to postgresql_rw group
node_id: 1
priority: 150
role: primary
postgresql2: # Maps to first postgresql_ro
node_id: 2
priority: 100
role: standby
postgresql3: # Maps to second postgresql_ro
node_id: 3
priority: 50
role: standby
# repmgr_node_config is defined in group_vars/postgresql/postgresql.yml

[elasticsearch:vars]
# elasticsearch_network_interface = enp1s0
Expand All @@ -110,8 +98,10 @@ repmgr_node_config:

# Rabbitmq specific variables
[rmq-cluster:vars]
# host name here must match each node's actual hostname
rabbitmq_cluster_master: rabbitmq1
# host name here must match each node's actual hostname (short hostname, not FQDN)
# e.g. if the node's FQDN is rabbitmq1.example.com, use: rabbitmq_cluster_master = rabbitmq1
# Using FQDN here will cause rabbitmqctl join_cluster to fail with a :badarg error
rabbitmq_cluster_master = rabbitmq1
# rabbitmq_network_interface = enp1s0

# For the following groups, add all nodes defined above to the sections below.
Expand Down
18 changes: 16 additions & 2 deletions ansible/inventory/offline/group_vars/postgresql/postgresql.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -24,8 +24,22 @@ repmgr_namespace: "{{ wire_namespace | default('default') }}"
wire_pg_secret_name: "wire-postgresql-external-secret"

# Node configuration for repmgr
# NOTE: repmgr_node_config is defined in the inventory file ansible/inventory/offline/99-static, ansible/inventory/offline/staging.yml and terraform/examples/wire-server-deploy-offline-hetzner/outputs.tf
# to allow environment-specific node mappings. Do not define here.
# NOTE: staging inventory and terraform-generated envs override this in their own vars:
# - ansible/inventory/offline/staging.yml
# - terraform/examples/wire-server-deploy-offline-hetzner/outputs.tf
repmgr_node_config:
postgresql1: # Maps to postgresql_rw group
node_id: 1
priority: 150
role: primary
postgresql2: # Maps to first postgresql_ro
node_id: 2
priority: 100
role: standby
postgresql3: # Maps to second postgresql_ro
node_id: 3
priority: 50
role: standby

# repmgr settings
# repmgrd monitoring and reconnection configuration
Expand Down
1 change: 1 addition & 0 deletions changelog.d/5-bug-fixes/fix-rabbitmq-repmgr-inventory-syntax
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
Fixed: rabbitmq_cluster_master and repmgr_node_config syntax errors in offline inventory 99-static
Loading