Skip to content

Commit

Permalink
chore(docs): add missing step around Slack notification setup
Browse files Browse the repository at this point in the history
  • Loading branch information
afeld committed Oct 27, 2022
1 parent 1f8c4d8 commit cc9d710
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 2 deletions.
5 changes: 5 additions & 0 deletions docs/deployment/infrastructure.md
Original file line number Diff line number Diff line change
Expand Up @@ -83,6 +83,8 @@ All resources in these Resource Groups should be reflected in Terraform in this
- Secrets, such as values under [Key Vault](https://azure.microsoft.com/en-us/services/key-vault/) and [App Service application settings](https://docs.microsoft.com/en-us/azure/app-service/configure-common#configure-app-settings). [`prevent_destroy`](https://developer.hashicorp.com/terraform/tutorials/state/resource-lifecycle#prevent-resource-deletion) is used on these Resources.
- [Things managed by DevSecOps](#ownership)

You'll see these represented in Terraform as [data sources](https://developer.hashicorp.com/terraform/language/data-sources).

For browsing the [Azure portal](https://portal.azure.com), you can [switch your `Default subscription filter`](https://docs.microsoft.com/en-us/azure/azure-portal/set-preferences).

## Monitoring
Expand Down Expand Up @@ -180,6 +182,9 @@ lifecycle {
The following steps are required to set up the environment, with linked issues to automate them:

- `terraform apply`
- Set up Slack notifications
1. [Create a Slack email](https://slack.com/help/articles/206819278-Send-emails-to-Slack) for the [#benefits-notify](https://cal-itp.slack.com/archives/C022HHSEE3F) channel
1. [Set it as a Secret in the Key Vault](https://learn.microsoft.com/en-us/azure/key-vault/secrets/quick-create-portal#add-a-secret-to-key-vault) named `slack-benefits-notify-email`
- Set required [App Service configuration](../configuration/environment-variables.md)
- Bind the certificate - [#704](https://github.com/cal-itp/benefits/issues/704)

Expand Down
2 changes: 0 additions & 2 deletions terraform/monitor.tf
Original file line number Diff line number Diff line change
Expand Up @@ -22,8 +22,6 @@ resource "azurerm_application_insights" "main" {
}
}

# created manually
# https://slack.com/help/articles/206819278-Send-emails-to-Slack
data "azurerm_key_vault_secret" "slack_benefits_notify_email" {
name = "slack-benefits-notify-email"
key_vault_id = azurerm_key_vault.main.id
Expand Down

0 comments on commit cc9d710

Please sign in to comment.