Skip to content

CloudResourceContextIntegration freezing in ECS Container #2376

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

Closed
jensenbox opened this issue Sep 16, 2023 · 2 comments · Fixed by #4120
Closed

CloudResourceContextIntegration freezing in ECS Container #2376

jensenbox opened this issue Sep 16, 2023 · 2 comments · Fixed by #4120
Assignees
Labels
Integration: Cloud Resource Context Triaged Has been looked at recently during old issue triage

Comments

@jensenbox
Copy link

How do you use Sentry?

Sentry Saas (sentry.io)

Version

1.31.0

Steps to Reproduce

Configure the CloudResourceContextIntegration without any parameters but run in an ECS container on AWS - which is running on EC2 hardware.

My suggestion is to either make the IMDS address configurable or do some detection to see if you are on ECS. There is an IMDS for ECS you could use.

At the very least make the http have a timeout so that things are not totally locked up.

Expected Result

Not freeze

Actual Result

Infinite freeze at https://github.com/getsentry/sentry-python/blob/master/sentry_sdk/integrations/cloud_resource_context.py#L74

@antonpirker
Copy link
Member

Hey @jensenbox !
Thanks for reporting this. I guess I never tested it on ECS. But it seams like a not-so-big fix. (regardless the request should not freece in the first place but just time out at some point...)

@jensenbox
Copy link
Author

While it is awesome that you added a timeout it does not fix the fact that this will not work for ECS tasks that are running on EC2 instances.

This will only allow it to proceed and not actually fix the root issue.

The Metadata address is still wrong. It needs to have a fallback or be configurable. There are environment variables available that tell you the address that can be used rather than hard coding it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Integration: Cloud Resource Context Triaged Has been looked at recently during old issue triage
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

3 participants