rohityadavcloud commented on PR #7084: URL: https://github.com/apache/cloudstack/pull/7084#issuecomment-1382001450
@DaanHoogland only if you insist, I prefer to set cmd to the default case (and not null; just the style to having less branching read better to me). I think it may not need additional testing; what we're doing is to enable the case when Ubuntu based KVM hosts. I've done testing of the pkg name and dpkg -l output manually to verify that it returns non-zero exit code when the pkg is not installed. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
