Skip to content

Conversation

benbz
Copy link
Member

@benbz benbz commented Feb 10, 2025

Have almost all of charts/matrix-stack/ci/*-values.yaml be generated from fragments that get merged together. This allows us to

  • Consistently use fragments (e.g. minimal config, checkov suppressions) between files
  • Remove Secrets that can be generated from all values files in 1 go
  • Build all values files on top of nothing-enabled-values.yaml to ensure components aren't accidentally enabled
  • Reduced c/p'ing a lot

When can also now enforce testing of Helm inlined credentials and external Secrets for components that have Secrets in addition to generated credentials

@benbz benbz requested a review from a team as a code owner February 10, 2025 11:47
@benbz benbz force-pushed the bbz/ci-values-file-rationalisations branch from ce19cfe to b1c75d0 Compare February 10, 2025 12:25
@benbz benbz force-pushed the bbz/ci-values-file-rationalisations branch 3 times, most recently from 5af584b to 0e6df38 Compare February 10, 2025 15:43
@gaelgatelement
Copy link
Member

The main README probably needs an update to describe the new CI values files generation.

@benbz benbz force-pushed the bbz/ci-values-file-rationalisations branch from 0e6df38 to 8f01e15 Compare February 11, 2025 09:16
@benbz benbz force-pushed the bbz/ci-values-file-rationalisations branch from 8f01e15 to ea94d39 Compare February 11, 2025 10:08
@benbz benbz force-pushed the bbz/ci-values-file-rationalisations branch from ea94d39 to 1ee06ce Compare February 11, 2025 14:42
@benbz benbz requested a review from gaelgatelement February 11, 2025 14:49
@benbz benbz merged commit 2af5863 into main Feb 13, 2025
14 checks passed
@benbz benbz deleted the bbz/ci-values-file-rationalisations branch February 13, 2025 09:25
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