Re: [sandbox PATCH v4 10/21] Image: Add get_command function to Source

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

 



On Wed, Sep 09, 2015 at 09:43:35AM +0200, Cedric Bosdonnat wrote:
> On Tue, 2015-09-08 at 17:21 +0100, Daniel P. Berrange wrote:
> > On Fri, Aug 28, 2015 at 01:47:38PM +0000, Eren Yagdiran wrote:
> > > Provide a way to know how a template can be started depending on the used source
> > > DockerSource will need to parse the topmost config file in order to find the igniter command
> > > ---
> > >  virt-sandbox-image/sources/DockerSource.py | 14 ++++++++++++++
> > >  virt-sandbox-image/sources/Source.py       |  4 ++++
> > >  2 files changed, 18 insertions(+)
> > > 
> > > diff --git a/virt-sandbox-image/sources/DockerSource.py b/virt-sandbox-image/sources/DockerSource.py
> > > index 760ba6c..3e0362b 100644
> > > --- a/virt-sandbox-image/sources/DockerSource.py
> > > +++ b/virt-sandbox-image/sources/DockerSource.py
> > > @@ -29,6 +29,15 @@ import os
> > >  import subprocess
> > >  import shutil
> > >  
> > > +class DockerConfParser():
> > > +
> > > +    def __init__(self,jsonfile):
> > > +        with open(jsonfile) as json_file:
> > > +            self.json_data = json.load(json_file)
> > > +    def getRunCommand(self):
> > > +        cmd = self.json_data['container_config']['Cmd'][2]
> > > +        return cmd[cmd.index('"') + 1:cmd.rindex('"')]
> > 
> > In testing, I found out we should be using 'config' here
> > not 'container_config', since the latter refers to the
> > configuration used when the image was built, which is
> > subtly different from the config that is intended to be
> > used when in it deployed.
> > 
> > In particular the 'Cmd' field does not need this string
> > munging when we use 'config' instead of 'container_config'.
> > 
> > In researching this, I found out that docker also has an
> > optional 'Entrypoints' array and if both Entrypoints and
> > Cmd are specified, we should concatenate them. If the
> > user provides command line args, then should replace
> > the 'Cmd' list, but not the 'Entrypoints' list. Finally
> > some images don'pt define any 'Cmd' or 'Entrypoints' at
> > all, in which case it seems we should just try using
> > /bin/sh, in the absence of any user supplied command.
> 
> Do we really want to have a default to /bin/sh when docker is giving an
> error to the user?

Yeah, I'm not sure really.

> Error response from daemon: No command specified

I guess the error is nicer in the csae where the user  does
not have a shell inside their image

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]