On 11/23/2016 09:30 AM, Eyal Edri wrote:
On Wed, Nov 23, 2016 at 10:26 AM, Juan Hernández <jhernand(a)redhat.com
<mailto:jhernand@redhat.com>> wrote:
On 11/23/2016 09:16 AM, Eyal Edri wrote:
> Maybe worth adding the build artifacts to check-patch.sh so you'll see
> it before merge?
>
The check-patch.sh script already runs the build of the artifacts. The
problem is that in this case the "bug" was in the automation
build-artifacts.sh script itself, and it only runs when the patch is
merged. The reason for running the build-artifacts.sh script only when
the patch is merged is that it runs a heavy process: generation of
documentation using Publican. I didn't want to do that for every patch,
as it consumes a lot of resources and doesn't add anything in terms of
patch verification.
Gating will help with that, we're working on it now, you can be a beta
tested for the api project if you want :)
So basically you'll have a flag in gerrit to trigger the pre-merge jobs
and if the job will fail the patch won't merge.
Looks interesting. What do I need to do to become beta tester?
Actually I am currently using the build-artifacts.sh script where I
should probably use the check-merged.sh script. I think I need to
reorganize the jobs and the automation scripts to make this clearer. I
am working on that.
> On Wed, Nov 23, 2016 at 10:10 AM, Juan Hernández <jhernand(a)redhat.com
<mailto:jhernand@redhat.com>
> <mailto:jhernand@redhat.com <mailto:jhernand@redhat.com>>>
wrote:
>
> On 11/23/2016 08:29 AM, Sandro Bonazzola wrote:
> >
>
http://jenkins.ovirt.org/job/ovirt-engine-api-model_4.0_build-artifacts-f...
<
http://jenkins.ovirt.org/job/ovirt-engine-api-model_4.0_build-artifacts-f...
>
<
http://jenkins.ovirt.org/job/ovirt-engine-api-model_4.0_build-artifacts-f...
<
http://jenkins.ovirt.org/job/ovirt-engine-api-model_4.0_build-artifacts-f...
> >
> > *
> > *
> >
> > *
> > *
> >
> > *00:04:35.993* + mv target/model.json exported-artifacts
> > *00:04:35.993* mv: cannot stat 'target/model.json': No such
file
> or directory
> > *00:04:35.993* Took 93 seconds
> >
> >
> > Can you please have a look?
> >
>
>
> That should be addressed by the following patch:
>
> Fix automation scripts after removing 'describe' profile
>
https://gerrit.ovirt.org/67208
>
> --
> Dirección Comercial: C/Jose Bardasano Baos, 9, Edif. Gorbea 3, planta
> 3ºD, 28016 Madrid, Spain
> Inscrita en el Reg. Mercantil de Madrid – C.I.F. B82657941 - Red Hat
> S.L.
> _______________________________________________
> Infra mailing list
> Infra(a)ovirt.org <mailto:Infra@ovirt.org>
<mailto:Infra@ovirt.org <mailto:Infra@ovirt.org>>
>
http://lists.ovirt.org/mailman/listinfo/infra
<
http://lists.ovirt.org/mailman/listinfo/infra>
> <
http://lists.ovirt.org/mailman/listinfo/infra
<
http://lists.ovirt.org/mailman/listinfo/infra>>
>
>
>
>
>
> --
> Eyal Edri
> Associate Manager
> RHV DevOps
> EMEA ENG Virtualization R&D
> Red Hat Israel
>
> phone: +972-9-7692018 <tel:%2B972-9-7692018>
> irc: eedri (on #tlv #rhev-dev #rhev-integ)
--
Dirección Comercial: C/Jose Bardasano Baos, 9, Edif. Gorbea 3, planta
3ºD, 28016 Madrid, Spain
Inscrita en el Reg. Mercantil de Madrid – C.I.F. B82657941 - Red Hat
S.L.
--
Eyal Edri
Associate Manager
RHV DevOps
EMEA ENG Virtualization R&D
Red Hat Israel
phone: +972-9-7692018
irc: eedri (on #tlv #rhev-dev #rhev-integ)
--
Dirección Comercial: C/Jose Bardasano Baos, 9, Edif. Gorbea 3, planta
3ºD, 28016 Madrid, Spain
Inscrita en el Reg. Mercantil de Madrid – C.I.F. B82657941 - Red Hat S.L.