Skip to content
This repository was archived by the owner on Oct 23, 2024. It is now read-only.

Conversation

@smashwilson
Copy link
Contributor

Using objectrocket 0.4.6, I'm seeing this exception when attempting to list ACLs:

ERROR:objectrocket.acls:Could not instantiate ACL document. You probably need to upgrade to a recent version of the client. Document which caused this error: {u'cidr_mask': u'10.209.71.176', u'_cls': u'Acl.ESAcl', u'description': u'6f19a4a-staging-node-02', u'zone': u'US-East-IAD1', u'instance': u'k8s_cluster', u'kibana': False, u'instance_id': u'57696bd7ffbbf86002753b7f', u'instance_type': u'elasticsearch', u'user': u'573e200b5b33521d7f4df5f6', u'service_type': u'elasticsearch', u'date_created': u'1467773249', u'login': u'1006813', u'_id': u'577c0ed1dd0d821893f7f320', u'port': 10478, u'transport': False, u'metadata': {}}
ERROR:objectrocket.acls:'id'
Traceback (most recent call last):
  File "/Users/ashl6947/.venv/rax/lib/python2.7/site-packages/objectrocket/acls.py", line 106, in _concrete_acl
    return Acl(document=acl_doc, acls=self)
  File "/Users/ashl6947/.venv/rax/lib/python2.7/site-packages/objectrocket/acls.py", line 159, in __init__
    self._id = document['id']
KeyError: 'id'

From the response dump and the API documentation, it seems that the id field has been renamed to _id. This corrects the Acl constructor to match.

@zds97 zds97 merged commit 0a7587f into objectrocket:master Dec 8, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants