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/ottl] Add StandardFuncs and StandardConverters to ottlfuncs #23190

Merged

Conversation

TylerHelmuth
Copy link
Member

Description:
Currently every time a function is added to ottlfuncs it must then be manually added to each component that uses OTTL (assuming that the component wants access to it). As OTTL's usage has grown across Contrib this has become more tedious.

To ease this process for components that want all available editors/converts from ottlfuncs I've added StandardFuncs and StandardConverters groups. These methods allow components access to all the standard functions (or only the converters) in ottlfuncs. Going forward, whenever a new function is added these lists can be updated and all the components that are opting in will get the components in the next release.

Copy link
Contributor

@evan-bradley evan-bradley left a comment

Choose a reason for hiding this comment

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

Looks good to me. Looking forward to this, I think it will make things easier.

@TylerHelmuth TylerHelmuth merged commit 8faa947 into open-telemetry:main Jun 8, 2023
@TylerHelmuth TylerHelmuth deleted the ottl-standard-functions branch June 8, 2023 19:38
@github-actions github-actions bot added this to the next release milestone Jun 8, 2023
Caleb-Hurshman pushed a commit to observIQ/opentelemetry-collector-contrib that referenced this pull request Jul 6, 2023
…n-telemetry#23190)

* Add StandardFuncs and StandardConverters to ottlfuncs

* changelog

* add license

* make goporto

* update changelog
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.

4 participants