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

[resourceprocessor] add support for profile signal type #36208

Merged
merged 43 commits into from
Mar 4, 2025

Conversation

bacherfl
Copy link
Contributor

@bacherfl bacherfl commented Nov 5, 2024

Description

This PR adds support for the profile signal type to the resource processor

Link to tracking issue

Fixes #35979

Testing

Extended the unit tests of the component

Documentation

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>
@github-actions github-actions bot added the processor/resource Resource processor label Nov 5, 2024
@github-actions github-actions bot requested a review from dmitryax November 5, 2024 13:13
Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>
Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>
Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>
Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>
@bacherfl bacherfl marked this pull request as ready for review November 6, 2024 08:05
@bacherfl bacherfl requested a review from a team as a code owner November 6, 2024 08:05
# Conflicts:
#	processor/resourceprocessor/README.md
#	processor/resourceprocessor/go.mod
#	processor/resourceprocessor/go.sum
Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>
Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>
# Conflicts:
#	processor/resourceprocessor/go.mod
#	processor/resourceprocessor/go.sum
Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>
Copy link
Contributor

github-actions bot commented Dec 4, 2024

This PR was marked stale due to lack of activity. It will be closed in 14 days.

@github-actions github-actions bot added the Stale label Dec 4, 2024
# Conflicts:
#	processor/resourceprocessor/go.mod
#	processor/resourceprocessor/go.sum
Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>
@github-actions github-actions bot removed the Stale label Dec 7, 2024
# Conflicts:
#	processor/resourceprocessor/go.mod
#	processor/resourceprocessor/go.sum
Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>
Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>
# Conflicts:
#	processor/resourceprocessor/go.mod
#	processor/resourceprocessor/go.sum
Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>
Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>
Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>
@atoulme
Copy link
Contributor

atoulme commented Jan 23, 2025

@dmitryax please review as codeowner

# Conflicts:
#	processor/resourceprocessor/go.mod
#	processor/resourceprocessor/go.sum
Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>
# Conflicts:
#	processor/resourceprocessor/go.mod
#	processor/resourceprocessor/go.sum
Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>
Copy link
Contributor

This PR was marked stale due to lack of activity. It will be closed in 14 days.

@github-actions github-actions bot added the Stale label Feb 19, 2025
@atoulme
Copy link
Contributor

atoulme commented Feb 20, 2025

Please resolve conflicts.

@atoulme atoulme removed the Stale label Feb 20, 2025
# Conflicts:
#	processor/resourceprocessor/go.mod
#	processor/resourceprocessor/go.sum
Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>
@@ -9,12 +9,16 @@ require (
go.opentelemetry.io/collector/component v0.120.1-0.20250218214003-dae5d199ffc6
go.opentelemetry.io/collector/component/componenttest v0.120.1-0.20250218214003-dae5d199ffc6
go.opentelemetry.io/collector/confmap v1.26.1-0.20250218214003-dae5d199ffc6
go.opentelemetry.io/collector/confmap/xconfmap v0.120.1-0.20250218214003-dae5d199ffc6
go.opentelemetry.io/collector/confmap/xconfmap v0.120.0
Copy link
Contributor

Choose a reason for hiding this comment

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

probably keep the old version for this, but we can also wait after release

bacherfl added 6 commits March 4, 2025 09:58
# Conflicts:
#	processor/resourceprocessor/go.mod
#	processor/resourceprocessor/go.sum
Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>
# Conflicts:
#	processor/resourceprocessor/go.mod
#	processor/resourceprocessor/go.sum
Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>
Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>
@atoulme atoulme merged commit 02491b8 into open-telemetry:main Mar 4, 2025
156 checks passed
@github-actions github-actions bot added this to the next release milestone Mar 4, 2025
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.

[processor/resource] Add support for profiles signal
4 participants