Skip to content

feat(ci): Make kai images distroless#745

Merged
enoodle merged 9 commits intoNVIDIA:mainfrom
dttung2905:make-kai-images-distroless
Feb 6, 2026
Merged

feat(ci): Make kai images distroless#745
enoodle merged 9 commits intoNVIDIA:mainfrom
dttung2905:make-kai-images-distroless

Conversation

@dttung2905
Copy link
Contributor

Description

This PR is inspired from the other PR in k8s-device-plugin NVIDIA/k8s-device-plugin#1291. We hope to make these images distroless and in turn remove any unecessary CVEs

Related Issues

nil

Checklist

Note: Ensure your PR title follows the Conventional Commits format (e.g., feat(scheduler): add new feature)

  • Self-reviewed
  • Added/updated tests (if needed)
  • Updated documentation (if needed)

Breaking Changes

nil

Additional Notes

nil

@dttung2905 dttung2905 force-pushed the make-kai-images-distroless branch 2 times, most recently from 9243e4a to f4423cf Compare December 11, 2025 21:24
@dttung2905 dttung2905 force-pushed the make-kai-images-distroless branch from f4423cf to 4dbe6b0 Compare December 24, 2025 18:25
Signed-off-by: dttung2905 <ttdao.2015@accountancy.smu.edu.sg>
Signed-off-by: dttung2905 <ttdao.2015@accountancy.smu.edu.sg>
@dttung2905 dttung2905 force-pushed the make-kai-images-distroless branch from 4dbe6b0 to 78d116a Compare January 7, 2026 22:38
@enoodle
Copy link
Collaborator

enoodle commented Jan 16, 2026

@dttung2905 I tested this locally and the issue is that crd-upgrader gets broken and the whole charts fails to load. You can use the script in hack directory to test it locally.

@dttung2905
Copy link
Contributor Author

@enoodle Thanks alot for the help. Let me find some time to check this 🙏

Signed-off-by: dttung2905 <ttdao.2015@accountancy.smu.edu.sg>
@dttung2905 dttung2905 requested a review from itsomri January 19, 2026 15:56
Signed-off-by: dttung2905 <ttdao.2015@accountancy.smu.edu.sg>
Signed-off-by: dttung2905 <ttdao.2015@accountancy.smu.edu.sg>
@dttung2905
Copy link
Contributor Author

@enoodle I reduced the scope of the PR and exclude the CRD upgrader image for now. Could you help to take a look at it again? 🙏

Signed-off-by: dttung2905 <ttdao.2015@accountancy.smu.edu.sg>
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 5, 2026

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

  • 🔍 Trigger a full review
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@enoodle enoodle enabled auto-merge February 5, 2026 12:24
@dttung2905
Copy link
Contributor Author

hmm. It is weird @enoodle last time the e2e passes, now it seems to be flaky. 🤔 Let me rebase from main again

@enoodle enoodle added this pull request to the merge queue Feb 6, 2026
Merged via the queue into NVIDIA:main with commit 042ec94 Feb 6, 2026
8 checks passed
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