[openstack-dev] [python-*client] moving to WebOb exceptions

Alexei Kornienko alexei.kornienko at gmail.com
Wed Feb 26 22:30:07 UTC 2014


Hi,

Could you please explain in more details what causes pain in adding
additional requirement to clients (in most cases when client is used inside
other openstack projects it's there already)?
My IMHO is that having to write ugly hacks to handle several exception
classes with the same name causes much more pain.

Regards,
Alexei


2014-02-26 20:02 GMT+02:00 Dean Troyer <dtroyer at gmail.com>:

> On Wed, Feb 26, 2014 at 11:20 AM, Andrey Kurilin <akurilin at mirantis.com>wrote:
>
>> While working on unification of clients code we try to unify various
>> exceptions in different client projects.
>> We have module apiclient.exceptions in oslo-incubator[1]. Since our
>> apiclient is an oslo-inclubator library and not a standalone lib this
>> doesn't help in case we need to process exceptions from several clients.
>>
> [...]
>
>> The solution would be to use exceptions from external library - Module
>> WebOb.exc[2] for example (since WebOb is already used in other openstack
>> projects). This exceptions cover all our custom http exceptions.
>>
>
> I would oppose adding WebOb as a requirement for the client libraries.  I
> see keystoneclient has it today but that is only because the middleware is
> still in that repo (moving those is another topic).
>
> The pain of installing the exiting client libs and their prereqs is bad
> enough, adding to it is not tenable and is part of what is motivating the
> SDK efforts.
>
> dt
>
> --
>
> Dean Troyer
> dtroyer at gmail.com
>
> _______________________________________________
> OpenStack-dev mailing list
> OpenStack-dev at lists.openstack.org
> 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/20140227/1d9e0b25/attachment.html>


More information about the OpenStack-dev mailing list