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

[chore] Update core dependancy #14159

Merged
merged 2 commits into from
Sep 18, 2022
Merged

Conversation

dmitryax
Copy link
Member

@dmitryax dmitryax commented Sep 16, 2022

It also includes removal of deprecated API as done in #14165

mx-psi
mx-psi previously approved these changes Sep 16, 2022
@mx-psi mx-psi dismissed their stale review September 16, 2022 08:30

tests failing

@mx-psi
Copy link
Member

mx-psi commented Sep 16, 2022

legacy_encode_json_test.go:82:19: NewValueBytesEmpty not declared by package pcommon (typecheck)
	bytes := pcommon.NewValueBytesEmpty()

(loki exporter)

@dmitryax
Copy link
Member Author

addressed in #14165

@dmitryax dmitryax force-pushed the upgrotel branch 2 times, most recently from 54f993d to fbf29c0 Compare September 16, 2022 17:11
@dmitryax dmitryax added the Skip Changelog PRs that do not require a CHANGELOG.md entry label Sep 16, 2022
@bogdandrutu bogdandrutu merged commit c06fda4 into open-telemetry:main Sep 18, 2022
djaglowski pushed a commit to djaglowski/opentelemetry-collector-contrib that referenced this pull request Sep 19, 2022
* [chore] Update core dependancy

* [processor/spanmetrics] Remove usage of deprecated pcommon.NewMapFromRaw
@dmitryax dmitryax deleted the upgrotel branch November 15, 2022 17:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Skip Changelog PRs that do not require a CHANGELOG.md entry
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants