Skip to content

Mini: fix tablet hostname#6509

Merged
deepthi merged 1 commit intovitessio:masterfrom
artemvovk:mini_topo_hostname
Jul 31, 2020
Merged

Mini: fix tablet hostname#6509
deepthi merged 1 commit intovitessio:masterfrom
artemvovk:mini_topo_hostname

Conversation

@artemvovk
Copy link
Copy Markdown

So that this container can work with a non-host network

Signed-off-by: avovk@recurly.com

I have a docker network and MySQL running in a container on that network.
Using the following docker run:

docker run -it --hostname vitess --name vitess -e "TOPOLOGY_SERVER=db:3306" -e "TOPOLOGY_USER=root" -e "TOPOLOGY_PASSWORD=dev" -e "MYSQL_SCHEMA=development" --network="development" vitess/mini:latest

I kept running into timeouts from vtctlclient.

vitess@23cc1adb9441:/$ vtctlclient TabletExternallyReparented zone1-0000000100
E0729 23:04:55.257453     914 main.go:67] remote error: rpc error: code = Unavailable desc = transport is closing

Finally, identified that hostname:port for that command were not the ones where vttablet was listening.

I retested this using --network=<custom> and --network=host.

So that this container can work with a non-host network

Signed-off-by:  <avovk@recurly.com>
Signed-off-by: Artem Vovk <avovk@recurly.com>
@artemvovk artemvovk force-pushed the mini_topo_hostname branch from 9049a30 to fca0764 Compare July 30, 2020 19:10
Copy link
Copy Markdown
Contributor

@dkhenry dkhenry left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like it will work under all circumstances.

@deepthi deepthi merged commit 2dd43fe into vitessio:master Jul 31, 2020
@askdba askdba added this to the v8.0 milestone Oct 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants