FAQ | This is a LIVE service | Changelog

Skip to content
Snippets Groups Projects

make use of our common monitoring module

Merged Dr Rich Wareham requested to merge issue-13-split-out-monitoring into master

Rather than ship our own monitoring module, make use of the gcp-site-monitoring module. This effectively makes this module require terraform 0.13 so a major version bump is required. Once this is merged, "release" by opening a v2 branch and having the previous master be v1 as outlined in the README.

This will add monitoring for the auto-generated Cloud Run domain as well as any domain mappings. Allow monitoring to be disabled if, for example, the service is configured to disallow direct invocations.

Tested by upgrading the tlscerts deployment to terraform 0.13 (https://gitlab.developers.cam.ac.uk/uis/devops/tls-certificates/google-cloud-deploy/-/merge_requests/16) and deploying development:

Screenshot_2021-01-26_Uptime_checks___Monitoring___TLS_Certificates_me____Google_Cloud_Platform_1_

Closes #13 (closed)

Edited by Dr Rich Wareham

Merge request reports

Pipeline #59082 passed

Pipeline passed for 1ff24a86 on issue-13-split-out-monitoring

Approval is optional

Merged by Dave HartDave Hart 4 years ago (Jan 27, 2021 4:27pm UTC)

Merge details

Pipeline #59148 passed

Pipeline passed for 3fdd4024 on master

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • added 1 commit

    • 8cb6c1c5 - make use of our common monitoring module

    Compare with previous version

  • Dr Rich Wareham added 5 commits

    added 5 commits

    Compare with previous version

  • Rebased on current master to fix merge conflicts.

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading