Re: [openstack-dev] [heat][novaclient] heat gate is broken because of new novaclient

2015-04-22 Thread Sean Dague
On 04/22/2015 07:07 AM, Sean Dague wrote:
 On 04/22/2015 04:09 AM, Angus Salkeld wrote:
 Hi

 Can we please have a new release of novaclient (after the below fix)?

 Heat's unit tests pass fine with:
 python-novaclient (2.23.0)

 but python-novaclient 2.24.0 introduces this bug:
 https://bugs.launchpad.net/python-novaclient/+bug/1437244

 We still need this patch in: https://review.openstack.org/176228

 We should also update requirements to exclude 2.24.0
 
 I've got an alternate fix here - https://review.openstack.org/#/c/176252/
 
 I was -1 for a long time on the complex repr stuff in the introducing
 patch, and I think that's just going to get us into other problems down
 the road. The repr fall back for Resource is totally fine, and we should
 just use that.

python-novaclient 2.24.1 has been released with
https://review.openstack.org/#/c/176252/ in place. Hopefully that fixes
things for Heat.

-Sean

-- 
Sean Dague
http://dague.net

__
OpenStack Development Mailing List (not for usage questions)
Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev


Re: [openstack-dev] [heat][novaclient] heat gate is broken because of new novaclient

2015-04-22 Thread Angus Salkeld
On Wed, Apr 22, 2015 at 11:49 PM, Sean Dague s...@dague.net wrote:

 On 04/22/2015 07:07 AM, Sean Dague wrote:
  On 04/22/2015 04:09 AM, Angus Salkeld wrote:
  Hi
 
  Can we please have a new release of novaclient (after the below fix)?
 
  Heat's unit tests pass fine with:
  python-novaclient (2.23.0)
 
  but python-novaclient 2.24.0 introduces this bug:
  https://bugs.launchpad.net/python-novaclient/+bug/1437244
 
  We still need this patch in: https://review.openstack.org/176228
 
  We should also update requirements to exclude 2.24.0
 
  I've got an alternate fix here -
 https://review.openstack.org/#/c/176252/
 
  I was -1 for a long time on the complex repr stuff in the introducing
  patch, and I think that's just going to get us into other problems down
  the road. The repr fall back for Resource is totally fine, and we should
  just use that.

 python-novaclient 2.24.1 has been released with
 https://review.openstack.org/#/c/176252/ in place. Hopefully that fixes
 things for Heat.


Thanks Sean!

Shouldn't we also exclude 2.24.0 from requirements?

-Angus



 -Sean

 --
 Sean Dague
 http://dague.net

 __
 OpenStack Development Mailing List (not for usage questions)
 Unsubscribe: openstack-dev-requ...@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-requ...@lists.openstack.org?subject:unsubscribe
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev


[openstack-dev] [heat][novaclient] heat gate is broken because of new novaclient

2015-04-22 Thread Angus Salkeld
Hi

Can we please have a new release of novaclient (after the below fix)?

Heat's unit tests pass fine with:
python-novaclient (2.23.0)

but python-novaclient 2.24.0 introduces this bug:
https://bugs.launchpad.net/python-novaclient/+bug/1437244

We still need this patch in: https://review.openstack.org/176228

We should also update requirements to exclude 2.24.0

Thanks
Angus
__
OpenStack Development Mailing List (not for usage questions)
Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev


Re: [openstack-dev] [heat][novaclient] heat gate is broken because of new novaclient

2015-04-22 Thread Sean Dague
On 04/22/2015 04:09 AM, Angus Salkeld wrote:
 Hi
 
 Can we please have a new release of novaclient (after the below fix)?
 
 Heat's unit tests pass fine with:
 python-novaclient (2.23.0)
 
 but python-novaclient 2.24.0 introduces this bug:
 https://bugs.launchpad.net/python-novaclient/+bug/1437244
 
 We still need this patch in: https://review.openstack.org/176228
 
 We should also update requirements to exclude 2.24.0

I've got an alternate fix here - https://review.openstack.org/#/c/176252/

I was -1 for a long time on the complex repr stuff in the introducing
patch, and I think that's just going to get us into other problems down
the road. The repr fall back for Resource is totally fine, and we should
just use that.

-Sean

-- 
Sean Dague
http://dague.net

__
OpenStack Development Mailing List (not for usage questions)
Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev