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

carbonexporter: add support for resourcetotelemetry #29879

Merged
merged 1 commit into from
Dec 14, 2023

Conversation

bogdandrutu
Copy link
Member

No description provided.

.chloggen/addresourcetotelemetry.yaml Outdated Show resolved Hide resolved
@bogdandrutu bogdandrutu deleted the addresourcetotelemetry branch December 14, 2023 15:58
@bogdandrutu bogdandrutu restored the addresourcetotelemetry branch December 14, 2023 15:58
@bogdandrutu bogdandrutu reopened this Dec 14, 2023
@codeboten

This comment was marked as resolved.

@bogdandrutu bogdandrutu force-pushed the addresourcetotelemetry branch from 4722f4d to 064d69d Compare December 14, 2023 17:32
Signed-off-by: Bogdan Drutu <bogdandrutu@gmail.com>
@bogdandrutu bogdandrutu force-pushed the addresourcetotelemetry branch from 064d69d to 20ea4cf Compare December 14, 2023 17:35
@bogdandrutu bogdandrutu merged commit 6160efb into open-telemetry:main Dec 14, 2023
73 of 77 checks passed
@github-actions github-actions bot added this to the next release milestone Dec 14, 2023
bogdandrutu added a commit that referenced this pull request Dec 14, 2023
In
#29879
I forgot to actually plugin the logic, only the config was added there.
Added a test to confirm it.

Signed-off-by: Bogdan Drutu <bogdandrutu@gmail.com>
cparkins pushed a commit to AmadeusITGroup/opentelemetry-collector-contrib that referenced this pull request Jan 10, 2024
…9879)

Signed-off-by: Bogdan Drutu <bogdandrutu@gmail.com>
cparkins pushed a commit to AmadeusITGroup/opentelemetry-collector-contrib that referenced this pull request Jan 10, 2024
…ry#29888)

In
open-telemetry#29879
I forgot to actually plugin the logic, only the config was added there.
Added a test to confirm it.

Signed-off-by: Bogdan Drutu <bogdandrutu@gmail.com>
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