Dan Kenigsberg has submitted this change and it was merged.
Change subject: Replace unused getFileList() with more flexible getFileStats()
......................................................................
Replace unused getFileList() with more flexible getFileStats()
1. Rename to getFileStats().
New name getFileStats() better describes the essence of that API.
This is safe to rename it because this API is not in use.
2. Add caseSensitive option.
Currently Engine is using getIsoList() and getFloppyList() APIs to
retrieve the list of iso and floppy files respectively. Both these APIs
are working in case-insensitive manner but don't provide file
statistics. getFileStats() returns file statistics. Adding
caseSensitive option allows using getFileStats() instead of
getIsoList() and getFloppyList().
Change-Id: I550827c7b4c7e11fe09e41745fcc9d91249c6c23
Bug-Url:
https://bugzilla.redhat.com/1005889
Signed-off-by: Sergey Gotliv <sgotliv(a)redhat.com>
Reviewed-on:
http://gerrit.ovirt.org/20476
Reviewed-by: Federico Simoncelli <fsimonce(a)redhat.com>
Reviewed-by: Ayal Baron <abaron(a)redhat.com>
Reviewed-by: Nir Soffer <nsoffer(a)redhat.com>
Reviewed-by: Dan Kenigsberg <danken(a)redhat.com>
---
M client/vdsClient.py
M vdsm/API.py
M vdsm/BindingXMLRPC.py
M vdsm/storage/hsm.py
M vdsm/storage/storage_exception.py
M vdsm_api/Bridge.py
M vdsm_api/vdsmapi-schema.json
7 files changed, 38 insertions(+), 30 deletions(-)
Approvals:
Ayal Baron: Looks good to me, approved
Nir Soffer: Looks good to me, but someone else must approve
Federico Simoncelli: Looks good to me, but someone else must approve
Sergey Gotliv: Verified
Dan Kenigsberg: Looks good to me, but someone else must approve
--
To view, visit
http://gerrit.ovirt.org/20476
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I550827c7b4c7e11fe09e41745fcc9d91249c6c23
Gerrit-PatchSet: 7
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Sergey Gotliv <sgotliv(a)redhat.com>
Gerrit-Reviewer: Ayal Baron <abaron(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce(a)redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer(a)redhat.com>
Gerrit-Reviewer: Sergey Gotliv <sgotliv(a)redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server