Skip to content
This repository has been archived by the owner on Oct 28, 2022. It is now read-only.

Add pagination support #639

Merged
merged 3 commits into from
Mar 7, 2021
Merged

Add pagination support #639

merged 3 commits into from
Mar 7, 2021

Conversation

dcoghlan
Copy link
Contributor

Pagination support for the following cmdlets. These are required when environments have over 1024 security tags or service composer security policies

  • Get-NsxSecurityPolicy
  • Get-NsxSecurityTag

@dcoghlan
Copy link
Contributor Author

jenkins test this please

@vmware-archive vmware-archive deleted a comment from powernsxbot Feb 16, 2021
@vmware-archive vmware-archive deleted a comment from powernsxbot Feb 16, 2021
@powernsxbot
Copy link

Tests Successful

@alagoutte
Copy link
Contributor

it is not possible to include directly on Invoke-NSXRestMethod ?

@dcoghlan
Copy link
Contributor Author

dcoghlan commented Mar 7, 2021

it is not possible to include directly on Invoke-NSXRestMethod ?

it is possible, but would take substantially more amount of time and effort to implement and make sure we didn't break anything, not to mention that there are multiple implementations of pagination implemented within the NSX-v API, so it would need to account for all the differing implementations.

@dcoghlan dcoghlan merged commit 018a4af into vmware-archive:master Mar 7, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants