From Yaniv Bronhaim <ybronhei(a)redhat.com>:
Yaniv Bronhaim has posted comments on this change.
Change subject: spec: make sure infra package is deleted on upgrade
......................................................................
Patch Set 2:
(1 comment)
https://gerrit.ovirt.org/#/c/78357/2/vdsm.spec.in
File vdsm.spec.in:
Line 119: Requires: safelease >= 1.0-7
Line 120: Requires: mom >= 0.5.8
Line 121: Requires(pre): shadow-utils
Line 122:
Line 123: Obsoletes: %{name}-infra <= %{version}-%{release}
why would we have vdsm-infra in 4.19? isn't <=4.18 enough?
so seems like < 4.19 is the right thing to do iiuc.. I prefer to be explicit in
the spec. since 4.19 you shouldn't see vdsm-infra
Line 124:
Line 125: %if 0%{?with_openvswitch}
Line 126: Requires: openvswitch >= 2.0.0
Line 127: %endif
--
To view, visit
https://gerrit.ovirt.org/78357
To unsubscribe, visit
https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I8c975bffe713b716dfa9a8bc1bbee6d9a24fb144
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Irit Goihman <igoihman(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Irit Goihman <igoihman(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
Gerrit-HasComments: Yes