Add Cloud Run runtime guard alerts#118
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 202c18cb46
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| message_lines.append(f"Workflow: {run_url}") | ||
| message = "\n".join(message_lines) | ||
| print(message) | ||
| _send_telegram(message[:3900]) |
There was a problem hiding this comment.
Handle failed Telegram sends before exiting cleanly
When Telegram credentials are missing or the POST fails, _send_telegram() returns False, but this call ignores that result. In environments using the supported RUNTIME_GUARD_FAIL_WORKFLOW_ON_ALERT=false input/variable, the guard exits 0 even though no Telegram alert was delivered, so Cloud Run/Scheduler failures can be completely unreported except in the workflow log; please make send failures affect the exit status or otherwise surface them reliably.
Useful? React with 👍 / 👎.
Summary
Validation