Skip to content

Conversation

@elelaysh
Copy link
Contributor

@elelaysh elelaysh commented Dec 4, 2025

  • show failure url and status on latest_timestamps.py failure

@elelaysh elelaysh requested a review from a team as a code owner December 4, 2025 09:02
Copy link
Contributor

@wtripp180901 wtripp180901 left a comment

Choose a reason for hiding this comment

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

one issue with repo paths but otherwise looks good (timestamp bumping script will need rerunning once fixed). As per https://github.com/stackhpc/ansible-slurm-appliance/blob/main/ansible/roles/dnf_repos/tasks/set_repos.yml#L19 the latest minor version snapshots will be used in the build automatically, so this can be tested with a fatimage run

@sjpb
Copy link
Collaborator

sjpb commented Dec 11, 2025

Does this need to incorporate #800? (or replace/close it?)

@sjpb sjpb changed the title Add Rocky Linux 9.7 repositories Update DNF packages & upgrade to Rocky Linux 9.7 Dec 12, 2025
@elelaysh
Copy link
Contributor Author

@sjpb

Does this need to incorporate #800? (or replace/close it?)

Incorporate, yes: I've cherry-picked your commit

@elelaysh
Copy link
Contributor Author

@elelaysh
Copy link
Contributor Author

@sjpb sjpb self-requested a review December 16, 2025 13:47
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, but needs #866 merging first and then an image rebuild again (and possibly, timestamps updating again, might as well)

@elelaysh
Copy link
Contributor Author

Tested on a small sms cluster:

[root@slurm-1-control ~]# uname -a
Linux slurm-1-control.slurm-1.internal 5.14.0-611.13.1.el9_7.x86_64 #1 SMP PREEMPT_DYNAMIC Fri Dec 12 11:55:11 UTC 2025 x86_64 x86_64 x86_64 GNU/Linux

sinfo, srun, openondemand work

it only includes the major version: ie rhel9 not rhel9.7
avoids loading the temporary inventory produced by tofu
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants