From: Royce Lv <lvroyce@linux.vnet.ibm.com> Update vm disk path with a non-existent path will raise 500 error, we need to raise invalid parameter error instead. Signed-off-by: Royce Lv <lvroyce@linux.vnet.ibm.com> --- src/kimchi/model/vmstorages.py | 3 +-- tests/test_model.py | 2 +- 2 files changed, 2 insertions(+), 3 deletions(-) diff --git a/src/kimchi/model/vmstorages.py b/src/kimchi/model/vmstorages.py index 1597d49..6f03e75 100644 --- a/src/kimchi/model/vmstorages.py +++ b/src/kimchi/model/vmstorages.py @@ -217,5 +217,4 @@ class VMStorageModel(object): stgModel.create(vm_name, backup_params) except: pass - - raise OperationFailed("KCHCDROM0009E", {'error': e.message}) + raise diff --git a/tests/test_model.py b/tests/test_model.py index 859f7bd..9ec7027 100644 --- a/tests/test_model.py +++ b/tests/test_model.py @@ -205,7 +205,7 @@ class ModelTests(unittest.TestCase): # update path of existing cd with # non existent iso - self.assertRaises(OperationFailed, inst.vmstorage_update, + self.assertRaises(InvalidParameter, inst.vmstorage_update, vm_name, cdrom_dev, {'path': wrong_iso_path}) # update path of existing cd with -- 1.8.1.2