Open Stack

Mark McLoughlin wrote:
>> To fix the stable/folsom issues, how about we say that the stable/folsom
>> tests only need to test the 0.2.0 version of keystoneclient, not master?
>>
>> That's as simple as this change:
>>
>>   @@ -789,7 +789,7 @@ class KeystoneClientTests(object):
>>  
>>    class KcMasterTestCase(CompatTestCase, KeystoneClientTests):
>>        def get_checkout(self):
>>   -        return KEYSTONECLIENT_REPO, 'master'
>>   +        return KEYSTONECLIENT_REPO, '0.2.0'
>>  
>>        def test_tenant_add_and_remove_user(self):
>>            client = self.get_client(admin=True)
> 
> Proposed here: https://review.openstack.org/18898

Mixed feelings: we are supposed to always use the latest client --
that's why there is no stable/folsom branch of the client.

The real issue here is the test itself. I'm fine with this as a
temporary workaround to unblock keystone stable/folsom, as long as it's
clear that this test needs to be rewritten (or moved to tempest).

-- 
Thierry Carrez (ttx)
Release Manager, OpenStack