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

removed conditional and modified RetentionInDays values to 30 #265

Merged
merged 3 commits into from
May 29, 2024

Conversation

kelvinosarenkhoe
Copy link
Contributor

@kelvinosarenkhoe kelvinosarenkhoe commented May 22, 2024

Increase log group's RetentionInDays to 30. Currently any log retention that is set under 30 days will be set to 30 by a Lambda attached to the LZA Lambda run - INCIDEN-627

What changed
RetentionInDays: changed RetentionInDays: 30

Check ticket https://govukverify.atlassian.net/jira/software/c/projects/PSREC/boards/555?selectedIssue=PSREC-243 and associated epic https://govukverify.atlassian.net/browse/PSREC-314

Why did it change
LZA lambda unpicks log retentions lower than 30 days.

Issue tracking
INCIDEN-627

Environment variables or secrets
[ ] No environment variables or secrets were added or changed

[ ] Documented in the https://github.com/govuk-one-login/ipv-cri-check-hmrc-api/blob/main/infrastructure/README.md
[ ] Added to deployment steps
[ ] Added to local startup config

Other considerations
[ ] Update [https://github.com/govuk-one-login/ipv-cri-check-hmrc-api/blob/main/infrastructure/README.md] with any new instructions or tasks

@kelvinosarenkhoe kelvinosarenkhoe requested review from a team as code owners May 22, 2024 11:21
Copy link

sonarcloud bot commented May 22, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@merlinc merlinc merged commit 64dec9e into main May 29, 2024
15 checks passed
@merlinc merlinc deleted the log-group-retention-cfn-PSREC-243 branch May 29, 2024 13:21
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.

3 participants