Skip to content

Commit 9683e0c

Browse files
authored
fix(logs): adjust naming for operator to work with -operator release … (#1251)
* fix(logs): adjust naming for operator to work with -operator release name * adjust test
1 parent 9fa0289 commit 9683e0c

File tree

6 files changed

+6
-11
lines changed

6 files changed

+6
-11
lines changed

logs/README.md

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -119,7 +119,6 @@ The **Logs** Plugin comes with a [Failover Connector](https://github.com/open-te
119119
| opentelemetry-operator.manager.image.tag | string | `"v0.131.0"` | overrides the default tag repository for the OpenTelemetry Operator image. |
120120
| opentelemetry-operator.manager.serviceMonitor.enabled | bool | `true` | Enable serviceMonitor for Prometheus metrics scrape |
121121
| opentelemetry-operator.manager.serviceMonitor.extraLabels | object | `{}` | Additional labels on the ServiceMonitor |
122-
| opentelemetry-operator.nameOverride | string | `"operator"` | Provide a name in place of the default name `opentelemetry-operator`. |
123122
| testFramework.enabled | bool | `true` | Activates the Helm chart testing framework. |
124123
| testFramework.image.registry | string | `"ghcr.io"` | Defines the image registry for the test framework. |
125124
| testFramework.image.repository | string | `"cloudoperators/greenhouse-extensions-integration-test"` | Defines the image repository for the test framework. |

logs/charts/Chart.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33

44
apiVersion: v2
55
name: logs
6-
version: 0.0.3
6+
version: 0.0.4
77
description: OpenTelemetry Operator Helm chart for Kubernetes
88
icon: https://raw.githubusercontent.com/cncf/artwork/a718fa97fffec1b9fd14147682e9e3ac0c8817cb/projects/opentelemetry/icon/color/opentelemetry-icon-color.png
99
type: application

logs/charts/ci/test-values.yaml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,6 @@
33

44
opentelemetry-operator:
55
enabled: true
6-
nameOverride: "operator"
76
crds:
87
create: false
98
admissionWebhooks:

logs/charts/templates/tests/test-opentelemetry-config.yaml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -22,11 +22,11 @@ data:
2222
{{- if ".Values.opentelemetry-operator.enabled" }}
2323
@test "Verify successful deployment and running status of the {{ .Release.Name }} pod" {
2424
verify "there is 1 deployment named '{{ .Release.Name }}'"
25-
try "at most 4 times every 5s to get pods named '{{ .Release.Name }}-[a-zA-Z0-9]{9,10}-[a-zA-Z0-9]{5}$' and verify that '.status.phase' is 'running'"
25+
try "at most 4 times every 5s to get pods named '{{ .Release.Name }}-opentelemetry-operator-[a-zA-Z0-9]{9,10}-[a-zA-Z0-9]{5}$' and verify that '.status.phase' is 'running'"
2626
}
2727
28-
@test "Verify successful deployment of the {{ .Release.Name }}-operator service" {
29-
verify "there is 1 service named '{{ .Release.Name }}-operator\$'"
28+
@test "Verify successful deployment of the {{ .Release.Name }}-opentelemetry-operator service" {
29+
verify "there is 1 service named '{{ .Release.Name }}-opentelemetry-operator\$'"
3030
}
3131
{{- end -}}
3232

logs/charts/values.yaml

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -8,9 +8,6 @@ commonLabels: {}
88
opentelemetry-operator:
99
# -- Set to true to enable the installation of the OpenTelemetry Operator.
1010
enabled: true
11-
# -- Provide a name in place of the default name `opentelemetry-operator`.
12-
nameOverride: "operator"
13-
1411
crds:
1512
# This is required because there is no other Plugin that would install them, and because automatic CRD installs are not supported due to a limitation with helm, see [HIP0011](https://github.com/helm/community/blob/main/hips/hip-0011.md).
1613
# -- If you want to use the upstream CRDs, set this variable to `true``.

logs/plugindefinition.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,14 +6,14 @@ kind: PluginDefinition
66
metadata:
77
name: logs
88
spec:
9-
version: 0.11.15
9+
version: 0.11.16
1010
displayName: Logs
1111
description: Observability framework for instrumenting, generating, collecting, and exporting logs.
1212
icon: https://raw.githubusercontent.com/cloudoperators/greenhouse-extensions/main/logs/logo.png
1313
helmChart:
1414
name: logs
1515
repository: oci://ghcr.io/cloudoperators/greenhouse-extensions/charts
16-
version: 0.0.3
16+
version: 0.0.4
1717
options:
1818
- default: true
1919
description: Set to true to enable the installation of the OpenTelemetry Operator.

0 commit comments

Comments
 (0)