Skip to content

Identity: Support include_names query arg in /role_assignments#2133

Merged
jtopjian merged 1 commit intogophercloud:masterfrom
objque:identity-support-include-names-flag-in-role-assignments-endpoint
Mar 26, 2021
Merged

Identity: Support include_names query arg in /role_assignments#2133
jtopjian merged 1 commit intogophercloud:masterfrom
objque:identity-support-include-names-flag-in-role-assignments-endpoint

Conversation

@objque
Copy link
Copy Markdown
Contributor

@objque objque commented Mar 25, 2021

Keystone may include names into RoleAssignment entity, if provide include_names=true as query arguemnt

For #2124

Links to the line numbers/files in the OpenStack source code that support the
code in this PR:

https://docs.openstack.org/api-ref/identity/v3/index.html?expanded=id627-detail#id627

Keystone may include names into RoleAssignment entity, if provide include_names=true as query arguemnt

doc: https://docs.openstack.org/api-ref/identity/v3/index.html?expanded=id627-detail#id627
@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage remained the same at 79.846% when pulling 425f6af on objque:identity-support-include-names-flag-in-role-assignments-endpoint into 61ead6a on gophercloud:master.

@jtopjian
Copy link
Copy Markdown
Contributor

@objque Thanks. Just for reference, API documentation isn't used as it may be outdated. Instead, we need to see the actual Keystone code that defines this feature. This is explained here.

For this PR, the following links are relevant:

Please let me know when this PR is ready for review.

@objque
Copy link
Copy Markdown
Contributor Author

objque commented Mar 25, 2021

Yes, you definitely right. In the next PR's I will reference on the actual Keystone code.
This PR is ready for review. No wip/draft tag here :)

Copy link
Copy Markdown
Contributor

@jtopjian jtopjian left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM - thank you!

@jtopjian jtopjian merged commit d26cb14 into gophercloud:master Mar 26, 2021
@objque objque deleted the identity-support-include-names-flag-in-role-assignments-endpoint branch March 26, 2021 09:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants