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

Always --pull during docker build #6

Merged
merged 1 commit into from
Jun 13, 2017

Conversation

ixdy
Copy link
Contributor

@ixdy ixdy commented Jun 13, 2017

Without --pull, it's possible to use old base images which may have fixed vulnerabilities upstream.

x-ref kubernetes/kubernetes#47386

After merging this, we should probably create a new tag and then rebuild the image.

@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label Jun 13, 2017
@dnardo
Copy link
Contributor

dnardo commented Jun 13, 2017

/assign

@dnardo
Copy link
Contributor

dnardo commented Jun 13, 2017

/lgtm

@ixdy
Copy link
Contributor Author

ixdy commented Jun 13, 2017

@dnardo can you merge and then possibly publish a new git tag (and build a new release)? I don't have write access on this repo.

@ixdy ixdy force-pushed the docker-build-always-pull branch from 4595ff0 to 6017b66 Compare June 13, 2017 18:53
@ixdy
Copy link
Contributor Author

ixdy commented Jun 13, 2017

also updated so we pull the build image; this way we'll get the latest 1.7 go patch release too.

@dnardo dnardo merged commit 1264dac into kubernetes-sigs:master Jun 13, 2017
@dnardo
Copy link
Contributor

dnardo commented Jun 14, 2017

@ixdy done and v2.0.2 is now built.

@ixdy
Copy link
Contributor Author

ixdy commented Jun 14, 2017

@dnardo where? I don't see a v2.0.2 tag in git or on gcr.io.

(also, a fix for CVE-2017-9526 just went into debian-iptables-amd64:v7, so it's good that you haven't published anything anywhere yet. if you do push an image, please be sure to rebuild first.)

@dnardo
Copy link
Contributor

dnardo commented Jun 14, 2017

That is because I don't permission to push to gcr. I hadn't notice that last night when I pushed it.
Let me see if I can get someone to push the new image up.

@dnardo
Copy link
Contributor

dnardo commented Jun 15, 2017

Ok done. Really :-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cncf-cla: yes Indicates the PR's author has signed the CNCF CLA.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants