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

Docker automatic builds #2308

Merged
merged 1 commit into from
Feb 10, 2016
Merged

Docker automatic builds #2308

merged 1 commit into from
Feb 10, 2016

Conversation

ghost
Copy link

@ghost ghost commented Feb 6, 2016

I've set up automatic builds of the ipfs/go-ipfs image, but it looks like they prevent the existing jbenet/go-ipfs image from being automatically built. This seems to be due to the integration being done using github services instead of generic webhooks, and docker hub doesn't seem to offer using these webhooks instead. That's why only one automatic build can be triggered by pushing to the repo.

This PR updates all mentions of jbenet/go-ipfs in the repo, but I wanna give the old automated build a grace period, it's used wildly (>300k pulls). I'm not 100% sure how to maintain the automation here, but when in doubt a daily cronjob will do I guess?

Any more thoughts? @jbenet @Luzifer

License: MIT
Signed-off-by: Lars Gierth <larsg@systemli.org>
@ghost ghost mentioned this pull request Feb 8, 2016
@ghost ghost added RFM and removed RFCR labels Feb 10, 2016
@ghost
Copy link
Author

ghost commented Feb 10, 2016

This one's ready -- I'm setting up a cronjob to keep the old image alive for a while

@whyrusleeping
Copy link
Member

cool, SGTM

whyrusleeping added a commit that referenced this pull request Feb 10, 2016
@whyrusleeping whyrusleeping merged commit c913105 into master Feb 10, 2016
@whyrusleeping whyrusleeping deleted the docker-auto-builds branch February 10, 2016 17:23
@ghost ghost mentioned this pull request Feb 12, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant