airbytehq/airbyte

update sentry client in airbyte-ci/connectors/pipelines

Open

#52,676 建立於 2025年1月31日

在 GitHub 查看
 (5 留言) (0 反應) (0 負責人)Python (12,832 star) (3,358 fork)batch import
area/airbyte-cigood first issueteam/extensibilityteam/triage

描述

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.

貢獻者指南

技術棧
python
領域
clitoolingdeveloper experience
議題類型
refactor
難度面向新貢獻者的預計實作難度，1 表示很小改動，5 表示專家級工作。
2
預計時間有經驗貢獻者完成調查、實作、測試並準備 pull request 的粗略時間範圍。
1-3 hours
活動狀態議題目前的可參與程度：新鮮、活躍、陳舊、阻塞或等待維護者輸入。
fresh
清晰度議題是否清楚說明預期改動、驗收標準和下一步。
clear
前置要求
Python programmingBasic understanding of Sentry SDK
新手友善度1-100 的估計分數，表示該議題對首次貢獻者的友善程度。
85
研究方向
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.