Re: [openstack-dev] [qa] [cinder] Do we now require schema response validation in tempest clients?

2014-06-05 Thread Ollie Leahy
I've submitted a patch that copies the compute http response validation structure for the 'cinder list' request. https://review.openstack.org/96440 I'm having trouble getting it through the gate, but input from the cinder group about whether this is useful work that I should persist with

Re: [openstack-dev] [qa] [cinder] Do we now require schema response validation in tempest clients?

2014-05-04 Thread Christopher Yeoh
On Thu, May 1, 2014 at 1:32 PM, Ken'ichi Ohmichi ken1ohmi...@gmail.comwrote: Hi David, 2014-05-01 5:44 GMT+09:00 David Kranz dkr...@redhat.com: There have been a lot of patches that add the validation of response dicts. We need a policy on whether this is required or not. For example,

Re: [openstack-dev] [qa] [cinder] Do we now require schema response validation in tempest clients?

2014-05-01 Thread Ghanshyam Mann
Hi David, Ken'ichi, Thursday, May 1, 2014 1:02 PM Ken'ichi Ohmichi ken1ohmi...@gmail.com: Hi David, 2014-05-01 5:44 GMT+09:00 David Kranz dkr...@redhat.com: There have been a lot of patches that add the validation of response dicts. We need a policy on whether this is required or not. For

[openstack-dev] [qa] [cinder] Do we now require schema response validation in tempest clients?

2014-04-30 Thread David Kranz
There have been a lot of patches that add the validation of response dicts. We need a policy on whether this is required or not. For example, this patch https://review.openstack.org/#/c/87438/5 is for the equivalent of 'cinder service-list' and is a basically a copy of the nova test which

Re: [openstack-dev] [qa] [cinder] Do we now require schema response validation in tempest clients?

2014-04-30 Thread Ken'ichi Ohmichi
Hi David, 2014-05-01 5:44 GMT+09:00 David Kranz dkr...@redhat.com: There have been a lot of patches that add the validation of response dicts. We need a policy on whether this is required or not. For example, this patch https://review.openstack.org/#/c/87438/5 is for the equivalent of