ground-android icon indicating copy to clipboard operation
ground-android copied to clipboard

Save config for google chat notifications

Open shobhitagarwal1612 opened this issue 3 years ago • 3 comments
trafficstars

Fixes #1177

Configures a google chat notifier when a build is passes / fails / timeouts on the master branch. https://cloud.google.com/build/docs/configuring-notifications/notifiers

How does it work?

image

Cloud run free tier: image

Tested by removing the "master" filter and triggering a cloud build. A notification was sent to the google chat space after the build was complete. image

@gino-m PTAL?

P.S. The same config can be extended to be used for web repo

shobhitagarwal1612 avatar Aug 07 '22 17:08 shobhitagarwal1612

Codecov Report

Merging #1194 (330aa81) into master (10e0938) will not change coverage. The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #1194   +/-   ##
=========================================
  Coverage     29.64%   29.64%           
  Complexity      783      783           
=========================================
  Files           291      291           
  Lines          7638     7638           
  Branches        346      346           
=========================================
  Hits           2264     2264           
  Misses         5266     5266           
  Partials        108      108           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

codecov-commenter avatar Aug 07 '22 17:08 codecov-commenter

The config looks much less intimidating than I expected! Qq - does this also enable email notifications?

gino-m avatar Aug 08 '22 15:08 gino-m

The config looks much less intimidating than I expected! Qq - does this also enable email notifications?

No, email notifications would require configuring a SMTP server and saving the sender email credentials in the secret manager. https://cloud.google.com/build/docs/configuring-notifications/configure-smtp

This config is the only script that can be checked-in. Everything else was one-time setup using CLI.

shobhitagarwal1612 avatar Aug 09 '22 03:08 shobhitagarwal1612

@gino-m Can we merge this now?

shobhitagarwal1612 avatar Aug 17 '22 16:08 shobhitagarwal1612

@JSunde Can you please approve this?

shobhitagarwal1612 avatar Aug 20 '22 13:08 shobhitagarwal1612