update jobs to include hystrix from EPEL

Description

oVirt 4.1 requires hystrix yet this package is filtered in jobs which is causing tests to fail:
http://jenkins.ovirt.org/job/ovirt-engine_master_upgrade-from-4.0_el7_merged/378/

http://jenkins.ovirt.org/job/ovirt-engine_master_upgrade-from-4.0_el7_merged/378/artifact/logs/_home_jenkins_workspace_ovirt-engine_master_upgrade-from-4.0_el7_merged_tmp_repo_rpmbuild.log.tgz

error: Failed build dependencies:
hystrix-core is needed by ovirt-engine-4.1.0-0.0.master.20160629135203.gitf1f59e0.el7.centos.noarch
hystrix-metrics-event-stream is needed by ovirt-engine-4.1.0-0.0.master.20160629135203.gitf1f59e0.el7.centos.noarch
Installing /home/jenkins/workspace/ovirt-engine_master_upgrade-from-4.0_el7_merged/tmp_repo/ovirt-engine-4.1.0-0.0.master.20160629135203.el7.centos.src.rpm

Activity

Show:

Barak Korren September 22, 2016 at 7:38 AM

Note: moving the build part to mock might not be as easy as I initially thought because IIRC the engine build_artifacts.sh script builds all GWT permutations which are hardly needed for this job. We might need to commit some stand-alone quick build script to the engine repo, and that may take some time.

So if the above is true, just moving to the experimental flow might be a faster approach.

Former user September 22, 2016 at 7:26 AM

can we move the build part to mock to get rid of this dependencies on hosts?

Former user September 19, 2016 at 8:46 AM

Thanks for following up on this - this still has to be puppetized as we've been adding el7 slaves recently and these packages may end up missing.

Here's the list of packages I installed via fabric:
hystrix-core archaius-core rxjava jctools hystrix-metrics-event-stream

Can't recall where I got this list of packages (likely from some job definition) but I'll add them as a new class like "ovirt_jenkins_slave::engine_builder" if you don't mind. Another option, as proposed by Barak, is to change the job to build inside mock and install the package there instead - sounds much better to me but I'm not that familiar with Jenkins jobs and will need some help if we go that way.

Eyal Edri July 28, 2016 at 6:59 AM

is there an action item on this via puppet or we can close it?

Eyal Edri July 12, 2016 at 2:27 PM

was this resolved or we are still waiting to add it to puppet?

Eyal Edri July 4, 2016 at 11:18 AM

please close this ticket or convert it to adding this as puppet code.

Former user June 30, 2016 at 2:07 PM

As a quick and dirty test I installed the RPM using a fabric task to EL7 VMs and now monitoring the latest build:
http://jenkins.ovirt.org/job/ovirt-engine_master_upgrade-from-4.0_el7_merged/398/console
It doesn't seem to fail immediately after trying to build the RPM so I assume we're good and the build is happening.

Barak Korren June 30, 2016 at 10:30 AM

If it is indeed a missing build dep (not sure hysterix should really be needed during the build process), then maybe the best way to deal with this once and for all is just to run the build part of the job inside mock.
(A quick hack to do this would be to copy the building part of check_patch.sh to a temporary script and run it with mock_runner.sh. A more stable solution would be to then submit the temp script is a patch to engine. We can also use build_articafts.sh but I think that does a slow full-permutation build)

Former user June 30, 2016 at 10:02 AM

Nadav Goldin June 30, 2016 at 9:47 AM

Might be as this is a none standard-ci job, it assumes the packages are already installed, if this is indeed the case, it is controlled by:
infra-puppet.git/site/ovirt_jenkins_slave/manifests/engine_test_runner.pp
adding it to there will cause the hystrix packages to be installed on all slaves.. wonder if we can avoid that though..

Fixed

Details

Assignee

Reporter

Priority

Created June 30, 2016 at 9:28 AM
Updated February 27, 2017 at 8:50 AM
Resolved December 8, 2016 at 3:06 PM