openstack
/
oslo.context
Code Issues Proposed changes

Change instance_uuid to resource_uuid 

Use an application-agnostic term for the resource the context is
attached to.

bp/app-agnostic-logging-parameters

Change-Id: I548120770d0f11a2af56f07d0902433a8c7c98fe
This commit is contained in:
Doug Hellmann 2014-11-19 11:04:52 -05:00
parent 384c0de994
commit cac31d9c6f
2 changed files with 6 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
oslo_context/context.py
View File

@ -41,7 +41,7 @@ class RequestContext(object):
def __init__(self, auth_token=None, user=None, tenant=None, domain=None,
user_domain=None, project_domain=None, is_admin=False,
read_only=False, show_deleted=False, request_id=None,
instance_uuid=None):
resource_uuid=None):
self.auth_token = auth_token
self.user = user
self.tenant = tenant
@ -51,7 +51,7 @@ class RequestContext(object):
self.is_admin = is_admin
self.read_only = read_only
self.show_deleted = show_deleted
self.instance_uuid = instance_uuid
self.resource_uuid = resource_uuid
if not request_id:
request_id = generate_request_id()
self.request_id = request_id
@ -74,7 +74,7 @@ class RequestContext(object):
'show_deleted': self.show_deleted,
'auth_token': self.auth_token,
'request_id': self.request_id,
'instance_uuid': self.instance_uuid,
'resource_uuid': self.resource_uuid,
'user_identity': user_idt}
@classmethod
@ -90,7 +90,7 @@ class RequestContext(object):
read_only=ctx.get("read_only", False),
show_deleted=ctx.get("show_deleted", False),
request_id=ctx.get("request_id"),
instance_uuid=ctx.get("instance_uuid"))
resource_uuid=ctx.get("resource_uuid"))
def get_admin_context(show_deleted=False):

4
oslo_context/tests/test_context.py
View File

@ -40,7 +40,7 @@ class ContextTest(test_base.BaseTestCase):
"read_only": True,
"show_deleted": True,
"request_id": "request1",
"instance_uuid": "instance1",
"resource_uuid": "instance1",
"extra_data": "foo"
}
ctx = context.RequestContext.from_dict(dct)
@ -54,4 +54,4 @@ class ContextTest(test_base.BaseTestCase):
self.assertTrue(ctx.read_only)
self.assertTrue(ctx.show_deleted)
self.assertEqual("request1", ctx.request_id)
self.assertEqual("instance1", ctx.instance_uuid)
self.assertEqual("instance1", ctx.resource_uuid)