Allow to push image by sroze · Pull Request #121 · docker-php/docker-php · GitHub
Skip to content
This repository was archived by the owner on Oct 26, 2019. It is now read-only.

Allow to push image#121

Merged
ubermuda merged 1 commit into
docker-php:masterfrom
sroze:feature/image-push
Aug 26, 2015
Merged

Allow to push image#121
ubermuda merged 1 commit into
docker-php:masterfrom
sroze:feature/image-push

Conversation

@sroze

@sroze sroze commented Jul 8, 2015

Copy link
Copy Markdown
Contributor

This PR add a push method on the image manager that allows to push an image to a registry.

@hacfi

hacfi commented Jul 8, 2015

Copy link
Copy Markdown
Contributor

@sroze

sroze commented Jul 8, 2015

Copy link
Copy Markdown
Contributor Author

@hacfi yes, it is.

@hacfi

hacfi commented Jul 8, 2015

Copy link
Copy Markdown
Contributor

Haven’t tried this but looks good to me!

@sroze Will this work with both Docker Hub and private registries?

@sroze

sroze commented Jul 8, 2015

Copy link
Copy Markdown
Contributor Author

@hacfi it is, for private repositories you have have to have an image name like address.of.my.repository:1234/image-name so Docker will know it have to push to address.of.my.repository:1234.

@sroze sroze force-pushed the feature/image-push branch from 25912e7 to 9674c4e Compare August 16, 2015 14:48
@sroze

sroze commented Aug 26, 2015

Copy link
Copy Markdown
Contributor Author

ping @hacfi

@hacfi

hacfi commented Aug 26, 2015

Copy link
Copy Markdown
Contributor

Looks good to me but I can’t merge it as I’m just a contributor.

@tasuki

tasuki commented Aug 26, 2015

Copy link
Copy Markdown
Contributor

@ubermuda, @joelwurtz: Pretty please!

@ubermuda

Copy link
Copy Markdown
Contributor

Hey @sroze, thanks for your contribution! Can you rebase this on master? I just merged in some fixes to unit test running in travis

@sroze sroze force-pushed the feature/image-push branch from 9674c4e to ca97551 Compare August 26, 2015 13:03
@sroze

sroze commented Aug 26, 2015

Copy link
Copy Markdown
Contributor Author

@ubermuda I did but still failing, without any relation with this PR.

@ubermuda

Copy link
Copy Markdown
Contributor

That's right, I'll have to take a look at this later :)

ubermuda added a commit that referenced this pull request Aug 26, 2015
@ubermuda ubermuda merged commit d75b6f1 into docker-php:master Aug 26, 2015
@sroze

sroze commented Aug 26, 2015

Copy link
Copy Markdown
Contributor Author

Thanks @ubermuda ! Anyway you can release a minor version ?

@sroze sroze deleted the feature/image-push branch August 26, 2015 13:25
@ubermuda

Copy link
Copy Markdown
Contributor

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants