> - return status
> + logger.error("------ FAILED: Invalid InstanceID Key
Value.------")
> It's better to return FAIL after logger.error above, otherwise, we
have to return
status instead of PASS below.
Others looks fine for me.
Oops! Yes, this should be a "return status" not a "return FAIL".
Thanks for catching this! I'll resubmit =)
--
Kaitlin Rupert
IBM Linux Technology Center
kaitlin(a)linux.vnet.ibm.com