Skip to content

Bump Slurm to v2.5, and caas-operator to 0.18.0 #947

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
merged 17 commits into from
Sep 11, 2025
Merged

Bump Slurm to v2.5, and caas-operator to 0.18.0 #947

merged 17 commits into from
Sep 11, 2025

Conversation

bertiethorpe
Copy link
Contributor

Want to get stackhpc/ansible-slurm-appliance#743 in to Azimuth slurm to try to improve CI reliability.

Change to ANSIBLE_INVENTORY variable reflects that now all environments should derive from common AND site.

@bertiethorpe bertiethorpe requested a review from a team as a code owner August 28, 2025 16:47
@bertiethorpe bertiethorpe changed the title Bump Slurm to v2.3 Bump Slurm to v2.4 Sep 2, 2025
@bertiethorpe bertiethorpe changed the title Bump Slurm to v2.4 Bump Slurm to v2.4, and caas-operator to 0.18.0 Sep 2, 2025
@bertiethorpe bertiethorpe marked this pull request as draft September 3, 2025 15:23
JohnGarbutt
JohnGarbutt reviewed Sep 8, 2025
View reviewed changes
JohnGarbutt
JohnGarbutt reviewed Sep 8, 2025
View reviewed changes
@bertiethorpe bertiethorpe changed the title Bump Slurm to v2.4, and caas-operator to 0.18.0 Bump Slurm to v2.4.2, and caas-operator to 0.18.0 Sep 9, 2025
@bertiethorpe bertiethorpe marked this pull request as ready for review September 10, 2025 08:17
@bertiethorpe bertiethorpe changed the title Bump Slurm to v2.4.2, and caas-operator to 0.18.0 Bump Slurm to v2.5, and caas-operator to 0.18.0 Sep 10, 2025
JohnGarbutt
JohnGarbutt approved these changes Sep 11, 2025
View reviewed changes
Copy link
Contributor

@JohnGarbutt JohnGarbutt left a comment

Choose a reason for hiding this comment

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

This looks OK, but I am waiting to see the test pass.

@JohnGarbutt
Copy link
Contributor

OK the tests look good:

------------------------------------------------------------------------------
Create slurm                                                          elapsed time: 0m 8s
| PASS |
------------------------------------------------------------------------------
Verify slurm                                                          elapsed time: 16m 33s
| PASS |
------------------------------------------------------------------------------
Delete slurm                                                          elapsed time: 2m 29s
| PASS |
------------------------------------------------------------------------------

sjpb
sjpb approved these changes Sep 11, 2025
View reviewed changes
Copy link
Contributor

@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 to me, image matches release

@bertiethorpe bertiethorpe merged commit 40401dd into main Sep 11, 2025
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@JohnGarbutt JohnGarbutt JohnGarbutt approved these changes

@sjpb sjpb sjpb approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@bertiethorpe @JohnGarbutt @sjpb