Re: [Development] Moving to Gerrit 2.16.9
On 2.7.2019 15.13, Kari Oikarinen wrote: > > On 2.7.2019 14.59, Cristian Adam wrote: >>> -Original Message- >>> From: Development On Behalf Of >>> Frederik Gladhorn >>> Sent: Thursday, 27 June 2019 16:11 >>> To: Qt Project Development Mailing-List >>> Subject: [Development] Moving to Gerrit 2.16.9 >>> >>> Hi, >>> >>> Just to keep the ball rolling, we prepared the upgrade to move from Gerrit >>> 2.16.7 to 2.16.9. I don't expect any real changes, but it's a good exercise >>> for us >>> to stay up to date and see if the scripting of the upgrade works. >>> The only real challenge was upgrading Bazel, since every Gerrit version >>> seems to only compile with one exact Bazel version (roughly). >>> >>> So far it seems to work nicely for me, running the script which stops >>> gerrit, >>> pushes the new release and restarts it takes around two minutes, so there >>> won't be any significant downtime this time around. >>> >>> Assuming there are no big concerns I'll just do the upgrade tomorrow, on the >>> test instance it works without problems. >>> >>> On a related note, now that things are generally working with the new >>> Gerrit, >>> I was wondering if we want to consider plugins. There is one to add >>> reviewers based on git blame https://gerrit- >>> review.googlesource.com/admin/repos/plugins/reviewers-by-blame >>> and I'll give Gravatar a spin: >>> https://gerrit-review.googlesource.com/admin/repos/plugins/avatars- >>> gravatar >>> >>> We should also consider the various webhooks plugins. Comments >>> appreciated. >>> >>> Cheers, >>> Frederik >>> >> >> Hi, >> >> I'm getting from Qt Sanity Bot for >> https://codereview.qt-project.org/c/qt-creator/qt-creator/+/266791 >> >> The following reply every 5 minutes: >> >> Worker 'sanity' produced an unreasonable amount of output. You should >> ask the bot maintainers for advice. >> >> The current behavior is a bit loopy, can we do something about it? > > I guess you've managed to get sanity into a situation where it doesn't give a > sanity score at all. > > Then once the bot looks at the commits it hasn't scored yet, it will repeat > that > again and again. This isn't actually sufficient to cause the issue, the bot remembers what commits it has looked at during current run and doesn't look at them again. So the new attempts only happen when sanity bot is started again and it looks at all open changes it hasn't reviewed yet. But looks like for some reason sanity bot's connection (with which it receives notifications about new patchsets) drops every 5 minutes and causes the bot to quit. When it is automatically restarted, it then tries to review again because of missing review. I manually put a negative review in place, so at least the spam should be over for this specific commit for now. > Could you run sanity bot locally and see what is happening to it? There is a > git > hook in qtrepotools to do it when you commit. > -- Kari ___ Development mailing list Development@qt-project.org https://lists.qt-project.org/listinfo/development
Re: [Development] Moving to Gerrit 2.16.9
On 2.7.2019 14.59, Cristian Adam wrote: >> -Original Message- >> From: Development On Behalf Of >> Frederik Gladhorn >> Sent: Thursday, 27 June 2019 16:11 >> To: Qt Project Development Mailing-List >> Subject: [Development] Moving to Gerrit 2.16.9 >> >> Hi, >> >> Just to keep the ball rolling, we prepared the upgrade to move from Gerrit >> 2.16.7 to 2.16.9. I don't expect any real changes, but it's a good exercise >> for us >> to stay up to date and see if the scripting of the upgrade works. >> The only real challenge was upgrading Bazel, since every Gerrit version >> seems to only compile with one exact Bazel version (roughly). >> >> So far it seems to work nicely for me, running the script which stops gerrit, >> pushes the new release and restarts it takes around two minutes, so there >> won't be any significant downtime this time around. >> >> Assuming there are no big concerns I'll just do the upgrade tomorrow, on the >> test instance it works without problems. >> >> On a related note, now that things are generally working with the new Gerrit, >> I was wondering if we want to consider plugins. There is one to add >> reviewers based on git blame https://gerrit- >> review.googlesource.com/admin/repos/plugins/reviewers-by-blame >> and I'll give Gravatar a spin: >> https://gerrit-review.googlesource.com/admin/repos/plugins/avatars- >> gravatar >> >> We should also consider the various webhooks plugins. Comments >> appreciated. >> >> Cheers, >> Frederik >> > > Hi, > > I'm getting from Qt Sanity Bot for > https://codereview.qt-project.org/c/qt-creator/qt-creator/+/266791 > > The following reply every 5 minutes: > > Worker 'sanity' produced an unreasonable amount of output. You should > ask the bot maintainers for advice. > > The current behavior is a bit loopy, can we do something about it? I guess you've managed to get sanity into a situation where it doesn't give a sanity score at all. Then once the bot looks at the commits it hasn't scored yet, it will repeat that again and again. Could you run sanity bot locally and see what is happening to it? There is a git hook in qtrepotools to do it when you commit. -- Kari ___ Development mailing list Development@qt-project.org https://lists.qt-project.org/listinfo/development
Re: [Development] Moving to Gerrit 2.16.9
> -Original Message- > From: Development On Behalf Of > Frederik Gladhorn > Sent: Thursday, 27 June 2019 16:11 > To: Qt Project Development Mailing-List > Subject: [Development] Moving to Gerrit 2.16.9 > > Hi, > > Just to keep the ball rolling, we prepared the upgrade to move from Gerrit > 2.16.7 to 2.16.9. I don't expect any real changes, but it's a good exercise > for us > to stay up to date and see if the scripting of the upgrade works. > The only real challenge was upgrading Bazel, since every Gerrit version > seems to only compile with one exact Bazel version (roughly). > > So far it seems to work nicely for me, running the script which stops gerrit, > pushes the new release and restarts it takes around two minutes, so there > won't be any significant downtime this time around. > > Assuming there are no big concerns I'll just do the upgrade tomorrow, on the > test instance it works without problems. > > On a related note, now that things are generally working with the new Gerrit, > I was wondering if we want to consider plugins. There is one to add > reviewers based on git blame https://gerrit- > review.googlesource.com/admin/repos/plugins/reviewers-by-blame > and I'll give Gravatar a spin: > https://gerrit-review.googlesource.com/admin/repos/plugins/avatars- > gravatar > > We should also consider the various webhooks plugins. Comments > appreciated. > > Cheers, > Frederik > Hi, I'm getting from Qt Sanity Bot for https://codereview.qt-project.org/c/qt-creator/qt-creator/+/266791 The following reply every 5 minutes: Worker 'sanity' produced an unreasonable amount of output. You should ask the bot maintainers for advice. The current behavior is a bit loopy, can we do something about it? Cheers, Cristian. ___ Development mailing list Development@qt-project.org https://lists.qt-project.org/listinfo/development
Re: [Development] Moving to Gerrit 2.16.9
On mandag 1. juli 2019 21:23:35 CEST André Hartmann wrote: > Hi all, > > I'm not sure if it's related to the gravatars, but Gerrit became > horrible slow here (this is on a 2MBit DSL line). It's almost unuseable. > > Has someone else experienced the same? Yes, it slowed to a crawl yesterday, we're trying to find out why, even though that's always a bit tricky post mortem. We do have some records of the resource usage and the logs, but the logs seem rather innocent from what I could tell yesterday. Cheers, Frederik > > Regards, > André > > Am 01.07.19 um 19:57 schrieb André Pönitz: > > On Mon, Jul 01, 2019 at 02:43:43PM +, Frederik Gladhorn wrote: > and I'll give Gravatar a spin: > https://gerrit-review.googlesource.com/admin/repos/plugins/avatars-grav > ata > r > >> > >> It's there, enjoy and put your avatar up at https://gravatar.com . > > > > I know it's a bit late and it won't change anything anyway. > > > > Still: Was there an explanation of the benefits of using gravatar > > somewhere, also perhaps in the light of discussions like > > > > https://meta.stackexchange.com/questions/4553/can-we-use-non-gravatar-avat > > ars/5658#5658 > > > > ? > > > > Andre' > > ___ > > Development mailing list > > Development@qt-project.org > > https://lists.qt-project.org/listinfo/development ___ Development mailing list Development@qt-project.org https://lists.qt-project.org/listinfo/development