Re: [openstack-dev] [qa] [Tempest] Abondoned old code reviews

[Ken'ichi Ohmichi]

Thanks for feedback here.
There are not any objections, so I will drop old code reviews on Tempest queue.

Thanks
Ken Ohmichi

---


2016-05-31 23:55 GMT-07:00 Masayuki Igawa :
> On Wed, Jun 1, 2016 at 3:05 AM, Andrea Frittoli
>  wrote:
>> On Mon, 30 May 2016, 6:25 p.m. Ken'ichi Ohmichi, 
>> wrote:
>>>
>>> Hi,
>>>
>>> There are many patches which are not updated in Tempest review queue
>>> even if having gotten negative feedback from reviewers or jenkins.
>>> Nova team is abandoning such patches like [1].
>>> I feel it would be nice to abandone such patches which are not updated
>>> since the end of 2015.
>>> Any thoughts?
>
> +1
> I think 5 months is enough to abandon :)
>
> Best Regards,
> -- Masayuki
>
>
>>
>>
>> I don't mind either way, if you prefer abandoning them it's ok with me.
>> I rely on gerrit dashboards and IRC communication to decide which patches I
>> should review; but I understand it would be nice to remove some clutter.
>>
>> Andrea
>>
>>>
>>> [1]:
>>> http://lists.openstack.org/pipermail/openstack-dev/2016-May/096112.html
>>>
>>> Thanks
>>> Ken Ohmichi
>>>
>>> __
>>> 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

__
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] [qa] [Tempest] Abondoned old code reviews

[Masayuki Igawa]

On Wed, Jun 1, 2016 at 3:05 AM, Andrea Frittoli
 wrote:
> On Mon, 30 May 2016, 6:25 p.m. Ken'ichi Ohmichi, 
> wrote:
>>
>> Hi,
>>
>> There are many patches which are not updated in Tempest review queue
>> even if having gotten negative feedback from reviewers or jenkins.
>> Nova team is abandoning such patches like [1].
>> I feel it would be nice to abandone such patches which are not updated
>> since the end of 2015.
>> Any thoughts?

+1
I think 5 months is enough to abandon :)

Best Regards,
-- Masayuki


>
>
> I don't mind either way, if you prefer abandoning them it's ok with me.
> I rely on gerrit dashboards and IRC communication to decide which patches I
> should review; but I understand it would be nice to remove some clutter.
>
> Andrea
>
>>
>> [1]:
>> http://lists.openstack.org/pipermail/openstack-dev/2016-May/096112.html
>>
>> Thanks
>> Ken Ohmichi
>>
>> __
>> 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] [qa] [Tempest] Abondoned old code reviews

[Andrea Frittoli]

On Mon, 30 May 2016, 6:25 p.m. Ken'ichi Ohmichi, 
wrote:

> Hi,
>
> There are many patches which are not updated in Tempest review queue
> even if having gotten negative feedback from reviewers or jenkins.
> Nova team is abandoning such patches like [1].
> I feel it would be nice to abandone such patches which are not updated
> since the end of 2015.
> Any thoughts?
>

I don't mind either way, if you prefer abandoning them it's ok with me.
I rely on gerrit dashboards and IRC communication to decide which patches I
should review; but I understand it would be nice to remove some clutter.

Andrea


> [1]:
> http://lists.openstack.org/pipermail/openstack-dev/2016-May/096112.html
>
> Thanks
> Ken Ohmichi
>
> __
> 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] [qa] [Tempest] Abondoned old code reviews

[Yaroslav Lobankov]

+1! But I think we can leave patches created in 2016 and abandon others.

Regards,
Yaroslav Lobankov.

On Mon, May 30, 2016 at 8:20 PM, Ken'ichi Ohmichi 
wrote:

> Hi,
>
> There are many patches which are not updated in Tempest review queue
> even if having gotten negative feedback from reviewers or jenkins.
> Nova team is abandoning such patches like [1].
> I feel it would be nice to abandone such patches which are not updated
> since the end of 2015.
> Any thoughts?
>
> [1]:
> http://lists.openstack.org/pipermail/openstack-dev/2016-May/096112.html
>
> Thanks
> Ken Ohmichi
>
> __
> 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-dev] [qa] [Tempest] Abondoned old code reviews

[Ken'ichi Ohmichi]

Hi,

There are many patches which are not updated in Tempest review queue
even if having gotten negative feedback from reviewers or jenkins.
Nova team is abandoning such patches like [1].
I feel it would be nice to abandone such patches which are not updated
since the end of 2015.
Any thoughts?

[1]: http://lists.openstack.org/pipermail/openstack-dev/2016-May/096112.html

Thanks
Ken Ohmichi

__
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