Skip to content

Commit

Permalink
virsh_nodedev_create_destroy: Use TestNAError
Browse files Browse the repository at this point in the history
If the host doesn't support vHBA's or as no available ports,
then return TestNAError(SKIP) rather than TestFail(FAIL).
  • Loading branch information
jferlan committed Dec 2, 2013
1 parent 514ab22 commit 815f63c
Showing 1 changed file with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -238,7 +238,7 @@ def run_virsh_nodedev_create_destroy(test, params, env):

# No HBA or no vHBA supporting
if not vport_ops_list:
raise error.TestFail("No HBAs to support vHBA on the host!")
raise error.TestNAError("No HBAs to support vHBA on the host!")

# Check ports connectivity
port_state_dict = check_port_connectivity(vport_ops_list)
Expand All @@ -249,7 +249,7 @@ def run_virsh_nodedev_create_destroy(test, params, env):

# No online port is available
if not port_online_list:
raise error.TestFail("No port is active!")
raise error.TestNAError("No port is active!")

if dev_name:
# Negative testing for creating device
Expand Down

0 comments on commit 815f63c

Please sign in to comment.