Change in vdsm[master]: misc: remove cp parameter
nsoffer at redhat.com
nsoffer at redhat.com
Thu Oct 8 20:47:33 UTC 2015
Nir Soffer has posted comments on this change.
Change subject: misc: remove cp parameter
......................................................................
Patch Set 2: Code-Review-1
(1 comment)
https://gerrit.ovirt.org/#/c/45613/2/vdsm/storage/misc.py
File vdsm/storage/misc.py:
Line 448: raise se.InvalidParameterException(name, number)
Line 449: return n
Line 450:
Line 451:
Line 452: def rotateFiles(directory, prefixName, gen, persist=False):
This function does not work and its tests are wrong. We also cannot use this code as it is extremely slow on node, and it is used for backing up domain metadata in /var/log/vdsm/backup/
Please remove this function.
Line 453: log.debug("dir: %s, prefixName: %s, versions: %s" %
Line 454: (directory, prefixName, gen))
Line 455: gen = int(gen)
Line 456: files = os.listdir(directory)
--
To view, visit https://gerrit.ovirt.org/45613
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I0906bfd7dfa128c323aa399810bbd75883618434
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yeela Kaplan <ykaplan at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Yeela Kaplan <ykaplan at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-HasComments: Yes
More information about the vdsm-patches
mailing list