From Dan Kenigsberg <dan...@redhat.com>:

Dan Kenigsberg has submitted this change and it was merged.

Change subject: API: move vm parameters fixup in a method
......................................................................


API: move vm parameters fixup in a method

After the VM parameter validation, the API.VM.create
method does some parameter fixing, meaning it adds
back some optional parameters if they are not provided
explicitely.
This patch moves this code on its own submethod.

Change-Id: Ia46a63ac8f321860a5718d1af9c412e537646cc9
Signed-off-by: Francesco Romani <from...@redhat.com>
---
M vdsm/API.py
1 file changed, 15 insertions(+), 11 deletions(-)

Approvals:
  Jenkins CI: Passed CI tests
  Dan Kenigsberg: Verified; Looks good to me, approved



-- 
To view, visit https://gerrit.ovirt.org/66416
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ia46a63ac8f321860a5718d1af9c412e537646cc9
Gerrit-PatchSet: 12
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <from...@redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com>
Gerrit-Reviewer: Francesco Romani <from...@redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: gerrit-hooks <automat...@ovirt.org>
_______________________________________________
vdsm-patches mailing list -- vdsm-patches@lists.fedorahosted.org
To unsubscribe send an email to vdsm-patches-le...@lists.fedorahosted.org

Reply via email to