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

Make configsource Watchable an optional interface #3792

Merged
merged 1 commit into from
Aug 9, 2021

Conversation

bogdandrutu
Copy link
Member

Signed-off-by: Bogdan Drutu [email protected]

@bogdandrutu bogdandrutu requested a review from a team as a code owner August 7, 2021 00:46
@bogdandrutu bogdandrutu requested a review from owais August 7, 2021 00:46
Copy link
Contributor

@pjanotti pjanotti left a comment

Choose a reason for hiding this comment

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

Cleaner

Copy link
Member

@tigrannajaryan tigrannajaryan left a comment

Choose a reason for hiding this comment

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

LGTM

@tigrannajaryan tigrannajaryan merged commit e79b691 into open-telemetry:main Aug 9, 2021
@bogdandrutu bogdandrutu deleted the optconfigsource branch August 9, 2021 15:33
dmitryax added a commit to signalfx/splunk-otel-collector that referenced this pull request Aug 18, 2021
Adapt code to changes in core:
- config.experimental.configsource.Watchable interface open-telemetry/opentelemetry-collector#3792
- Rename CustomUnmarshable to Unmarshallable open-telemetry/opentelemetry-collector#3774
- Switched metrics from Value() to DoubleVal() open-telemetry/opentelemetry-collector#3740
emaderer added a commit to signalfx/splunk-otel-collector that referenced this pull request Aug 18, 2021
* Bump smart agent version to v5.11.3

* Update CHANGELOG.md

* Update core/contrib deps

* Fix broken code due to renaming of CustomUnmarshable to Unmarshallable

* Fix broken code due to renaming of CustomUnmarshable to Unmarshallable

* Revert "Fix broken code due to renaming of CustomUnmarshable to Unmarshallable"

This reverts commit de03f5e.

* Revert "Fix broken code due to renaming of CustomUnmarshable to Unmarshallable"

This reverts commit e09a9e3.

* Update code according to changes in core

Adapt code to changes in core:
- config.experimental.configsource.Watchable interface open-telemetry/opentelemetry-collector#3792
- Rename CustomUnmarshable to Unmarshallable open-telemetry/opentelemetry-collector#3774
- Switched metrics from Value() to DoubleVal() open-telemetry/opentelemetry-collector#3740

Co-authored-by: Eyal Maderer <[email protected]>
Co-authored-by: Dmitry <[email protected]>
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.

None yet

3 participants