Skip to content

Commit 8f69598

Browse files
authored
Merge pull request #468 from RedHatInsights/CCXDEV-14676
[CCXDEV-14676]Adding option to disable sending alerts to sentry
2 parents af552df + 0d7e0f6 commit 8f69598

2 files changed

Lines changed: 21 additions & 14 deletions

File tree

ccx_messaging/command_line.py

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -77,7 +77,10 @@ def ccx_messaging() -> None:
7777
sys.exit(0)
7878

7979
init_sentry(
80-
os.environ.get("SENTRY_DSN", None), None, os.environ.get("SENTRY_ENVIRONMENT", None)
80+
os.environ.get("SENTRY_DSN", None),
81+
None,
82+
os.environ.get("SENTRY_ENVIRONMENT", None),
83+
os.environ.get("SENTRY_ENABLED", "false").lower() == "true",
8184
)
8285

8386
if args.config:

ccx_messaging/utils/sentry.py

Lines changed: 17 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -28,17 +28,21 @@ def get_event_level():
2828
return logging.ERROR
2929

3030

31-
def init_sentry(dsn=None, transport=None, environment=None):
31+
def init_sentry(dsn=None, transport=None, environment=None, enabled=False):
3232
"""Configure and initialize sentry SDK for this project."""
33-
if dsn:
34-
logging.getLogger(__name__).info("Initializing sentry")
35-
sentry_logging = LoggingIntegration(level=logging.INFO, event_level=get_event_level())
36-
37-
sentry_sdk.init(
38-
dsn=dsn,
39-
ca_certs="/etc/pki/tls/certs/ca-bundle.crt",
40-
integrations=[sentry_logging],
41-
max_breadcrumbs=15,
42-
transport=transport,
43-
environment=environment,
44-
)
33+
if enabled:
34+
if dsn:
35+
logging.getLogger(__name__).info("Initializing sentry")
36+
sentry_logging = LoggingIntegration(level=logging.INFO, event_level=get_event_level())
37+
38+
sentry_sdk.init(
39+
dsn=dsn,
40+
ca_certs="/etc/pki/tls/certs/ca-bundle.crt",
41+
integrations=[sentry_logging],
42+
max_breadcrumbs=15,
43+
transport=transport,
44+
environment=environment,
45+
)
46+
else:
47+
logger = logging.getLogger(__name__)
48+
logger.warning("Configuration Warning: Sentry is enabled, but no DSN was provided.")

0 commit comments

Comments
 (0)