[Helm] Move collector service account template to separate collector … (

#4312)
Dynatrace · Jan 27, 2025 · cef9690 · cef9690
1 parent f00e14a
commit cef9690
Show file tree

Hide file tree

Showing 9 changed files with 14 additions and 14 deletions.
diff --git a/...e-extensions-opentelemetry-collector.yaml → ...ons/clusterole-extensions-prometheus.yaml b/...e-extensions-opentelemetry-collector.yaml → ...ons/clusterole-extensions-prometheus.yaml
@@ -18,7 +18,7 @@ metadata:
   name: dynatrace-extensions-prometheus
   namespace: {{ .Release.Namespace }}
   labels:
-    {{- include "dynatrace-operator.extensionsOpenTelemetryCollectorLabels" . | nindent 4 }}
+    {{- include "dynatrace-operator.openTelemetryCollectorLabels" . | nindent 4 }}
 rules:
   - apiGroups:
       - ""
@@ -66,13 +66,13 @@ metadata:
   name: dynatrace-extensions-prometheus
   namespace: {{ .Release.Namespace }}
   labels:
-    {{- include "dynatrace-operator.extensionsOpenTelemetryCollectorLabels" . | nindent 4 }}
+    {{- include "dynatrace-operator.openTelemetryCollectorLabels" . | nindent 4 }}
 roleRef:
   apiGroup: rbac.authorization.k8s.io
   kind: ClusterRole
   name: dynatrace-extensions-prometheus
 subjects:
   - kind: ServiceAccount
-    name: dynatrace-extensions-collector
+    name: dynatrace-opentelemetry-collector
     namespace: {{ .Release.Namespace }}
 {{ end }}
diff --git a/...ontroller/role-extensions-controller.yaml → ...xtensions/role-extensions-controller.yaml b/...ontroller/role-extensions-controller.yaml → ...xtensions/role-extensions-controller.yaml
diff --git a/...serviceaccount-extensions-controller.yaml → ...serviceaccount-extensions-controller.yaml b/...serviceaccount-extensions-controller.yaml → ...serviceaccount-extensions-controller.yaml
diff --git a/...t-extensions-opentelemetry-collector.yaml → ...rviceaccount-opentelemetry-collector.yaml b/...t-extensions-opentelemetry-collector.yaml → ...rviceaccount-opentelemetry-collector.yaml
@@ -15,12 +15,12 @@
 apiVersion: v1
 kind: ServiceAccount
 metadata:
-  name: dynatrace-extensions-collector
+  name: dynatrace-opentelemetry-collector
   namespace: {{ .Release.Namespace }}
   {{- if .Values.rbac.extensions.annotations }}
   annotations:
     {{- toYaml .Values.rbac.extensions.annotations | nindent 4 }}
   {{- end }}
   labels:
-    {{- include "dynatrace-operator.extensionsOpenTelemetryCollectorLabels" . | nindent 4 }}
+    {{- include "dynatrace-operator.openTelemetryCollectorLabels" . | nindent 4 }}
 {{ end }}
diff --git a/config/helm/chart/default/templates/_labels.tpl b/config/helm/chart/default/templates/_labels.tpl
@@ -110,9 +110,9 @@ app.kubernetes.io/component: dynatrace-extensions-controller
 {{- end -}}
 
 {{/*
-Extensions OpenTelemetry Collector (OTelC) labels
+OpenTelemetry Collector (OTelC) labels
 */}}
-{{- define "dynatrace-operator.extensionsOpenTelemetryCollectorLabels" -}}
+{{- define "dynatrace-operator.openTelemetryCollectorLabels" -}}
 {{ include "dynatrace-operator.commonLabels" . }}
 app.kubernetes.io/component: dynatrace-extensions-collector
 {{- end -}}

diff --git a/...ensions-opentelemetry-collector_test.yaml → ...usterrole-extensions-prometheus_test.yaml b/...ensions-opentelemetry-collector_test.yaml → ...usterrole-extensions-prometheus_test.yaml
@@ -1,6 +1,6 @@
-suite: test clusterrole for the extensions OpenTelemetry collector
+suite: test clusterrole for the dynatrace OpenTelemetry collector
 templates:
-  - Common/extensions-opentelemetry-collector/clusterrole-extensions-opentelemetry-collector.yaml
+  - Common/extensions/clusterole-extensions-prometheus.yaml
 tests:
   - it: ClusterRole and ClusterRoleBinding exists
     asserts:

diff --git a/...ller/role-extensions-controller_test.yaml → ...ions/role-extensions-controller_test.yaml b/...ller/role-extensions-controller_test.yaml → ...ions/role-extensions-controller_test.yaml
@@ -1,6 +1,6 @@
 suite: test role for the extensions controller
 templates:
-  - Common/extensions-controller/role-extensions-controller.yaml
+  - Common/extensions/role-extensions-controller.yaml
 tests:
   - it: should not exist by default
     asserts:

diff --git a/...ceaccount-extensions-controller_test.yaml → ...ceaccount-extensions-controller_test.yaml b/...ceaccount-extensions-controller_test.yaml → ...ceaccount-extensions-controller_test.yaml
@@ -1,6 +1,6 @@
 suite: test service account for extensions controller
 templates:
-  - Common/extensions-controller/serviceaccount-extensions-controller.yaml
+  - Common/extensions/serviceaccount-extensions-controller.yaml
 tests:
   - it: should exist
     set:

diff --git a/...ensions-opentelemetry-collector_test.yaml → ...account-opentelemetry-collector_test.yaml b/...ensions-opentelemetry-collector_test.yaml → ...account-opentelemetry-collector_test.yaml
@@ -1,6 +1,6 @@
-suite: test service account for extensions OpenTelemetry collector
+suite: test service account for dynatrace OpenTelemetry collector
 templates:
-  - Common/extensions-opentelemetry-collector/serviceaccount-extensions-opentelemetry-collector.yaml
+  - Common/opentelemetry-collector/serviceaccount-opentelemetry-collector.yaml
 tests:
   - it: should exist
     set:
@@ -10,7 +10,7 @@ tests:
           of: ServiceAccount
       - equal:
           path: metadata.name
-          value: dynatrace-extensions-collector
+          value: dynatrace-opentelemetry-collector
       - equal:
           path: metadata.namespace
           value: NAMESPACE