Add Get to placement resourceproviders#2358
Merged
mandre merged 3 commits intogophercloud:masterfrom Mar 18, 2022
Merged
Conversation
|
Build failed.
|
d8f3c7b to
eba6c0b
Compare
|
Build succeeded.
|
mandre
approved these changes
Feb 26, 2022
| defer DeleteResourceProvider(t, client, resourceProvider.UUID) | ||
|
|
||
| resourceProvider, err = CreateResourceProviderWithParent(t, client, resourceProvider.UUID) | ||
| resourceProvider2, err := CreateResourceProviderWithParent(t, client, resourceProvider.UUID) |
Contributor
There was a problem hiding this comment.
Oops, this belongs to #2357 but not a big deal as we'll likely merge your PRs very close to each other.
Contributor
|
please rebase on master to run the new tests for placement API |
Contributor
Author
Dont have much time atm, but will try to do at the end of this week |
Add ParentProviderUUID to placement/v1/resourceproviders createOpts. Moreover restructure acceptance tests for consistency and for future additions. [Docs](https://docs.openstack.org/api-ref/placement/?expanded=create-resource-provider-detail) Relates to: gophercloud#526
eba6c0b to
60fab0f
Compare
|
Build succeeded.
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Docs
Relates to: #526
PR-train: requires #2357