On Wed, Jul 13, 2016 at 10:44:54AM +0300, Nir Soffer wrote:
On Wed, Jul 13, 2016 at 10:08 AM, Eyal Edri <eedri@redhat.com> wrote:
It might be a good idea to add an install test to check-patch.sh so we'll avoid such errors in the future...
We have such test - if you modified the spec, we build and install rpms.
From check-patch.sh:
# In case of vdsm specfile or any Makefile.am file modification in commit, # try to build and install all new created packages if git diff-tree --no-commit-id --name-only -r HEAD | egrep --quiet 'vdsm.spec.in|Makefile.am' ; then ./automation/build-artifacts.sh yum -y install "$EXPORT_DIR/"!(*.src).rpm fi
On Wed, Jul 13, 2016 at 10:04 AM, Sandro Bonazzola <sbonazzo@redhat.com> wrote:
DEBUG util.py:417: Error: Package: vdsm-4.18.999-232.git6e84643.el7.centos.x86_64 (ovirt-master-snapshot) DEBUG util.py:417: Requires: safelease >= 1.0-7 DEBUG util.py:417: Available: safelease-1.0-5.el7.x86_64 (centos-epel) DEBUG util.py:417: safelease = 1.0-5.el7
And specifically, https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2016-277e8cf30d says that safelease >= 1.0-7 was pushed to stable 4 months ago, 2016-03-24 15:32:12 (UTC). So the probalem is in a specific rpm mirror, probably.