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

(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
难度: 2
预计时间: 1-3 hours
活动状态: fresh
清晰度: clear
前置要求: Python programmingBasic understanding of Sentry SDK
新手友好度: 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.