Skip to content

Update ANSIBLE_CONFIG path for consistency in scripts#531

Open
floatingman wants to merge 4 commits intorancher:mainfrom
floatingman:fix/harvester-recurring-ansible
Open

Update ANSIBLE_CONFIG path for consistency in scripts#531
floatingman wants to merge 4 commits intorancher:mainfrom
floatingman:fix/harvester-recurring-ansible

Conversation

@floatingman
Copy link
Copy Markdown
Contributor

Standardize the ANSIBLE_CONFIG path across multiple scripts to ensure consistency in configuration.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR standardizes the ANSIBLE_CONFIG environment variable path across multiple infrastructure automation scripts to use a centralized configuration file. Previously, scripts used different paths (such as ansible/rke2/default/ansible.cfg), but now all scripts consistently point to ansible/ansible.cfg at the root of the ansible directory in the qa-infra-automation repository.

Changes:

  • Updated ANSIBLE_CONFIG path from feature-specific location to centralized ansible/ansible.cfg
  • Applied changes consistently across shell scripts and Jenkins pipeline files
  • Ensures all infrastructure automation uses the same Ansible configuration

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File Description
validation/pipeline/scripts/build_qa_infra.sh Updated ANSIBLE_CONFIG default path in documentation and variable assignment from ansible/rke2/default/ansible.cfg to ansible/ansible.cfg
validation/pipeline/scripts/build_elemental_qa_infra.sh Added export of ANSIBLE_CONFIG pointing to centralized ansible/ansible.cfg
validation/pipeline/scripts/build_elemental_harvester_qa_infra.sh Added export of ANSIBLE_CONFIG pointing to centralized ansible/ansible.cfg
validation/pipeline/Jenkinsfile.setup.airgap.rke2 Added ANSIBLE_CONFIG environment variable assignment to use centralized configuration file

rancher-max
rancher-max previously approved these changes Feb 25, 2026
Copy link
Copy Markdown
Contributor

@lscalabrini01 lscalabrini01 left a comment

Choose a reason for hiding this comment

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

Looks good! Could you run the elemental_harvester job and the pit.daily job pointing to this branch?

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

Labels

team/pit-crew slack notifier for pit crew

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants