Skip to content

Conversation

shwejanraj
Copy link
Contributor

Description

Adds NoOpTracerProvider test cases for botocore instrumentation.

Fixes #988

How Has This Been Tested?

tox -e test-instrumentation-botocore

Does This PR Require a Core Repo Change?

No.

@shwejanraj shwejanraj changed the title Adds NoOpTracer test case for botocore instrumentation Adds NoOpTracerProvider test case for botocore instrumentation Jun 7, 2024
Copy link
Member

@emdneto emdneto left a comment

Choose a reason for hiding this comment

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

In #2605 we updated moto package. You'll need to fix conflicts and update your tests accordingly

@brianwarner brianwarner force-pushed the 988-Adds-NoOpTracerProvider-test-cases-for-botocore-instrumentation branch from be1e3ca to b2b1d22 Compare June 21, 2024 12:00
@brianwarner brianwarner requested a review from a team June 21, 2024 12:00
@shwejanraj
Copy link
Contributor Author

@emdneto
I have updated the test cases accordingly. Could you please take a moment to review it.

@shwejanraj shwejanraj force-pushed the 988-Adds-NoOpTracerProvider-test-cases-for-botocore-instrumentation branch 2 times, most recently from 4e4ffa0 to 8612ef7 Compare June 22, 2024 06:54
@shwejanraj shwejanraj requested a review from emdneto June 22, 2024 06:54
@shwejanraj shwejanraj force-pushed the 988-Adds-NoOpTracerProvider-test-cases-for-botocore-instrumentation branch from 8612ef7 to 82a64da Compare June 24, 2024 07:22
@brianwarner brianwarner force-pushed the 988-Adds-NoOpTracerProvider-test-cases-for-botocore-instrumentation branch from 82a64da to bae1df1 Compare July 9, 2024 15:15
Copy link
Contributor

@lzchen lzchen left a comment

Choose a reason for hiding this comment

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

Please rebase and then allow maintainer edits so we can rebase for you.

@lzchen
Copy link
Contributor

lzchen commented Jul 22, 2024

@shwejanraj

Are you still working on this?

@Sunilwali679
Copy link

@shwejanraj

Are you still working on this?

@lzchen . I am working on this and @shwejanraj was working with me on this issue . I will rebase and update the PR . Thanks

@Sunilwali679
Copy link

@lzchen #2846 . created for the same

@xrmx xrmx closed this Sep 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Audit and test opentelemetry-instrumentation-botocore

5 participants