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

update sentry client in airbyte-ci/connectors/pipelines #52676

Open
natikgadzhi opened this issue Jan 31, 2025 · 2 comments · May be fixed by #53398
Open

update sentry client in airbyte-ci/connectors/pipelines #52676

natikgadzhi opened this issue Jan 31, 2025 · 2 comments · May be fixed by #53398

Comments

@natikgadzhi
Copy link
Contributor

Problem

We have this warning show up in build logs whenever we build airbyte-ci/pipelines for testing:

  /airbyte/airbyte-ci/connectors/pipelines/pipelines/helpers/sentry_utils.py:41: DeprecationWarning: sentry_sdk.configure_scope is deprecated and will be removed in the next major version. Please consult our migration guide to learn how to migrate to the new API: https://docs.sentry.io/platforms/python/migration/1.x-to-2.x#scope-configuring
    with sentry_sdk.configure_scope() as scope:

To reproduce this, you can fetch the repo and then:

cd airbyte-ci/connectors/pipelines
poetry install 
poetry run pytest .

Solution

Upgrade poetry client to the recent version and clean up the code that sets sentry up. This should be a good first issue.

@manojks1999
Copy link

@natikgadzhi can I work on this ?

saimadib added a commit to saimadib/airbyte that referenced this issue Feb 8, 2025
@saimadib
Copy link

saimadib commented Feb 8, 2025

PR #53398
CC: @natikgadzhi

saimadib added a commit to saimadib/airbyte that referenced this issue Feb 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants