Skip to content

Commit

Permalink
Refactor sentry and add tests
Browse files Browse the repository at this point in the history
  • Loading branch information
Lipu Fei committed Mar 18, 2024
1 parent 3e174f1 commit 6655f04
Show file tree
Hide file tree
Showing 5 changed files with 307 additions and 198 deletions.
198 changes: 0 additions & 198 deletions airflow/sentry.py

This file was deleted.

29 changes: 29 additions & 0 deletions airflow/sentry/__init__.py
Original file line number Diff line number Diff line change
@@ -0,0 +1,29 @@
#
# Licensed to the Apache Software Foundation (ASF) under one
# or more contributor license agreements. See the NOTICE file
# distributed with this work for additional information
# regarding copyright ownership. The ASF licenses this file
# to you under the Apache License, Version 2.0 (the
# "License"); you may not use this file except in compliance
# with the License. You may obtain a copy of the License at
#
# http://www.apache.org/licenses/LICENSE-2.0
#
# Unless required by applicable law or agreed to in writing,
# software distributed under the License is distributed on an
# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
# KIND, either express or implied. See the License for the
# specific language governing permissions and limitations
# under the License.
"""Sentry Integration."""
from __future__ import annotations

from airflow.configuration import conf
from airflow.sentry.blank import BlankSentry

Sentry: BlankSentry = BlankSentry()
d = conf.getboolean("sentry", "sentry_on", fallback=False)
if conf.getboolean("sentry", "sentry_on", fallback=False):
from airflow.sentry.configured import ConfiguredSentry

Sentry = ConfiguredSentry()
40 changes: 40 additions & 0 deletions airflow/sentry/blank.py
Original file line number Diff line number Diff line change
@@ -0,0 +1,40 @@
#
# Licensed to the Apache Software Foundation (ASF) under one
# or more contributor license agreements. See the NOTICE file
# distributed with this work for additional information
# regarding copyright ownership. The ASF licenses this file
# to you under the Apache License, Version 2.0 (the
# "License"); you may not use this file except in compliance
# with the License. You may obtain a copy of the License at
#
# http://www.apache.org/licenses/LICENSE-2.0
#
# Unless required by applicable law or agreed to in writing,
# software distributed under the License is distributed on an
# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
# KIND, either express or implied. See the License for the
# specific language governing permissions and limitations
# under the License.
from __future__ import annotations

from typing import TYPE_CHECKING

if TYPE_CHECKING:
from sqlalchemy.orm import Session


class BlankSentry:
"""Blank class for Sentry."""

def add_tagging(self, task_instance):
"""Blank function for tagging."""

def add_breadcrumbs(self, task_instance, session: Session | None = None):
"""Blank function for breadcrumbs."""

def enrich_errors(self, run):
"""Blank function for formatting a TaskInstance._run_raw_task."""
return run

def flush(self):
"""Blank function for flushing errors."""

0 comments on commit 6655f04

Please sign in to comment.