On 7/23/14, 11:41 AM, "Ihar Hrachyshka" <ihrachys@redhat.com> wrote:
-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA512
On 23/07/14 00:03, Aaron Rosen wrote:
Hi Ihar,
I had directly approved those patches as they were just reviewed and merged into master (they were also pretty trivial bugs imo). I'll wait for another +2 though in the future.
Not everything from master belongs to stable branches though, and there is a reason why stable maintainers' team is different from per-project core team.
The patches above were something that was needed to get into the stable version. The problem with the stable version at the moment is that patches are back ported (which are fixing issues that break open stack) and these are not being reviewed. I think that in this case Aaron took the initiative and pushed code that was an important fix. It would be nice if we were all involved in getting he stable review cadence back again. If not then we really need to reevaluate the situation. A luta continua
I hope that Neutron backports will get more love from now on, so feedback should not take too much time and block valuable bug fixes.
And thanks for considering my argument.
/Ihar
Best,
Aaron
On Tue, Jul 22, 2014 at 10:07 AM, Ihar Hrachyshka <ihrachys@redhat.com <mailto:ihrachys@redhat.com>> wrote:
I didn't really want to call out any names, but since you ask, those patches are here:
- https://review.openstack.org/#/c/108534/ - https://review.openstack.org/#/c/108533/ - https://review.openstack.org/#/c/106199/
Probably other patches of the same author are of the same nature.
/Ihar
On 22/07/14 18:28, Gary Kotton wrote:
Which patch was this?
On 7/22/14, 2:37 PM, "Ihar Hrachyshka" <ihrachys@redhat.com <mailto:ihrachys@redhat.com>> wrote:
Hi, I'm walking thru the list of stable backports for Neutron, and I've found an instance when a patch was pushed to gate by its author (or better, backporter) without receiving any feedback from other team members.
Let me remind you that (quoting [1]):
"Each backported commit proposed to gerrit should be reviewed and +2ed by two stable-maint members before it is approved. Where a stable-maint member has backported a fix, a single other +2 is sufficient for approval."
So please refrain from pushing patches before another maintainer checks your backport. Without following that rule, we miss opportunity for peer review.
Cheers, /Ihar
_______________________________________________ Openstack-stable-maint mailing list Openstack-stable-maint@lists.openstack.org
<mailto:Openstack-stable-maint@lists.openstack.org>
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-stable-main t
_______________________________________________ Openstack-stable-maint mailing list Openstack-stable-maint@lists.openstack.org <mailto:Openstack-stable-maint@lists.openstack.org>
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-stable-main t
-----BEGIN PGP SIGNATURE----- Version: GnuPG/MacGPG2 v2.0.22 (Darwin) Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/
iQEcBAEBCgAGBQJTz3UuAAoJEC5aWaUY1u572RYIAIw7wJkRxmKXILMoA6BkrCE/ 8Jt04FEuiKPW+7jZ/UgTbZCAyzjWuB7bUkPCYy9ITHQ9m8KaD9hsjyyGRwiLs8P1 y0TLwmNVq3b7lTn+qFGnCAHtKozv7t7qTn5HXyIf/v3JYiYCHUBlfT5XDyLqos7X mtRIUzBJj6zVnKkgjJS5FQVWbw7ZWzi7lLX+bEUTDT9YROnM1SlOzHPOAg2P6M7V 1p8EPewBlxdoI6/D7LJQraP4KPc4ijY0wNju11VKUjYd21nWI1kWCgWFlql7z1wv 0auGknz4RI/T2H9zK3CP87kZLAm9FSd+BNbaUhjSl9fmm6sAq0WDoUm30zVD4og= =zixh -----END PGP SIGNATURE-----