Skip to content

Use ansible-core over ansible and upgrade to v2.16 #853

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

Jump to bottom
Merged
sjpb merged 22 commits into from
Dec 11, 2025
Merged

Use ansible-core over ansible and upgrade to v2.16 #853

sjpb merged 22 commits into from
Dec 11, 2025

Conversation

@bertiethorpe
Copy link
Member

@bertiethorpe bertiethorpe commented Nov 24, 2025 •
edited by sjpb
Loading

  • Upgrade supported CI controlled host Ubuntu version from 22 to 24.
  • Upgraded the controller host venv python version to 3.12 for all OSs (RockyLinux 8, RockyLinux 9 and Ubuntu 24).
  • Unpinned selinux in requirements.txt.
  • Pinned ansible-core 2.16 and pip freeze other dependencies.
  • Updated forks of grafana and prometheus roles for newer ansible versions.
  • Fixes compute-init inject files sync task with incorrect rsync options (not enforced in earlier ansible versions)

CI:

  • Now uses actions/python-setup to set python version separately for both latest release and current branch setup-env.
  • Provisionally uses python 3.10 for previous release deployment so as not break CI. Must be upgraded to 3.12 once a including this PR is made.

@bertiethorpe bertiethorpe force-pushed the upgrade/ansible-core-2.16 branch from 73c3752 to 9c15a95 Compare November 25, 2025 14:14
@bertiethorpe bertiethorpe marked this pull request as ready for review November 25, 2025 14:54
@bertiethorpe bertiethorpe requested a review from a team as a code owner November 25, 2025 14:54
@bertiethorpe bertiethorpe force-pushed the upgrade/ansible-core-2.16 branch from eb87e27 to 2e80e5e Compare November 25, 2025 18:39
sjpb
sjpb requested changes Dec 10, 2025
View reviewed changes
Copy link
Collaborator

@sjpb sjpb 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 pending a couple of releases and a minor comment to help us later ...

@bertiethorpe
Copy link
Member Author

bertiethorpe commented Dec 10, 2025

Image builds bumped above ^:

https://github.com/stackhpc/ansible-slurm-appliance/actions/runs/20101546177

@bertiethorpe bertiethorpe requested a review from sjpb December 10, 2025 15:28
sjpb
sjpb approved these changes Dec 11, 2025
View reviewed changes
Copy link
Collaborator

@sjpb sjpb left a comment

Choose a reason for hiding this comment

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

LGTM

@sjpb sjpb merged commit 7856c23 into main Dec 11, 2025
34 checks passed
@sjpb sjpb deleted the upgrade/ansible-core-2.16 branch December 11, 2025 14:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@sjpb sjpb sjpb approved these changes

Assignees

No one assigned

Labels

requires-imagebuild

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@bertiethorpe @sjpb