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

Pkger support multiple pkg import #16657

Closed
3 of 6 tasks
jsteenb2 opened this issue Jan 23, 2020 · 2 comments · Fixed by #16765
Closed
3 of 6 tasks

Pkger support multiple pkg import #16657

jsteenb2 opened this issue Jan 23, 2020 · 2 comments · Fixed by #16765
Assignees

Comments

@jsteenb2
Copy link
Contributor

jsteenb2 commented Jan 23, 2020

As a user, I want to provide multiple pages for inserting, so that I can create resources and maintain my sanity when my pkgs get very large.

I can see something like:

my_pkgs_dir/
  - labels.yml
  - buckets.yml

AC

  • should be able to reference associations in other files
  • errors should include which pkg had the issue and where in it the issue happened specifically
  • should support a directory structure
  • shoudl be supported via the CLI like following:
influx pkg -f labels.yml -f buckets.yml -f endpoints.yml -f telegrafs.yml

# should see that it is a directory and then run all pkgs
influx pkg -f my_pkg_dirs

Use case:
This would help provide easier organization of pkgs and easier composition of pkgs together and have them all be part of the same application/dry run.

Acceptance criteria

  • support multiple file uploads via cli
  • support dir upload via cli
  • support multiple url upload svia cli
  • support multiple pkgs via HTTP API
  • support multiple urls via HTTP API
  • both cli and HTTP API should support both urls and pkgs together
@jsteenb2 jsteenb2 self-assigned this Jan 23, 2020
@russorat
Copy link
Contributor

awesome! could we have the cli work like kubectl apply -f <directory>? https://kubernetes.io/docs/reference/kubectl/overview/#examples-common-operations

@jsteenb2
Copy link
Contributor Author

jsteenb2 commented Jan 23, 2020

sure @russorat add it to the AC 👍 donezo

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants