Skip to content

Commit d0cd9aa

Browse files
authored
Support custom domains for ECO's goldpinger support bundle spec (#2022)
1 parent 32c20a3 commit d0cd9aa

File tree

5 files changed

+15
-4
lines changed

5 files changed

+15
-4
lines changed

cmd/buildtools/embeddedclusteroperator.go

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,10 @@ var operatorImageComponents = map[string]addonComponent{
2121
"docker.io/library/busybox": {
2222
name: "utils",
2323
},
24+
"docker.io/bloomberg/goldpinger": {
25+
name: "goldpinger",
26+
useUpstreamImage: true,
27+
},
2428
}
2529

2630
var updateOperatorAddonCommand = &cli.Command{
@@ -130,9 +134,9 @@ func updateOperatorAddonImages(ctx context.Context, hcli helm.Client, chartURL s
130134
return fmt.Errorf("failed to get images from embedded cluster operator chart: %w", err)
131135
}
132136

133-
// make sure we include the operator util image as it does not show up when rendering the helm
134-
// chart.
137+
// make sure we include the operator util and goldpinger images as they don't show up when rendering the helm chart.
135138
images = append(images, "docker.io/library/busybox:latest")
139+
images = append(images, "docker.io/bloomberg/goldpinger:latest")
136140

137141
metaImages, err := UpdateImages(ctx, operatorImageComponents, embeddedclusteroperator.Metadata.Images, images)
138142
if err != nil {

operator/charts/embedded-cluster-operator/templates/embedded-cluster-troubleshoot-goldpinger.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -19,9 +19,9 @@ data:
1919
collectors:
2020
- goldpinger:
2121
namespace: goldpinger
22-
image: proxy.replicated.com/anonymous/bloomberg/goldpinger@sha256:70416f19f1cbeedd344d37b08e64114779976b99905e0d018e71c437cde750dc
22+
image: {{ .Values.goldpingerImage }}
2323
podLaunchOptions:
24-
image: proxy.replicated.com/anonymous/library/busybox@sha256:768e5c6f5cb6db0794eec98dc7a967f40631746c32232b78a3105fb946f3ab83
24+
image: {{ .Values.utilsImage }}
2525
exclude: {{ .Values.isAirgap }}
2626
analyzers:
2727
- goldpinger:

operator/charts/embedded-cluster-operator/values.yaml.tmpl

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,7 @@ image:
1313
pullPolicy: IfNotPresent
1414

1515
utilsImage: busybox:latest
16+
goldpingerImage: bloomberg/goldpinger:latest
1617

1718
extraEnv: []
1819
# - name: HTTP_PROXY

pkg/addons/embeddedclusteroperator/static/metadata.yaml

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,11 @@ images:
1313
tag:
1414
amd64: v1.19.0-k8s-1.30
1515
arm64: v1.19.0-k8s-1.30
16+
goldpinger:
17+
repo: proxy.replicated.com/anonymous/bloomberg/goldpinger
18+
tag:
19+
amd64: latest
20+
arm64: latest
1621
utils:
1722
repo: proxy.replicated.com/anonymous/replicated/ec-utils
1823
tag:

pkg/addons/embeddedclusteroperator/static/values.tpl.yaml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,4 +7,5 @@ image:
77
repository: '{{ (index .Images "embedded-cluster-operator").Repo }}'
88
tag: '{{ index (index .Images "embedded-cluster-operator").Tag .GOARCH }}'
99
utilsImage: '{{ ImageString (index .Images "utils") }}'
10+
goldpingerImage: '{{ ImageString (index .Images "goldpinger") }}'
1011
{{- end }}

0 commit comments

Comments
 (0)