vultr / vultr-cli

Official command line tool for Vultr services
Apache License 2.0
472 stars 75 forks source link

[Feature] - Support paging for repositories in container registry #454

Open F21 opened 3 months ago

F21 commented 3 months ago

Is your feature request related to a problem? Please describe. When listing the repositories in a container registry, the results returned are pagable, however, the cli doesn't seem to support paging or cursors:

❯ docker run -e VULTR_API_KEY vultr/vultr-cli:latest container-registry repository list REDACTED --per-page=1
Error: unknown flag: --per-page
❯ docker run -e VULTR_API_KEY vultr/vultr-cli:latest container-registry repository list REDACTED --cursor=xxx
Error: unknown flag: --cursor

Describe the solution you'd like Implement paging and cursors for container registry

Describe alternatives you've considered n/a

Additional context n/a

optik-aper commented 3 months ago

That would be a limitation of the API, I'm afraid.

F21 commented 3 months ago

I think the API supports paging as it contains the meta object in the response.

optik-aper commented 3 months ago

Yeah, in the response but it doesn't appear the controller has it on the request. I'll look into it on the API side.