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

Decouple UBI images for Plus and NAP #3540

Merged
merged 1 commit into from Feb 10, 2023
Merged

Decouple UBI images for Plus and NAP #3540

merged 1 commit into from Feb 10, 2023

Conversation

lucacome
Copy link
Member

This will decouple the UBI images so that NAP images don't depend on the Plus image. By doing this we can update the Plus image to UBI 9 and benefit from the newer image with less CVEs. It's also now possible to use the minimal image.

This also fixes CVE-2022-47629 in NAP images.

Closes https://github.com/nginxinc/kubernetes-ingress/security/code-scanning/10037

This will decouple the UBI images so that NAP images don't depend on the Plus image. By doing this we can update the Plus image to UBI 9 and benefit from the newer image with less CVEs. It's also now possible to use the minimal image.

This also fixes CVE-2022-47629 in NAP images.
@lucacome lucacome requested a review from a team as a code owner February 10, 2023 04:30
@lucacome lucacome self-assigned this Feb 10, 2023
@github-actions github-actions bot added chore Pull requests for routine tasks documentation Pull requests/issues for documentation labels Feb 10, 2023
@codecov-commenter
Copy link

Codecov Report

Merging #3540 (5fe2e2f) into main (2bb4006) will not change coverage.
The diff coverage is n/a.

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@           Coverage Diff           @@
##             main    #3540   +/-   ##
=======================================
  Coverage   52.00%   52.00%           
=======================================
  Files          60       60           
  Lines       16816    16816           
=======================================
  Hits         8746     8746           
  Misses       7775     7775           
  Partials      295      295           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Copy link
Contributor

@vepatel vepatel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍🏼

@lucacome lucacome merged commit 876a0f6 into main Feb 10, 2023
@lucacome lucacome deleted the chore/decouple-nap branch February 10, 2023 18:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore Pull requests for routine tasks documentation Pull requests/issues for documentation
Projects
Status: Done 🚀
Development

Successfully merging this pull request may close these issues.

None yet

4 participants