<div dir="ltr">Folks,<div><br></div><div>First, let me report current feature status: we continued the work with Bogdan Dobrelya and Sergii Golovatiuk. I have incorporated their feedback into the change. Also, I have fully tested it on custom ISO and Fuel CI passes successfully. Also, I have an approval from Bogdan on the implementation of the change (see his +2 before he casted -2 because of FF).</div><div><br></div><div>Now, I still would like to ask the change to be merged into 8.0 due to the following reasons:</div><div>1. It is small and isolated</div><div>2. It is disabled by default and marked as experimental</div><div>3. It promises big value by reducing load on RabbitMQ, which becomes bottleneck on big environments</div><div><br></div><div>Thanks,</div><div><br></div><div>Dmitry</div><div><br></div><div>The CR: <a href="https://review.openstack.org/#/c/249180">https://review.openstack.org/#/c/249180</a></div></div><div class="gmail_extra"><br><div class="gmail_quote">2015-12-08 13:11 GMT+03:00 Igor Kalnitsky <span dir="ltr"><<a href="mailto:ikalnitsky@mirantis.com" target="_blank">ikalnitsky@mirantis.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hey Dmitry,<br>
<br>
Despite the fact the feature promises performance boost (IIUC) and<br>
it's really nice to have it, I agree with Mike's opinion - it's late<br>
to continue working on features. Each delay means less time to test<br>
it, and we need to focus on quality.<br>
<br>
I'm sorry, but I have to say "No" on requested exception.<br>
<span class="HOEnZb"><font color="#888888"><br>
- Igor<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
On Tue, Dec 8, 2015 at 9:55 AM, Mike Scherbakov<br>
<<a href="mailto:mscherbakov@mirantis.com">mscherbakov@mirantis.com</a>> wrote:<br>
> Hi Dmitry,<br>
> as much as I support the change, and glad that we got time for it, my<br>
> opinion is that we should not extend a FFE. I have following reasons to<br>
> think this way:<br>
><br>
> 1) Feature Freeze is time based milestone, with the rational "FF ensures<br>
> that sufficient share of the ReleaseCycle is dedicated to QA, until we<br>
> produce the first release candidates. Limiting the changes that affect the<br>
> behavior of the software allow for consistent testing and efficient<br>
> bugfixing" [1]. Even though this feature will be disabled by default, it is<br>
> important to note the first part of this rationale - we need to focus on<br>
> stability now, not on features.<br>
> 2) 7 FFEs for Fuel [2] I'd subjectively consider as high number, as in total<br>
> there are ~25 major blueprints to be delivered. Dmitry, our PTL,<br>
> unfortunately is absent for a couple of weeks, but his opinion is quite<br>
> similar: "The list of exceptions is much longer than I'd like, and some have<br>
> larger impact than I'd like, lets all of us make sure we don't come to<br>
> regret granting these exceptions." [3]. Taking any exception further means<br>
> moving FF, in fact. That means moving of release date, which I don't think<br>
> we should even consider doing.<br>
> 3) Exception to exception, in my opinion, should only be allowed in<br>
> extremely rare cases for essential features only. When it becomes clear that<br>
> the whole release has a major gap or serious issue, which can only be<br>
> resolved by finishing an essential feature. I have no evidence to think that<br>
> this functionality, which will be disabled by default, can fall into this<br>
> category.<br>
> 4) Changeset [4] has a change to the packaging spec. Any small change to<br>
> packaging after FF imposes additional risk, as there is no good test<br>
> automation for such kind of changes. Even if it's just include of a new<br>
> file. In case of regression, we may easily lose a day for figuring out what<br>
> is wrong and reverting a change.<br>
><br>
> I'd like to hear component leads while PTL is absent these days....<br>
><br>
> [1] <a href="https://wiki.openstack.org/wiki/FeatureFreeze" rel="noreferrer" target="_blank">https://wiki.openstack.org/wiki/FeatureFreeze</a><br>
> [2]<br>
> <a href="http://lists.openstack.org/pipermail/openstack-dev/2015-December/081131.html" rel="noreferrer" target="_blank">http://lists.openstack.org/pipermail/openstack-dev/2015-December/081131.html</a><br>
> [3]<br>
> <a href="http://lists.openstack.org/pipermail/openstack-dev/2015-December/081149.html" rel="noreferrer" target="_blank">http://lists.openstack.org/pipermail/openstack-dev/2015-December/081149.html</a><br>
> [4] <a href="https://review.openstack.org/#/c/249180/" rel="noreferrer" target="_blank">https://review.openstack.org/#/c/249180/</a><br>
><br>
> On Mon, Dec 7, 2015 at 2:30 PM Adam Heczko <<a href="mailto:aheczko@mirantis.com">aheczko@mirantis.com</a>> wrote:<br>
>><br>
>> Hello Dmitry,<br>
>> I like this idea and very much appreciate it.<br>
>> +1 from me :)<br>
>><br>
>> A.<br>
>><br>
>> On Mon, Dec 7, 2015 at 9:48 PM, Dmitry Mescheryakov<br>
>> <<a href="mailto:dmescheryakov@mirantis.com">dmescheryakov@mirantis.com</a>> wrote:<br>
>>><br>
>>> Hello folks,<br>
>>><br>
>>> I'd like to request extension of current FFE for the feature [1]. During<br>
>>> the three FFE days we merged the spec [2] after big discussion and made a<br>
>>> couple iterations over the implementation [3]. We had a chat with Bogdan on<br>
>>> how to progress and here are the action items still need to be done:<br>
>>>  * part of the change responsible for RabbitMQ policy need to be<br>
>>> upstreamed first to RabbitMQ repo.<br>
>>>  * the change needs to be review and merged by our library folks.<br>
>>><br>
>>> Overall I think that 2-3 more days should be enough to finish it.<br>
>>><br>
>>> What do you think folks?<br>
>>><br>
>>> Dmitry<br>
>>><br>
>>> [1]<br>
>>> <a href="https://blueprints.launchpad.net/fuel/+spec/rabbitmq-disable-mirroring-for-rpc" rel="noreferrer" target="_blank">https://blueprints.launchpad.net/fuel/+spec/rabbitmq-disable-mirroring-for-rpc</a><br>
>>> [2] <a href="https://review.openstack.org/247517" rel="noreferrer" target="_blank">https://review.openstack.org/247517</a><br>
>>> [3] <a href="https://review.openstack.org/249180" rel="noreferrer" target="_blank">https://review.openstack.org/249180</a><br>
>>><br>
>>><br>
>>> __________________________________________________________________________<br>
>>> OpenStack Development Mailing List (not for usage questions)<br>
>>> Unsubscribe:<br>
>>> <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" rel="noreferrer" target="_blank">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</a><br>
>>> <a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
>>><br>
>><br>
>><br>
>><br>
>> --<br>
>> Adam Heczko<br>
>> Security Engineer @ Mirantis Inc.<br>
>> __________________________________________________________________________<br>
>> OpenStack Development Mailing List (not for usage questions)<br>
>> Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" rel="noreferrer" target="_blank">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</a><br>
>> <a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
><br>
> --<br>
> Mike Scherbakov<br>
> #mihgen<br>
><br>
> __________________________________________________________________________<br>
> OpenStack Development Mailing List (not for usage questions)<br>
> Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" rel="noreferrer" target="_blank">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</a><br>
> <a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
><br>
<br>
__________________________________________________________________________<br>
OpenStack Development Mailing List (not for usage questions)<br>
Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" rel="noreferrer" target="_blank">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
</div></div></blockquote></div><br></div>