That would explain it. The commit messages were quite long (especially the second one) and a commit with a smaller message has shown up. Thanks for letting me know. I'll try to keep them a bit more reasonable in future.> -----Original Message----- > From: NAKAMURA Takumi [mailto:geek4civic at gmail.com] > Sent: 15 November 2013 16:28 > To: Daniel Sanders > Cc: LLVM Developers Mailing List; Galina Kistanova > Subject: Re: [LLVMdev] Buildbot skipping commits? > > It's a bug in buildbot. Longer (than 1024, I guess) commit message might let > buildbot commits ignored. > When buildbot is configured with mysql, such a check is done. > > FYI, my buildbot tweaks longer commit log, for example; > http://bb.pgr.jp/changes/24995 > (Snipping is done not in the buildmaster, but on the commit feeder) > > That said, I wish every guys would not be made afraid of commit message... > > Galina, FYI, it's the place to check length; master/buildbot/db/base.py > > CREATE TABLE `changes` ( > (snip) > `comments` varchar(1024) NOT NULL, > (snip) > ) ENGINE=MyISAM AUTO_INCREMENT=25011 DEFAULT CHARSET=utf8; > > ...I don't know why this is created as myisam and the column "comments" > would be not "text" but varchar... > > 2013/11/15 Daniel Sanders <Daniel.Sanders at imgtec.com>: > > Hi, > > > > > > > > Neither of my commits so far today have shown up in the buildbot > > console view. They aren't showing up in the list of changes reported > > by the changesource either. > > > > For example: http://lab.llvm.org:8011/changes/45724 shows r194805, > > then > > http://lab.llvm.org:8011/changes/45725 shows r194807. My commit is the > > r194806 that should be between them. > > > > Similarly change 45728 is r194810, and change 45729 is r194812. There > > should be a commit of mine (r194811) between these. > > > > > > > > Does anyone have any idea what's going on here? > > > > > > _______________________________________________ > > LLVM Developers mailing list > > LLVMdev at cs.uiuc.edu http://llvm.cs.uiuc.edu > > http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev > >
Ah, ...no. please don't hesitate to write logs ;) It must be the subject in the buildbot, not in developer guys. Galina, I'll write up later, how to tweak this issue. It is not resolved in released versions, though. The easiest way is to tweak svnpoller. 2013/11/16 Daniel Sanders <Daniel.Sanders at imgtec.com>:> That would explain it. The commit messages were quite long (especially the second one) and a commit with a smaller message has shown up. > > Thanks for letting me know. I'll try to keep them a bit more reasonable in future. > >> -----Original Message----- >> From: NAKAMURA Takumi [mailto:geek4civic at gmail.com] >> Sent: 15 November 2013 16:28 >> To: Daniel Sanders >> Cc: LLVM Developers Mailing List; Galina Kistanova >> Subject: Re: [LLVMdev] Buildbot skipping commits? >> >> It's a bug in buildbot. Longer (than 1024, I guess) commit message might let >> buildbot commits ignored. >> When buildbot is configured with mysql, such a check is done. >> >> FYI, my buildbot tweaks longer commit log, for example; >> http://bb.pgr.jp/changes/24995 >> (Snipping is done not in the buildmaster, but on the commit feeder) >> >> That said, I wish every guys would not be made afraid of commit message... >> >> Galina, FYI, it's the place to check length; master/buildbot/db/base.py >> >> CREATE TABLE `changes` ( >> (snip) >> `comments` varchar(1024) NOT NULL, >> (snip) >> ) ENGINE=MyISAM AUTO_INCREMENT=25011 DEFAULT CHARSET=utf8; >> >> ...I don't know why this is created as myisam and the column "comments" >> would be not "text" but varchar... >> >> 2013/11/15 Daniel Sanders <Daniel.Sanders at imgtec.com>: >> > Hi, >> > >> > >> > >> > Neither of my commits so far today have shown up in the buildbot >> > console view. They aren't showing up in the list of changes reported >> > by the changesource either. >> > >> > For example: http://lab.llvm.org:8011/changes/45724 shows r194805, >> > then >> > http://lab.llvm.org:8011/changes/45725 shows r194807. My commit is the >> > r194806 that should be between them. >> > >> > Similarly change 45728 is r194810, and change 45729 is r194812. There >> > should be a commit of mine (r194811) between these. >> > >> > >> > >> > Does anyone have any idea what's going on here? >> > >> > >> > _______________________________________________ >> > LLVM Developers mailing list >> > LLVMdev at cs.uiuc.edu http://llvm.cs.uiuc.edu >> > http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev >> > >
Don't worry. I'm not going to stop writing good (and occasionally long) commit messages. I'm only saying that my 2.5KB commit message might have been a little bit excessive. I agree that any length limits imposed by buildbot shouldn't prevent it from testing the commit.> -----Original Message----- > From: NAKAMURA Takumi [mailto:geek4civic at gmail.com] > Sent: 15 November 2013 17:03 > To: Daniel Sanders > Cc: LLVM Developers Mailing List; Galina Kistanova > Subject: Re: [LLVMdev] Buildbot skipping commits? > > Ah, ...no. please don't hesitate to write logs ;) It must be the subject in the > buildbot, not in developer guys. > > Galina, I'll write up later, how to tweak this issue. It is not resolved in > released versions, though. > The easiest way is to tweak svnpoller. > > 2013/11/16 Daniel Sanders <Daniel.Sanders at imgtec.com>: > > That would explain it. The commit messages were quite long (especially the > second one) and a commit with a smaller message has shown up. > > > > Thanks for letting me know. I'll try to keep them a bit more reasonable in > future. > > > >> -----Original Message----- > >> From: NAKAMURA Takumi [mailto:geek4civic at gmail.com] > >> Sent: 15 November 2013 16:28 > >> To: Daniel Sanders > >> Cc: LLVM Developers Mailing List; Galina Kistanova > >> Subject: Re: [LLVMdev] Buildbot skipping commits? > >> > >> It's a bug in buildbot. Longer (than 1024, I guess) commit message > >> might let buildbot commits ignored. > >> When buildbot is configured with mysql, such a check is done. > >> > >> FYI, my buildbot tweaks longer commit log, for example; > >> http://bb.pgr.jp/changes/24995 > >> (Snipping is done not in the buildmaster, but on the commit feeder) > >> > >> That said, I wish every guys would not be made afraid of commit > message... > >> > >> Galina, FYI, it's the place to check length; > >> master/buildbot/db/base.py > >> > >> CREATE TABLE `changes` ( > >> (snip) > >> `comments` varchar(1024) NOT NULL, > >> (snip) > >> ) ENGINE=MyISAM AUTO_INCREMENT=25011 DEFAULT CHARSET=utf8; > >> > >> ...I don't know why this is created as myisam and the column "comments" > >> would be not "text" but varchar... > >> > >> 2013/11/15 Daniel Sanders <Daniel.Sanders at imgtec.com>: > >> > Hi, > >> > > >> > > >> > > >> > Neither of my commits so far today have shown up in the buildbot > >> > console view. They aren't showing up in the list of changes > >> > reported by the changesource either. > >> > > >> > For example: http://lab.llvm.org:8011/changes/45724 shows r194805, > >> > then > >> > http://lab.llvm.org:8011/changes/45725 shows r194807. My commit is > >> > the > >> > r194806 that should be between them. > >> > > >> > Similarly change 45728 is r194810, and change 45729 is r194812. > >> > There should be a commit of mine (r194811) between these. > >> > > >> > > >> > > >> > Does anyone have any idea what's going on here? > >> > > >> > > >> > _______________________________________________ > >> > LLVM Developers mailing list > >> > LLVMdev at cs.uiuc.edu http://llvm.cs.uiuc.edu > >> > http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev > >> > > >
Hi Takumi, Thanks! I also have a fix for the issue staging. Should go to the production soon. Thanks Galina On Fri, Nov 15, 2013 at 9:02 AM, NAKAMURA Takumi <geek4civic at gmail.com>wrote:> Ah, ...no. please don't hesitate to write logs ;) It must be the > subject in the buildbot, not in developer guys. > > Galina, I'll write up later, how to tweak this issue. It is not > resolved in released versions, though. > The easiest way is to tweak svnpoller. > > 2013/11/16 Daniel Sanders <Daniel.Sanders at imgtec.com>: > > That would explain it. The commit messages were quite long (especially > the second one) and a commit with a smaller message has shown up. > > > > Thanks for letting me know. I'll try to keep them a bit more reasonable > in future. > > > >> -----Original Message----- > >> From: NAKAMURA Takumi [mailto:geek4civic at gmail.com] > >> Sent: 15 November 2013 16:28 > >> To: Daniel Sanders > >> Cc: LLVM Developers Mailing List; Galina Kistanova > >> Subject: Re: [LLVMdev] Buildbot skipping commits? > >> > >> It's a bug in buildbot. Longer (than 1024, I guess) commit message > might let > >> buildbot commits ignored. > >> When buildbot is configured with mysql, such a check is done. > >> > >> FYI, my buildbot tweaks longer commit log, for example; > >> http://bb.pgr.jp/changes/24995 > >> (Snipping is done not in the buildmaster, but on the commit feeder) > >> > >> That said, I wish every guys would not be made afraid of commit > message... > >> > >> Galina, FYI, it's the place to check length; master/buildbot/db/base.py > >> > >> CREATE TABLE `changes` ( > >> (snip) > >> `comments` varchar(1024) NOT NULL, > >> (snip) > >> ) ENGINE=MyISAM AUTO_INCREMENT=25011 DEFAULT CHARSET=utf8; > >> > >> ...I don't know why this is created as myisam and the column "comments" > >> would be not "text" but varchar... > >> > >> 2013/11/15 Daniel Sanders <Daniel.Sanders at imgtec.com>: > >> > Hi, > >> > > >> > > >> > > >> > Neither of my commits so far today have shown up in the buildbot > >> > console view. They aren't showing up in the list of changes reported > >> > by the changesource either. > >> > > >> > For example: http://lab.llvm.org:8011/changes/45724 shows r194805, > >> > then > >> > http://lab.llvm.org:8011/changes/45725 shows r194807. My commit is > the > >> > r194806 that should be between them. > >> > > >> > Similarly change 45728 is r194810, and change 45729 is r194812. There > >> > should be a commit of mine (r194811) between these. > >> > > >> > > >> > > >> > Does anyone have any idea what's going on here? > >> > > >> > > >> > _______________________________________________ > >> > LLVM Developers mailing list > >> > LLVMdev at cs.uiuc.edu http://llvm.cs.uiuc.edu > >> > http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev > >> > > > >-------------- next part -------------- An HTML attachment was scrubbed... URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20131118/1971dfec/attachment.html>