You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The #969 PR introduced the pagination support, however the pagination didn't make sense, since the AllPages() method was used, which automatically fetches all pages.
This PR uses a proper pagination and requests only one page, when the MaxItems limit is provided.
What you expected to happen:
When ListVolumes is called with a limit, the function should return max the limited amount of volumes.
How to reproduce it:
Not possible. Limit was introduced as a spec and not used by any CSI controllers. However this bug can appear, when controllers start to set the limit option.
Anything else we need to know?:
Environment:
openstack-cloud-controller-manager(or other related binary) version:
OpenStack version:
Others:
The text was updated successfully, but these errors were encountered:
Is this a BUG REPORT or FEATURE REQUEST?:
/kind bug
What happened:
The #969 PR introduced the pagination support, however the pagination didn't make sense, since the
AllPages()
method was used, which automatically fetches all pages.This PR uses a proper pagination and requests only one page, when the
MaxItems
limit is provided.What you expected to happen:
When
ListVolumes
is called with a limit, the function should return max the limited amount of volumes.How to reproduce it:
Not possible. Limit was introduced as a spec and not used by any CSI controllers. However this bug can appear, when controllers start to set the limit option.
Anything else we need to know?:
Environment:
The text was updated successfully, but these errors were encountered: