Skip to content

[OTel] Refactor of pymysql implementation#598

Merged
CagriYonca merged 2 commits into
otel_migrationfrom
otel/pymysql
Sep 4, 2024
Merged

[OTel] Refactor of pymysql implementation#598
CagriYonca merged 2 commits into
otel_migrationfrom
otel/pymysql

Conversation

@CagriYonca

Copy link
Copy Markdown
Contributor
  • 100% coverage

@CagriYonca CagriYonca added the OTel_migration Migration the code dependency from OpenTracing to OpenTelemetry label Aug 26, 2024
@CagriYonca CagriYonca added this to the H2-2024 milestone Aug 26, 2024
@CagriYonca CagriYonca requested review from GSVarsha and pvital August 26, 2024 19:27
@CagriYonca CagriYonca self-assigned this Aug 26, 2024
Comment thread src/instana/span/registered_span.py Outdated
Comment thread tests/clients/test_pymysql.py Outdated
Comment thread tests/clients/test_pymysql.py Outdated

@GSVarsha GSVarsha left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Few requests.

@CagriYonca CagriYonca requested a review from GSVarsha August 30, 2024 08:18
@CagriYonca CagriYonca changed the title [OTel] pymysql implementation [OTel] Refactor of pymysql implementation Aug 30, 2024
@CagriYonca CagriYonca force-pushed the otel/pymysql branch 2 times, most recently from 226fa8c to 81c1863 Compare September 2, 2024 15:47

@pvital pvital left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

A couple of change requests.

Comment thread src/instana/instrumentation/pymysql.py Outdated
Comment thread tests/clients/test_pymysql.py Outdated
@pvital

pvital commented Sep 2, 2024

Copy link
Copy Markdown
Member

One more thing, are you not forgetting to remove the following line from the tests/conftest.pyfile?

collect_ignore_glob.append("*clients/test_pym*")

Signed-off-by: Cagri Yonca <cagriyonca@gmail.com>
Signed-off-by: Cagri Yonca <cagriyonca@gmail.com>

@pvital pvital left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

It looks good to me.

@CagriYonca CagriYonca merged commit 0dd9b2d into otel_migration Sep 4, 2024
@CagriYonca CagriYonca deleted the otel/pymysql branch September 4, 2024 09:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

OTel_migration Migration the code dependency from OpenTracing to OpenTelemetry

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants