Skip to content
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

Rename parameter groups for Splunk OTel Collector components #301

Merged
merged 6 commits into from
Nov 18, 2021

Conversation

dmitryax
Copy link
Contributor

@dmitryax dmitryax commented Nov 18, 2021

Rename the following parameter groups according to Splunk GDI Specification:

  • otelAgent -> agent
  • otelCollector -> gateway
  • otelK8sClusterReceiver -> clusterReceiver

@dmitryax dmitryax requested review from a team as code owners November 18, 2021 08:32
@dmitryax dmitryax force-pushed the change-component-config-groups branch from d9abeee to 07bc2a1 Compare November 18, 2021 08:33
Rename the following parameter groups according to [Splunk GDI Specification](https://github.com/signalfx/gdi-specification/blob/main/specification/configuration.md#kubernetes-package-management-solutions):
- `otelAgent` -> `agent`
- `otelCollector` -> `gateway`
- `otelK8sClusterReceiver` -> `clusterReceiver`
@dmitryax dmitryax force-pushed the change-component-config-groups branch from 07bc2a1 to 18b699f Compare November 18, 2021 08:38
@chaitanyaphalak
Copy link
Contributor

I am guessing this change is for more clarity and reducing ambiguity?

@dmitryax
Copy link
Contributor Author

Yes. Decision was made here to establish this configuration interface for helm chart and operator prior to declaring GA

@rockb1017
Copy link
Contributor

let me see if i can push a commit to here

@rockb1017
Copy link
Contributor

oh i can. please review @dmitryax

@dmitryax
Copy link
Contributor Author

Awesome! Thanks @rockb1017!

@dmitryax dmitryax merged commit 533de07 into main Nov 18, 2021
@dmitryax dmitryax deleted the change-component-config-groups branch November 18, 2021 22:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants