chain-build broken with the lastest code

[Nathan Blackham]

I just updated to the latest code in the repo and it appears that
chain-builds are now broken.  When I have specified more than one URL to
build before doing a "waitrepo" it will only build the first one and then
move on to the wait repo step without trying to build the rest of the
packages specified.

Anyone else seen this?  Any ideas on where it might be broken?

Nathan

[Mike Bonnet]

On 05/11/2010 07:28 PM, Nathan Blackham wrote:
>  I just updated to the latest code in the repo and it appears that
>  chain-builds are now broken.  When I have specified more than one URL to
>  build before doing a "waitrepo" it will only build the first one and then
>  move on to the wait repo step without trying to build the rest of the
>  packages specified.
>  
>  Anyone else seen this?  Any ideas on where it might be broken? 
The chain-build code hasn't changed in a while.  What's the exact
command you're running?

[Nathan Blackham]

I run something akin to :

koji chain-build rhel-5-epel cvs://URL/packageA#HEAD cvs://URL/packageB#HEAD
: cvs://URL/packageC#HEAD.

It will build packageA and then do a wait repo and try to build packageC.
 It works fine with the waitrepo but it won't do the parallel builds.  I
don't see any errors either.  It was working when I pulled the code on March
23 ish, but the lastest code doesn't build the parallel builds.

Nathan

On Wed, May 12, 2010 at 6:33 AM, Mike Bonnet <mikeb(a)redhat.com&gt; wrote:

>  On 05/11/2010 07:28 PM, Nathan Blackham wrote:
>  > I just updated to the latest code in the repo and it appears that
>  > chain-builds are now broken.  When I have specified more than one URL to
>  > build before doing a "waitrepo" it will only build the first one and then
>  > move on to the wait repo step without trying to build the rest of the
>  > packages specified.
>  >
>  > Anyone else seen this?  Any ideas on where it might be broken?
>
>  The chain-build code hasn't changed in a while.  What's the exact
>  command you're running?
>  --
>  buildsys mailing list
>  buildsys(a)lists.fedoraproject.org
>  https://admin.fedoraproject.org/mailman/listinfo/buildsys
>

[Nathan Blackham]

So we dug into the changes a bit and it looks like the change was last year.
 I thought I had the head of the git tree, but it looks like I had code
previous.  It appears that the change on 2009-08-05 caused the issue.  It
changes line 1702.  It changes the label from the SCMUrl to "build #" where
the number is the level in the build system.

This is not unique when you are running parallel builds in each level.  So
it submits the first URL in the parallel build but when it tries to submit
the next one, it thinks that it already exists.

Nathan



On Wed, May 12, 2010 at 10:06 AM, Nathan Blackham <kemotaha(a)gmail.com&gt;wrote:

>  I run something akin to :
>
>  koji chain-build rhel-5-epel cvs://URL/packageA#HEAD
>  cvs://URL/packageB#HEAD : cvs://URL/packageC#HEAD.
>
>  It will build packageA and then do a wait repo and try to build packageC.
>   It works fine with the waitrepo but it won't do the parallel builds.  I
>  don't see any errors either.  It was working when I pulled the code on March
>  23 ish, but the lastest code doesn't build the parallel builds.
>
>  Nathan
>
>
>  On Wed, May 12, 2010 at 6:33 AM, Mike Bonnet <mikeb(a)redhat.com&gt; wrote:
>
> > On 05/11/2010 07:28 PM, Nathan Blackham wrote:
> > > I just updated to the latest code in the repo and it appears that
> > > chain-builds are now broken.  When I have specified more than one URL to
> > > build before doing a "waitrepo" it will only build the first one and
> > then
> > > move on to the wait repo step without trying to build the rest of the
> > > packages specified.
> > >
> > > Anyone else seen this?  Any ideas on where it might be broken?
> >
> > The chain-build code hasn't changed in a while.  What's the exact
> > command you're running?
> > --
> > buildsys mailing list
> > buildsys(a)lists.fedoraproject.org
> > https://admin.fedoraproject.org/mailman/listinfo/buildsys
> >
>
>

[Mike McLean]

On 05/12/2010 03:03 PM, Nathan Blackham wrote:
>  So we dug into the changes a bit and it looks like the change was
> last year.
>    I thought I had the head of the git tree, but it looks like I had code
>  previous.  It appears that the change on 2009-08-05 caused the issue.  It
>  changes line 1702.  It changes the label from the SCMUrl to "build #" where
>  the number is the level in the build system.
>
>  This is not unique when you are running parallel builds in each level.  So
>  it submits the first URL in the parallel build but when it tries to submit
>  the next one, it thinks that it already exists. 
Sorry about that. Will get a fix out by tomorrow.

[Nathan Blackham]

No worries.  Thanks,

Nathan

On Wed, May 12, 2010 at 2:20 PM, Mike McLean <mikem(a)redhat.com&gt; wrote:

>  On 05/12/2010 03:03 PM, Nathan Blackham wrote:
>  > So we dug into the changes a bit and it looks like the change was last
>  year.
>  >   I thought I had the head of the git tree, but it looks like I had code
>  > previous.  It appears that the change on 2009-08-05 caused the issue.  It
>  > changes line 1702.  It changes the label from the SCMUrl to "build #"
>  where
>  > the number is the level in the build system.
>  >
>  > This is not unique when you are running parallel builds in each level.
>   So
>  > it submits the first URL in the parallel build but when it tries to
>  submit
>  > the next one, it thinks that it already exists.
>
>  Sorry about that. Will get a fix out by tomorrow.
>
>  --
>  buildsys mailing list
>  buildsys(a)lists.fedoraproject.org
>  https://admin.fedoraproject.org/mailman/listinfo/buildsys
>