Skip to content
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.

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

Fix some invalid generated YAML. #21

Merged
merged 1 commit into from
Jun 28, 2023
Merged

Conversation

amrc-benmorrow
Copy link
Contributor

Excessive chomping on Helm templates was causing the YAML divider --- to end up appended to the end of the previous line.

Excessive chomping on Helm templates was causing the YAML divider `---`
to end up appended to the end of the previous line.
@AlexGodbehere AlexGodbehere merged commit 9840bb8 into main Jun 28, 2023
@AlexGodbehere AlexGodbehere deleted the fix/bm-fix-principals branch June 28, 2023 15:15
@amrc-benmorrow amrc-benmorrow restored the fix/bm-fix-principals branch July 3, 2023 09:16
@amrc-benmorrow amrc-benmorrow deleted the fix/bm-fix-principals branch July 3, 2023 09:19
AlexGodbehere pushed a commit that referenced this pull request Mar 22, 2024
We were sometimes ending up with an `undefined` flux-system manifest,
because cluster-manager had checked out the repo before git had had a
chance to act on its auto-pull instructions. Rather than messing about
just check out every time.

Fix a small bug which occurred when checking out an empty repo, and make
sure we refresh our cluster list every 10 minutes in case we miss a
ConfigDB update.

Fixes: #21
Fixes: #11
AlexGodbehere added a commit that referenced this pull request Mar 22, 2024
AlexGodbehere added a commit that referenced this pull request Mar 22, 2024
AlexGodbehere pushed a commit that referenced this pull request Mar 22, 2024
I'm not sure why I did this in the first place.
amrc-benmorrow pushed a commit that referenced this pull request Oct 16, 2024
Environment schema v1
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