openstack
/
charm-interface-keystone
Code Issues Proposed changes

Fix checking auto_accessors for none

This commit is contained in:
Liam Young 2015-12-07 10:57:58 +00:00
parent 3ff28b362e
commit fab262721a
1 changed files with 2 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
requires.py
View File

@ -35,8 +35,6 @@ class KeystoneRequires(RelationBase):
self.set_state('{relation_name}.available')
else:
self.remove_state('{relation_name}.available')
if not self.keystone_hosts():
self.remove_state('{relation_name}.connected')
@hook('{requires:keystone}-relation-changed')
def changed(self):
@ -51,7 +49,8 @@ class KeystoneRequires(RelationBase):
Get the connection string, if available, or None.
"""
for field in self.auto_accessors:
if not self.get(field):
attr_method = getattr(self, field.replace('-', '_'))
if not attr_method():
return False
return True