Skip to content

opentelemetry tracer: Not crash envoy when environment resource detector can't detect attributes#32229

Merged
wbpcode merged 2 commits intoenvoyproxy:mainfrom
dynatrace-oss-contrib:feat/otel-res-detector-no-fail
Feb 8, 2024
Merged

opentelemetry tracer: Not crash envoy when environment resource detector can't detect attributes#32229
wbpcode merged 2 commits intoenvoyproxy:mainfrom
dynatrace-oss-contrib:feat/otel-res-detector-no-fail

Conversation

@joaopgrassi
Copy link
Copy Markdown
Contributor

Commit Message: Not crash envoy when env resource detector can't detect attributes
Additional Description:
Risk Level: Low
Testing: Unit/Integration/Manual
Docs Changes: N/A
Release Notes: N/A
Platform Specific Features: N/A
[Optional Runtime guard:]
[Optional Fixes #32225]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

Signed-off-by: Joao Grassi <joao.grassi@dynatrace.com>
@joaopgrassi joaopgrassi requested a review from htuch as a code owner February 6, 2024 17:54
@joaopgrassi
Copy link
Copy Markdown
Contributor Author

joaopgrassi commented Feb 6, 2024

CC @wbpcode as we discussed offline.

@joaopgrassi
Copy link
Copy Markdown
Contributor Author

/retest

@wbpcode
Copy link
Copy Markdown
Member

wbpcode commented Feb 7, 2024

/wait add warning log

Signed-off-by: Joao Grassi <joao.grassi@dynatrace.com>
Copy link
Copy Markdown
Member

@wbpcode wbpcode left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks.

@wbpcode wbpcode merged commit b788e1a into envoyproxy:main Feb 8, 2024
@joaopgrassi joaopgrassi deleted the feat/otel-res-detector-no-fail branch February 8, 2024 12:42
joaopgrassi added a commit to dynatrace-oss-contrib/envoy that referenced this pull request Feb 9, 2024
…tor can't detect attributes (envoyproxy#32229)

Signed-off-by: Joao Grassi <joao.grassi@dynatrace.com>
phlax pushed a commit that referenced this pull request Feb 9, 2024
…tor can't detect attributes (#32229)

Signed-off-by: Joao Grassi <joao.grassi@dynatrace.com>
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.

OTEL Environment resource detector is unsafe/non-compliant due to error handling

3 participants