Commit 51894c0
authored
fix: Remove the hardcoded image repository from values.yaml (#445)
commons-operator was the only operator setting image.repository in its own
values.yaml. The other 13 non-CSI operators leave it to the registry overlay in
deploy/helm/commons-operator/values/, which publish-helm-chart merges on top at
package time.
The value was also wrong in shape. operator.image renders
"{repository}/{Chart.Name}:{tag}", so repository is the namespace and the chart
name is appended. The hardcoded value already contained the name, so had it ever
taken effect it would have produced .../sdp/commons-operator/commons-operator.
No behaviour change for published charts: the overlay wins the merge either way.1 parent b1dc358 commit 51894c0
1 file changed
Lines changed: 0 additions & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | 1 | | |
2 | 2 | | |
3 | 3 | | |
4 | | - | |
5 | 4 | | |
6 | 5 | | |
7 | 6 | | |
| |||
0 commit comments