MetaCell / cloud-harness

Other
14 stars 5 forks source link

Improve output harness-deployment, showing 2x ip address + (sub)domains #543

Open zsinnema opened 2 years ago

zsinnema commented 2 years ago

at the end of the output of the harness-deployment command it shows the message To test locally, update your hosts file and then the ip address and the list of the (sub)domains however at the end of the output a new list with ip address 127.0.0.1 and the (sub)domains is show. The last list of subdomains actually shows the application name concatenated with the namespace

e.g.

127.0.0.1       notifications.ch accounts.ch volumemanager.ch sentry.ch jupyterhub.ch argo.ch samples.ch common.ch events.ch workflows.ch

this list "works" (resolves) in the kubernetes cluster but not on your local host.

it is very confusing that the second list is show and it also doesn't add much value.

suggestion: hide the last list from the output

filippomc commented 2 years ago

This list concatenated to the namespace is meant for cases when forwarding is used to emulate the hosts inside the k8s cluster. With the debug the forwarding is less necessary though, and I agree the output is confusing. I propose to address this issue to generally improve the output information and formatting, in order to be more understandable to the user. I would also like to control the verbosity so that only the relevant information is shown unless needed