Skip to content

Send a Google Chat notification when nightly E2E fails - #86

Merged
brionmario merged 1 commit into
thunder-id:mainfrom
janithjay:add-e2e-nightly-chat-notification
Aug 20, 2026
Merged

Send a Google Chat notification when nightly E2E fails#86
brionmario merged 1 commit into
thunder-id:mainfrom
janithjay:add-e2e-nightly-chat-notification

Conversation

@janithjay

@janithjay janithjay commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

Purpose

The nightly E2E workflow (e2e-nightly.yml) runs on a schedule against the latest ThunderID release, but a failure was only ever visible if someone happened to check the Actions tab. Nothing surfaced it proactively.

Approach

Added a Google Chat notification step, gated on if: failure(), mirroring thunder-id/thunderid's own nightly-build-validation.yml pattern (jq-built card payload, curl -f, skips gracefully if the webhook secret isn't configured).

One addition beyond thunderid's version: threading. messageReplyOption=REPLY_MESSAGE_FALLBACK_TO_NEW_THREAD is appended to the webhook URL and a fixed thread.threadKey (javascript-sdks-e2e-nightly) is included in the payload, so every nightly failure replies into the same Google Chat thread instead of posting a new top-level message each time.

Requires a GOOGLE_CHAT_WEBHOOK repository secret (pointing at the same Chat space thunderid notifies) to actually send anything until it's set, the step logs and exits 0 without failing the job.

Related Issues

Related PRs

Checklist

  • Followed the contribution guidelines.
  • Manual test round performed and verified.
  • Documentation provided. (Add links if there are any)
  • Tests provided. (Add links if there are any)
    • Unit Tests
    • Integration Tests
  • Breaking changes. (Fill if applicable)
    • Breaking changes section filled.
    • breaking change label added.

Security checks

  • Followed secure coding standards.
  • Confirmed that this PR doesn't commit any keys, passwords, tokens, usernames, or other secrets.

Copilot AI lite review requested due to automatic review settings August 19, 2026 04:18

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@coderabbitai

coderabbitai Bot commented Aug 19, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: acfb89da-38a0-4c31-9f81-3d247dafecbf


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@janithjay
janithjay force-pushed the add-e2e-nightly-chat-notification branch 4 times, most recently from 45ca9c2 to ebe7368 Compare August 19, 2026 07:13
Mirrors thunder-id/thunderid's nightly-build-validation.yml pattern, with
threading added: every failure carries the same threadKey, so repeated
nightly failures reply into one Google Chat thread instead of each posting
a separate top-level message.

Signed-off-by: janithjay <janithjayashan018@gmail.com>
@janithjay
janithjay force-pushed the add-e2e-nightly-chat-notification branch from ebe7368 to cc12c4e Compare August 20, 2026 08:50
@brionmario
brionmario merged commit 8bc3e4e into thunder-id:main Aug 20, 2026
6 checks passed
@janithjay
janithjay deleted the add-e2e-nightly-chat-notification branch August 20, 2026 09:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants