Skip to content

Commit

Permalink
fix: update to README file, adjust iterator age alarm threshold and f…
Browse files Browse the repository at this point in the history
…ix to vault data integrity check local lambda test script (#525)
  • Loading branch information
craigzour authored Nov 1, 2023
1 parent 4ff5b4d commit 0761ad0
Show file tree
Hide file tree
Showing 3 changed files with 15 additions and 9 deletions.
14 changes: 10 additions & 4 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,13 @@ Pre-requisites:
```bash
/bin/bash -c "$(curl -fsSL https://raw.githubusercontent.com/Homebrew/install/HEAD/install.sh)"
```
- LocalStack : `brew install localstack`

- LocalStack:

1. `brew install localstack`

Please note that the latest tested version of Localstack with your infrastructure code was `2.3.2`.

- Terragrunt:

1. `brew install warrensbox/tap/tfswitch`
Expand All @@ -32,10 +38,10 @@ Pre-requisites:

- AWS SAM CLI

Please run these commands to install the aws sam cli using homebrew. The AWS SAM CLI is what we use to run the lambda functions locally and invoke them.
1. If you previously had it installed with Brew then you should first uninstall this package using `brew uninstall aws-sam-cli`
1. To install the AWS SAM CLI tool, follow the instructions on https://docs.aws.amazon.com/serverless-application-model/latest/developerguide/install-sam-cli.html under "macOS" and "Command line - All users" (you will have to download the .pkg file and run a command to install it on your machine)

1. `brew tap aws/tap`
1. `brew install aws-sam-cli`
Please note that the latest tested version of AWS SAM CLI with your infrastructure code was `1.99.0`.

- Postgres and PGAdmin
1. `brew install postgresql`
Expand Down
4 changes: 2 additions & 2 deletions aws/alarms/cloudwatch.tf
Original file line number Diff line number Diff line change
Expand Up @@ -433,11 +433,11 @@ resource "aws_cloudwatch_metric_alarm" "vault_data_integrity_check_lambda_iterat
metric_name = "IteratorAge"
statistic = "Average"
comparison_operator = "GreaterThanThreshold"
threshold = "30000"
threshold = "90000" // 90 seconds (Lambda consumes events every 60 seconds if low traffic. Adding 30 seconds on top of that for the processing time of that Lambda)
period = "60"
evaluation_periods = "2"

alarm_description = "The Vault data integrity check lambda function is unable to keep up with the amount of events sent by the Vault DynamoDB stream"
alarm_description = "Warning - Vault data integrity check lambda is unable to keep up with the amount of events sent by the Vault DynamoDB stream"
alarm_actions = [var.sns_topic_alert_warning_arn]
ok_actions = [var.sns_topic_alert_ok_arn]

Expand Down
6 changes: 3 additions & 3 deletions aws/app/lambda/invoke_test_vault_data_integrity_check.sh
Original file line number Diff line number Diff line change
Expand Up @@ -346,12 +346,12 @@ echo '{
"FormID": {
"S": "clmqej5xf0002mi8krkfmim02"
},
"NAME_OR_CONF": {
"S": "NAME#19-09-0043"
},
"ConfirmationCode": {
"S": "fb303d9e-3afd-4ef3-bfd8-fc4596f70223"
},
"CreatedAt": {
"N": "1695139169449"
},
"FormSubmission": {
"S": "{\"1\":\"\"}"
},
Expand Down

0 comments on commit 0761ad0

Please sign in to comment.