Dan Kenigsberg has submitted this change and it was merged.

Change subject: virt: Make Vm.devSpecMapFromConf() private
......................................................................


virt: Make Vm.devSpecMapFromConf() private

There is currently no reason to expose the method outside Vm.
Actually we are going to move device processing out of Vm and thus
isolating the use of this method is a preparation step for that.

Change-Id: I6dffb82bde715453ff4ae3eae3b82b4890f04f8b
Signed-off-by: Milan Zamazal <mzama...@redhat.com>
Reviewed-on: https://gerrit.ovirt.org/53483
Reviewed-by: Francesco Romani <from...@redhat.com>
Continuous-Integration: Jenkins CI
Reviewed-by: Nir Soffer <nsof...@redhat.com>
---
M tests/deviceTests.py
M tests/vmTests.py
M tests/vmfakelib.py
M vdsm/virt/vm.py
4 files changed, 11 insertions(+), 11 deletions(-)

Approvals:
  Nir Soffer: Looks good to me, but someone else must approve
  Jenkins CI: Passed CI tests
  Francesco Romani: Looks good to me, approved
  Milan Zamazal: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I6dffb82bde715453ff4ae3eae3b82b4890f04f8b
Gerrit-PatchSet: 8
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Milan Zamazal <mzama...@redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com>
Gerrit-Reviewer: Francesco Romani <from...@redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Polednik <mpoled...@redhat.com>
Gerrit-Reviewer: Milan Zamazal <mzama...@redhat.com>
Gerrit-Reviewer: Nir Soffer <nsof...@redhat.com>
Gerrit-Reviewer: gerrit-hooks <automat...@ovirt.org>
_______________________________________________
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to