Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Various CLI updates #324

Closed
6 of 8 tasks
gadkins opened this issue Sep 7, 2023 · 4 comments · Fixed by #331 · May be fixed by UffizziCloud/cluster-action#40
Closed
6 of 8 tasks

Various CLI updates #324

gadkins opened this issue Sep 7, 2023 · 4 comments · Fixed by #331 · May be fixed by UffizziCloud/cluster-action#40
Assignees
@zipofar
Labels
depends on cluster-action

Comments

@gadkins
Copy link
Member

gadkins commented Sep 7, 2023 •
edited by ALEXANCHESSS
Loading

  • Make --update-current-context default to true (i.e. user should set --update-current-context=false)
  • Do not require --name / -n option (use positional argument)
  • Make --delete-config default to true (i.e. user should set --delete-config=false)
  • Handle ctl+c (currently get a stack trace)
  • Default to account that corresponds to last created cluster when user runs uffizzi login
  • Do not list projects that user does not have access to in UI
  • CLI typo “Login successfull” — should be "successful"
  • Do not prompt to enter Username when user logs in via email uffizzi login --email [email protected]
@axisofentropy
Copy link
Member

axisofentropy commented Sep 8, 2023 •
edited by zipofar
Loading

  • Could we also not ask for the server and assume https://app.uffizzi.com unless otherwise specified?

@zipofar
Copy link
Contributor

zipofar commented Sep 12, 2023 •
edited
Loading

Do not list projects that user does not have access to in UI

Can't reproduce

@zipofar zipofar linked a pull request Sep 13, 2023 that will close this issue
[324] Various CLI updates #331
Merged
@ALEXANCHESSS
Copy link

ALEXANCHESSS commented Sep 14, 2023 •
edited
Loading

  • Make --update-current-context default to true (i.e. user should set --update-current-context=false)
  • Do not require --name / -n option (use positional argument)
  • Make --delete-config default to true (i.e. user should set --delete-config=false)
  • Handle ctl+c (currently get a stack trace)
  • CLI typo “Login successfull” — should be "successful"
  • Do not prompt to enter Username when user logs in via email uffizzi login --email [email protected]
  • Could we also not ask for the server and assume https://app.uffizzi.com unless otherwise specified?

@ALEXANCHESSS ALEXANCHESSS reopened this Sep 14, 2023
@ALEXANCHESSS
Copy link

ALEXANCHESSS commented Sep 14, 2023 •
edited by zipofar
Loading

@zipofar

  • fix text u cluster create -h
267925963-f38d0812-e50e-401a-a178-4f628c734717
  • When deleting a cluster, an error occurs and the cluster is not removed from the kubeconfig
image

@zipofar zipofar linked a pull request Sep 14, 2023 that will close this issue
[324] Refactor cluster create command UffizziCloud/cluster-action#40
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@zipofar zipofar

Labels
depends on cluster-action
Projects
Working Project Board
Status: Closed
Milestone
No milestone
4 participants
@axisofentropy @gadkins @zipofar @ALEXANCHESSS