-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Create opamp extension #16462
Comments
@open-telemetry/collector-contrib-approvers is anyone interested in being the sponsor for this component? I believe @portertech is interested in implementing it (provided we agree on the design) and I am happy to be a co-owner of the extension and will also review the PRs. |
I'll sponsor this. |
I can help with reviewing PRs. @portertech feel free to ping me as well |
@tigrannajaryan To confirm, the extension will now live in contrib and not in the collector itself? I'll move my WIP if that's the case 👍 |
contrib is probably a better place, @open-telemetry/collector-maintainers WDYT? |
i'd assume that in the future this would become a core component, but for now i think it's ok for it to live in contrib. there are other components I imagine will likely move from contrib to core in the future (jaeger/zipkin/prometheus receivers/exporters) |
part of open-telemetry#16462 Signed-off-by: Sean Porter <portertech@gmail.com>
This issue has been inactive for 60 days. It will be closed in 60 days if there is no activity. To ping code owners by adding a component label, see Adding Labels via Comments, or if you are unsure of which component this issue relates to, please ping |
This issue has been inactive for 60 days. It will be closed in 60 days if there is no activity. To ping code owners by adding a component label, see Adding Labels via Comments, or if you are unsure of which component this issue relates to, please ping |
This issue has been inactive for 60 days. It will be closed in 60 days if there is no activity. To ping code owners by adding a component label, see Adding Labels via Comments, or if you are unsure of which component this issue relates to, please ping |
The extension is now at alpha stability and is in the contrib distro, so I think we can say it is in a good state. |
Hi have tried the same steps given by you @brightzheng100 , error opampextension@v0.98.0/logger.go:26 Connection failed (tls: first record does not look like a TLS handshake), will retry. {"kind": "extension", "name": "opamp", "client": "ws"} The Other way i tried was , started the opampsupervisor from /root/OTEL98/opentelemetry-collector-contrib-main/cmd/opampsupervisor using the command ./opampsupervisor --config supervisor.yaml and my yaml is below capabilities: reports_effective_config: true # true if unspecified accepts_packages: true # false if unspecified reports_own_metrics: # true if unspecified reports_own_logs: # true if unspecified accepts_other_connection_settings: true # false if unspecified accepts_restart_command: true # true if unspecified reports_health: true # true if unspecified storage: args: --config /root/OTEL98/opentelemetry-collector-contrib-main/cmd/otelcontribcol/config.yaml env: run_as: root please help me out to solve these issues |
@MSA0208 please file a separate issue for this, thanks! This issue has been completed and is about creating the extension |
See design document.
Depends on:
Todo
Edit (by Evan): more comprehensive health reporting will be tracked by this issue: #32304
The text was updated successfully, but these errors were encountered: