Re: CI does not run tests post merge

2016-01-26 Thread Dan Kenigsberg
On Mon, Jan 25, 2016 at 02:39:24PM +0200, Eyal Edri wrote: > I'm guessing there was a reason to remove/disable them? > Adding Danken. > > Anton can help to re-enable them if needed. We should simply add automation/check_merge to vdsm.git; and we should start by having them as a softlink to

Re: CI does not run tests post merge

2016-01-26 Thread Edward Haas
<bkor...@redhat.com>, "Edward Haas" <edwa...@redhat.com>, "infra" <infra@ovirt.org> | Sent: Wednesday, January 27, 2016 8:31:21 AM | Subject: Re: CI does not run tests post merge | | On Mon, Jan 25, 2016 at 02:39:24PM +0200, Eyal Edri wrote: | > I'm guessin

Re: CI does not run tests post merge

2016-01-26 Thread Barak Korren
> > Done, https://gerrit.ovirt.org/#/c/52779 > > Is there a way to test it before merging it? You can use mock_runner.sh (see "Testing the scripts locally" here: [1]) You will also need to merge this patch to get this actually running in Jenkins: https://gerrit.ovirt.org/#/c/52780/ [1]:

Re: CI does not run tests post merge

2016-01-25 Thread Eyal Edri
I'm guessing there was a reason to remove/disable them? Adding Danken. Anton can help to re-enable them if needed. e. On Mon, Jan 25, 2016 at 11:48 AM, Yaniv Bronheim wrote: > Yes, we never ran check_patch for vdsm > > Can we add back our el7,f22,f21 runs for our current

Re: CI does not run tests post merge

2016-01-25 Thread Eyal Edri
The tests did run, see the last comment on the patch. Though they took some time, probably due to heavy load on jenkins - that usually happens when a feature branch or a massive patch is merged, adding to the jenkins queue. We are working on ways to handle it, but it would be better if the people

Re: CI does not run tests post merge

2016-01-25 Thread Eyal Edri
Jenkins CI Jan 12 12:34 PM ↩ Patch Set 1: Continuous-Integration-1 Build Failed http://jenkins.ovirt.org/job/vdsm_master_check-patch-fc23-x86_64/2089/ : FAILURE Shahar Havivi Patch Set 1: Verified-1 On Mon, Jan 25, 2016 at 10:27 AM, Edward Haas wrote: > On 01/25/2016 10:13

Re: CI does not run tests post merge

2016-01-25 Thread Barak Korren
On 25 January 2016 at 10:27, Edward Haas wrote: > On 01/25/2016 10:13 AM, Eyal Edri wrote: >> The tests did run, see the last comment on the patch. > > I meant the check_patch (it runs the tests) > check_patch runs for patch before merge. check_merged runs after merge. So if

Re: CI does not run tests post merge

2016-01-25 Thread Barak Korren
> > Yes sorry, check_merge is the one. > Was it removed intentionally? > Looks like there aren't check_merged jobs for vdsm, were there? Did anyone made any at some point? -- Barak Korren bkor...@redhat.com RHEV-CI Team ___ Infra mailing list

Re: CI does not run tests post merge

2016-01-25 Thread Yaniv Bronheim
Yes, we never ran check_patch for vdsm Can we add back our el7,f22,f21 runs for our current check_patch? I asked for that few weeks ago, and I think its important to avoid regressions Thanks. On Mon, Jan 25, 2016 at 11:20 AM, Barak Korren wrote: > > > > Yes sorry,

Re: CI does not run tests post merge

2016-01-25 Thread Edward Haas
On 01/25/2016 10:33 AM, Barak Korren wrote: > On 25 January 2016 at 10:27, Edward Haas wrote: >> On 01/25/2016 10:13 AM, Eyal Edri wrote: >>> The tests did run, see the last comment on the patch. >> >> I meant the check_patch (it runs the tests) >> > > check_patch runs for

CI does not run tests post merge

2016-01-24 Thread Edward Haas
Hello, Seems like no tests are running after merging to master. Is it intentionally? Master got broken due to one of the merges. Can we push the tests back in? Thanks, Edy. ___ Infra mailing list Infra@ovirt.org

Re: CI does not run tests post merge

2016-01-24 Thread Edward Haas
On 01/25/2016 09:56 AM, Edward Haas wrote: > Hello, > > Seems like no tests are running after merging to master. > Is it intentionally? > > Master got broken due to one of the merges. > Can we push the tests back in? https://gerrit.ovirt.org/#/c/51704/ > > Thanks, > Edy. >