bump dev charts to staging #273
Conversation
- add opensearch chart to staging - bump cert-manager to 1.15.3 - fix inconsistency with chatops values - `env` needs to be a cluster-specific value This PR does not affect infrastructure charts like capi/capi-addons. See PR #272
anish-mudaraddi
force-pushed
the
bump-capi-staging
branch
from
02eb324 to
8d62791
Compare
| @@ -1,4 +1,4 @@ | |||
| env: staging | |||
| env: dev | |||
There was a problem hiding this comment.
Think this needs to be set back to staging
There was a problem hiding this comment.
Since this a cluster-specific value I override the value here - clusters/staging/worker/chatops-values.yaml
This means that staging/chatops/values.yaml matches exactly with dev/chatops/values.yaml enabling promotion workflow to be a simple copy-paste
There was a problem hiding this comment.
Can we change to this something like NotSet and add something to the helm chart to error with something like
env is 'NotSet', please specify this using the override param
Since this is something I can see other people getting confused by
There was a problem hiding this comment.
leave it for another PR - @khalford should look at this
There was a problem hiding this comment.
I'm not sure what the solution is for this PR but this is the current status.
ChatOps is going to stay on staging for now and not be promoted to production. This is because I willl be moving ChatOps off of ArgoCD onto a different deployment
|
(some review comments might need to be backported to dev, if you throw them into this PR we can do both at the same time) |
envneeds to be a cluster-specific valueThis PR does not affect infrastructure charts like capi/capi-addons. See PR #272