Re: [openstack-dev] [python-openstacksdk][shade] Reviewing the merge-shade patches

2017-11-13 Thread David Shrewsbury
Hi! Thanks for the explanations. On Mon, Nov 13, 2017 at 11:30 AM, Monty Taylor wrote: I also think that the OpenStackCloud/OperatorCloud split from shade wound > up being more confusing than helpful. > Yep. At the time, the classes were doing different auth things, so

Re: [openstack-dev] [python-openstacksdk][shade] Reviewing the merge-shade patches

2017-11-13 Thread Monty Taylor
On 11/13/2017 09:06 AM, David Shrewsbury wrote: Hi! On Mon, Nov 13, 2017 at 9:54 AM, Monty Taylor > wrote: Hey everybody, You may have noticed a giant patch series up: https://review.openstack.org/#/q/topic:merge-shade

Re: [openstack-dev] [python-openstacksdk][shade] Reviewing the merge-shade patches

2017-11-13 Thread David Shrewsbury
Hi! On Mon, Nov 13, 2017 at 9:54 AM, Monty Taylor wrote: > Hey everybody, > > You may have noticed a giant patch series up: > > https://review.openstack.org/#/q/topic:merge-shade > > One thing I don't see covered here is the current set of Ansible module tests. Unless

[openstack-dev] [python-openstacksdk][shade] Reviewing the merge-shade patches

2017-11-13 Thread Monty Taylor
Hey everybody, You may have noticed a giant patch series up: https://review.openstack.org/#/q/topic:merge-shade It's a big beastie, so I thought a synopsis of what's going would be useful. Also, there's a pile of patches towards the end that we want to squash before landing but which have