civo / cli

Our Command Line Interface (CLI) for interacting with your Civo resources
Apache License 2.0
178 stars 87 forks source link

When adding applications to a cluster, the feedback needs to be more specific #396

Open fernando-villalba opened 6 months ago

fernando-villalba commented 6 months ago

Issue

When adding applications to a cluster, the feedback is insufficient:

Consider these two examples:

civo kubernetes create  --size g4s.kube.xsmall --nodes 1 test-app
civo kubernetes applications add postgresql:5GB,uptime-kuma:1GB,volcano  --cluster test-app

output:

% civo kubernetes create  --size g4s.kube.xsmall --nodes 1 test-app                                                     
civo kubernetes applications add postgresql:5GB,uptime-kuma:1GB,volcano  --cluster test-app
The cluster test-app (74e90d95-b159-45db-8b1c-93e26162302c) has been created
The application was installed in the Kubernetes cluster test-app

And this:

civo kubernetes create --size g4s.kube.xsmall --nodes 1 --applications postgresql:5GB,uptime-kuma:1GB,volcano test-app

Output:

 % civo kubernetes create --size g4s.kube.xsmall --nodes 1 --applications postgresql:5GB,uptime-kuma:1GB,volcano test-app
The cluster test-app (3aad83f1-ad19-43db-98e0-c019b7287ee6) has been created

Acceptance Criteria

dipu989 commented 3 weeks ago

@fernando-villalba I am working on this issue.

I have formatted the command output to display a list of all the installed applications on the cluster. Let me know if you think this output looks good. The API returns a list of all installed apps, so it’s convenient to show them all in the CLI and informative for the end user as well.

Alternatively, let me know if we only want to show the applications that were created by the specific command (e.g., postgresql, uptime-kuma, volcano), excluding the ones that were installed by default (e.g., metrics-server, traefik2-nodeport) when the cluster is spun up.

I am attaching a screenshot for your reference. If this looks good to you, I will raise the PR.

image

CC: @uzaxirr @alessandroargentieri