Skip to content

[release-1.16] feat: add gzip to container image for logrotate compression - #7239

Merged
zhangzujian merged 1 commit into
kubeovn:release-1.16from
zhangzujian:backport-1.16-gzip-logrotate
Aug 15, 2026
Merged

[release-1.16] feat: add gzip to container image for logrotate compression#7239
zhangzujian merged 1 commit into
kubeovn:release-1.16from
zhangzujian:backport-1.16-gzip-logrotate

Conversation

@zhangzujian

Copy link
Copy Markdown
Member

Pull Request

  • Make sure you have followed Kube-OVN Code Style.

What type of this PR

  • Bug fixes

What this PR does

Backport merged PR #7217 to release-1.16.

Explicitly install gzip in both the standard and DPDK runtime base images so the existing logrotate configurations can honor their compress directive.

Testing

  • Backport stable patch ID matches source PR head commit 615893ac27f1572cdcd85ff38d53eeae73a3b853
  • Verified that only dist/images/Dockerfile.base and dist/images/Dockerfile.base-dpdk change
  • Verified that both runtime package lists contain one explicit gzip logrotate declaration
  • Verified that all existing dist/images/logrotate/* configurations enable compress
  • git diff --check origin/release-1.16...HEAD

Full-repository make lint CI=true was run under the required 2 GiB local memory limit. CRD generation and verification passed; the Go lint stage could not complete within the local resource limit. This backport changes only runtime package lists, and GitHub Actions will run the branch checks.

Which issue(s) this PR fixes

N/A

Backport of #7217.

Signed-off-by: Max Weiss <maxweiss@microsoft.com>
(cherry picked from commit 615893a)
@dosubot dosubot Bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Aug 15, 2026
@coveralls

Copy link
Copy Markdown

Coverage Report for CI Build 31857809789

Coverage decreased (-0.004%) to 30.385%

Details

  • Coverage decreased (-0.004%) from the base build.
  • Patch coverage: No coverable lines changed in this PR.
  • 2 coverage regressions across 1 file.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

2 previously-covered lines in 1 file lost coverage.

File Lines Losing Coverage Coverage
pkg/bfddsupervisor/socket.go 2 51.58%

Coverage Stats

Coverage Status
Relevant Lines: 60858
Covered Lines: 18492
Line Coverage: 30.39%
Coverage Strength: 0.35 hits per line

💛 - Coveralls

@zhangzujian
zhangzujian merged commit d756e11 into kubeovn:release-1.16 Aug 15, 2026
73 of 79 checks passed
@zhangzujian
zhangzujian deleted the backport-1.16-gzip-logrotate branch August 15, 2026 04:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants