Skip to content

Add missing otel signal viewer plugin for the otel Agent#532

Open
ralphm wants to merge 1 commit into
masterfrom
missing-plugins
Open

Add missing otel signal viewer plugin for the otel Agent#532
ralphm wants to merge 1 commit into
masterfrom
missing-plugins

Conversation

@ralphm

@ralphm ralphm commented Jun 18, 2026

Copy link
Copy Markdown
Member

As the default configuration explicitly disables "new" plugins, all plugins that are not explicitly enabled will be disabled. Since the set of enabled plugins for use within k8s has been created a while ago, new plugins such as the otel-signal-viewer plugin are not mentioned, even though we do expect them to be there.

@CLAassistant

CLAassistant commented Jun 18, 2026

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@ralphm

ralphm commented Jun 18, 2026

Copy link
Copy Markdown
Member Author

@M4itee We may want to add some other plugins as well, like network-viewer. This problem will continue to pop up when we add new plugins.

We currently have enable running new plugin = no, which means 'disable all by default', so all the explicit = no entries we have now are basically no-ops. Possibly, we could have that setting as 'yes' and then only disables the plugins we do not want?

@M4itee

M4itee commented Jun 18, 2026

Copy link
Copy Markdown
Contributor

we can, the goal was for this endpoint to ONLY host logs (no overhead)

@ralphm ralphm changed the title Add missing Agent plugins Add missing otel signal viewer plugin for the otel Agent Jun 18, 2026
@ralphm ralphm marked this pull request as ready for review June 18, 2026 11:35
@ralphm ralphm requested review from a team and ilyam8 as code owners June 18, 2026 11:35
@ralphm ralphm requested review from M4itee and witalisoft June 18, 2026 11:35
@ralphm

ralphm commented Jun 18, 2026

Copy link
Copy Markdown
Member Author

Per out-of-band discussion, this change should be sufficient for the time being. Please review.

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