-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
fix: handle DOS-style path for Dockerfile in minikube build #15074
Conversation
Can one of the admins verify this patch? |
Welcome @ComradeProgrammer! |
Hi @ComradeProgrammer. Thanks for your PR. I'm waiting for a kubernetes member to verify that this patch is reasonable to test. If it is, they should reply with Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
bc51c57
to
fd0d0c7
Compare
Here is an example to reproduce the problem. The folder is
The content of .dockerignore
The content of dockerfile
Output:
When I execute
|
Tested this and confirms it works Before:
After:
|
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.
@ComradeProgrammer Lets make sure this replace is only suggested by minikube and not automatically done, if minikube gonna replace it, it has to get user permission to clearly say "Yes" it is better to leave this operation to the user
04aa149
to
10744e8
Compare
This feature has been added, now the we need user's permission to replace the \ |
10744e8
to
be5c203
Compare
be5c203
to
4302ab3
Compare
@spowelljr Indeed you are right, so I changed the code. Instead of asking for user's input, I switched to command line arguments. |
/ok-to-test |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
7d3f127
to
4f6e197
Compare
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
4f6e197
to
65a516c
Compare
65a516c
to
9b342d2
Compare
9b342d2
to
76950eb
Compare
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
@spowelljr It seems that the test under Docker_Windows has passed, is there anything else I should do? |
Co-authored-by: Steven Powell <44844360+spowelljr@users.noreply.github.com>
kvm2 driver with docker runtime
Times for minikube start: 52.6s 49.1s 48.8s 48.5s 48.7s Times for minikube ingress: 27.7s 27.1s 27.1s 27.1s 28.2s docker driver with docker runtime
Times for minikube start: 21.0s 23.8s 24.4s 23.5s 21.0s Times for minikube ingress: 20.8s 21.8s 19.8s 20.8s 20.8s docker driver with containerd runtime
Times for minikube start: 22.6s 20.4s 19.6s 22.3s 23.6s Times for minikube ingress: 30.3s 31.3s 31.3s 31.3s 47.3s |
These are the flake rates of all failed tests.
To see the flake rates of all tests by environment, click here. |
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.
PR looks good, thanks for all the changes
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: ComradeProgrammer, spowelljr The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
fixes #15007
Here is an example to reproduce the problem. The folder is
The content of .dockerignore
The content of dockerfile
Output:
When I execute
.\minikube-windows-amd64.exe image build -t aaa:latest -f test\Dockerfile .
, using DOS-style pathWhen I execute
.\minikube-windows-amd64.exe image build -t aaa:latest -f test/Dockerfile .
, using UNIX-style path