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

Upgrade otel compenents to v0.0.0-20240528184218-531527333157 #1473

Merged
merged 8 commits into from
Dec 17, 2024
Merged

Conversation

Paramadon
Copy link
Contributor

@Paramadon Paramadon commented Dec 17, 2024

Description of the Issue

This PR updates OpenTelemetry components from v0.0.0-20241206205307-66e9942f29e0 to v0.0.0-20240528184218-531527333157. The updated version includes health-related improvements to the following components:

  • awscontainerinsightsreceiver
  • resourcedetectionprocessor
  • emf_exporter
  • awsxrayexporter
  • awscloudwatchlogsexporter

Additionally, this update adds the statuscode middleware to the configuration of these components.

Relevant PRs for context:


Description of Changes

This change ensures that the updated components incorporate health monitoring improvements and statuscode middleware, improving reliability and observability.


Tests

Please describe what tests were performed to verify these changes. For example:

  • Ran agent to produce metrics
Screenshot 2024-12-17 at 4 06 22 PM ---

License

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution under the terms of your choice.


Requirements

Before merging this PR, please ensure the following steps are completed:

  1. Run make fmt and make fmt-sh.
  2. Run make lint to ensure there are no linting issues.

Let me know if you need further refinements!

@Paramadon Paramadon requested a review from a team as a code owner December 17, 2024 14:53
@musa-asad
Copy link
Contributor

musa-asad commented Dec 17, 2024

Some unit tests failures.

            	            	Diff:
            	            	--- Expected
            	            	+++ Actual
            	            	@@ -552,3 +552,8 @@
            	            	  Attributes: ([]string) <nil>,
            	            	- MiddlewareID: (*component.ID)(<nil>)
            	            	+ MiddlewareID: (*component.ID)({
            	            	+  typeVal: (component.Type) {
            	            	+   name: (string) (len=11) "agenthealth"
            	            	+  },
            	            	+  nameVal: (string) (len=10) "statuscode"
            	            	+ })
            	            	 })
            	Test:       	TestTranslate/WithAppSignalsEnabledOnECS

@Paramadon Paramadon merged commit 3f78406 into main Dec 17, 2024
7 checks passed
@Paramadon Paramadon deleted the otelBump branch December 17, 2024 23:13
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.

3 participants