-
Notifications
You must be signed in to change notification settings - Fork 40k
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
allow wrapped binaries to use a configurable image as the base image #17194
allow wrapped binaries to use a configurable image as the base image #17194
Conversation
Labelling this PR as size/M |
c98f65e
to
fa59c31
Compare
GCE e2e test build/test passed for commit fa59c312b633d89b1d947b1357f16354861d4d47. |
@@ -737,27 +739,36 @@ function kube::release::sha1() { | |||
function kube::release::create_docker_images_for_server() { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
While you're at it, document the args this function takes. (See kube::release::gcs::publish_ci
.)
LGTM with nits. |
fa59c31
to
d68c36a
Compare
GCE e2e test build/test passed for commit d68c36a. |
LGTM. |
Continuous integration appears to have missed, closing and re-opening to trigger it |
@k8s-bot test this [submit-queue is verifying that this PR is safe to merge] |
GCE e2e build/test failed for commit d68c36a. |
@k8s-bot ok to test |
GCE e2e build/test failed for commit d68c36a. |
@k8s-bot test this please |
GCE e2e test build/test passed for commit d68c36a. |
@k8s-bot unit test this please |
@k8s-bot ok to test |
Automatic merge from submit-queue |
Auto commit by PR queue bot
GCE e2e build/test failed for commit d68c36a. |
I'm attempting to dockerize kube-proxy, and failing since it requires iptables. This will allow me to use a custom base image to wrap kube-proxy.
@ihmccreery related to build. Do you mind reviewing?