Skip to content

Add proxy to log rotation and update syndic rotation for SUSE - #57354

Merged
dwoz merged 6 commits into
saltstack:masterfrom
dmurphy18:proxy_log
May 22, 2020
Merged

Add proxy to log rotation and update syndic rotation for SUSE#57354
dwoz merged 6 commits into
saltstack:masterfrom
dmurphy18:proxy_log

Conversation

@dmurphy18

Copy link
Copy Markdown
Contributor

What does this PR do?

Adds proxy logs to log rotation

What issues does this PR fix or reference?

Fixes: #57353

Previous Behavior

Proxy logs could fill available disk space

New Behavior

Proxy logs are now part of log rotation

Merge requirements satisfied?

[NOTICE] Bug fixes or features added to Salt require tests.

Commits signed with GPG?

Yes

Please review Salt's Contributing Guide for best practices.

See GitHub's page on GPG signing for more information about signing commits with GPG.

@dmurphy18
dmurphy18 requested a review from a team as a code owner May 18, 2020 14:21
@ghost
ghost requested review from DmitryKuzmenko and removed request for a team May 18, 2020 14:21
@dmurphy18
dmurphy18 requested review from Akm0d and s0undt3ch May 18, 2020 14:21
@dmurphy18

Copy link
Copy Markdown
Contributor Author

Needs 3 reviewers

s0undt3ch
s0undt3ch previously approved these changes May 18, 2020
Comment thread pkg/suse/salt-common.logrotate Outdated
@sagetherage sagetherage added the ZRelease-Sodium retired label label May 18, 2020
@dwoz dwoz added the needs-testcase PR needs test cases written, or the issue is about a bug/feature that needs test cases label May 18, 2020
Akm0d
Akm0d previously approved these changes May 18, 2020
@dmurphy18
dmurphy18 dismissed stale reviews from Akm0d and s0undt3ch via 852e94a May 19, 2020 15:31
@dmurphy18

Copy link
Copy Markdown
Contributor Author

@dwoz this is a simple fix and needs 3 reviewers, mocking a log rotation does not compute.
Adding previously altered log rotation for syndic to SUSE which was missed at the time.

@dwoz dwoz removed the needs-testcase PR needs test cases written, or the issue is about a bug/feature that needs test cases label May 22, 2020
@dwoz
dwoz merged commit 0ab763c into saltstack:master May 22, 2020
@dmurphy18
dmurphy18 deleted the proxy_log branch August 4, 2020 17:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ZRelease-Sodium retired label

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] Proxy logs are not rotated

6 participants