-
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!: remove replacing values in configuration marked by 'replace' suffix #2615
Conversation
512986f
to
ed9ece8
Compare
ed9ece8
to
f37b99a
Compare
d5d1a27
to
f37b99a
Compare
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.
Great. Not sure if we need a migration for that change.
I need to add at least information to migration doc that the mechanism with replacing some configuration values with 'replace' suffix was removed. related change in update-collection-v3: SumoLogic/sumologic-kubernetes-tools#382 |
f37b99a
to
32a635c
Compare
…ace' suffix in configuration for otelcol and otelagent related to SumoLogic/sumologic-kubernetes-collection#2615
…ace' suffix in configuration for otelcol and otelagent related to SumoLogic/sumologic-kubernetes-collection#2615
32a635c
to
4892a55
Compare
98dee38
to
f9daff2
Compare
…ace' suffix in configuration for otelcol and otelagent related to SumoLogic/sumologic-kubernetes-collection#2615
f9daff2
to
2e7d283
Compare
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.
🎉
…ace' suffix in configuration for otelcol and otelagent related to SumoLogic/sumologic-kubernetes-collection#2615
chore: remove replacing values in configuration marked by 'replace' suffix
make values in configuration for otelcol dependended on otelcol parameters in values.yaml
removes "otelcol.k8s.cluster.name" which was added in #1721 and it is no longer in use
this pull request reverts changes introduced in #1721
related to #1729
Generated templates have been compared using output of
helm template . -f values.yaml -f values_custom.yaml
wherevalues_custom.yaml
had following form:related change in
update-collection-v3
: SumoLogic/sumologic-kubernetes-tools#382