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

Fix: update python unit test to use a valid lambda arn #1267

Merged
merged 2 commits into from
Apr 22, 2024

Conversation

rapphil
Copy link
Contributor

@rapphil rapphil commented Apr 22, 2024

This has to be done because the lambda instrumentation was updated to extract the account id from the arn present in the lambda context in this PR: open-telemetry/opentelemetry-python-contrib#2367

This made the CI in this dependabot PR to fail.

After this pr is merged, the previous dependabot pr should succeed

This has to be done because the lambda instrumentation was updated to
extract the account id from the arn present in the lambda context
in this PR: open-telemetry/opentelemetry-python-contrib#2367
@rapphil rapphil requested a review from a team April 22, 2024 05:36
@rapphil rapphil changed the title Fix: update unit test to use real lambda arn Fix: update python unit test to use real lambda arn Apr 22, 2024
@rapphil rapphil changed the title Fix: update python unit test to use real lambda arn Fix: update python unit test to use a valid lambda arn Apr 22, 2024
@rapphil rapphil merged commit 629c198 into open-telemetry:main Apr 22, 2024
13 checks passed
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.

2 participants