airbytehq/airbyte

update sentry client in airbyte-ci/connectors/pipelines

Open

#52,676 opened on Jan 31, 2025

View on GitHub
 (5 comments) (0 reactions) (0 assignees)Python (12,832 stars) (3,358 forks)batch import
area/airbyte-cigood first issueteam/extensibilityteam/triage

Description

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.

Contributor guide

Tech stack
python
Domain
clitoolingdeveloper experience
Issue type
refactor
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
2
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-3 hours
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
fresh
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
clear
Prerequisites
Python programmingBasic understanding of Sentry SDK
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
85
Research direction
The deprecation warning is in `pipelines/helpers/sentry utils.py` at line 41. Update the `sentry sdk` dependency in `pyproject.toml` to a version >=2.0. Then, replace `sentry sdk.configure scope()` with the new API per the Sentry migration guide (e.g., `sentry sdk.isolation scope()`). After updating, run `poetry lock` and tests to verify the warning is gone.
update sentry client in airbyte-ci/connectors/pipelines · airbytehq/airbyte#52676 | Good First Issue