Resolve "Transition all BB values to Secret objects parsed by `HelmRelease`"
Summary
- Removes all inline
HelmRelease
values and replaces with{{ .Release.Name }}-<package>-values
secret loaded viavaluesFrom
- These values are loaded before the
values
passthroughs so they get overridden by any customer defined values - the secret creation is handled by a named template
values-secret
- These values are loaded before the
- refactors all customer overlay values into the same secret with bb defaults but using a different
overlays
key - introduces
common
values that are set in thevalues-secret
named template that all charts can use (currently empty) - Refactors
haproxy-sso
helmrelease into it's own file (instead of with theauthservice
namespace.yaml
) - ensure istiooperator components are tied to
.Values.istiooperator.enabled
(instead of.Values.istio.enabled
)
Closes #223 (closed)
Merge request reports
Activity
added 10 commits
-
48a2e47e...0cabda1b - 9 commits from branch
master
- 88ab0e25 - move all bigbang set helm values to secrets and load with valuesFrom
-
48a2e47e...0cabda1b - 9 commits from branch
changed milestone to %1.3.0
requested review from @runyontr
requested review from @micah.nagel and @ryan.j.garcia
added 29 commits
-
1adb8376...386aecd3 - 25 commits from branch
master
- c1737fd2 - move all bigbang set helm values to secrets and load with valuesFrom
- 3da35f32 - LALALALA SQUASHED
- 31d677a4 - remove valuesfrom for haproxy
- 7a007dca - typos for days
Toggle commit list-
1adb8376...386aecd3 - 25 commits from branch
added bigbang statusreview labels
- Resolved by joshwolf
- Resolved by joshwolf
Only comment I have is that this is a somewhat big change that we need to make sure is communicated well especially for MRs that are in progress right now (even if it's basically just moving things between files, it'll mess with any MRs that modify the helmrelease for values stuff and could definitely cause confusion).
Now that !316 (merged) is merged you can update the developer docs as part of this!
- Automatically resolved by joshwolf
added 9 commits
-
7a007dca...c2ec4d2c - 8 commits from branch
master
- fb7f0c18 - move all bigbang set helm values to secrets and load with valuesFrom
-
7a007dca...c2ec4d2c - 8 commits from branch
added 1 commit
- fb3002a5 - move leftover passthrough values to valuesFrom secrets