Skip to content

Fix/rabbitmq inventory syntax#893

Open
sghosh23 wants to merge 2 commits intomasterfrom
fix/rabbitmq-inventory-syntax
Open

Fix/rabbitmq inventory syntax#893
sghosh23 wants to merge 2 commits intomasterfrom
fix/rabbitmq-inventory-syntax

Conversation

@sghosh23
Copy link
Copy Markdown
Contributor

@sghosh23 sghosh23 commented Apr 2, 2026

  • Fix rabbitmq_cluster_master using YAML colon syntax instead of INI
    key=value syntax, which caused Ansible parse warnings and the wrong
    node name being used when joining the RabbitMQ cluster
  • Move repmgr_node_config from 99-static (INI, unsupported YAML dict)
    into group_vars/postgresql/postgresql.yml (proper YAML)

Change type

  • Fix
  • Feature
  • Documentation
  • Security / Upgrade

Basic information

  • THIS CHANGE REQUIRES A DEPLOYMENT PACKAGE RELEASE
  • THIS CHANGE REQUIRES A WIRE-DOCS RELEASE

Testing

  • I ran/applied the changes myself, in a test environment.
  • The CI job attached to this repo will test it for me.

Offline Build CI (label-based)

Add one or more labels to trigger offline builds:

  • build-default - Full production build (ansible, terraform, all packages)
  • build-dev - WIAB/dev build
  • build-wiab-staging - WIAB-staging build
  • build-min - Minimal build (fastest, essential charts only)
  • build-all - Run all three builds

Note: No builds run by default. Add a label to trigger CI.

Tracking

  • I added a new entry in an appropriate subdirectory of changelog.d
  • I mentioned this PR in Jira, OR I mentioned the Jira ticket in this PR.
  • I mentioned this PR in one of the issues attached to one of our repositories.

Knowledge Transfer

  • An Asciinema session is attached to the Jira ticket.

Motivation

Objective

Reason

Use case

sghosh23 and others added 2 commits April 2, 2026 15:42
…pmgr

- Fix rabbitmq_cluster_master using YAML colon syntax instead of INI
  key=value syntax, which caused Ansible parse warnings and the wrong
  node name being used when joining the RabbitMQ cluster
- Move repmgr_node_config from 99-static (INI, unsupported YAML dict)
  into group_vars/postgresql/postgresql.yml (proper YAML)

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@sghosh23 sghosh23 requested review from a team and julialongtin as code owners April 2, 2026 14:06
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Apr 2, 2026

Copy link
Copy Markdown
Contributor

@mohitrajain mohitrajain left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The PR #885 has resolved the problem of repmgr_node_config and rabbitmq_network_interface.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants