Skip to content

Commit

Permalink
Merge pull request #1261 from DFE-Digital/enable_monitoring
Browse files Browse the repository at this point in the history
Enable monitoring
  • Loading branch information
vipin-dfe authored Dec 2, 2024
2 parents eb163ea + 3460a51 commit b59932f
Show file tree
Hide file tree
Showing 5 changed files with 12 additions and 7 deletions.
5 changes: 5 additions & 0 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -242,6 +242,11 @@ deploy-arm-resources: arm-deployment ## Validate ARM resource deployment. Usage:

validate-arm-resources: set-what-if arm-deployment ## Validate ARM resource deployment. Usage: make domains validate-arm-resources

action-group-resources: set-azure-account # make env action-group-resources [email protected] . Must be run before setting enable_monitoring=true for each subscription
$(if $(ACTION_GROUP_EMAIL), , $(error Please specify a notification email for the action group))
az group create -l uksouth -g ${AZURE_RESOURCE_PREFIX}-${SERVICE_SHORT}-mn-rg --tags "Product=Refer Serious Misconduct"
az monitor action-group create -n ${AZURE_RESOURCE_PREFIX}-${SERVICE_NAME} -g ${AZURE_RESOURCE_PREFIX}-${SERVICE_SHORT}-mn-rg --action email ${AZURE_RESOURCE_PREFIX}-${SERVICE_SHORT}-email ${ACTION_GROUP_EMAIL}

aks-terraform-init: composed-variables bin/terrafile aks-set-azure-account
$(if ${DOCKER_IMAGE_TAG}, , $(eval DOCKER_IMAGE_TAG=main))

Expand Down
3 changes: 2 additions & 1 deletion terraform/application/config/production.tfvars.json
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,8 @@
"account_replication_type": "GRS",
"cluster": "production",
"enable_logit": true,
"enable_monitoring": false,
"enable_monitoring": true,
"statuscake_contact_groups": [288912, 282453],
"namespace": "tra-production",
"allegations_container_delete_retention_days": 30
}
1 change: 1 addition & 0 deletions terraform/application/config/production.yml
Original file line number Diff line number Diff line change
@@ -1 +1,2 @@
---
EXTERNAL_DOMAIN: refer-serious-misconduct.education.gov.uk
6 changes: 2 additions & 4 deletions terraform/application/statuscake.tf
Original file line number Diff line number Diff line change
@@ -1,12 +1,10 @@
/*
module "statuscake" {
count = var.enable_monitoring ? 1 : 0

source = "./vendor/modules/aks//monitoring/statuscake"

uptime_urls = compact([module.web_application.probe_url, var.external_url])
ssl_urls = compact([var.external_url])
uptime_urls = compact([module.web_application.probe_url, "https://${local.external_domain}/health"])
ssl_urls = compact(["https://${local.external_domain}/"])

contact_groups = var.statuscake_contact_groups
}
*/
4 changes: 2 additions & 2 deletions terraform/application/terraform.tf
Original file line number Diff line number Diff line change
Expand Up @@ -41,6 +41,6 @@ provider "kubernetes" {
}
}

/*provider "statuscake" {
provider "statuscake" {
api_token = module.infrastructure_secrets.map.STATUSCAKE-API-TOKEN
}*/
}

0 comments on commit b59932f

Please sign in to comment.