Skip to content

Commit

Permalink
Merge pull request #5768 from drew2a/sentry_fixes_for_release
Browse files Browse the repository at this point in the history
Sentry fixes for release
  • Loading branch information
drew2a committed Dec 2, 2020
2 parents e8c3e1c + a6793c5 commit 3f63a93
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 3 deletions.
6 changes: 4 additions & 2 deletions build/update_version_from_git.py
Original file line number Diff line number Diff line change
Expand Up @@ -31,9 +31,11 @@ def run_command(cmd):

sentry_url = os.environ.get('SENTRY_URL', None)
logger.info(f'Sentry url: {sentry_url}')
if not sentry_url:
if sentry_url is None:
logger.critical('Sentry url is not defined. To define sentry url use:'
'EXPORT SENTRY_URL=<sentry_url>')
'EXPORT SENTRY_URL=<sentry_url>\n'
'If you want to disable sentry, then define the following:'
'EXPORT SENTRY_URL=')
sys.exit(1)

build_date = ctime()
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -94,7 +94,7 @@ def scrub_event(self, event):
integrations=[
LoggingIntegration(
level=logging.INFO, # Capture info and above as breadcrumbs
event_level=logging.ERROR, # Send errors as events
event_level=None, # Send no errors as events
),
ThreadingIntegration(propagate_hub=True),
],
Expand Down

0 comments on commit 3f63a93

Please sign in to comment.