Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Deploy multiple SSL Checks from using StatusCake #184

Merged
merged 1 commit into from
Oct 18, 2023

Conversation

DrizzlyOwl
Copy link
Collaborator

  • The StatusCake module requires us to specify the minimum terraform of 1.6.1
  • Updating to version 0.1.2 of StatusCake module allows us to associate multiple SSL checks with a single contact group, rather than having a 1:1 relationship.
  • Setting the API token to 00000000000000000000000000000 means that we can skip the validation that requires us to specify this. Essentially, this makes the StatusCake module conditionally deployable.
  • Specifying statuscake_contact_group_email_addresses instead of using the email addresses used for infrastructure monitoring ensures that the list doesnt change per environment. e.g. In Production, there are more email receivers for infra monitoring compared to Dev. Whereas we want everyone to be notified about TLS certs expiry.

@DrizzlyOwl DrizzlyOwl temporarily deployed to development October 17, 2023 16:29 — with GitHub Actions Inactive
@DrizzlyOwl DrizzlyOwl temporarily deployed to development October 17, 2023 16:32 — with GitHub Actions Inactive
Stretch96
Stretch96 previously approved these changes Oct 17, 2023
@DrizzlyOwl DrizzlyOwl force-pushed the tf-statuscake-tls-monitor branch from 110f734 to 0759257 Compare October 17, 2023 16:48
@DrizzlyOwl DrizzlyOwl temporarily deployed to development October 17, 2023 16:49 — with GitHub Actions Inactive
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@DrizzlyOwl DrizzlyOwl temporarily deployed to development October 17, 2023 16:51 — with GitHub Actions Inactive
@DrizzlyOwl DrizzlyOwl requested a review from Stretch96 October 18, 2023 08:42
@DrizzlyOwl DrizzlyOwl merged commit 3e94c67 into main Oct 18, 2023
9 checks passed
@DrizzlyOwl DrizzlyOwl deleted the tf-statuscake-tls-monitor branch October 18, 2023 08:49
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.

2 participants