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

chore: enable CI for the experimental branch #41

Merged

Conversation

jedel1043
Copy link
Contributor

@jedel1043 jedel1043 commented Jul 19, 2024

Description

Since the experimental branch and the main branch use different versions of SLURM, we need to use the experimental branch for slurmctld when pushing new changes to slurmd.

How was the code tested?

N/A. Tested when running CI.

Checklist

  • I am the author of these changes, or I have the rights to submit them.
  • I have added the relevant changes to the README and/or documentation.
  • I have self reviewed my own code.
  • All requested changes and/or review comments have been resolved.

@jedel1043 jedel1043 changed the base branch from main to experimental July 22, 2024 22:04
@jedel1043 jedel1043 force-pushed the allow-ci-experimental branch from c21e999 to 0f7145d Compare July 22, 2024 22:05
Since the experimental branch and the main branch use different versions of SLURM, we need to use the experimental branch for `slurmctld` when pushing new changes to `slurmctld`.
@jedel1043 jedel1043 force-pushed the allow-ci-experimental branch from 0f7145d to b11c4cf Compare July 26, 2024 19:28
@NucciTheBoss NucciTheBoss self-requested a review July 26, 2024 19:41
@NucciTheBoss NucciTheBoss added the bug Something isn't working label Jul 26, 2024
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! We'll consolidate the workflow files when we reassemble the Slurm monorepo.

@NucciTheBoss NucciTheBoss merged commit 28267fd into charmed-hpc:experimental Jul 26, 2024
5 checks passed
@jedel1043 jedel1043 deleted the allow-ci-experimental branch July 26, 2024 19:43
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants