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

Fixup influx CLI config create/update command #68

Closed
1 task
jsteenb2 opened this issue Apr 24, 2020 · 4 comments
Closed
1 task

Fixup influx CLI config create/update command #68

jsteenb2 opened this issue Apr 24, 2020 · 4 comments
Labels
wontfix This will not be worked on

Comments

@jsteenb2
Copy link

the current API looks as follows
$ influx config create 
--active        -a  -- Set it to be the active config
--help          -h  -- Help for the create command
--hide-headers      -- Hide the table headers; defaults false; Maps to env var $INFLUX_HIDE_HEADERS
--host              -- HTTP address of Influx; Maps to env var $INFLUX_HOST
--json              -- Output data as json; defaults false; Maps to env var $INFLUX_OUTPUT_JSON
--local             -- Run commands locally against the filesystem
--name          -n  -- The config name (required)
--org           -o  -- The optional organization name
--skip-verify       -- SkipVerify controls whether a client verifies the server's certificate chain and host name.
--token         -t  -- API token to be used throughout client calls; Maps to env var $INFLUX_TOKEN
--url           -u  -- The config url (required)
updated CLI
$ influx config create 
--active        -a  -- Set it to be the active config
--help          -h  -- Help for the create command
--hide-headers      -- Hide the table headers; defaults false; Maps to env var $INFLUX_HIDE_HEADERS
--json              -- Output data as json; defaults false; Maps to env var $INFLUX_OUTPUT_JSON
--config-name          -n  -- The config name (required)
--config-org           -o  -- The optional organization name
--config-token          -- API token to be used throughout client calls; Maps to env var $INFLUX_TOKEN
--config-host-address          -u  -- The config url (required)

AC

  • flag names and descriptions updated to match details above
@jsteenb2 jsteenb2 changed the title Fixup influx CLI config create command Fixup influx CLI config create/update command Apr 24, 2020
@mhall119
Copy link

Should there be a --config-org too?

@jsteenb2
Copy link
Author

yes, great catch 👍

@jsteenb2
Copy link
Author

the descriptions should be updated too. We can peel that onion when we pull this in

@danxmoran danxmoran transferred this issue from influxdata/influxdb May 11, 2021
@danxmoran danxmoran added the wontfix This will not be worked on label Aug 3, 2021
@danxmoran
Copy link
Contributor

Closing this as a "won't fix" since we've had multiple releases with the old-style flags, and I doubt we'd break them now

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
wontfix This will not be worked on
Projects
None yet
Development

No branches or pull requests

3 participants