Re: [openstack-dev] [mistral] Clearing out old gerrit reviews
Awesome! We need to do it periodically :) It’ll be easier to sort out patches. Thanks Renat Akhmerov @Nokia On 6 Aug 2018, 16:24 +0700, Dougal Matthews , wrote: > > > > On 3 August 2018 at 10:45, Dougal Matthews wrote: > > > > On 9 July 2018 at 16:13, Dougal Matthews wrote: > > > > > Hey folks, > > > > > > > > > > I'd like to propose that we start abandoning old Gerrit reviews. > > > > > > > > > > This report shows how stale and out of date some of the reviews are: > > > > > http://stackalytics.com/report/reviews/mistral-group/open > > > > > > > > > > I would like to initially abandon anything without any activity for a > > > > > year, but we might want to consider a shorter limit - maybe 6 months. > > > > > Reviews can be restored, so the risk is low. > > > > > > > > > > What do you think? Any objections or counter suggestions? > > > > > > > > > > If I don't hear any complaints, I'll go ahead with this next week (or > > > > > maybe the following week). > > > > > > > > That time line was ambitious. I didn't get started :-) > > > > > > > > However, I did decide it would be best to formalise this plan > > > > somewhere. So I quickly wrote up the plan in a Mistral policy spec. If > > > > we can agree there and merge it, then I'll go ahead and start the > > > > cleanup. > > > > > > > > https://review.openstack.org/#/c/588492/ > > > > The spec merged today, so I did a first pass and abandoned 24 reviews that > > met the criteria. > > > > It will be interesting to see if any of them are restored. > > > > Dougal > > > > > > > > > > > > > > > > > > > > > > Cheers, > > > > > Dougal > > > > > __ > OpenStack Development Mailing List (not for usage questions) > Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe > http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev __ OpenStack Development Mailing List (not for usage questions) Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
Re: [openstack-dev] [mistral] Clearing out old gerrit reviews
On 3 August 2018 at 10:45, Dougal Matthews wrote: > On 9 July 2018 at 16:13, Dougal Matthews wrote: > >> Hey folks, >> >> I'd like to propose that we start abandoning old Gerrit reviews. >> >> This report shows how stale and out of date some of the reviews are: >> http://stackalytics.com/report/reviews/mistral-group/open >> >> I would like to initially abandon anything without any activity for a >> year, but we might want to consider a shorter limit - maybe 6 months. >> Reviews can be restored, so the risk is low. >> >> What do you think? Any objections or counter suggestions? >> >> If I don't hear any complaints, I'll go ahead with this next week (or >> maybe the following week). >> > > That time line was ambitious. I didn't get started :-) > > However, I did decide it would be best to formalise this plan somewhere. > So I quickly wrote up the plan in a Mistral policy spec. If we can agree > there and merge it, then I'll go ahead and start the cleanup. > > https://review.openstack.org/#/c/588492/ > The spec merged today, so I did a first pass and abandoned 24 reviews that met the criteria. It will be interesting to see if any of them are restored. Dougal > > > >> >> Cheers, >> Dougal >> > > __ OpenStack Development Mailing List (not for usage questions) Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
Re: [openstack-dev] [mistral] Clearing out old gerrit reviews
Same. On 3 August 2018 at 13:15, Renat Akhmerov wrote: > Dougal, the policy looks good for me. I gave it the second +2 but didn’t > approve yet so that others could also review (e.g. Adriano and Vitalii). > > Thanks > > Renat Akhmerov > @Nokia > On 3 Aug 2018, 16:46 +0700, Dougal Matthews , wrote: > > On 9 July 2018 at 16:13, Dougal Matthews wrote: > >> Hey folks, >> >> I'd like to propose that we start abandoning old Gerrit reviews. >> >> This report shows how stale and out of date some of the reviews are: >> http://stackalytics.com/report/reviews/mistral-group/open >> >> I would like to initially abandon anything without any activity for a >> year, but we might want to consider a shorter limit - maybe 6 months. >> Reviews can be restored, so the risk is low. >> >> What do you think? Any objections or counter suggestions? >> >> If I don't hear any complaints, I'll go ahead with this next week (or >> maybe the following week). >> > > That time line was ambitious. I didn't get started :-) > > However, I did decide it would be best to formalise this plan somewhere. > So I quickly wrote up the plan in a Mistral policy spec. If we can agree > there and merge it, then I'll go ahead and start the cleanup. > > https://review.openstack.org/#/c/588492/ > > > >> >> Cheers, >> Dougal >> > > __ > OpenStack Development Mailing List (not for usage questions) > Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe > http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev > > > __ > OpenStack Development Mailing List (not for usage questions) > Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe > http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev > > __ OpenStack Development Mailing List (not for usage questions) Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
Re: [openstack-dev] [mistral] Clearing out old gerrit reviews
Dougal, the policy looks good for me. I gave it the second +2 but didn’t approve yet so that others could also review (e.g. Adriano and Vitalii). Thanks Renat Akhmerov @Nokia On 3 Aug 2018, 16:46 +0700, Dougal Matthews , wrote: > > On 9 July 2018 at 16:13, Dougal Matthews wrote: > > > Hey folks, > > > > > > I'd like to propose that we start abandoning old Gerrit reviews. > > > > > > This report shows how stale and out of date some of the reviews are: > > > http://stackalytics.com/report/reviews/mistral-group/open > > > > > > I would like to initially abandon anything without any activity for a > > > year, but we might want to consider a shorter limit - maybe 6 months. > > > Reviews can be restored, so the risk is low. > > > > > > What do you think? Any objections or counter suggestions? > > > > > > If I don't hear any complaints, I'll go ahead with this next week (or > > > maybe the following week). > > > > That time line was ambitious. I didn't get started :-) > > > > However, I did decide it would be best to formalise this plan somewhere. So > > I quickly wrote up the plan in a Mistral policy spec. If we can agree there > > and merge it, then I'll go ahead and start the cleanup. > > > > https://review.openstack.org/#/c/588492/ > > > > > > > > > > Cheers, > > > Dougal > > __ > OpenStack Development Mailing List (not for usage questions) > Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe > http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev __ OpenStack Development Mailing List (not for usage questions) Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
Re: [openstack-dev] [mistral] Clearing out old gerrit reviews
On 9 July 2018 at 16:13, Dougal Matthews wrote: > Hey folks, > > I'd like to propose that we start abandoning old Gerrit reviews. > > This report shows how stale and out of date some of the reviews are: > http://stackalytics.com/report/reviews/mistral-group/open > > I would like to initially abandon anything without any activity for a > year, but we might want to consider a shorter limit - maybe 6 months. > Reviews can be restored, so the risk is low. > > What do you think? Any objections or counter suggestions? > > If I don't hear any complaints, I'll go ahead with this next week (or > maybe the following week). > That time line was ambitious. I didn't get started :-) However, I did decide it would be best to formalise this plan somewhere. So I quickly wrote up the plan in a Mistral policy spec. If we can agree there and merge it, then I'll go ahead and start the cleanup. https://review.openstack.org/#/c/588492/ > > Cheers, > Dougal > __ OpenStack Development Mailing List (not for usage questions) Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
Re: [openstack-dev] [mistral] Clearing out old gerrit reviews
Agreed. On 10 July 2018 at 05:42, Renat Akhmerov wrote: > Dougal, I’m totally OK with this idea. > > Thanks > > Renat Akhmerov > @Nokia > On 9 Jul 2018, 22:14 +0700, Dougal Matthews , wrote: > > Hey folks, > > I'd like to propose that we start abandoning old Gerrit reviews. > > This report shows how stale and out of date some of the reviews are: > http://stackalytics.com/report/reviews/mistral-group/open > > I would like to initially abandon anything without any activity for a > year, but we might want to consider a shorter limit - maybe 6 months. > Reviews can be restored, so the risk is low. > > What do you think? Any objections or counter suggestions? > > If I don't hear any complaints, I'll go ahead with this next week (or > maybe the following week). > > Cheers, > Dougal > __ > OpenStack Development Mailing List (not for usage questions) > Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe > http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev > > > __ > OpenStack Development Mailing List (not for usage questions) > Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe > http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev > > __ OpenStack Development Mailing List (not for usage questions) Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
Re: [openstack-dev] [mistral] Clearing out old gerrit reviews
Dougal, I’m totally OK with this idea. Thanks Renat Akhmerov @Nokia On 9 Jul 2018, 22:14 +0700, Dougal Matthews , wrote: > Hey folks, > > I'd like to propose that we start abandoning old Gerrit reviews. > > This report shows how stale and out of date some of the reviews are: > http://stackalytics.com/report/reviews/mistral-group/open > > I would like to initially abandon anything without any activity for a year, > but we might want to consider a shorter limit - maybe 6 months. Reviews can > be restored, so the risk is low. > > What do you think? Any objections or counter suggestions? > > If I don't hear any complaints, I'll go ahead with this next week (or maybe > the following week). > > Cheers, > Dougal > __ > OpenStack Development Mailing List (not for usage questions) > Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe > http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev __ OpenStack Development Mailing List (not for usage questions) Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
