Signed-off-by: Alex Jia <ajia(a)redhat.com&gt; --- bin/virt-sandbox-service | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/bin/virt-sandbox-service b/bin/virt-sandbox-service index 03873c9..26b4a40 100755 --- a/bin/virt-sandbox-service +++ b/bin/virt-sandbox-service @@ -453,8 +453,12 @@ WantedBy=multi-user.target def get_name(self): if self.config: - return self.config.get_name() - raise ValueError([_("Name not configured")]) + name = self.config.get_name() + if not name: + raise ValueError([_("Name not configured")]) + return name + sys.stderr.write("The configuration %s does not exist\n" % self.config) + sys.exit(1) def set_copy(self, copy): self.copy = copy

On Fri, Aug 09, 2013 at 06:26:47PM +0800, Alex Jia wrote: > Signed-off-by: Alex Jia&lt;ajia(a)redhat.com&gt; > --- > bin/virt-sandbox-service | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/bin/virt-sandbox-service b/bin/virt-sandbox-service > index 26b4a40..cb40f6a 100755 > --- a/bin/virt-sandbox-service > +++ b/bin/virt-sandbox-service > @@ -965,6 +965,9 @@ def upgrade_config(args): > configfile = get_legacy_config_path(args.name) > if os.path.exists(configfile): > upgrade_config_legacy(configfile) > + else: > + sys.stderr.write("No legacy '%s' configuration\n" % args.name) > + sys.exit(1) This isn't desired. This command is intended to be a no-op if nothing needs changing. It is not just about upgrading from this legacy config file layout. In the future I expet us to add more code here as we make further changes. So it is right to silently exit with success here, not report an error.

Daniel