Open Stack

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On 03/17/2015 05:22 PM, Ihar Hrachyshka wrote:
> On 03/13/2015 04:36 PM, Doug Hellmann wrote:
> 
> 
>> On Fri, Mar 13, 2015, at 07:25 AM, Ihar Hrachyshka wrote: On 
>> 03/13/2015 01:37 AM, Nikhil Manchanda wrote:
>>>>> Looking back at the meeting eavesdrop, the primary reasons 
>>>>> why we deferred this work to Liberty was twofold:
>>>>> 
>>>>> - It wasn't a set plan based on information available to
>>>>> us at the time. This being the case, we decided to wait
>>>>> until we had more information regarding the requirements
>>>>> around this from oslo.
>>>>> 
>>>>> - We wanted to ensure that we had a corresponding hacking 
>>>>> rule in place to prevent future patch-sets from using the 
>>>>> deprecated module names.
>>>>> 
> 
>> For hacking check, I have a patch in review for 'hacking' repo
>> to add checks (both for stable branches where oslo.* namespace
>> is used, and new branches where oslo_* is expected):
> 
>> - https://review.openstack.org/157894
> 
>> Also, neutron has a (test covered) hacking check at:
> 
>> http://git.openstack.org/cgit/openstack/neutron/tree/neutron/hacking/checks.py#n119
>
>>  Feel free to adopt.
> 
>>> I wish we, as a community, were less obsessed with creating so 
>>> many hacking rules. These are really minor changes and it's
>>> going to be a relatively short-lived issue that could just be
>>> fixed once. If there's a regression, fixing *that* won't be
>>> hard or take long.
> 
> That relatively short-lived issue already resulted in multiple 
> backports to stable branches with new namespaces being used. F.e.
> see:
> 
> https://bugs.launchpad.net/nova/+bug/1432685
> 
> There is no safe way to communicate the issue to all parties
> involved, so if automation is good at catching those issues, it
> should be applied. It's wrong to rely on people when a hacking
> check is enough.
> 
> 

OK, that was a wrong example. Though we still had bugs before when a
patch that used oslo_* namespace was backported to Juno (which is wrong).

/Ihar
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1

iQEcBAEBAgAGBQJVCFgjAAoJEC5aWaUY1u57Hd4H/32X6Utp1aLm0KnF2HgW63ah
DWk+PJj4Ku5TfZZ7IAXH/Rk6C4crCidfqEguwFuCkhlwQ+9bqsZ2INUjjGIfeY4B
GJaG5lBIn8Pcvbtf3X2oU8ByE4/GTHNP91XRv2/mQ5+PnhFF57GW4b3qVBy/DQp8
7+CBUqKwxPVx0rgCcotRiGmJs6tjLtlm/8iFhkYzu9Xr5Ti1b+nAm7NP9HzK1279
sEbbv68zRdYhOAm06CAoy/WoEgftoFR6xSc9hhTkakf6t7zE5EMeMWdJATrILjsE
pZjD39g8ye6ni3OiUUQtx8NosMJs+ORroNgeAn0WnFYW63kZoHvd2iJ/91CFNyg=
=m5pk
-----END PGP SIGNATURE-----