Repository navigation
Support explicit pagination in VMware collection reads - #63
Merged
Merged
Conversation
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
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.
Scope
Add optional
query_params=NonetoVMwareRead.list()for explicit collection pagination through the existing transport. Preserve omitted/None/empty defaults, complete response envelope and Decimal values, and composition-only surface. No automatic traversal or lifecycle expansion.Exactly three files: reader, six synthetic offline regression tests, finite CI count 216→222. Dependencies and workflows unchanged.
Qualification
Frozen base f1543d3; accepted tree 35e9d77cf0356d847c924cac0f3edd3a2d0ad3f0; patch SHA256 0924fb4589b3ab3ac15af79713c9e6aef0b3d34b5d4104e7085d889eea6d9597. Independent SAFE review SHA256 b748f49d1a47fce096baa622aea5d501fbc64b5533ccdaf8970a4ec2cd2e8c74.
Retained native baseline regression fails; candidate focused 6 PASS/full 222 PASS. Independent bounded VMware suite 11 PASS. Current-head installed-wheel Python 3.10/3.11/3.12 hosted checks required before merge.
Limits
VMware integration remains BETA. New fixtures are synthetic; historical malformed documentation response caveat remains. No nested-schema, arbitrary-filter, live availability, whole-operation, provider or release qualification. No live cloud/resource operations. Existing holds and OTP blocker remain unchanged.