-
Notifications
You must be signed in to change notification settings - Fork 283
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
Publish docker image on DockerHub #51
Comments
@frosforever thanks for looking further into this and for the followup. I'll see if I can add this to DockerHub in the next week or so. |
@donnemartin sure thing. If it's any help, I can attempt to run through the steps and publish on my account working out the kinks and then hand over the steps I took. I don't know if there's a better way. It would be cool if one could publish on behalf of a user/repo via docker hub, but I don't know if that's possible. Also, I realized the current Dockerfile installs via |
@frosforever I think this would be a big help, I've been pretty tied up lately.
Hmm interesting catch. Thanks for your previous contribution, I released it to GitHub and PyPI today and of course credited you :) https://github.com/donnemartin/saws/releases/tag/0.3.2 |
@donnemartin it ebbs and flows. I'm currently a bit swamped as well but I'll try and get on this sometime. |
@frosforever no rush! |
This would be super useful. Not sure if this helps or not but codeclimate do something similar with a wrapper script for the docker process (although that's a ruby app): |
FWIW I have published my image to the docker hub: https://hub.docker.com/r/sysadmiral/saws/ A few things to note:
Please let me know what you think! If you think the Dockerfile is suitable I can create a PR to update the official one in this repo so that the one on docker hub can be from the official |
@sysadmiral thanks for the help on docker support!
I'm not really a docker user myself so I'm not sure I'm the best person to comment. Perhaps our other docker users (@frosforever @cliveza @stilvoid) might share their thoughts :) |
Wow, I completely forgot about this! @sysadmiral Thanks for bringing this up. A few things as I see it:
I've submitted #84 that uses the official python alpine image and builds from sources instead of installing from pip should it be decided that that's the way to go. I've also set up an automated build https://hub.docker.com/r/frosforever/saws/ @sysadmiral What do you think? It seems that your image is a bit smaller (104.8MB vs 97.91MB) but that might just be splitting hairs. |
Glad I got this moving! 😄 @frosforever just to expand on some of my decisions/your points:
I will check out PR #84 later this evening (GMT). |
You can install the latest from GitHub source which can contain changes not yet pushed to PyPI:
I haven't refreshed the version on PyPI in awhile (5 months and 35 commits behind), although I plan to do so soon. Pushing to PyPI is more involved than pushing to master :) I think installing from source is probably a better option if you always want the latest changes. |
Leverage DockerHub's Automated build to pick up changes and deploy public
saws
image to dockerhub.My understanding is this should be done under @donnemartin's account as he is the owner of
saws
unless someone else has a better idea on this.This would require:
The text was updated successfully, but these errors were encountered: