Bug 765558 (GLUSTER-3826) - enable parallel builds
Summary: enable parallel builds
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: GLUSTER-3826
Product: GlusterFS
Classification: Community
Component: build
Version: mainline
Hardware: x86_64
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Amar Tumballi
QA Contact: Sachidananda Urs
URL:
Whiteboard:
Depends On:
Blocks: 817967
TreeView+ depends on / blocked
 
Reported: 2011-11-23 19:56 UTC by Kaleb KEITHLEY
Modified: 2013-12-19 00:07 UTC (History)
2 users (show)

Fixed In Version: glusterfs-3.4.0
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2013-07-24 17:59:30 UTC
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Embargoed:


Attachments (Terms of Use)
tweak the libglusterfs/src/Makefile.* to avoid race in parallel makes (873 bytes, text/plain)
2011-11-23 16:58 UTC, Kaleb KEITHLEY
no flags Details


Links
System ID Private Priority Status Summary Last Updated
Red Hat Bugzilla 756510 0 low CLOSED can't use parallel make when building 2021-02-22 00:41:40 UTC

Description Kaleb KEITHLEY 2011-11-23 16:58:17 UTC
Created attachment 721 [details]
My XF86Config file

Comment 1 Kaleb KEITHLEY 2011-11-23 17:45:19 UTC
just to be clear——

The attached patch patches both the Makefile.in and Makefile.am because it's a patch against the sources in the glusterfs-3.2.5.tgz distributed by Gluster. The fedora build only runs configure which only uses the Makefile.in. The patch patches the Makefile.am for the situation where someone runs automake before running configure. Hope that makes sense

Comment 2 Kaleb KEITHLEY 2011-11-23 19:56:57 UTC
Description of problem:
When using parallel make there's a race condition between running yacc on
graph.y and compiling the resulting y.tab.c.

Version-Release number of selected component (if applicable):
All versions, but 3.2.5 in particular.

How reproducible:
run a `fedpkg build` or a `koji --scratch` build. The epel build seems to
encounter the race condition more than the rawhide or f16 builds

Steps to Reproduce:
1. See above

See also https://bugzilla.redhat.com/show_bug.cgi?id=756510

Comment 3 Amar Tumballi 2011-11-24 03:22:04 UTC
Kelab,

Can you go ahead and submit just the Makefile.am patch to gerrit. When we make a release out of git, the Makefile.in will be auto generated and added in tar.gz anyways.

If fedora build of glusterfs-3.2.5 is maintained separately, then you can go ahead and submit this patch there, but for glusterfs, lets get it to upstream (master branch) first, and then also send a patch to release-3.2 branch.

Regards,
Amar

Comment 4 Anand Avati 2011-11-30 09:57:12 UTC
CHANGE: http://review.gluster.com/763 (builds of glusterfs in the fedora build system often fail due to a) merged in master by Anand Avati (avati)

Comment 5 Kaleb KEITHLEY 2011-12-01 15:33:03 UTC
Jeff Darcy found that the previous fix still manifested the race condition.

I have submitted a revised fix to gerrit http://review.gluster.com/765

Comment 6 Anand Avati 2011-12-02 04:19:35 UTC
CHANGE: http://review.gluster.com/765 (builds of glusterfs in the fedora build system often fail due to a) merged in master by Anand Avati (avati)

Comment 7 Kaleb KEITHLEY 2011-12-13 12:37:16 UTC
I'm not sure why comment #1 and attachment 721 [details] are XF86Config files. Looks like something was borken in the bugzilla migration.

And I thought this was closed already.

Comment 8 Amar Tumballi 2011-12-20 06:39:26 UTC
Lets close once QA verifies this is working.

Comment 9 Kaleb KEITHLEY 2011-12-21 13:03:45 UTC
Agreed we should close once QA verifies the official gluster build.

https://bugzilla.redhat.com/show_bug.cgi?id=756510 is the BZ for the canonical f16 build, and it's already been auto-closed by Koji.

Comment 10 Sachidananda Urs 2012-05-31 07:31:39 UTC
Not present in release 3.3.0, will verify later.

Comment 11 Kaleb KEITHLEY 2012-06-18 15:28:18 UTC
I don't see that this has been addressed in any branch of glusterfs.

I still have to patch the libglusterfs/src/Makefile.am (and Makefile.in) to build in koji for fedora for all releases of 3.2.x and 3.3.0


Note You need to log in before you can comment on or make changes to this bug.