diff options
author | Michael Johnson <johnsomor@gmail.com> | 2021-07-14 22:50:06 +0000 |
---|---|---|
committer | Michael Johnson <johnsomor@gmail.com> | 2021-10-20 15:43:30 +0000 |
commit | b124eb7df2617be56a23819f670bd6e054f87ce7 (patch) | |
tree | 0d309e74774aa3919631dca41776ce6744e29fec | |
parent | be0e41e0fdb6c4f08712abb5176123c8dcc63f46 (diff) | |
download | oslo-context-stable/wallaby.tar.gz |
Fix context from_dict() for system_scopewallaby-em3.2.1stable/wallaby
A previous patch[1] added "system_scope" to the context for to_dict()
but the from_dict() method was not updated. This caused the
system_scope to always be None.
This patch corrects that by adding "system_scope" to the list of
values that may need to be extracted from the context dict.
[1] https://review.opendev.org/c/openstack/oslo.context/+/530509
Change-Id: Ica23d5c4183a692de3cb65a7ad72b19f47988ca6
(cherry picked from commit 8290621b058081754db55c2584c00b8b166369aa)
(cherry picked from commit 7187c672d41fc54a0368b766c0fa7a59677964d8)
-rw-r--r-- | oslo_context/context.py | 1 | ||||
-rw-r--r-- | oslo_context/tests/test_context.py | 4 | ||||
-rw-r--r-- | releasenotes/notes/Fix-system-scope-from_dict-29615af90723ba4d.yaml | 4 |
3 files changed, 8 insertions, 1 deletions
diff --git a/oslo_context/context.py b/oslo_context/context.py index 2102791..94c2f8d 100644 --- a/oslo_context/context.py +++ b/oslo_context/context.py @@ -419,6 +419,7 @@ class RequestContext(object): values.get('project_domain_name')) kwargs.setdefault('is_admin_project', values.get('is_admin_project', True)) + kwargs.setdefault('system_scope', values.get('system_scope')) for key in cls.FROM_DICT_EXTRA_KEYS: kwargs.setdefault(key, values.get(key)) return cls(**kwargs) diff --git a/oslo_context/tests/test_context.py b/oslo_context/tests/test_context.py index fa44763..ef19369 100644 --- a/oslo_context/tests/test_context.py +++ b/oslo_context/tests/test_context.py @@ -143,7 +143,8 @@ class ContextTest(test_base.BaseTestCase): "request_id": "request1", "global_request_id": "req-uuid", "resource_uuid": "instance1", - "extra_data": "foo" + "extra_data": "foo", + "system_scope": "all" } ctx = context.RequestContext.from_dict(dct) self.assertEqual(dct['auth_token'], ctx.auth_token) @@ -163,6 +164,7 @@ class ContextTest(test_base.BaseTestCase): self.assertEqual(dct['domain_name'], ctx.domain_name) self.assertEqual(dct['user_domain_name'], ctx.user_domain_name) self.assertEqual(dct['project_domain_name'], ctx.project_domain_name) + self.assertEqual(dct['system_scope'], ctx.system_scope) def test_from_dict_unknown_keys(self): dct = { diff --git a/releasenotes/notes/Fix-system-scope-from_dict-29615af90723ba4d.yaml b/releasenotes/notes/Fix-system-scope-from_dict-29615af90723ba4d.yaml new file mode 100644 index 0000000..6d18376 --- /dev/null +++ b/releasenotes/notes/Fix-system-scope-from_dict-29615af90723ba4d.yaml @@ -0,0 +1,4 @@ +--- +fixes: + - | + Fixes context from_dict() to properly handle system_scope. |