-
Notifications
You must be signed in to change notification settings - Fork 18.6k
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
integration-cli: fix --net=none tests #24490
Conversation
Example: 21:28:01 [d68573521] waiting for daemon to start 21:28:01 [d68573521] daemon started 21:28:01 docker_api_swarm_test.go:163: 21:28:01 c.Assert(d1.Init(map[string]bool{"worker": true}, ""), checker.IsNil) 21:28:01 ... value *errors.errorString = &errors.errorString{s:"initializing swarm: invalid statuscode 500, \"{\\\"message\\\":\\\"could not determine local IP address: dial udp 8.8.8.8:53: connect: network is unreachable\\\"}\\n\"} ("initializing swarm: invalid statuscode 500, \"{\\\"message\\\":\\\"could not determine local IP address: dial udp 8.8.8.8:53: connect: network is unreachable\\\"}\\n\"") 21:28:01 21:28:01 [d68573521] exiting daemon"} Signed-off-by: Antonio Murdaca <runcom@redhat.com>
LGTM 🐮 |
LGTM |
This doesn't look like a right fix. We should support these tests without external networking. The error seems to come from the listen-address detection that is already being changed. This error should not be fatal or we should just specify a specific listening address in tests. |
@tonistiigi wasn't aware of that, happy to change it again |
removed the cherry-pick label |
@thaJeztah We may want to include it to avoid fixing cherry pick conflicts. I created a new issue #24547 so we don't forget to fix it. |
Example:
Signed-off-by: Antonio Murdaca runcom@redhat.com