<div dir="ltr">Hi Igor,<div><br></div><div>I don't agree with you, some basic validation is essential part of</div><div>any handler and our API, currently it's easy to get meaningless 500 error</div><div>(which is unhandled exception) from the backend or get the error that there</div><div>is something wrong with the template only after you press deploy button.</div><div>It's a bad UX and contradicts to our attempts to develop good api.</div><div><br></div><div>Thanks,</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Jul 24, 2015 at 12:02 PM, Igor Kalnitsky <span dir="ltr"><<a href="mailto:ikalnitsky@mirantis.com" target="_blank">ikalnitsky@mirantis.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Greetings,<br>
<br>
The issue [1] looks like a feature to me. I'd move it to next release.<br>
Let's focus on what's important right now - stability.<br>
<br>
Thanks,<br>
Igor<br>
<br>
[1]: <a href="https://bugs.launchpad.net/fuel/+bug/1476779" rel="noreferrer" target="_blank">https://bugs.launchpad.net/fuel/+bug/1476779</a><br>
<span class=""><br>
On Fri, Jul 24, 2015 at 11:53 AM, Evgeniy L <<a href="mailto:eli@mirantis.com">eli@mirantis.com</a>> wrote:<br>
> Hi,<br>
><br>
> Since the feature is essential, and changes are small, we can accept it as<br>
> a,<br>
> feature freeze exceptions.<br>
><br>
> But as far as I know there is a very important ticket [1] which was created<br>
> in<br>
> order to get patches merged faster, also I still have concerns regarding to<br>
> ERB style template "<% if3 %>" which is in fact Jinja. So it's not only<br>
> about<br>
> fixes in the client.<br>
><br>
> [1] <a href="https://bugs.launchpad.net/fuel/+bug/1476779" rel="noreferrer" target="_blank">https://bugs.launchpad.net/fuel/+bug/1476779</a><br>
><br>
</span>> On Thu, Jul 23, 2015 at 9:18 PM, Mike Scherbakov <<a href="mailto:mscherbakov@mirantis.com">mscherbakov@mirantis.com</a>><br>
<span class="">> wrote:<br>
>><br>
>> Looks like the only CLI part left:<br>
</span><span class="">>> <a href="https://review.openstack.org/#/c/204321/" rel="noreferrer" target="_blank">https://review.openstack.org/#/c/204321/</a>, and you guys did a great job<br>
>> finishing the other two.<br>
>><br>
>> Looks like we'd need to give FF exception, as this is essential feature.<br>
>> It's glad that we merged all other thousands lines of code. This is the most<br>
>> complex feature, and seems like the only small thing is left.<br>
>><br>
>> I'd like to hear feedback from Nailgun cores & fuel client SMEs. For me,<br>
>> it seems it is lower risk, and patch is relatively small. How long would it<br>
>> take to complete it? If it takes a couple of days, then it is fine. If it is<br>
>> going to take week or two, then we will have to have it as a risk for HCF<br>
>> deadline. Spending resources on features now, not on bugs, means less<br>
>> quality or slip of the release.<br>
>><br>
</span>>> On Wed, Jul 22, 2015 at 2:36 PM Aleksey Kasatkin <<a href="mailto:akasatkin@mirantis.com">akasatkin@mirantis.com</a>><br>
<span class="">>> wrote:<br>
>>><br>
>>> Team,<br>
>>><br>
>>> I would like to request an exception from the Feature Freeze for<br>
>>> "Templates for Networking" feature [1].<br>
>>><br>
>>> Exception is required for two CRs to python-fuelclient: [2],[3] and one<br>
>>> CR to fuel-web (Nailgun): [4].<br>
>>> These CRs are for adding ability to create/remove networks via API [4]<br>
>>> and for supporting new API functionality via CLI.<br>
>>> These patchsets are for adding new templates-related functionality and<br>
>>> they do not change existing functionality.<br>
>>> Patchsets [3],[4] are in deep review and they will hopefully be merged on<br>
>>> Thursday.<br>
>>><br>
>>> Please, respond if you have any questions or concerns related to this<br>
>>> request.<br>
>>><br>
>>> Thanks in advance.<br>
>>><br>
>>> [1] <a href="https://blueprints.launchpad.net/fuel/+spec/templates-for-networking" rel="noreferrer" target="_blank">https://blueprints.launchpad.net/fuel/+spec/templates-for-networking</a><br>
</span>>>> [2] <a href="https://review.openstack.org/#/c/204321/" rel="noreferrer" target="_blank">https://review.openstack.org/#/c/204321/</a><br>
>>> [3] <a href="https://review.openstack.org/#/c/203602/" rel="noreferrer" target="_blank">https://review.openstack.org/#/c/203602/</a><br>
>>> [4] <a href="https://review.openstack.org/#/c/201217/" rel="noreferrer" target="_blank">https://review.openstack.org/#/c/201217/</a><br>
<span class="">>>><br>
>>> --<br>
>>> Best regards,<br>
>>> Aleksey Kasatkin<br>
>>><br>
>>> __________________________________________________________________________<br>
>>> OpenStack Development Mailing List (not for usage questions)<br>
>>> Unsubscribe:<br>
</span>>>> <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>
<span class="">>>> <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>
</span>>> 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>
<span class="">>><br>
><br>
><br>
> __________________________________________________________________________<br>
> OpenStack Development Mailing List (not for usage questions)<br>
</span>> 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>
<span class="">><br>
<br>
__________________________________________________________________________<br>
OpenStack Development Mailing List (not for usage questions)<br>
</span>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>
</blockquote></div><br></div>