Skip to content

Add resource provider inventories API for placement#1862

Merged
jtopjian merged 9 commits intogophercloud:masterfrom
dkt26111:placement_rp_inventories
Feb 29, 2020
Merged

Add resource provider inventories API for placement#1862
jtopjian merged 9 commits intogophercloud:masterfrom
dkt26111:placement_rp_inventories

Conversation

@dkt26111
Copy link
Copy Markdown
Contributor

For #526

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

API doc:
https://docs.openstack.org/api-ref/placement/#list-resource-provider-inventories

API code:
https://github.com/openstack/placement/blob/master/placement/handlers/inventory.py#L259

@dkt26111
Copy link
Copy Markdown
Contributor Author

This PR is based on top of #1861 so it contains the resource provider usage changes as well.

@coveralls
Copy link
Copy Markdown

coveralls commented Feb 27, 2020

Coverage Status

Coverage increased (+0.02%) to 77.235% when pulling 0ea0694 on dkt26111:placement_rp_inventories into ab34fe0 on gophercloud:master.

@dkt26111
Copy link
Copy Markdown
Contributor Author

@jtopjian This is ready for review as well.

@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci bot commented Feb 28, 2020

Build succeeded.

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.

@dkt26111 Thanks for this, too! Similar comment. Please let me know if I've overlooked something.

@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci bot commented Feb 29, 2020

Build failed.

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
Copy link
Copy Markdown
Contributor

I'm pretty sure the OpenLab error is transient/temporary.

@jtopjian jtopjian merged commit b79ada0 into gophercloud:master Feb 29, 2020
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