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

Convert influxd binary to use the cobra CLI package #12602

Closed
3 tasks done
stuartcarnie opened this issue Mar 13, 2019 · 0 comments
Closed
3 tasks done

Convert influxd binary to use the cobra CLI package #12602

stuartcarnie opened this issue Mar 13, 2019 · 0 comments
Assignees
@stuartcarnie
Labels
area/cli

Comments

@stuartcarnie
Copy link
Contributor

stuartcarnie commented Mar 13, 2019 •
edited
Loading

Future versions of the influxd binary will provide tooling via subcommands. To simplify adding these subcommands, migrate to the cobra CLI package.

TODOs:

  • add cobra CLI package
  • add run subcommand the launches the influxd server
    • If no subcommand is specified when executing influxd, the default will be to execute the run subcommand.

Some examples of future commands include

  • influxd inspect - analyze various data files created by InfluxDB
  • influxd gen - generate data for testing
@stuartcarnie stuartcarnie self-assigned this Mar 13, 2019
stuartcarnie added a commit that referenced this issue Mar 13, 2019
@stuartcarnie
feat(influxd): Migrate influxd binary to cobra Command package
7317fea 
This commit consists of several improvements or changes:

* migrate the influxd binary to cobra.Command
* introduce a default run sub-command to start the server
* register the run sub-command flags with viper
  to maintain compatibility with the existing behavior of automatic
  binding of flags to environment variables.

Closes #12602
@stuartcarnie stuartcarnie mentioned this issue Mar 13, 2019
Merged
stuartcarnie added a commit that referenced this issue Mar 13, 2019
@stuartcarnie
feat(influxd): Migrate influxd binary to cobra Command package
f7dc808 
This commit consists of several improvements or changes:

* migrate the influxd binary to cobra.Command
* introduce a default run sub-command to start the server
* register the run sub-command flags with viper
  to maintain compatibility with the existing behavior of automatic
  binding of flags to environment variables.

Closes #12602
stuartcarnie added a commit that referenced this issue Mar 14, 2019
@stuartcarnie
feat(influxd): Migrate influxd binary to cobra Command package
e8045ae 
This commit consists of several improvements or changes:

* migrate the influxd binary to cobra.Command
* introduce a default run sub-command to start the server
* register the run sub-command flags with viper
  to maintain compatibility with the existing behavior of automatic
  binding of flags to environment variables.

Closes #12602
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@stuartcarnie stuartcarnie

Labels
area/cli
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@stuartcarnie