Re: Change in ovirt-engine[master]: core: mask fencing alerts when disabled in cluster

2016-05-30 Thread Martin Perina
On Mon, May 30, 2016 at 9:35 AM, Eyal Edri wrote: > both check-patch and check-merge are using the following rpmbuild params: > > rpmbuild \ > -D "_rpmdir $PWD/output" \ > -D "_topmdir $PWD/rpmbuild" \ > -D "release_suffix ${SUFFIX}" \ > -D "ovirt_build_ut

Re: Change in ovirt-engine[master]: core: mask fencing alerts when disabled in cluster

2016-05-30 Thread Eyal Edri
both check-patch and check-merge are using the following rpmbuild params: rpmbuild \ -D "_rpmdir $PWD/output" \ -D "_topmdir $PWD/rpmbuild" \ -D "release_suffix ${SUFFIX}" \ -D "ovirt_build_ut $BUILD_UT" \ -D "ovirt_build_extra_flags $EXTRA_BUILD_FLAGS" \ -D

Re: Change in ovirt-engine[master]: core: mask fencing alerts when disabled in cluster

2016-05-30 Thread Barak Korren
Adding Rgolan because he is discussing this in a different thread. On 30 May 2016 at 10:25, Martin Perina wrote: > > > On Mon, May 30, 2016 at 9:05 AM, Barak Korren wrote: >> >> On 30 May 2016 at 09:51, Martin Perina wrote: >> > Hi,

Re: Change in ovirt-engine[master]: core: mask fencing alerts when disabled in cluster

2016-05-30 Thread Martin Perina
On Mon, May 30, 2016 at 9:05 AM, Barak Korren wrote: > On 30 May 2016 at 09:51, Martin Perina wrote: > > Hi, > > > > I just verified that local build is is broken by the same error in > > HostedEngineDeployConfiguration. > > I will post a fix for that,

Re: Change in ovirt-engine[master]: core: mask fencing alerts when disabled in cluster

2016-05-30 Thread Barak Korren
On 30 May 2016 at 09:51, Martin Perina wrote: > Hi, > > I just verified that local build is is broken by the same error in > HostedEngineDeployConfiguration. > I will post a fix for that, but upgrade-job build is correct (raised build > failure) and check-merged jobs > are

Re: Change in ovirt-engine[master]: core: mask fencing alerts when disabled in cluster

2016-05-30 Thread Martin Perina
Hi, I just verified that local build is is broken by the same error in HostedEngineDeployConfiguration. I will post a fix for that, but upgrade-job build is correct (raised build failure) and check-merged jobs are broken (producing success although build is broken), could you please fix them

Re: Change in ovirt-engine[master]: core: mask fencing alerts when disabled in cluster

2016-05-30 Thread Martin Perina
Hi, I'm missing something here, how can it fail on engine build when both build jobs for the patch were successful? http://jenkins.ovirt.org/job/ovirt-engine_master_check-merged-el7-x86_64/419/ http://jenkins.ovirt.org/job/ovirt-engine_master_check-merged-fc23-x86_64/419/ Martin On Mon, May

Re: Change in ovirt-engine[master]: core: mask fencing alerts when disabled in cluster

2016-05-30 Thread Barak Korren
Hi, It is failing on engine build: [INFO] Compiling module org.ovirt.engine.ui.webadmin.WebAdmin [INFO]Validating units: [INFO] [ERROR] Errors in

Re: Change in ovirt-engine[master]: core: mask fencing alerts when disabled in cluster

2016-05-30 Thread Martin Perina
Hi, could you please take a look why upgrade-from-master job failed? Thanks Martin On Mon, May 30, 2016 at 8:12 AM, Jenkins CI wrote: > Jenkins CI has posted comments on this change. > > Change subject: core: mask fencing alerts when disabled in cluster >