Errors / warnings in ovirt-engine-nodejs-modules build log
Oved Ourfali
oourfali at redhat.com
Tue Aug 2 06:17:13 UTC 2016
Eyal - see Juan's comment in his patch:
Juan Hernandez
12:07 AM
↩
Patch Set 4:
This is going to fail as well, the old .src.rpm file exists outside of the
chroot where the build.sh script runs, so that script can't remove it. We
definitively need help from the CI team to clean that slave.
Can you clean it?
Thanks,
Oved
On Tue, Aug 2, 2016 at 9:06 AM, Sandro Bonazzola <sbonazzo at redhat.com>
wrote:
>
>
> On Mon, Aug 1, 2016 at 9:56 PM, Vojtech Szocs <vszocs at redhat.com> wrote:
>
>>
>>
>> ----- Original Message -----
>> > From: "Juan Hernández" <jhernand at redhat.com>
>> > To: "Eyal Edri" <eedri at redhat.com>, "Vojtech Szocs" <vszocs at redhat.com>,
>> "Anton Marchukov" <amarchuk at redhat.com>
>> > Cc: "infra" <infra at ovirt.org>
>> > Sent: Monday, August 1, 2016 9:30:23 PM
>> > Subject: Re: Errors / warnings in ovirt-engine-nodejs-modules build log
>> >
>> > On 08/01/2016 09:01 PM, Eyal Edri wrote:
>> > > Anton, can you sit with Vojtech tomorrow and see if you can help?
>> > >
>> > > Vojtech
>> > > Maybe we're hitting: http://rpm.org/ticket/862
>> > > https://bugzilla.redhat.com/show_bug.cgi?id=913099
>> > >
>> > > Anyway, what I meant was to try and run it on el7 slaves, not el7
>> mock,
>> > > to see if its a regression on mock running on fc24.
>> > > trying it here to see if it
>> > > works:
>> > >
>> http://jenkins.ovirt.org/job/ovirt-engine-nodejs-modules_master_create-rpms-el7-x86_64_created/22/console
>> > >
>> >
>> > As this seems to be a RPM limitation, and one that isn't going to be
>> > fixed soon, I'd suggest to change the Node.js modules RPM so that it
>> > packages a single .tar file containing all the modules. It can then,
>> > during installation (in the %post) section extract the content to
>> > /var/lib/ovirt-engine-nodejs-modules, so users only have to change the
>> > directory they use. Actually, as Vojtech introduced a "setup-env.sh"
>> > script that does all the preparations, only this script needs to be
>> > changed. This patch reflects my suggestion:
>> >
>> > Package tarball containing Node.js modules
>> > https://gerrit.ovirt.org/61790
>>
>> +1 this is great!
>>
>> Proposed solution doesn't require any change in Dashboard code.
>>
>> Seems like it was only a matter of time till we hit that ~100k
>> files limit when creating ovirt-engine-nodejs-modules RPM.
>>
>
> You may also consider spliitng the rpm in subpackages in a per-module
> fashion.
>
>
>
>>
>> Juan++
>>
>> >
>> > > On Mon, Aug 1, 2016 at 9:42 PM, Vojtech Szocs <vszocs at redhat.com
>> > > <mailto:vszocs at redhat.com>> wrote:
>> > >
>> > >
>> > >
>> > > ----- Original Message -----
>> > > > From: "Eyal Edri" <eedri at redhat.com <mailto:eedri at redhat.com>>
>> > > > To: "Vojtech Szocs" <vszocs at redhat.com <mailto:
>> vszocs at redhat.com>>
>> > > > Cc: "infra" <infra at ovirt.org <mailto:infra at ovirt.org>>
>> > > > Sent: Monday, August 1, 2016 8:25:50 PM
>> > > > Subject: Re: Errors / warnings in ovirt-engine-nodejs-modules
>> build
>> > > > log
>> > > >
>> > > > Does it work on el7?
>> > >
>> > > It fails on el7 as well:
>> > >
>> > >
>> http://jenkins.ovirt.org/job/ovirt-engine-nodejs-modules_master_create-rpms-el7-x86_64_created/21/console
>> > >
>> > > error: Unable to create immutable header region.
>> > >
>> > > >
>> > > > On Mon, Aug 1, 2016 at 8:21 PM, Vojtech Szocs <
>> vszocs at redhat.com
>> > > <mailto:vszocs at redhat.com>> wrote:
>> > > >
>> > > > > Forwarding to infra, TL;DR we seem to have an issue with
>> rpmbuild
>> > > > > (see below) and I'm not sure how to fix that, is there anyone
>> who
>> > > > > faced such issue in past?
>> > > > >
>> > > > > Vojtech
>> > > > >
>> > > > >
>> > > > > ----- Forwarded Message -----
>> > > > > From: "Vojtech Szocs" <vszocs at redhat.com
>> > > > > <mailto:vszocs at redhat.com>>
>> > > > > To: "Sandro Bonazzola" <sbonazzo at redhat.com
>> > > <mailto:sbonazzo at redhat.com>>
>> > > > > Cc: "Oved Ourfali" <oourfali at redhat.com
>> > > <mailto:oourfali at redhat.com>>, "Alexander Wels" <
>> > > > > awels at redhat.com <mailto:awels at redhat.com>>, "Greg Sheremeta"
>> > > <gshereme at redhat.com <mailto:gshereme at redhat.com>>, "Juan
>> > > > > Hernández" <jhernand at redhat.com <mailto:jhernand at redhat.com
>> >>,
>> > > "Ryan Barry" <rbarry at redhat.com <mailto:rbarry at redhat.com>>
>> > > > > Sent: Monday, August 1, 2016 6:49:08 PM
>> > > > > Subject: Errors / warnings in ovirt-engine-nodejs-modules
>> build log
>>
>> > > > >
>> > > > > Hi Sandro,
>> > > > >
>> > > > > I've looked into the build log [1].
>> > > > >
>> > > > > Adding Juan & Ryan as well. Your feedback is highly
>> appreciated.
>> > > > >
>> > > > > npm-specific issues (don't block the build):
>> > > > >
>> > > > > npm WARN package.json dependencies@ No repository field.
>> > > > > npm WARN package.json dependencies@ No license field.
>> > > > > - these warnings are harmless
>> > > > > - TODO update modules' package.json
>> > > > >
>> > > > > npm WARN deprecated ...
>> > > > > - some (possibly transitive) dependency in package.json
>> relies
>> > > > > on a deprecated npm package
>> > > > > - TODO find out which dependencies are causing this
>> > > > >
>> > > > > npm ERR! registry error parsing json
>> > > > > - this might indicate corrupt npm cache
>> > > > > - should be fixed by `npm cache clean`
>> > > > > - TODO update modules' build.sh
>> > > > >
>> > > > > npm WARN optional dep failed, continuing ...
>> > > > > - these warnings are harmless
>> > > > > - some (possibly transitive) dependency in package.json
>> relies
>> > > > > on an optional npm package that is platform-specific but
>> it
>> > > > > is NOT available for current platform
>> > > > > - e.g. "fsevents" is MacOSX only
>> > > > > - this should be fixed by `npm install --no-optional`
>> > > > > - TODO update modules' build.sh
>> > > > >
>> > > > > sh: bower: command not found
>> > > > > npm WARN optional dep failed, continuing
>> bootstrap-treeview at 1.2.0
>> > > > > - bootstrap-treeview does `bower install` in its `install`
>> script
>> > > > > - this is BAD practice (bootstrap-treeview's fault)
>> > > > > - TODO latest commit on May 9, 2015 -- do we need this at
>> all (?)
>> > > > >
>> > > > > CI-specific issues (which block the build):
>> > > > >
>> > > > > Wrote:
>> > > > >
>> > >
>> /tmp/ovirt-engine-nodejs-modules/ovirt-engine-nodejs-modules-0.0.10-1.fc24.src.rpm
>> > > > > error: Unable to create immutable header region.
>> > > > > - at this point, spec's %install phase has finished
>> executing
>> > > > > and RPM was created
>> > > > > - it is rpmbuild related, see
>> > > > > https://bugzilla.redhat.com/show_bug.cgi?id=913099#c2
>> > > > >
>> > > > > For ^^ error, seems like too many files in RPM package will
>> cause
>> > > > > "artificial cap on header size" to be exceeded and rpmbuild
>> fails.
>> > > > >
>> > > > > Juan, what do you think? I don't have any idea how to solve
>> this.
>> > > > >
>> > > > > I don't understand the actual issue as we're creating single
>> .tar
>> > > > > file from `node_modules` directory during the RPM build..
>> > > > >
>> > > > > [1]
>> > > > >
>> > >
>> http://jenkins.ovirt.org/job/ovirt-engine-nodejs-modules_master_create-rpms-fc24-x86_64_created/4/console
>> > > > >
>> > > > > Thanks,
>> > > > > Vojtech
>> > > > > _______________________________________________
>> > > > > Infra mailing list
>> > > > > Infra at ovirt.org <mailto:Infra at ovirt.org>
>> > > > > 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)
>> > > >
>> > >
>> > >
>> > >
>> > >
>> > > --
>> > > 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)
>> > >
>> > >
>> > > _______________________________________________
>> > > Infra mailing list
>> > > Infra at ovirt.org
>> > > http://lists.ovirt.org/mailman/listinfo/infra
>> > >
>> >
>> >
>> > --
>> > 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 at ovirt.org
>> http://lists.ovirt.org/mailman/listinfo/infra
>>
>
>
>
> --
> Sandro Bonazzola
> Better technology. Faster innovation. Powered by community collaboration.
> See how it works at redhat.com
>
> _______________________________________________
> Infra mailing list
> Infra at ovirt.org
> http://lists.ovirt.org/mailman/listinfo/infra
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.ovirt.org/pipermail/infra/attachments/20160802/d87387aa/attachment.html>
More information about the Infra
mailing list