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

How do users set up OTEL initialization configuration? #29581

Closed
mike9421 opened this issue Nov 30, 2023 · 3 comments
Closed

How do users set up OTEL initialization configuration? #29581

mike9421 opened this issue Nov 30, 2023 · 3 comments
Labels
cmd/opampsupervisor question Further information is requested

Comments

@mike9421
Copy link

Component(s)

cmd/opampsupervisor

Describe the issue you're reporting

Consider this scenario: The user deploys OTEL as a DaemonSet in Kubernetes, and the user wants to specify the initial configuration of OTEL in the k8s configuration file. What should the user do?

  1. According to opampsupervisor document,When Supervisor is launched for the first time, users can only initialize the configuration for each OTEL instance remotely.

  2. According to opampsupervisor source code, when supervisor starts, it will try to read the local configuration file (which can be considered as initialization configuration) to start OTEL.

Which of the above methods or both will opampsupervisor ultimately adopt?

@mike9421 mike9421 added the needs triage New item requiring triage label Nov 30, 2023
Copy link
Contributor

Pinging code owners:

See Adding Labels via Comments if you do not have permissions to add labels yourself.

@tigrannajaryan
Copy link
Member

Which of the above methods or both will opampsupervisor ultimately adopt?

Both, as described by the design document. Quote from design document:

The config file is created by merging the local config file (optional) and the 
remote config received from OpAMP backend (also optional).

@crobert-1 crobert-1 added question Further information is requested and removed needs triage New item requiring triage labels Dec 1, 2023
@mike9421
Copy link
Author

mike9421 commented Dec 4, 2023

@tigrannajaryan Thanks for your answer.
I was a little confused about the opampsupervisor document before. It's clear now

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cmd/opampsupervisor question Further information is requested
Projects
None yet
Development

No branches or pull requests

3 participants