Merge "identity: Remove TagMixin from Domain"
This commit is contained in:
commit
76829ff274
3 files changed
+7
-7
No files matched your search
@@ -14,7 +14,6 @@ from typing import TYPE_CHECKING
|
||||
|
||||
from keystoneauth1 import adapter
|
||||
|
||||
from openstack.common import tag
|
||||
from openstack import resource
|
||||
from openstack import utils
|
||||
|
||||
@@ -24,7 +23,7 @@ if TYPE_CHECKING:
|
||||
from openstack.identity.v3 import user as _user
|
||||
|
||||
|
||||
class Domain(resource.Resource, tag.TagMixin):
|
||||
class Domain(resource.Resource):
|
||||
resource_key = 'domain'
|
||||
resources_key = 'domains'
|
||||
base_path = '/domains'
|
||||
@@ -40,7 +39,6 @@ class Domain(resource.Resource, tag.TagMixin):
|
||||
_query_mapping = resource.QueryParameters(
|
||||
'name',
|
||||
is_enabled='enabled',
|
||||
**tag.TagMixin._tag_query_parameters,
|
||||
)
|
||||
|
||||
# Properties
|
||||
|
||||
@@ -67,10 +67,6 @@ class TestDomain(base.TestCase):
|
||||
'is_enabled': 'enabled',
|
||||
'limit': 'limit',
|
||||
'marker': 'marker',
|
||||
'tags': 'tags',
|
||||
'any_tags': 'tags-any',
|
||||
'not_tags': 'not-tags',
|
||||
'not_any_tags': 'not-tags-any',
|
||||
},
|
||||
sot._query_mapping._mapping,
|
||||
)
|
||||
|
||||
@@ -0,0 +1,6 @@
|
||||
---
|
||||
fixes:
|
||||
- |
|
||||
Remove incorrect ``TagMixin`` from ``Domain`` resource. The Keystone
|
||||
Identity API does not support tags on domains.
|
||||
[`bug 2166512 <https://bugs.launchpad.net/openstacksdk/+bug/2166512>`_]
|
||||
Reference in new issue
Block a user