openstack/karbor
Code Issues Proposed changes

Fix implicit reason for 'provider_id'.

Browse Source 
The reasons are the same while checking 'provider_id' of
operation_definition. But they are two different cases.
So fix it in this patch.

Change-Id: Id92f1b16d8dc59975c7f2e8d1fefe3f6f58558ec
Closes-Bug: #1716226
This commit is contained in:
ericxiett 2017-09-11 14:54:25 +08:00
parent e9160d22ce
commit 73058d0316
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
karbor/services/operationengine/operations/protect_operation.py
View File

@ -38,7 +38,7 @@ class ProtectOperation(base.Operation):
plan = objects.Plan.get_by_id(context.get_admin_context(), plan_id)
if provider_id != plan.provider_id:
reason = _("Provider_id is invalid")
reason = _("Provider_id is conflict")
raise exception.InvalidOperationDefinition(reason=reason)
def _execute(self, operation_definition, param):