Tyler Helmuth
Tyler Helmuth
Oh I see we don't set a default tag in the chart because we are relying on the operator's defaults. I believe the operator _requires_ both repository and image in...
@alita1991 I am not sure how the chart can handle setting the `--target-allocator-image` command line argument (or any of the command line images) if only a repository is configured but...
@siegenthalerroger would you like to submit a PR for this?
Manual change.
Hmm. We dont really want to include the examples in the chart package since it bloats the release: https://github.com/open-telemetry/opentelemetry-helm-charts/pull/1620.
@jaronoff97 should we add a feature to the chart to automatically create this scrap configs file on install?
If helm does honor the `required` section in the schema file then I agree they aren't needed.
@julianocosta89 I suspect this is a result of the result bump you did.
One alternative I want to throw out there, does this work? ``` transform/string_to_boolean: log_statements: - statements: - set(log.attributes["my.attribute"], ToLowerCase(log.attributes["my.attribute"]) == "true") ```
I think that would be possible.