Errors / warnings in ovirt-engine-nodejs-modules build log
Sandro Bonazzola
sbonazzo at redhat.com
Tue Aug 2 06:32:41 UTC 2016
On Tue, Aug 2, 2016 at 8:17 AM, Oved Ourfali <oourfali at redhat.com> wrote:
> 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?
>
Done
>
> 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
>>
>>
>
--
Sandro Bonazzola
Better technology. Faster innovation. Powered by community collaboration.
See how it works at redhat.com
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.ovirt.org/pipermail/infra/attachments/20160802/2a8dd0d8/attachment.html>
More information about the Infra
mailing list