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

base image version bump #818

Merged
merged 3 commits into from
Jul 23, 2019
Merged

base image version bump #818

merged 3 commits into from
Jul 23, 2019

Conversation

vsamidurai
Copy link
Contributor

Dockerfile base image version bump to fix multiple CVE.

gliderlabs/docker-alpine#430

#567

ldelossa
ldelossa previously approved these changes Jul 6, 2019
Copy link
Contributor

@ldelossa ldelossa left a comment

Choose a reason for hiding this comment

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

LGTM



# change ownership of ssl directory to allow custom cert in OpenShift
RUN chgrp -R 0 /etc/ssl/certs && \
Copy link
Contributor

Choose a reason for hiding this comment

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

@vsamidurai can you explain this a bit more? Are you expecting to use a volume to mount certs to this path?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@ldelossa that was added part of another change #755 . I just updated PR with conflict.

KeyboardNerd
KeyboardNerd previously approved these changes Jul 19, 2019
Dockerfile Outdated
COPY --from=build /go/src/github.com/coreos/clair/clair /clair
RUN apk add --no-cache git rpm xz ca-certificates dumb-init


Copy link
Contributor

Choose a reason for hiding this comment

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

nit: extra space

Copy link
Contributor Author

Choose a reason for hiding this comment

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

thank you.. removed extra space

@KeyboardNerd KeyboardNerd merged commit 517e3e5 into quay:master Jul 23, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants