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

Bash *? wildcard support for remote resource export/import #20

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

jgoff
Copy link

@jgoff jgoff commented May 4, 2017

hi Miłosz,
So here is the basic working first version. No doubt there will be a bunch of stylistic concerns (in some cases I have steered clear of changing certain things that seem obvious to change, for example not hacking at 'list' rather just adding 'list_path').

Outside of the readme, I'll mention again my use case which is entirely batch mode. I use some dev machine to create dashboards, then export them out so they can be stored in SCM. Once in SCM (in the future will have a commit trigger to do this automagically), I push (import) them out to the production hosts, e.g.

`
./grafcli export remote/dev/* $SCMROOT/grafana/dashboards

for h in cat prod_hosts.txt; do
./grafcli import $SCMROOT/grafana/dashboards remote/$h
done
`

There are a few caveats listed in the Readme.md.

By all means suggest ways you would prefer to see this refactored or changed to fit your code base or code style, it's your project.

thx
-jeff

@damoun
Copy link

damoun commented Jan 30, 2018

Hi @m110, what's the state of this PR ? It would be great to have this feature.

@m110
Copy link
Owner

m110 commented Jan 31, 2018

Hey. Sorry for letting this hang so long. I had some plans to add the wildcard support for all commands at the time, but never found some time to work on it. I'll check if this still works as expected with master and we're good to go!

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

Successfully merging this pull request may close these issues.

3 participants