Skip to content

[MOSIP-44390] Fix deprecated Ansible configurations#145

Open
bhumi46 wants to merge 1 commit intomosip:developfrom
bhumi46:develop
Open

[MOSIP-44390] Fix deprecated Ansible configurations#145
bhumi46 wants to merge 1 commit intomosip:developfrom
bhumi46:develop

Conversation

@bhumi46
Copy link
Member

@bhumi46 bhumi46 commented Feb 5, 2026

  • Replace ANSIBLE_STDOUT_CALLBACK=yaml with default callback and ANSIBLE_CALLBACK_RESULT_FORMAT=yaml
  • Replace ANSIBLE_CALLBACK_WHITELIST with ANSIBLE_CALLBACKS_ENABLED
  • Replace ANSIBLE_COLLECTIONS_PATH with ANSIBLE_COLLECTIONS_PATHS

This resolves RKE2 installation failures due to removed community.general.yaml plugin in version 12.0.0

Summary by CodeRabbit

  • Chores
    • Updated Ansible environment variable configuration to ensure proper collections path resolution.
    • Refined Ansible callback settings to improve output formatting and task profiling during cluster deployments.

- Replace ANSIBLE_STDOUT_CALLBACK=yaml with default callback and ANSIBLE_CALLBACK_RESULT_FORMAT=yaml
- Replace ANSIBLE_CALLBACK_WHITELIST with ANSIBLE_CALLBACKS_ENABLED
- Replace ANSIBLE_COLLECTIONS_PATH with ANSIBLE_COLLECTIONS_PATHS

This resolves RKE2 installation failures due to removed community.general.yaml plugin in version 12.0.0

Signed-off-by: Bhuminathan M <bn46@Bhuminathans-MacBook-Air.local>
@coderabbitai
Copy link

coderabbitai bot commented Feb 5, 2026

Walkthrough

Two Ansible configuration shell scripts updated with revised environment variable names and settings. ANSIBLE_COLLECTIONS_PATH renamed to ANSIBLE_COLLECTIONS_PATHS in the PostgreSQL setup, and callback configuration restructured in the RKE2 cluster Ansible runner with stdout callback and result format adjustments.

Changes

Cohort / File(s) Summary
PostgreSQL Setup
terraform/modules/aws/postgresql-setup/postgresql-setup.sh
Renamed environment variable ANSIBLE_COLLECTIONS_PATH to ANSIBLE_COLLECTIONS_PATHS for Ansible collections path configuration.
RKE2 Cluster Ansible
terraform/modules/aws/rke2-cluster/ansible/run-ansible.sh
Updated Ansible callback configuration: changed stdout callback from yaml to default, replaced ANSIBLE_CALLBACK_WHITELIST with ANSIBLE_CALLBACKS_ENABLED, and added ANSIBLE_CALLBACK_RESULT_FORMAT=yaml setting.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Poem

🐰 Hop, hop, through the config we go,
Variables renamed, let the playbooks flow,
Callbacks reconfigured with style and grace,
Ansible's now dancing at quite the pace!

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'MOSIP-44390 Fix deprecated Ansible configurations' accurately describes the main change: replacing deprecated Ansible configuration variables with their modern equivalents.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

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.

1 participant

Comments