There was a problem hiding this comment.
Adds support and documentation for Nova server-create scheduler hints (specifically for placing servers into an existing server group) in the Compute v2 SDK workflow.
Changes:
Copilot reviewed 8 out of 8 changed files in this pull request and generated no comments.
Show a summary per file| tests/unit/Compute/v2/ServiceTest.php | Adds unit coverage asserting os:scheduler_hints is sent alongside server on create. |
| tests/sample/Compute/v2/TestCase.php | Ensures server addresses are retrieved before deletion so dependent network cleanup can run after delete. |
| tests/sample/Compute/v2/ServerGroupTest.php | Adds an integration/sample test demonstrating server creation inside a server group via scheduler hints. |
| src/Compute/v2/Params.php | Introduces schedulerHints parameter schema (sent as os:scheduler_hints). |
| src/Compute/v2/Api.php | Refactors postServer JSON shaping to use per-param JSON paths under server, enabling top-level scheduler hints. |
| samples/Compute/v2/server_groups/create_server.php | New executable sample showing schedulerHints.group usage on server create. |
| doc/services/compute/v2/server-groups.rst | Documents the new “Create a server in a group” workflow and links the sample. |
| AGENTS.md | Adds guidance for reusing resources in sample tests via @depends patterns. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
You can also share your feedback on Copilot code review. Take the survey.
Sorry, something went wrong.
for #434
add scheduler hints for server