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.

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

The DCE leases create prompt implies budget-currency is not required because it has a default of USD. It is required. #90

Open
nelsonjchen opened this issue Jan 6, 2021 · 0 comments

Comments

@nelsonjchen
Copy link

Version information

0.5.0 "reports as 0.4.0" due to #86
Ubuntu 20.04 on WSL2
Unknown Go Version
Bundled Terraform

Describe the bug

The DCE leases create prompt implies budget-currency is not required because it has a default of USD. It is required.

To Reproduce

$ dce leases create -b 10.0 -e [email protected] -E 1d -p nelson                                                                                                                                                                            
Error: required flag(s) "budget-currency" not set
Usage:
  dce leases create [flags]

Flags:
  -b, --budget-amount float      The leased accounts budget amount
  -c, --budget-currency string   The leased accounts budget currency (default "USD")
  -e, --email stringArray        The email address that budget notifications will be sent to
  -E, --expires-on string        The leased accounts expiry date as a long (UNIX epoch) or string (eg., '7d', '8h' (default "7d")
  -h, --help                     help for create
  -p, --principal-id string      Principle ID for the user of the leased account

Global Flags:
      --config string   config file (default is "$HOME/.dce/config.yaml")

Expected behavior

dce leases create -b 10.0 -e [email protected] -E 1d -p nelson                                                                                                                                                                            

and it just does its thing.

The command assumes USD and does not fatally exit with a help prompt.

Additional context

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

No branches or pull requests

1 participant