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

[DO NOT MERGE] cli: add OCI to image save/load #122

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

runcom
Copy link
Contributor

@runcom runcom commented May 23, 2017

Client side stuff split from moby/moby#26369

/cc @stevvooe @thaJeztah @cpuguy83 @vbatts

server side is moby/moby#33355

Signed-off-by: Antonio Murdaca [email protected]

@runcom
Copy link
Contributor Author

runcom commented May 23, 2017

This PR vendors moby/moby#33355 to build the binary, so don't merge it yet

Signed-off-by: Antonio Murdaca <[email protected]>
@runcom runcom changed the title cli: add OCI to image save/load [DO NOT MERGE] cli: add OCI to image save/load May 23, 2017
@stevvooe
Copy link
Contributor

@runcom Do you have time to update this with a workflow description? :)

@runcom
Copy link
Contributor Author

runcom commented Aug 22, 2017

Where do I have to add a workflow description? The commands have been designed/discussed here mostly moby/moby#26369

@stevvooe
Copy link
Contributor

@runcom Just to the description of this PR. If they are the same, just copy them over so people don't have to guess.

I'll still build this version and give it a try in bit.

@thaJeztah
Copy link
Member

@stevvooe have you been able to give this one a go?

also ping @runcom for the updated description 😄

@stevvooe
Copy link
Contributor

stevvooe commented Oct 9, 2017

@thaJeztah The other PR needs to be rebased but we haven't heard back from @runcom. I am wondering if we are going to have to carry these PRs.

@runcom
Copy link
Contributor Author

runcom commented Nov 16, 2017

I have fixed, rebased and added tests on the Moby side. I'd love to have that merged first before diving into docker itself. I'll rebase this as I get a chance though.

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