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

Support dokku cli #227

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

Conversation

morizyun
Copy link

@morizyun morizyun commented Jan 9, 2016

Hi Steve,

I've recently moved from Heroku to Dokku, but love Figaro's approach to configuration management, so I've added some initial support for Figaro to push configurations to a Dokku app.

This pull request includes code, specs and documentation updates - hope this is useful. It's my first rodeo with Thor so I've based it on your Heroku code, and I'm sure there are improvements to be made.

(It's late, so I'll skip the animated gif!)

Regards,
morizyun

Jin Hian Lee and others added 6 commits February 15, 2015 18:14
Usage:
figaro dokku:set --server=<example-dokku-server.com> --app=<example-app>

Limitations:
1. Requires Dokku server and app to be specified via --app and --server
2. Requires Dokku server to be accessible via ssh from the client machine for the 'dokku' user

Future enhancements could include supporting arbitrary users (which would require dynamically changing the command) and potentially reading the app and server values from the client/shell ENV
Also fix (as best as possible) minor command line help formatting issues
@joshco
Copy link

joshco commented Jan 10, 2016

I like this. It works for me using dokku-client on win10x64 talking to a linux dokku server.

@morizyun
Copy link
Author

thanks!

@scarroll32
Copy link

I'm looking forward to seeing this merged!

@scarroll32
Copy link

Will this be merged?

@beauraF
Copy link

beauraF commented Apr 8, 2017

Hey, is it possible to merge this ? :D
Great feature !

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.

6 participants