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

Added container-common for #100 #102

Merged
merged 2 commits into from
Jan 25, 2021

Conversation

PavelKuzub
Copy link
Contributor

Initial release of container-common section that includes setting a limit of container log size any container can have, to prevent filling up UDM storage with excessive logging.

Initial release of container-common section that includes setting a limit of container log size any container can have, to prevent filling up UDM storage with excessive logging.
Copy link
Contributor

@SamErde SamErde left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good addition. Please note in the readme that this will truncate container logs at the specified limit, but will not set a limit application-specific logs.

Clarified description of max log size
@PavelKuzub
Copy link
Contributor Author

That's a good point, Sam. ReadMe has been clarified in an additional commit. Thanks for your review. Cheers

@boostchicken boostchicken merged commit 4162127 into unifi-utilities:master Jan 25, 2021
brandonsoto pushed a commit to brandonsoto/udm-utilities that referenced this pull request Jan 27, 2021
* Added container-common

Initial release of container-common section that includes setting a limit of container log size any container can have, to prevent filling up UDM storage with excessive logging.

* Update README.md

Clarified description of max log size

Co-authored-by: TRUPaC <[email protected]>
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.

4 participants