Skip to content
This repository has been archived by the owner on Aug 9, 2024. It is now read-only.

feat!: overhaul slurmctld API part 2 #27

Merged
merged 1 commit into from
Jun 28, 2024
Merged

feat!: overhaul slurmctld API part 2 #27

merged 1 commit into from
Jun 28, 2024

Conversation

jamesbeedy
Copy link
Contributor

These changes follow up the last pr with the last of my local changes:

  • remove fluentbit integration and associated code
  • adds back creation of the the state dir
  • fix path to jwt_key

These changes:
* remove fluentbit integration and associated code
* adds back creation of the the state dir
* fix path to jwt_key
@NucciTheBoss NucciTheBoss self-requested a review June 28, 2024 00:30
Copy link
Member

@NucciTheBoss NucciTheBoss left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks James 🎉

@NucciTheBoss NucciTheBoss changed the title push last of local changes for slurmdbd-operator feat!: overhaul slurmdbd API part 2 Jun 28, 2024
@NucciTheBoss NucciTheBoss changed the title feat!: overhaul slurmdbd API part 2 feat!: overhaul slurmctld API part 2 Jun 28, 2024
@NucciTheBoss NucciTheBoss merged commit 71a32c4 into charmed-hpc:main Jun 28, 2024
4 of 5 checks passed
@jamesbeedy jamesbeedy deleted the small_fixes branch July 1, 2024 14:25
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants