Skip to content

Add Ironic node allocation helpers#3786

Open
Sharpz7 wants to merge 4 commits into
gophercloud:mainfrom
Sharpz7:ironic-node-allocations
Open

Add Ironic node allocation helpers#3786
Sharpz7 wants to merge 4 commits into
gophercloud:mainfrom
Sharpz7:ironic-node-allocations

Conversation

@Sharpz7

@Sharpz7 Sharpz7 commented Jun 1, 2026

Copy link
Copy Markdown
Contributor

Summary

This PR adds the remaining Ironic node allocation API helpers:

  • get the allocation associated with a node by UUID or name
  • delete the allocation associated with a node by UUID or name
  • support the documented fields selector for node allocation lookup
  • add acceptance coverage for node-associated allocation lookup and deletion

This was created with assistance from OpenAI Codex.

Closes part of #1429.

Testing

  • env GOCACHE=/tmp/go-build GOMODCACHE=/tmp/go-mod go test ./openstack/baremetal/v1/allocations/testing
  • env GOCACHE=/tmp/go-build GOMODCACHE=/tmp/go-mod go test -tags 'acceptance baremetal allocations' ./internal/acceptance/openstack/baremetal/v1 -run TestNonexistent -count=0

@Sharpz7

Sharpz7 commented Jun 1, 2026

Copy link
Copy Markdown
Contributor Author

This PR was created with assistance from OpenAI Codex. Maintainers are free to close this PR with no explanation.

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

coveralls commented Jun 1, 2026

Copy link
Copy Markdown

Coverage Status

coverage: 63.976% (-0.005%) from 63.981% — Sharpz7:ironic-node-allocations into gophercloud:main

@Sharpz7 Sharpz7 marked this pull request as ready for review June 2, 2026 16:57
@Sharpz7 Sharpz7 changed the title [experimental] Add Ironic node allocation helpers Add Ironic node allocation helpers Jul 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport-v2 This PR will be backported to v2 edit:baremetal This PR updates baremetal code semver:minor Backwards-compatible change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants