shahar1 opened a new pull request, #92: URL: https://github.com/apache/airflow-ci-infra/pull/92
Changes the default `alert_email` of `terraform/runs-on-flex` from `[email protected]` to `[email protected]`, so RunsOn cost reports and alerts no longer go to one person. The old code comment explained why a personal address was used, so this replaces it with the constraint that still applies. Before approving the apply on `runs-on-prod`: - **The list must accept SNS mail.** SNS emails come from outside the list, and moderation would hold both the confirmation email and later alerts. PonyMail shows no messages for this list, which does not tell us whether it exists yet or how it is moderated. - **A subscriber has to confirm.** Someone on the list must click the link in the SNS confirmation email. Until then, alerts are not delivered. - **Check the plan.** It should show only the email change and what follows from it. I did not read how the RunsOn module uses `email`, so the plan job on this PR is the first check of that. Tested: `terraform fmt -check` and `prek run --files terraform/runs-on-flex/variables.tf` pass. I did not run `terraform plan` locally. Generated-by: Claude Opus 5.5 🤖 Generated with [Claude Code](https://claude.com/claude-code) https://claude.ai/code/session_01BVx6g8Kg5DwKFE749s5xKG -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
