Re: [PATCH v2 sandbox] virt-sandbox-image: switch to use URI to identify templates

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Sep 21, 2015 at 10:11:48PM +0200, Cedric Bosdonnat wrote:
> On Mon, 2015-09-21 at 15:45 +0100, Daniel P. Berrange wrote:
> > Currently the CLI syntax is somewhat docker specific requiring
> > inclusion of --registry arg to identify the docker download
> > server. Other app containers have a notion of download server,
> > but don't separate it from the template name.
> > 
> > This patch removes that docker-ism by changing to use a URI
> > for identifying the template image. So instead of
> > 
> >   virt-sandbox-image download \
> >       --source docker --registry index.docker.io
> >       --username dan --password 123456 ubuntu:15.04
> > 
> > You can use
> > 
> >   virt-sandbox-image download docker://dan:123456@xxxxxxxxxxxxxxx/ubuntu?tag=15.04
> > 
> > The only mandatory part is the source prefix and image name, so
> > that can shorten to just
> > 
> >   virt-sandbox-image download docker:///ubuntu
> > 
> > to pull down the latest ubuntu image, from the default registry
> > using no authentication.
> > ---
> > 
> > Changed in v2:
> > 
> >  - Rebase against master, instead of (unpushed) docker volume patch
> > 
> >  libvirt-sandbox/image/cli.py                  |  71 +++++--------
> >  libvirt-sandbox/image/sources/DockerSource.py | 142 ++++++++++++++------------
> >  libvirt-sandbox/image/sources/Source.py       |  29 +++---
> >  libvirt-sandbox/image/template.py             | 110 ++++++++++++++++++++
> 
> Missing change in libvirt-sandbox/image/Makefile.am to add template.py.
> As is that file isn't installed.
> 
> I'm also just realizing that we didn't add Eren't commit for the
> virt-sandbox-image man page. Adding it later is fine, but we need to
> keep that on our radar.

Yep, that's in my tree to update & pyush.

> > @@ -151,7 +150,7 @@ def run(args):
> >  
> >  def requires_template(parser):
> >      parser.add_argument("template",
> > -                        help=_("name of the template"))
> > +                        help=_("URI of the template"))
> 
> Shouldn't we provide some examples here? As those URIs can't be invented
> we need to give the user some chances to discover them without having to
> read our code ;)

I wasn't sure this was the best place. We'll certainly put examples in
the man pages though.

> ACK, with the help improvement + Makefile.am fix.

Regards,
Daniel
-- 
|: http://berrange.com      -o-    http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org              -o-             http://virt-manager.org :|
|: http://autobuild.org       -o-         http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org       -o-       http://live.gnome.org/gtk-vnc :|

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list



[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]