From a2385daddb890aae049a49869aea5ae7e90ce46f Mon Sep 17 00:00:00 2001 From: Martin Korbel Date: Tue, 27 Feb 2018 10:58:46 +0100 Subject: [PATCH 1/2] Better error messages in snapshot --- vminute/vminute.py | 12 ++++++++---- 1 file changed, 8 insertions(+), 4 deletions(-) diff --git a/vminute/vminute.py b/vminute/vminute.py index 327f49d..5af4b5e 100755 --- a/vminute/vminute.py +++ b/vminute/vminute.py @@ -788,9 +788,13 @@ def __parse_params(self, opts, argv): elif key in ('--metadata', '-m'): if params['metadata'] is None: params['metadata'] = {} - for it in val.split(';'): - col, val = it.split('=') - params['metadata'][col] = val.replace(',', ' ') + try: + for it in val.split(';'): + col, val = it.split('=') + params['metadata'][col] = val.replace(',', ' ') + except Exception: + die("The problem with parsing of metadata parameter. " + "Please try 5minute snapshot --help.") elif key in ('--umount', '-u'): params['umount'] = True elif key in ('--volume'): @@ -801,7 +805,7 @@ def __parse_params(self, opts, argv): params['delete'] = True else: die("Bad parameter '%s'. Please try 5minute " - "screenshot --help." % key) + "snapshot --help." % key) if 'delete' in params: if len(argv) != 1: die("Name of the snapshot is ambiguous or empty.") From 6687163b6fc91cbcc74ea3dfcda0c8c40bfef18c Mon Sep 17 00:00:00 2001 From: Martin Korbel Date: Tue, 27 Feb 2018 10:59:50 +0100 Subject: [PATCH 2/2] Fix spec file --- 5minute.spec | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/5minute.spec b/5minute.spec index 9393169..1bf9d10 100644 --- a/5minute.spec +++ b/5minute.spec @@ -1,5 +1,5 @@ Name: 5minute -Version: 0.2.27 +Version: 0.2.28 Release: 1%{?dist} Summary: Command line tool for getting instance from OpenStack