From Dan Kenigsberg danken@redhat.com:
Dan Kenigsberg has submitted this change and it was merged.
Change subject: API: replace validateMinimalKeySet ......................................................................
API: replace validateMinimalKeySet
Now that we have proper exception support, we can replace this utility function with new smaller code, achieving the same goal with less boilerplate.
We took this chance to bootstrap the 'validate' module, which will hold any validation function we may need in the future.
Change-Id: Ibc88ea8ec405974cc022be358f1ecb64242505b8 Signed-off-by: Francesco Romani fromani@redhat.com --- M lib/vdsm/common/exception.py A lib/vdsm/common/validate.py M lib/vdsm/utils.py M tests/Makefile.am A tests/validate_test.py M vdsm/API.py 6 files changed, 92 insertions(+), 65 deletions(-)
Approvals: Piotr Kliczewski: Looks good to me, approved Jenkins CI: Passed CI tests Dan Kenigsberg: Looks good to me, approved Francesco Romani: Verified
vdsm-patches@lists.fedorahosted.org