Commit 72db3fd7 authored by Iustin Pop's avatar Iustin Pop
Browse files

Two more fixes for OS params and opcode defaults



If the OS is not using API v20, the parameter verification should be
entirely skipped.

The second change is a simple typo.
Signed-off-by: default avatarIustin Pop <iustin@google.com>
Reviewed-by: default avatarGuido Trotter <ultrotter@google.com>
parent 40684c3a
...@@ -2521,6 +2521,9 @@ def ValidateOS(required, osname, checks, osparams): ...@@ -2521,6 +2521,9 @@ def ValidateOS(required, osname, checks, osparams):
else: else:
return False return False
if max(tbv.api_versions) < constants.OS_API_V20:
return True
if constants.OS_VALIDATE_PARAMETERS in checks: if constants.OS_VALIDATE_PARAMETERS in checks:
_CheckOSPList(tbv, osparams.keys()) _CheckOSPList(tbv, osparams.keys())
......
...@@ -2523,7 +2523,7 @@ class LUSetClusterParams(LogicalUnit): ...@@ -2523,7 +2523,7 @@ class LUSetClusterParams(LogicalUnit):
("add_uids", None), ("add_uids", None),
("remove_uids", None), ("remove_uids", None),
("hvparams", None), ("hvparams", None),
("ov_hvp", None), ("os_hvp", None),
] ]
REQ_BGL = False REQ_BGL = False
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment