Skip to content

Implement resource provider allocations - #3713

Merged
mandre merged 5 commits into
gophercloud:mainfrom
Etua:implement-resource-provider-allocations
May 4, 2026
Merged

Implement resource provider allocations#3713
mandre merged 5 commits into
gophercloud:mainfrom
Etua:implement-resource-provider-allocations

Conversation

@Etua

@Etua Etua commented Apr 14, 2026

Copy link
Copy Markdown
Contributor

@github-actions github-actions Bot added edit:placement This PR updates placement code semver:minor Backwards-compatible change backport-v2 This PR will be backported to v2 labels Apr 14, 2026
@coveralls

coveralls commented Apr 14, 2026 •
edited
Loading

Copy link
Copy Markdown

Coverage Status

coverage: 63.996% (-0.03%) from 64.022% — Etua:implement-resource-provider-allocations into gophercloud:main

Etua added 4 commits April 14, 2026 18:20
@Etua
Implement Placement allocations GET
376810a 
Related gophercloud#526

API reference:
https://docs.openstack.org/api-ref/placement/?expanded=#list-allocations

Code reference:
https://opendev.org/openstack/placement/src/branch/stable/2026.1/placement/handlers/allocation.py#L260-L286
@Etua
Implement Placement allocations PUT
6991fdf 
Related gophercloud#526

API reference:
https://docs.openstack.org/api-ref/placement/?expanded=#update-allocations

Code reference:
https://opendev.org/openstack/placement/src/branch/stable/2026.1/placement/handlers/allocation.py#L391-L413
@Etua
Implement Placement allocations DELETE
0b535ca 
Related gophercloud#526

API reference:
https://docs.openstack.org/api-ref/placement/?expanded=#delete-allocations

Code reference:
https://opendev.org/openstack/placement/src/branch/stable/2026.1/placement/handlers/allocation.py#L376-L388
@Etua
Implement Placement allocations POST
ce2a0d5 
Related gophercloud#526

API reference:
https://docs.openstack.org/api-ref/placement/?expanded=#manage-allocations

Code reference:
https://opendev.org/openstack/placement/src/branch/stable/2026.1/placement/handlers/allocation.py#L166-L205
@Etua
Etua force-pushed the implement-resource-provider-allocations branch from c662a8d to ce2a0d5 Compare April 14, 2026 16:24
@Etua

Etua commented Apr 28, 2026

Copy link
Copy Markdown
Contributor Author

@mandre Hello, did you have time to look at this PR (and the subsequent #3715)? :)

mandre
mandre reviewed Apr 29, 2026
View reviewed changes

@mandre mandre left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Just a nit, from a quick review. Let me know if you want to address it, otherwise I'll do another pass.

Comment thread internal/acceptance/openstack/placement/v1/allocations_test.go Outdated
@github-actions github-actions Bot added the edit:testinfra This PR updates testing infrastructure code label May 4, 2026
@mandre
mandre enabled auto-merge May 4, 2026 10:10
@mandre
mandre added this pull request to the merge queue May 4, 2026
Merged via the queue into gophercloud:main with commit 7c95518 May 4, 2026
64 checks passed
@gophercloud-backport-bot gophercloud-backport-bot Bot mentioned this pull request May 4, 2026
Merged
@Etua
Etua deleted the implement-resource-provider-allocations branch May 4, 2026 10:17
@Etua Etua mentioned this pull request May 4, 2026
Merged
@gophercloud-backport-bot gophercloud-backport-bot Bot mentioned this pull request May 5, 2026
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mandre mandre mandre approved these changes

Assignees

No one assigned

Labels

backport-v2 This PR will be backported to v2 edit:placement This PR updates placement code edit:testinfra This PR updates testing infrastructure code semver:minor Backwards-compatible change

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@Etua @coveralls @mandre