airbytehq/airbyte

update sentry client in airbyte-ci/connectors/pipelines

Open

#52.676 aberto em 31 de jan. de 2025

Ver no 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.

Guia do colaborador

Pilha de tecnologia
python
Domain
clitoolingdeveloper experience
Tipo Issue
refactor
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 hours
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
fresh
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
clear
Prerequisites
Python programmingBasic understanding of Sentry SDK
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
85
Direção de pesquisa
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.