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

update the minio server #79

Merged
merged 1 commit into from
Feb 25, 2016
Merged

update the minio server #79

merged 1 commit into from
Feb 25, 2016

Conversation

kmala
Copy link
Contributor

@kmala kmala commented Feb 25, 2016

The current minio master is not very stable and the release version of minio doesn't have the fix for multi-part upload.So we cloned the minio repo and cherrypicked the commit.
This PR is to use the deis version of the minio server.

Update by mboersma
Closes deis/workflow#367.

@mboersma
Copy link
Member

This PR changes deis/minio to build from the "deis" branch of https://github.com/deis/minio-src, which is minio tag OFFICIAL.2016-02-08T00-12-28Z plus a multipart upload fix. I've tried the before/after tests several times and this does fix deploying example-ruby-sinatra. I've tested the other Deis example buildpack and Dockerfile apps with git push deis master too and had success.

@arschles
Copy link
Member

💥 !!

Sent from my iPhone

On Feb 25, 2016, at 17:44, Matt Boersma notifications@github.com wrote:

This PR changes deis/minio to build from the "deis" branch of https://github.com/deis/minio-src, which is minio tag OFFICIAL.2016-02-08T00-12-28Z plus a multipart upload fix. I've tried the before/after tests several times and this does fix deploying example-ruby-sinatra. I've tested the other Deis example buildpack and Dockerfile apps with git push deis master too and had success.


Reply to this email directly or view it on GitHub.

kmala added a commit that referenced this pull request Feb 25, 2016
update the minio server
@kmala kmala merged commit 5b4871c into deis:master Feb 25, 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.

5 participants