Skip to content

[Multi-Cloud] Fix: Add missing list pagination across 8 cloud drivers - #1808

Merged
cb-spider merged 1 commit into
cloud-barista:masterfrom
powerkimhub:fix/multi-cloud-list-pagination
Aug 17, 2026
Merged

[Multi-Cloud] Fix: Add missing list pagination across 8 cloud drivers#1808
cb-spider merged 1 commit into
cloud-barista:masterfrom
powerkimhub:fix/multi-cloud-list-pagination

Conversation

@powerkimhub

Copy link
Copy Markdown
Member

Problem: As the number of created resources grows, missing pagination currently causes incomplete listings or failed deletions for many resource types.

Test Note: Basic CRUD testing was completed for all affected resources, but pagination testing itself was excluded due to practical constraints such as account quotas.

  • GCP: Fix ListVMSpec() and VMHandler/NICHandler/AnyCallHandler/TagHandler/DiskHandler/SecurityHandler/VPCHandler/RDBMSHandler/NLBHandler/MyImageHandler to page through all zones via the Compute API's Pages() callback instead of a single unpaginated call
  • AWS: Convert NLB ListNLB/ListIID to DescribeLoadBalancersPages, and EFS ListFileSystem/TagHandler to DescribeFileSystemsPages (EFS defaults to MaxItems=100 per call)
  • Azure: Fix a WaitGroup double-Done() panic risk in ListRegionZone's zone-pager error branch (missing return after wg.Done())
  • Alibaba: Add PageNumber/PageSize and NextToken pagination loops across VM/Image (ListVM, ListMyImage), ListVMStatus, ListVPC, ListSecurity, ListNLB, NAS ListIID, VMSpec, PriceInfoHandler, Cluster/NodeGroup/VSwitch queries, and ECS/VPC/Subnet tag-search helpers
  • Tencent: Fix ListCluster()'s pre-sized-slice nil-pointer crash risk (slice was sized to TotalCount while only one page was fetched), and add Offset/Limit pagination to Disk, KeyPair, VPC/Subnet, NLB, VM, SecurityGroup, Image, and Tag handlers
  • IBM: Add shared getAllSnapshots/getAllShares/getAllSubnets cursor-pagination helpers across MyImageHandler, VMHandler, FileSystemHandler, NICHandler, NLBHandler; fix RDBMS listResourceInstances() to loop via Start/NextURL; fix TagHandler.FindTag() to actually reuse its SearchCursor; add gateway-listing helpers to ClusterHandler
  • NHN: Add nhnListAllNASVolumes() pagination helper for the NAS Volume list API, fixing 4 call sites that previously missed volumes beyond page one (causing false "not found" on by-name lookups)
  • KT Cloud VPC: Fix NLB ListIID() to loop via EachPage() instead of FirstPage(), matching its sibling ListNLB()

@powerkimhub
powerkimhub requested a review from cb-spider August 17, 2026 17:51
@powerkimhub powerkimhub added bug Something isn't working CloudDriver labels Aug 17, 2026
@cb-spider
cb-spider merged commit 5929735 into cloud-barista:master Aug 17, 2026
3 checks passed
@powerkimhub
powerkimhub deleted the fix/multi-cloud-list-pagination branch August 18, 2026 04:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working CloudDriver

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants