feat(Dockerfile): modify base to use ubi-micro to reduce attack vectors - #3439
feat(Dockerfile): modify base to use ubi-micro to reduce attack vectors#3439jcantrill wants to merge 1 commit into
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Enterprise Run ID: 📒 Files selected for processing (2)
Included review availability: Your plan provides up to 12 included reviews per hour; 11 remain after this review. 📝 WalkthroughWalkthroughThe Dockerfiles update Go builder images to version 1.26. Runtime images switch from UBI Minimal with direct ChangesContainer image refresh
Estimated code review effort: 2 (Simple) | ~15 minutes Merge Risk: ⚪ Minimal · up to The PR changes the final container base image to UBI Micro to reduce unnecessary CVE exposure, and no actionable merge-blocking risk remains beyond normal checks and review. Suggested reviewers: 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
Full details: Description checkExplanation The description states the primary intent and references the replaced pull request, but it omits the mandatory /cc and /assign directives. It also does not include the template's Links section or sufficient implementation context. Resolution Add at least one reviewer with /cc and one approver with /assign from the top-level OWNERS file. Add the Links section and include related issues, dependent pull requests, or explicitly state that none apply. Expand the description with the implementation summary and rationale for the UBI Micro transition. Full details: Docstring CoverageExplanation No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (2 skipped: 2 unsupported.)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: jcantrill The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
/hold |
|
/retest-required |
|
/test e2e-using-bundle |
|
@jcantrill can you please also change go version in go.mod to |
|
@jcantrill: all tests passed! Full PR test history. Your PR dashboard. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here. |
|
/label verified |
Description
This PR:
Replaces #3432
Summary by CodeRabbit