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

[pkg/stanze, receiver/filelog] Allow opting into scope name and version #31771

Conversation

TylerHelmuth
Copy link
Member

Description:
This PR adds the option for Stanza users to opt in to setting the scope name and version by implementing extra interfaces.

Link to tracking Issue:

Closes #31737

Comment on lines +84 to +86
func withPackageName(name string) converterOption {
return packageNameOption{name}
}
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The interface pattern allows this to be an opt-in solution, but feels kinda magical. I'm not sold on this idea, but it does provide the value without any breaking changes and I think we do need the component to provide the value since the ID is not the same as the package name.

Copy link
Member

@djaglowski djaglowski left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd like to make a decision on the issue before this PR moves forward, so am blocking this for now.

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.

[receiver/filelog] Set instrumentation scope name and version
3 participants