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

Add StatusCake for the test environment #2979

Merged
merged 6 commits into from
Jul 10, 2024

Conversation

vacabor
Copy link
Contributor

@vacabor vacabor commented Jul 8, 2024

Enables the StatusCake module and uses the existing contact groups for alerting.

@vacabor vacabor self-assigned this Jul 8, 2024
Copy link
Contributor

@AbigailMcP AbigailMcP left a comment

Choose a reason for hiding this comment

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

Do we need to set the external_url variable in test.tfvars.json to be our healthcheck url?

@vacabor
Copy link
Contributor Author

vacabor commented Jul 9, 2024

Do we need to set the external_url variable in test.tfvars.json to be our healthcheck url?

statuscake.tf takes care of setting a check on the healthcheck URL by getting it from the application module, but I've also set external_url so we can have an extra SSL check on /additional-payments/claim

@vacabor
Copy link
Contributor Author

vacabor commented Jul 9, 2024

Do we need to set the external_url variable in test.tfvars.json to be our healthcheck url?

statuscake.tf takes care of setting a check on the healthcheck URL by getting it from the application module, but I've also set external_url so we can have an extra SSL check on /additional-payments/claim

I've reverted this - it's something we can set on production, but for test the endpoint is protected by HTTP basic (so the StatusCake check always fails).

@vacabor vacabor requested review from slorek and saliceti July 9, 2024 15:39
@vacabor vacabor merged commit b0422a4 into master Jul 10, 2024
16 checks passed
@vacabor vacabor deleted the CAPT-1726-add-statuscake-to-test branch July 10, 2024 11:16
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