-
Notifications
You must be signed in to change notification settings - Fork 357
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
Validate and publicize Docker support #78
Comments
What about this?
It would be great to see volume support for docker image. |
Agree - I'll make sure this works with volumes before releasing. I think I originally put that Dockerfile together on a whim to test something. 👍 |
Do you have a branch you're currently testing any improvements for Docker on that I can contribute to? I'd like to help on this but I'm a beginner on docker/ruby/git/etc and I don't want to mess up your awesome app! |
Hi @christag - you won't mess it up - feel free to make any pull-requests. They are a big help as I juggle my free time working on this. Let me try to resurrect my docker tests and I'll post an update here soon on where pwpush is currently with docker. |
It's already with docker in the current stateless state (see pull request). We need to decide how we proceed further, ether add mysql support instead of sqlite3 or work towards placing sqlite data on persistent storage in Openshift/Kubernetes. Image here: https://hub.docker.com/r/sfarosu/passwordpusher-ephemeral/ |
Just merged #82 and updated the front page README as a first pass. I suspect I'll update documentation more as time passes but for now we clearly point out the new container support. If either of you see any issues, please let me know. |
https://hub.docker.com/r/pglombardo/passwordpusher/
https://github.com/pglombardo/PasswordPusher/blob/pwpush.com/Dockerfile
Add documentation adding/explaining:
The text was updated successfully, but these errors were encountered: