terraform test: New version with local helm chart support#31527
terraform test: New version with local helm chart support#31527def- merged 1 commit intoMaterializeInc:mainfrom
Conversation
5c5d247 to
678517d
Compare
b50dabd to
d5bb254
Compare
|
@bobbyiliev I'm wondering why this doesn't work: https://buildkite.com/materialize/test/builds/99223#01951a82-de29-48b1-979f-897baf998e06 I'm explicitly setting the defaultReplicationFactor in helm_values, I assume that should work together with use_local_helm_chart. This is somehow related to #31452, but I don't understand how we can both use a new operator version that sets the defaults to 0, but doesn't accept me to override them. cc @SangJunBak maybe you have an idea too |
|
@def- from first glance, it seems like |
03bf6bb to
b9651ba
Compare
|
I tried that but no difference still: |
| probe = 1 | ||
| support = 1 | ||
| analytics = 1 | ||
| clusters = { |
There was a problem hiding this comment.
The clusters section is under the operator key in the values file:
https://github.com/MaterializeInc/materialize/blob/main/misc/helm-charts/operator/values.yaml#L11
There was a problem hiding this comment.
So is it true that if I set anything in operator I have to override everything?
There was a problem hiding this comment.
No, this should not be the case. It will override only the values that you are modifying:
Are you seeing a different behavior?
There was a problem hiding this comment.
I didn't get it into a working state, I'm not sure what I'm doing wrong with the helm_values or if they don't work. I just manually alter the cluster now and that works for me. So can be merged in this state.
a5fbf7b to
838f77d
Compare
838f77d to
53123b4
Compare
53123b4 to
5ef5199
Compare
Checklist
$T ⇔ Proto$Tmapping (possibly in a backwards-incompatible way), then it is tagged with aT-protolabel.