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

Bump alpine image for security fix #852

Merged
merged 1 commit into from
Sep 21, 2018
Merged

Conversation

nrb
Copy link
Contributor

@nrb nrb commented Sep 17, 2018

This change includes the fix at docker-library/official-images#4834

Signed-off-by: Nolan Brubaker [email protected]

@nrb nrb requested review from carlisia, wwitzel3 and skriss September 17, 2018 21:26
Copy link
Contributor

@carlisia carlisia left a comment

Choose a reason for hiding this comment

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

This upgrade seems fine to me.

This change includes the fix at
alpinelinux/apk-tools@6484ed9

Signed-off-by: Nolan Brubaker <[email protected]>
Copy link
Contributor

@skriss skriss left a comment

Choose a reason for hiding this comment

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

LGTM

@skriss
Copy link
Contributor

skriss commented Sep 20, 2018

@nrb do you want to include this in v0.9.6? As long as you've smoke-tested the new images I'm fine with it

@nrb
Copy link
Contributor Author

nrb commented Sep 21, 2018

@skriss Probably doesn't hurt. I confirmed the images build. Let me test the result in a cluster before we decide.

@nrb
Copy link
Contributor Author

nrb commented Sep 21, 2018

Tested, this appears to backup/restore correctly using this. There's a separate issue with the fsfreeze image, which I'll open a new PR for.

@skriss
Copy link
Contributor

skriss commented Sep 21, 2018

cool, thanks for testing!

@skriss skriss merged commit 5161890 into vmware-tanzu:master Sep 21, 2018
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.

3 participants