search

google / exposure-notifications-server

Exposure Notification Reference Server | Covid-19 Exposure Notifications
https://www.google.com/covid19/exposurenotifications/
Apache License 2.0
2.46k stars 312 forks source link

Set all cron jobs to retry 3 times #1374

Closed mariliamelo closed 3 years ago

mariliamelo commented 3 years ago

Standardize all cloud scheduler jobs to retry 3 times.

google-oss-robot commented 3 years ago

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: mariliamelo, sethvargo

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files: - ~~[OWNERS](https://github.com/google/exposure-notifications-server/blob/main/OWNERS)~~ [mariliamelo,sethvargo] Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment
mikehelmick commented 3 years ago

What's the retry strategy that cloud scheduler uses?

sethvargo commented 3 years ago

The default is immediately after failure. We can customize it if we want: https://registry.terraform.io/providers/hashicorp/google/latest/docs/resources/cloud_scheduler_job#retry_count