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

Automated cherry pick of #2522: Log maintenance bug fix in the Flow Aggregator #2524

Conversation

srikartati
Copy link
Member

Cherry pick of #2522 on release-1.2.

#2522: Log maintenance bug fix in the Flow Aggregator

For details on the cherry pick process, see the cherry pick requests page.

Periodic deletion of log files is missing in the Flow Aggreegator.
This logic periodically deletes extra files by making sure that max
number of files to be 4. This will ensure that no disk space is wasted
on the Kubernetes node running the Flow Aggregator.

This logic was missed initially when the logging in the Flow Aggregator is
added in the beginning adopting the same framework as the Antrea Agent and the
Antrea Controller.

Signed-off-by: Srikar Tati <stati@vmware.com>
@srikartati srikartati added the kind/cherry-pick Categorizes issue or PR as related to the cherry-pick of a bug fix from the main branch to a release label Aug 2, 2021
@codecov-commenter
Copy link

codecov-commenter commented Aug 2, 2021

Codecov Report

Merging #2524 (7d40189) into release-1.2 (36d5d6c) will decrease coverage by 0.04%.
The diff coverage is n/a.

Impacted file tree graph

@@               Coverage Diff               @@
##           release-1.2    #2524      +/-   ##
===============================================
- Coverage        59.86%   59.82%   -0.05%     
===============================================
  Files              284      284              
  Lines            22235    22235              
===============================================
- Hits             13312    13303       -9     
- Misses            7508     7512       +4     
- Partials          1415     1420       +5     
Flag Coverage Δ
kind-e2e-tests 46.88% <ø> (-0.05%) ⬇️
unit-tests 42.22% <ø> (-0.03%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
pkg/apiserver/certificate/certificate.go 69.86% <0.00%> (-6.85%) ⬇️
pkg/agent/flowexporter/connections/connections.go 75.00% <0.00%> (-6.82%) ⬇️
pkg/agent/metrics/prometheus.go 25.71% <0.00%> (-5.72%) ⬇️
...gent/controller/networkpolicy/status_controller.go 72.60% <0.00%> (-2.74%) ⬇️
pkg/agent/flowexporter/exporter/exporter.go 80.00% <0.00%> (-0.94%) ⬇️
pkg/agent/openflow/network_policy.go 75.92% <0.00%> (-0.15%) ⬇️
pkg/agent/cniserver/server.go 69.87% <0.00%> (+0.96%) ⬆️
pkg/controller/networkpolicy/status_controller.go 72.90% <0.00%> (+1.93%) ⬆️
pkg/log/log_file.go 71.05% <0.00%> (+2.63%) ⬆️

@antoninbas
Copy link
Contributor

/skip-all

@antoninbas antoninbas merged commit 91a5831 into antrea-io:release-1.2 Aug 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/cherry-pick Categorizes issue or PR as related to the cherry-pick of a bug fix from the main branch to a release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants