Skip to content

Commit

Permalink
feat: use Sumo Logic repository for all images (#3677)
Browse files Browse the repository at this point in the history
* feat: use Sumo Logic repository for all images

Signed-off-by: Dominik Rosiek <drosiek@sumologic.com>

* feat: use Sumo Logic repository for all images

Signed-off-by: Dominik Rosiek <drosiek@sumologic.com>

* chore: changelog

Signed-off-by: Dominik Rosiek <drosiek@sumologic.com>

* chore: update prometheus image

Signed-off-by: Dominik Rosiek <drosiek@sumologic.com>

* chore: regenerate golden files

Signed-off-by: Dominik Rosiek <drosiek@sumologic.com>

* chore: fix typo

Signed-off-by: Dominik Rosiek <drosiek@sumologic.com>

---------

Signed-off-by: Dominik Rosiek <drosiek@sumologic.com>
  • Loading branch information
sumo-drosiek authored Apr 25, 2024
1 parent 03efe65 commit 957f3b1
Show file tree
Hide file tree
Showing 11 changed files with 65 additions and 40 deletions.
1 change: 1 addition & 0 deletions .changelog/3677.changed.txt
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
feat: use Sumo Logic repository for all images
21 changes: 14 additions & 7 deletions deploy/helm/sumologic/README.md

Large diffs are not rendered by default.

35 changes: 26 additions & 9 deletions deploy/helm/sumologic/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -119,7 +119,7 @@ sumologic:
tag: 3.13.0
pullPolicy: IfNotPresent
initContainerImage:
repository: public.ecr.aws/docker/library/busybox
repository: public.ecr.aws/sumologic/busybox
tag: latest
## Optionally specify an array of pullSecrets.
## They will be added to serviceaccount that is used for Sumo Logic's
Expand Down Expand Up @@ -744,6 +744,9 @@ sumologic:
## Configure metrics-server
## ref: https://github.com/bitnami/charts/blob/master/bitnami/metrics-server/values.yaml
metrics-server:
image:
registry: public.ecr.aws
repository: sumologic/metrics-server
## Set the enabled flag to true for enabling metrics-server.
## This is required before enabling autoscaling unless you have an existing metrics-server in the cluster.
enabled: false
Expand Down Expand Up @@ -1020,6 +1023,8 @@ kube-prometheus-stack:
defaultDashboardsEnabled: false
prometheusOperator:
enabled: false
image:
repository: public.ecr.aws/sumologic/prometheus-operator
## Labels to add to the operator pod
podLabels: {}
## Annotations to add to the operator pod
Expand Down Expand Up @@ -1065,6 +1070,7 @@ kube-prometheus-stack:
## uses version 2.6.0 of kube-state-metrics, but this version has some critical vulnerabilities,
## so we bump the image manually.
image:
repository: public.ecr.aws/sumologic/kube-state-metrics
tag: "v2.7.0"
prometheus:
monitor:
Expand Down Expand Up @@ -1131,6 +1137,8 @@ kube-prometheus-stack:
prometheus-node-exporter:
## Put here the new name if you want to override the full name used for Prometheus Node exporter components.
# fullnameOverride: ""
image:
repository: public.ecr.aws/sumologic/node-exporter

nodeSelector:
kubernetes.io/os: linux
Expand Down Expand Up @@ -1180,6 +1188,8 @@ kube-prometheus-stack:
enabled: false
additionalServiceMonitors: []
prometheusSpec:
image:
repository: public.ecr.aws/sumologic/prometheus
## Prometheus default scrape interval, default from upstream Kube Prometheus Stack Helm chart
## NOTE changing the scrape interval to be >1m can result in metrics
## from recording rules to be missing and empty panels in Sumo Logic Kubernetes apps.
Expand Down Expand Up @@ -2068,7 +2078,7 @@ otellogs:
initContainers:
changeowner:
image:
repository: "public.ecr.aws/docker/library/busybox"
repository: "public.ecr.aws/sumologic/busybox"
tag: "1.36.0"
pullPolicy: IfNotPresent
securityContext:
Expand Down Expand Up @@ -2291,6 +2301,7 @@ telegraf-operator:
# fullnameOverride: ""
image:
sidecarImage: public.ecr.aws/sumologic/telegraf:1.21.2
repository: public.ecr.aws/sumologic/telegraf-operator
replicaCount: 1
classes:
secretName: "telegraf-operator-classes"
Expand Down Expand Up @@ -2323,7 +2334,7 @@ falco:

image:
registry: public.ecr.aws
# repository: falcosecurity/falco-no-driver
repository: sumologic/falco-no-driver

## Add kernel-devel package through MachineConfig, required to enable building of missing falco modules (only for OpenShift)
addKernelDevel: true
Expand All @@ -2332,7 +2343,7 @@ falco:
initContainers:
## Add initContainer to wait until kernel-devel is installed on host
- name: init-falco
image: public.ecr.aws/docker/library/busybox:1.36.0
image: public.ecr.aws/sumologic/busybox:1.36.0
command:
- "sh"
- "-c"
Expand All @@ -2357,7 +2368,7 @@ falco:
initContainer:
image:
registry: public.ecr.aws
# repository: falcosecurity/falco-driver-loader
repository: sumologic/falco-driver-loader
falco:
load_plugins:
- json
Expand Down Expand Up @@ -2438,7 +2449,7 @@ opentelemetry-operator:

instrumentation:
dotnet:
repository: ghcr.io/open-telemetry/opentelemetry-operator/autoinstrumentation-dotnet
repository: public.ecr.aws/sumologic/autoinstrumentation-dotnet
tag: 0.7.0

## image is deprecated, it should be specified in ${repository}:${tag} format
Expand All @@ -2454,7 +2465,7 @@ opentelemetry-operator:
# value: my_value

java:
repository: ghcr.io/open-telemetry/opentelemetry-operator/autoinstrumentation-java
repository: public.ecr.aws/sumologic/autoinstrumentation-java
tag: 1.26.0

## image is deprecated, it should be specified in ${repository}:${tag} format
Expand All @@ -2470,7 +2481,7 @@ opentelemetry-operator:
# value: my_value

python:
repository: ghcr.io/open-telemetry/opentelemetry-operator/autoinstrumentation-python
repository: public.ecr.aws/sumologic/autoinstrumentation-python
tag: 0.39b0

## image is deprecated, it should be specified in ${repository}:${tag} format
Expand All @@ -2488,7 +2499,7 @@ opentelemetry-operator:
## Current instrumentation doesn't support customization
## for nodejs. Traces are always enabled.
nodejs:
repository: ghcr.io/open-telemetry/opentelemetry-operator/autoinstrumentation-nodejs
repository: public.ecr.aws/sumologic/autoinstrumentation-nodejs
tag: 0.40.0

## image is deprecated, it should be specified in ${repository}:${tag} format
Expand Down Expand Up @@ -2516,6 +2527,8 @@ opentelemetry-operator:
issuerRef: {}

manager:
image:
repository: public.ecr.aws/sumologic/opentelemetry-operator
collectorImage:
repository: "public.ecr.aws/sumologic/sumologic-otel-collector"
tag: "0.92.0-sumo-0"
Expand All @@ -2534,6 +2547,10 @@ opentelemetry-operator:
image:
repository: public.ecr.aws/sumologic/kube-rbac-proxy

testFramework:
image:
repository: public.ecr.aws/sumologic/busybox

## pvcCleaner deletes unused PVCs
pvcCleaner:
metrics:
Expand Down
2 changes: 1 addition & 1 deletion tests/helm/testdata/goldenfile/cleanup/basic.output.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@ spec:
emptyDir: {}
initContainers:
- name: copy-files
image: public.ecr.aws/docker/library/busybox:latest
image: public.ecr.aws/sumologic/busybox:latest
command: ["sh", "-c", "cp /configmap/* /etc/terraform"]
volumeMounts:
- name: configmap
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@ spec:
emptyDir: {}
initContainers:
- name: copy-files
image: public.ecr.aws/docker/library/busybox:latest
image: public.ecr.aws/sumologic/busybox:latest
command: ["sh", "-c", "cp /configmap/* /etc/terraform"]
volumeMounts:
- name: configmap
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -120,7 +120,7 @@ spec:
initContainers: # ensure the host path is owned by the otel user group
- name: changeowner
# yamllint disable-line rule:line-length
image: public.ecr.aws/docker/library/busybox:1.36.0
image: public.ecr.aws/sumologic/busybox:1.36.0
imagePullPolicy: IfNotPresent
securityContext:
capabilities:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -103,7 +103,7 @@ spec:
initContainers: # ensure the host path is owned by the otel user group
- name: changeowner
# yamllint disable-line rule:line-length
image: public.ecr.aws/docker/library/busybox:1.36.0
image: public.ecr.aws/sumologic/busybox:1.36.0
imagePullPolicy: IfNotPresent
securityContext:
capabilities:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -94,7 +94,7 @@ spec:
initContainers: # ensure the host path is owned by the otel user group
- name: changeowner
# yamllint disable-line rule:line-length
image: public.ecr.aws/docker/library/busybox:1.36.0
image: public.ecr.aws/sumologic/busybox:1.36.0
imagePullPolicy: IfNotPresent
securityContext:
capabilities:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -112,7 +112,7 @@ spec:
initContainers: # ensure the host path is owned by the otel user group
- name: changeowner
# yamllint disable-line rule:line-length
image: public.ecr.aws/docker/library/busybox:1.36.0
image: public.ecr.aws/sumologic/busybox:1.36.0
imagePullPolicy: IfNotPresent
securityContext:
capabilities:
Expand Down
20 changes: 10 additions & 10 deletions tests/helm/testdata/goldenfile/metrics-server/basic.output.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -29,9 +29,6 @@ spec:
helm.sh/chart: metrics-server-6.13.1
spec:
serviceAccountName: RELEASE-NAME-metrics-server
volumes:
- emptyDir: {}
name: empty-dir
affinity:
podAffinity:

Expand All @@ -50,7 +47,7 @@ spec:
dnsPolicy: ClusterFirst
containers:
- name: metrics-server
image: docker.io/bitnami/metrics-server:0.7.0-debian-12-r8
image: public.ecr.aws/sumologic/metrics-server:0.7.0-debian-12-r8
imagePullPolicy: "IfNotPresent"
securityContext:
allowPrivilegeEscalation: false
Expand Down Expand Up @@ -96,9 +93,12 @@ spec:
port: https
scheme: HTTPS
volumeMounts:
- mountPath: "/tmp"
name: "empty-dir"
subPath: "tmp-dir"
- mountPath: "/opt/bitnami/metrics-server/apiserver.local.config"
name: "empty-dir"
subPath: "app-tmp-dir"
- name: empty-dir
mountPath: /tmp
subPath: tmp-dir
- name: empty-dir
mountPath: /opt/bitnami/metrics-server/apiserver.local.config
subPath: app-tmp-dir
volumes:
- name: empty-dir
emptyDir: {}
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@ data:
- name: OTEL_RESOURCE_ATTRIBUTES
value: application=ot-operator1
dotnet:
image: ghcr.io/open-telemetry/opentelemetry-operator/autoinstrumentation-dotnet:0.7.0
image: public.ecr.aws/sumologic/autoinstrumentation-dotnet:0.7.0
env:
- name: OTEL_METRICS_EXPORTER
value: otlp
Expand All @@ -48,7 +48,7 @@ data:
- name: OTEL_EXPORTER_OTLP_ENDPOINT
value: http://RELEASE-NAME-sumologic-otelagent.sumologic:4318
python:
image: ghcr.io/open-telemetry/opentelemetry-operator/autoinstrumentation-python:0.39b0
image: public.ecr.aws/sumologic/autoinstrumentation-python:0.39b0
env:
- name: OTEL_METRICS_EXPORTER
value: otlp
Expand All @@ -59,12 +59,12 @@ data:
- name: OTEL_EXPORTER_OTLP_ENDPOINT
value: http://RELEASE-NAME-sumologic-otelagent.sumologic:4318
nodejs:
image: ghcr.io/open-telemetry/opentelemetry-operator/autoinstrumentation-nodejs:0.40.0
image: public.ecr.aws/sumologic/autoinstrumentation-nodejs:0.40.0
env:
- name: OTEL_EXPORTER_OTLP_ENDPOINT
value: http://RELEASE-NAME-sumologic-otelagent.sumologic:4317
java:
image: ghcr.io/open-telemetry/opentelemetry-operator/autoinstrumentation-java:1.26.0
image: public.ecr.aws/sumologic/autoinstrumentation-java:1.26.0
env:
- name: OTEL_METRICS_EXPORTER
value: otlp
Expand Down Expand Up @@ -97,7 +97,7 @@ data:
- name: OTEL_RESOURCE_ATTRIBUTES
value: application=ot-operator2
dotnet:
image: ghcr.io/open-telemetry/opentelemetry-operator/autoinstrumentation-dotnet:0.7.0
image: public.ecr.aws/sumologic/autoinstrumentation-dotnet:0.7.0
env:
- name: OTEL_METRICS_EXPORTER
value: otlp
Expand All @@ -110,7 +110,7 @@ data:
- name: OTEL_EXPORTER_OTLP_ENDPOINT
value: http://RELEASE-NAME-sumologic-otelagent.sumologic:4318
python:
image: ghcr.io/open-telemetry/opentelemetry-operator/autoinstrumentation-python:0.39b0
image: public.ecr.aws/sumologic/autoinstrumentation-python:0.39b0
env:
- name: OTEL_METRICS_EXPORTER
value: otlp
Expand All @@ -121,12 +121,12 @@ data:
- name: OTEL_EXPORTER_OTLP_ENDPOINT
value: http://RELEASE-NAME-sumologic-otelagent.sumologic:4318
nodejs:
image: ghcr.io/open-telemetry/opentelemetry-operator/autoinstrumentation-nodejs:0.40.0
image: public.ecr.aws/sumologic/autoinstrumentation-nodejs:0.40.0
env:
- name: OTEL_EXPORTER_OTLP_ENDPOINT
value: http://RELEASE-NAME-sumologic-otelagent.sumologic:4317
java:
image: ghcr.io/open-telemetry/opentelemetry-operator/autoinstrumentation-java:1.26.0
image: public.ecr.aws/sumologic/autoinstrumentation-java:1.26.0
env:
- name: OTEL_METRICS_EXPORTER
value: otlp
Expand Down

0 comments on commit 957f3b1

Please sign in to comment.