Nir Soffer has uploaded a new change for review.
Change subject: multipath: Remove unused constant ......................................................................
multipath: Remove unused constant
We don't rotate files for couple of month now, so there is no need to keep this constant.
Change-Id: I08bd8ae2f860fcbdf3be45cb372ae44f9bd4b589 Signed-off-by: Nir Soffer nsoffer@redhat.com --- M lib/vdsm/tool/configurators/multipath.py 1 file changed, 0 insertions(+), 2 deletions(-)
git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/60/44860/1
diff --git a/lib/vdsm/tool/configurators/multipath.py b/lib/vdsm/tool/configurators/multipath.py index bced531..5ad0840 100644 --- a/lib/vdsm/tool/configurators/multipath.py +++ b/lib/vdsm/tool/configurators/multipath.py @@ -88,8 +88,6 @@
""" % {"current_tag": _MPATH_CONF_TAG}
-_MAX_CONF_COPIES = 5 - # conf file configured by vdsm should contain a tag # in form of "RHEV REVISION X.Y" _OLD_TAGS = ["# RHAT REVISION 0.2", "# RHEV REVISION 0.3",
automation@ovirt.org has posted comments on this change.
Change subject: multipath: Remove unused constant ......................................................................
Patch Set 1:
* Update tracker::IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url. * Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
Nir Soffer has posted comments on this change.
Change subject: multipath: Remove unused constant ......................................................................
Patch Set 1: Verified+1
Dan Kenigsberg has posted comments on this change.
Change subject: multipath: Remove unused constant ......................................................................
Patch Set 1: Code-Review+2
Dan Kenigsberg has submitted this change and it was merged.
Change subject: multipath: Remove unused constant ......................................................................
multipath: Remove unused constant
We don't rotate files for couple of month now, so there is no need to keep this constant.
Change-Id: I08bd8ae2f860fcbdf3be45cb372ae44f9bd4b589 Signed-off-by: Nir Soffer nsoffer@redhat.com Reviewed-on: https://gerrit.ovirt.org/44860 Continuous-Integration: Jenkins CI Reviewed-by: Dan Kenigsberg danken@redhat.com --- M lib/vdsm/tool/configurators/multipath.py 1 file changed, 0 insertions(+), 2 deletions(-)
Approvals: Nir Soffer: Verified Jenkins CI: Passed CI tests Dan Kenigsberg: Looks good to me, approved
automation@ovirt.org has posted comments on this change.
Change subject: multipath: Remove unused constant ......................................................................
Patch Set 2:
* Update tracker::IGNORE, no Bug-Url found * Set MODIFIED::IGNORE, no Bug-Url found.
vdsm-patches@lists.fedorahosted.org