Skip to content

Commit

Permalink
Fix for CI failure
Browse files Browse the repository at this point in the history
  • Loading branch information
jim-sheldon committed May 11, 2022
1 parent e863141 commit 38a06dd
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions ingestion/monitoring/daily_metrics.py
Original file line number Diff line number Diff line change
Expand Up @@ -91,8 +91,6 @@ def send_slack_message(slack_message: str) -> None:
if response.status_code != 200:
logger.error(f"Slack notification failed with {response.status_code}: {response.text}")
sys.exit(1)
if "⚠️" in ret:
sys.exit(1) # Trigger CI failure as an additional notification


if __name__ == "__main__":
Expand All @@ -119,3 +117,6 @@ def send_slack_message(slack_message: str) -> None:

logger.info(slack_message)
send_slack_message(slack_message)

if "⚠️" in slack_message:
sys.exit(1) # Trigger CI failure as an additional notification

0 comments on commit 38a06dd

Please sign in to comment.