Skip to content

[chore] Add display name to skywalker encoder#45649

Merged
atoulme merged 12 commits into
open-telemetry:mainfrom
jaydeluca:skywalker-display-name
Feb 10, 2026
Merged

[chore] Add display name to skywalker encoder#45649
atoulme merged 12 commits into
open-telemetry:mainfrom
jaydeluca:skywalker-display-name

Conversation

@jaydeluca
Copy link
Copy Markdown
Member

@jaydeluca jaydeluca commented Jan 26, 2026

Description

Uses the new display_name and description metadata fields for the skywalker encoder.

This module was missing the doc.go file so the mdatagen tasks were not executing properly, so I added that and it also generated the other files. The generated component test failed due to the factory returning nil for the config, so I added an empty struct to prevent that and for the tests to pass.

Related to open-telemetry/opentelemetry-collector#14114 and open-telemetry/opentelemetry-collector#14400

Continuation from #45537 #45554 #45599

@atoulme
Copy link
Copy Markdown
Contributor

atoulme commented Feb 9, 2026

Please address conflicts.

@atoulme atoulme marked this pull request as draft February 9, 2026 17:23
@jaydeluca jaydeluca marked this pull request as ready for review February 10, 2026 00:41
@atoulme atoulme merged commit b111895 into open-telemetry:main Feb 10, 2026
197 checks passed
geekdave pushed a commit to oso-team/opentelemetry-collector-contrib that referenced this pull request Feb 20, 2026
**Description**

Uses the new
[display_name](open-telemetry/opentelemetry-collector#14115)
and description metadata fields for the skywalker encoder.

This module was missing the `doc.go` file so the mdatagen tasks were not
executing properly, so I added that and it also generated the other
files. The generated component test failed due to the factory returning
nil for the config, so I added an empty struct to prevent that and for
the tests to pass.

Related to
open-telemetry/opentelemetry-collector#14114
and open-telemetry/opentelemetry-collector#14400

Continuation from open-telemetry#45537 open-telemetry#45554 open-telemetry#45599
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants