-
Notifications
You must be signed in to change notification settings - Fork 183
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore(opentelemetry-operator): upgrade opentelemetry-operator subchart #2577
Conversation
…t to 0.13.0 (#2561) * chore(chart): update version * chore(tls): remove certificate creation, supported by operator chart * chore(clusterole): add opentelemetry-operator api group * chore(instrumentation): add instrumentation configmap and job * chore(instrumentatio-cr): move instrumentation cr to confs * chore(tests): update tests * chore(opentelemetry-operator): split instrumentation resource creation * chore(readme): update parameters list (cherry picked from commit d4d7599)
@@ -303,8 +303,8 @@ The following table lists the configurable parameters of the Sumo Logic chart an | |||
| `telegraf-operator.default` | Name of the default output configuration. | `sumologic-prometheus` | | |||
| `telegraf-operator.data` | Telegraf sidecar configuration. | See [values.yaml] | | |||
| `opentelemetry-operator.enabled` | Flag to control deploying OpenTelemetry Operator Helm sub-chart. | `false` | | |||
| `opentelemetry-operator.createDefaultInstrumentation` | Flag to control creation of default Instrumentation object | `true` | | |||
| `opentelemetry-operator.manager.env.WATCH_NAMESPACE` | Used to set value for `WATCH_NAMESPACE` environment variable which specifies Namespace to watch and create Instrumentation objects. | `Nil` | | |||
| `opentelemetry-operator.createDefaultInstrumentation` | Flag to control creation of default Instrumentation object | `false` | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Isn't this kind of a breaking change? 😅
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, that seems like a breaking change to me to.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could be a breaking change, but we were treating OT Operator for Tracing as BETA
. Appropriate comments in the next PR with changelog and release description will be added about this change.
admissionWebhooks: | ||
failurePolicy: Fail | ||
enabled: false | ||
enabled: true |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Isn't this a breaking change?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's not as in version before admissionWebhooks were created by us with certificates.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Although this is a breaking change I'm giving 👍 because it is beta as of now.
Description
(cherry picked from commit d4d7599)
This PR:
Instrumentation
CR generation to confs (config map created for this purpose)Instrumentation
resource -opentelemetry-operator.createDefaultInstrumentation=false
opentelemetry-operator.instrumentationNamespaces
parameter forInstrumentation
resource creationChecklist
Testing performed