[openstack-dev] [fuel] FF Exception request for Templates for Networking feature

Sebastian Kalinowski skalinowski at mirantis.com
Mon Jul 27 08:58:38 UTC 2015


Yes, exactly like that.

+1

2015-07-27 10:53 GMT+02:00 Evgeniy L <eli at mirantis.com>:

> So, to summarise, +1 from me, we accept the changes which are required
> for the feature as feature freeze exceptions:
>
> 1. Fuel client changes [1]
> 2. Validation [2]
> 3. Change tokens in template language
>
> Sebastian, Igor, correct?
>
> [1] https://review.openstack.org/#/c/204321/
> [2] https://bugs.launchpad.net/fuel/+bug/1476779
>
> On Sat, Jul 25, 2015 at 1:25 AM, Andrew Woodward <xarses at gmail.com> wrote:
>
>> Igor,
>>
>> https://bugs.launchpad.net/fuel/+bug/1476779 must be included in the FFE
>> if you think it's a feature. Networking is the most complicated and
>> frustrating thing the user can work with. If we cant provide usable
>> feedback from bad data in the template then the feature is useless. I could
>> argue that its a critical UX defect.
>>
>>
>> On Fri, Jul 24, 2015 at 7:16 AM Evgeniy L <eli at mirantis.com> wrote:
>>
>>> Aleksey,
>>>
>>> Yes, my point is those parts should be also included in the scope of FFE.
>>> Regarding to template format, it's easy to fix and after release you
>>> will not
>>> be able to change it, or you can change it, but you will have to support
>>> both
>>> format, not to brake backward compatibility. So I would prefer to see it
>>> fixed
>>> in 7.0.
>>>
>>> Thanks,
>>>
>>> On Fri, Jul 24, 2015 at 3:14 PM, Aleksey Kasatkin <
>>> akasatkin at mirantis.com> wrote:
>>>
>>>> I agree, guys, we need at least some basic validation for template when
>>>> it is being loaded.
>>>> Ivan Kliuk started to work on this task.
>>>> And we agreed to test other types of delimiters (it is regarding ERB
>>>> style template) but we have some more important issues.
>>>> Evgeniy, is your meaning to include those to FFE ?
>>>>
>>>>
>>>> Aleksey Kasatkin
>>>>
>>>>
>>>> On Fri, Jul 24, 2015 at 2:12 PM, Sebastian Kalinowski <
>>>> skalinowski at mirantis.com> wrote:
>>>>
>>>>> I agree here with Evgeniy. Even if it's not a trivial change, we
>>>>> cannot leave a new API in such shape.
>>>>>
>>>>> 2015-07-24 11:41 GMT+02:00 Evgeniy L <eli at mirantis.com>:
>>>>>
>>>>>> Hi Igor,
>>>>>>
>>>>>> I don't agree with you, some basic validation is essential part of
>>>>>> any handler and our API, currently it's easy to get meaningless 500
>>>>>> error
>>>>>> (which is unhandled exception) from the backend or get the error that
>>>>>> there
>>>>>> is something wrong with the template only after you press deploy
>>>>>> button.
>>>>>> It's a bad UX and contradicts to our attempts to develop good api.
>>>>>>
>>>>>> Thanks,
>>>>>>
>>>>>> On Fri, Jul 24, 2015 at 12:02 PM, Igor Kalnitsky <
>>>>>> ikalnitsky at mirantis.com> wrote:
>>>>>>
>>>>>>> Greetings,
>>>>>>>
>>>>>>> The issue [1] looks like a feature to me. I'd move it to next
>>>>>>> release.
>>>>>>> Let's focus on what's important right now - stability.
>>>>>>>
>>>>>>> Thanks,
>>>>>>> Igor
>>>>>>>
>>>>>>> [1]: https://bugs.launchpad.net/fuel/+bug/1476779
>>>>>>>
>>>>>>> On Fri, Jul 24, 2015 at 11:53 AM, Evgeniy L <eli at mirantis.com>
>>>>>>> wrote:
>>>>>>> > Hi,
>>>>>>> >
>>>>>>> > Since the feature is essential, and changes are small, we can
>>>>>>> accept it as
>>>>>>> > a,
>>>>>>> > feature freeze exceptions.
>>>>>>> >
>>>>>>> > But as far as I know there is a very important ticket [1] which
>>>>>>> was created
>>>>>>> > in
>>>>>>> > order to get patches merged faster, also I still have concerns
>>>>>>> regarding to
>>>>>>> > ERB style template "<% if3 %>" which is in fact Jinja. So it's not
>>>>>>> only
>>>>>>> > about
>>>>>>> > fixes in the client.
>>>>>>> >
>>>>>>> > [1] https://bugs.launchpad.net/fuel/+bug/1476779
>>>>>>> >
>>>>>>> > On Thu, Jul 23, 2015 at 9:18 PM, Mike Scherbakov <
>>>>>>> mscherbakov at mirantis.com>
>>>>>>> > wrote:
>>>>>>> >>
>>>>>>> >> Looks like the only CLI part left:
>>>>>>> >> https://review.openstack.org/#/c/204321/, and you guys did a
>>>>>>> great job
>>>>>>> >> finishing the other two.
>>>>>>> >>
>>>>>>> >> Looks like we'd need to give FF exception, as this is essential
>>>>>>> feature.
>>>>>>> >> It's glad that we merged all other thousands lines of code. This
>>>>>>> is the most
>>>>>>> >> complex feature, and seems like the only small thing is left.
>>>>>>> >>
>>>>>>> >> I'd like to hear feedback from Nailgun cores & fuel client SMEs.
>>>>>>> For me,
>>>>>>> >> it seems it is lower risk, and patch is relatively small. How
>>>>>>> long would it
>>>>>>> >> take to complete it? If it takes a couple of days, then it is
>>>>>>> fine. If it is
>>>>>>> >> going to take week or two, then we will have to have it as a risk
>>>>>>> for HCF
>>>>>>> >> deadline. Spending resources on features now, not on bugs, means
>>>>>>> less
>>>>>>> >> quality or slip of the release.
>>>>>>> >>
>>>>>>> >> On Wed, Jul 22, 2015 at 2:36 PM Aleksey Kasatkin <
>>>>>>> akasatkin at mirantis.com>
>>>>>>> >> wrote:
>>>>>>> >>>
>>>>>>> >>> Team,
>>>>>>> >>>
>>>>>>> >>> I would like to request an exception from the Feature Freeze for
>>>>>>> >>> "Templates for Networking" feature [1].
>>>>>>> >>>
>>>>>>> >>> Exception is required for two CRs to python-fuelclient: [2],[3]
>>>>>>> and one
>>>>>>> >>> CR to fuel-web (Nailgun): [4].
>>>>>>> >>> These CRs are for adding ability to create/remove networks via
>>>>>>> API [4]
>>>>>>> >>> and for supporting new API functionality via CLI.
>>>>>>> >>> These patchsets are for adding new templates-related
>>>>>>> functionality and
>>>>>>> >>> they do not change existing functionality.
>>>>>>> >>> Patchsets [3],[4] are in deep review and they will hopefully be
>>>>>>> merged on
>>>>>>> >>> Thursday.
>>>>>>> >>>
>>>>>>> >>> Please, respond if you have any questions or concerns related to
>>>>>>> this
>>>>>>> >>> request.
>>>>>>> >>>
>>>>>>> >>> Thanks in advance.
>>>>>>> >>>
>>>>>>> >>> [1]
>>>>>>> https://blueprints.launchpad.net/fuel/+spec/templates-for-networking
>>>>>>> >>> [2] https://review.openstack.org/#/c/204321/
>>>>>>> >>> [3] https://review.openstack.org/#/c/203602/
>>>>>>> >>> [4] https://review.openstack.org/#/c/201217/
>>>>>>> >>>
>>>>>>> >>> --
>>>>>>> >>> Best regards,
>>>>>>> >>> Aleksey Kasatkin
>>>>>>> >>>
>>>>>>> >>>
>>>>>>> __________________________________________________________________________
>>>>>>> >>> OpenStack Development Mailing List (not for usage questions)
>>>>>>> >>> Unsubscribe:
>>>>>>> >>> OpenStack-dev-request at lists.openstack.org?subject:unsubscribe
>>>>>>> >>>
>>>>>>> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
>>>>>>> >>
>>>>>>> >> --
>>>>>>> >> Mike Scherbakov
>>>>>>> >> #mihgen
>>>>>>> >>
>>>>>>> >>
>>>>>>> __________________________________________________________________________
>>>>>>> >> OpenStack Development Mailing List (not for usage questions)
>>>>>>> >> Unsubscribe:
>>>>>>> OpenStack-dev-request at 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-request at 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-request at 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-request at 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-request at 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-request at 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-request at lists.openstack.org?subject:unsubscribe
>>> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
>>>
>> --
>>
>> --
>>
>> Andrew Woodward
>>
>> Mirantis
>>
>> Fuel Community Ambassador
>>
>> Ceph Community
>>
>> __________________________________________________________________________
>> OpenStack Development Mailing List (not for usage questions)
>> Unsubscribe:
>> OpenStack-dev-request at 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-request at lists.openstack.org?subject:unsubscribe
> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstack.org/pipermail/openstack-dev/attachments/20150727/7991d000/attachment.html>


More information about the OpenStack-dev mailing list