flyte icon indicating copy to clipboard operation
flyte copied to clipboard

[Flytectl Feature] flytectl demo cluster - configurable port number

Open shekhar-kotekar opened this issue 2 years ago • 9 comments

Problem Description:

flytectl demo start command fails with below error when a developer has Docker Desktop which allows us to have a Kubernetes local cluster. Kubernetes API server runs on port 6443. So we have to either turn off local Kubernetes cluster (not a good option) or allow demo cluster to use some other port.

{"status":"Status: Downloaded newer image for cr.flyte.org/flyteorg/flyte-sandbox-bundled:sha-1ae254f8683699b68ecddc89d775fc5d39cc3d84"}
🧑‍🏭 booting Flyte-sandbox container
😬 Something went wrong: Failed to start Sandbox container 🐋, Please check your docker client and try again. 
Error: Error response from daemon: driver failed programming external connectivity on endpoint flyte-sandbox (5eaf6609c86e6d818291e2f060e42f8002dfd4e288393da6e01109cfa5a60bd7): listen tcp4 0.0.0.0:6443: bind: address already in use

Requested feature: Allow demo cluster to use other port preferrably via config file (config.yaml or config-sandbox.yaml or some other file) so that if a developer has some other container / process using the same port, they can change it and quickly get the demo cluster up and running.

Environment details:

  • Docker Desktop with Kubernetes enabled
  • Mac with M1 chip
  • Discussion about this issue on slack is here
flytectl version  
{
  "App": "flytectl",
  "Build": "0cb2bfa",
  "Version": "0.6.39",
  "BuildTime": "2023-06-20 14:45:22.664715 +0200 CEST m=+0.016518584"
}

shekhar-kotekar avatar Jun 20 '23 12:06 shekhar-kotekar

I do vote in favor of this feature request. Also, flytectl demo start seems to insist on serving on localhost while I'd need it to serve on 0.0.0.0, should that also be configurable?

enricorotundo avatar Aug 11 '23 12:08 enricorotundo

Hey @jeevb any luck with solving this issue? I can give a try to fix this.

shekhar-kotekar avatar Oct 09 '23 15:10 shekhar-kotekar

Haven't had a chance to look into this yet. Please feel free to take a crack at it! 🙏

jeevb avatar Oct 09 '23 15:10 jeevb

I can take a look at this feature and will tell you the result soon.

Future-Outlier avatar Oct 11 '23 14:10 Future-Outlier

I will create a PR today, I know how to do it.

Future-Outlier avatar Oct 12 '23 06:10 Future-Outlier

@Future-Outlier was there a PR for this feature?

guyarad avatar Mar 03 '24 07:03 guyarad

@Future-Outlier was there a PR for this feature?

Nope, you can take it, thank you!

Future-Outlier avatar Mar 03 '24 10:03 Future-Outlier

Is there a workaround for this?

Masame avatar Aug 21 '24 00:08 Masame

This issue seems to be starting when one updates docker / docker desktop version.

rmadanykgw avatar Aug 23 '24 15:08 rmadanykgw