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.

Sign up for GitHub

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

Jump to bottom

NEXUS-20366 - Use UBI8 instead of CentOS #111

Merged
merged 4 commits into from
Jul 26, 2019
Merged

NEXUS-20366 - Use UBI8 instead of CentOS #111

merged 4 commits into from
Jul 26, 2019

Conversation

targeter21
Copy link
Collaborator

Switched the base image to UBI 8 instead of CentOS which resolved the OS level vulnerabilities.

@mpiggott mpiggott changed the title Use UBI8 instead of CentOS NEXUS-20366 - Use UBI8 instead of CentOS Jul 16, 2019
@joedragons
Copy link
Contributor

I think this also takes care of #110 ?

@mpiggott
Copy link
Contributor

I think this also takes care of #110 ?

Looks like

Blacktiger
Blacktiger approved these changes Jul 16, 2019
View reviewed changes
Copy link
Contributor

@Blacktiger Blacktiger left a comment

Choose a reason for hiding this comment

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

Not super-familiar with docker, but this looks fine to me.

@kevchu3 kevchu3 mentioned this pull request Jul 18, 2019
Closed
kevchu3
kevchu3 suggested changes Jul 18, 2019
View reviewed changes
Copy link
Contributor

@kevchu3 kevchu3 left a comment •
edited
Loading

Choose a reason for hiding this comment

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

I opened #112 as a PR into the use-ubi8 branch to update the other Dockerfiles missed in the #110 PR as well

@mpiggott mpiggott merged commit 37a9fcd into master Jul 26, 2019
@mpiggott mpiggott deleted the use-ubi8 branch August 16, 2022 14:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kevchu3 kevchu3 kevchu3 requested changes

@Blacktiger Blacktiger Blacktiger approved these changes

@mpiggott mpiggott mpiggott approved these changes

@mattjohnson mattjohnson mattjohnson approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@targeter21 @joedragons @mpiggott @Blacktiger @mattjohnson @kevchu3