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.

Sign up for GitHub

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

Jump to bottom

Update logrotate config #4236

Merged
merged 4 commits into from
Jul 27, 2022
Merged

Update logrotate config #4236

merged 4 commits into from
Jul 27, 2022

Conversation

cthulhuplus
Copy link
Contributor

@cthulhuplus cthulhuplus commented Jul 27, 2022 •
edited
Loading

Resolves # 4168

Description

Updating logrotate configuration to prevent logs filling up.

Significant changes or possible side effects

This is an update to the AMI, could potentially break application/instance

Steps to manually verify this change

  1. Log into instance, check logrotate config
  2. Check application is still working

This pull request is ready to review when

  • Automated tests are updated (and all tests are passing)
  • Pull request has been labeled, if applicable with feature, content, bug, tests, refactor
  • Associated OpenAPI documentation has been updated
  • The experience passes a basic manual accessibility audit (keyboard nav, screenreader, text scaling) OR an exemption is documented

This pull request can be merged when

  • Code has been reviewed by someone other than the original author
  • QA has verified the accessibility and functionality related to the change
  • Design has approved the experience
  • Product has approved the experience

@cthulhuplus cthulhuplus requested review from tbolt and thetif and removed request for tbolt July 27, 2022 16:07
@cthulhuplus cthulhuplus marked this pull request as ready for review July 27, 2022 16:07
@cms-eapd-bot
Copy link

See this pull request in action: https://ec2-52-2-79-95.compute-1.amazonaws.com

@cthulhuplus cthulhuplus merged commit b6ea9e9 into main Jul 27, 2022
@cthulhuplus cthulhuplus deleted the bbrooks/4168-Logrotate-Update branch July 27, 2022 16:58
@jeromeleecms jeromeleecms mentioned this pull request Jul 27, 2022
Closed
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@thetif thetif thetif approved these changes

@tbolt tbolt Awaiting requested review from tbolt

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@cthulhuplus @cms-eapd-bot @thetif