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.

Sign up for GitHub

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

Jump to bottom

docker: TestFunctional/parallel/ServiceCmd broken due to tunnel requirement #7383

Closed
tstromberg opened this issue Apr 2, 2020 · 1 comment
Closed

docker: TestFunctional/parallel/ServiceCmd broken due to tunnel requirement #7383

tstromberg opened this issue Apr 2, 2020 · 1 comment
Labels
co/docker-driver Issues related to kubernetes in container kind/failing-test Categorizes issue or PR as related to a consistently or frequently failing test. os/macos os/windows

Comments

@tstromberg
Copy link
Contributor

On macOS and Windows, the "service" command needs a tunnel to function.

TestFunctional/parallel/ServiceCmd was not updated to take this into account, so it hangs for a long time, waiting for the command to finish.
@tstromberg tstromberg added co/docker-driver Issues related to kubernetes in container kind/failing-test Categorizes issue or PR as related to a consistently or frequently failing test. os/macos labels Apr 2, 2020
@tstromberg
Copy link
Contributor Author

tstromberg commented Apr 2, 2020 •
edited
Loading

For example:

TestFunctional/parallel/ServiceCmd: functional_test.go:621: (dbg) Run: out/minikube -p minikube service --namespace=default --https --url hello-node

This command also corrupts stdout, making the URL unparseable.

@tstromberg tstromberg mentioned this issue Apr 2, 2020
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
co/docker-driver Issues related to kubernetes in container kind/failing-test Categorizes issue or PR as related to a consistently or frequently failing test. os/macos os/windows
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@tstromberg @sharifelgamal