Skip to content

Commit

Permalink
Skip uploading metrics on error setting up lambda notification
Browse files Browse the repository at this point in the history
Signed-off-by: Davanum Srinivas <[email protected]>
  • Loading branch information
dims committed Sep 16, 2023
1 parent eac3134 commit 2e1b9db
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion hack/e2e/metrics/metrics.sh
Original file line number Diff line number Diff line change
Expand Up @@ -198,7 +198,7 @@ upload_metrics() {
aws s3api put-bucket-notification-configuration \
--bucket "$SOURCE_BUCKET" \
--notification-configuration "$notification_config" \
--region "$AWS_REGION"
--region "$AWS_REGION" || (echo -e "\nERROR: Skipping uploading metrics ...\n" && return)

log "Uploading metrics to S3 bucket $SOURCE_BUCKET"
aws s3 sync "$METRICS_DIR_PATH" "s3://$SOURCE_BUCKET/$METRICS_DIR_NAME" --region "$AWS_REGION"
Expand Down

0 comments on commit 2e1b9db

Please sign in to comment.