-
Notifications
You must be signed in to change notification settings - Fork 880
add support to import image from docker-daemon in rkt #3939
base: master
Are you sure you want to change the base?
Conversation
|
This PR first fetches the image from machine's running docker daemon into a tar format. This is similar to running a Then it is converted into |
4e78b87 to
72d6995
Compare
Now you can import an image that is available in docker's image store on a local machine into rkt's imagestore. To do so run following command: ``` rkt fetch --insecure-options=image docker-daemon://docker.io/library/alpine ``` or directly run it the usual way ``` rkt run --insecure-options=image docker-daemon://docker.io/library/alpine ``` This will import the image `docker.io/library/alpine` into rkt's store.
72d6995 to
8891832
Compare
| if err != nil { | ||
| return "", err | ||
| } | ||
| defer aciCleaner() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
aciCleaner() called twice? (line 45 and 52)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
oops, I can't read. ignore me
|
That's a nice change to automatically invoke equivalent of two commands ( However I recommend not to merge it as it is, for numerous reasons:
|
Add support to import image from docker-daemon in rkt
Now you can import an image that is available in docker's image store on
a local machine into rkt's imagestore.
To do so run following command:
or directly run it the usual way
This will import the image
docker.io/library/alpineinto rkt's store.