Skip to content

Conversation

sjpb
Copy link
Collaborator

@sjpb sjpb commented Jan 15, 2025

Integrate the currently-used parts of slurm_openstack_tools collection directly into the appliance, to simplify changes.

@sjpb sjpb requested a review from a team as a code owner January 15, 2025 15:08
@sjpb
Copy link
Collaborator Author

sjpb commented Jan 15, 2025

@sjpb sjpb force-pushed the refactor/slurm-tools branch from ef5da31 to 6f19074 Compare January 15, 2025 16:59
@sjpb sjpb force-pushed the refactor/slurm-tools branch from 6f19074 to 62778b3 Compare January 16, 2025 09:35
@sjpb
Copy link
Collaborator Author

sjpb commented Jan 16, 2025

Image build actually wasn't needed, the slurm_stats stuff doesn't run from fatimage.yml

@sjpb sjpb force-pushed the refactor/slurm-tools branch from 62778b3 to 8405570 Compare January 16, 2025 10:14
Copy link
Member

@bertiethorpe bertiethorpe 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 3f9f880 into main Jan 17, 2025
5 of 7 checks passed
@sjpb sjpb deleted the refactor/slurm-tools branch January 17, 2025 13:37
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.

2 participants