node-feature-discovery icon indicating copy to clipboard operation
node-feature-discovery copied to clipboard

Pagination size cmd line arg

Open ivelichkovich opened this issue 1 year ago • 6 comments

What would you like to be added:

configurable pagination for nfd-gc list request via command line arg

configurable pagination for nfd-master informer sync initial list via command line arg

related to this: https://github.com/kubernetes-sigs/node-feature-discovery/pull/1815

Why is this needed:

more control over request behavior to api-server

ivelichkovich avatar Jan 02 '25 23:01 ivelichkovich

/assign ivelichkovich

ivelichkovich avatar Jan 02 '25 23:01 ivelichkovich

Thanks @ivelichkovich for the request. Makes sense to me. Simplest solution would be to add a config file option /help

marquiz avatar Jan 03 '25 15:01 marquiz

@marquiz: This request has been marked as needing help from a contributor.

Guidelines

Please ensure that the issue body includes answers to the following questions:

  • Why are we solving this issue?
  • To address this issue, are there any code changes? If there are code changes, what needs to be done in the code and what places can the assignee treat as reference points?
  • Does this issue have zero to low barrier of entry?
  • How can the assignee reach out to you for help?

For more details on the requirements of such an issue, please see here and ensure that they are met.

If this request no longer meets these requirements, the label can be removed by commenting with the /remove-help command.

In response to this:

Thanks @ivelichkovich for the request. Makes sense to me. Simplest solution would be to add a config file option /help

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

k8s-ci-robot avatar Jan 03 '25 15:01 k8s-ci-robot

I'm happy to work on this :)

ivelichkovich avatar Jan 03 '25 19:01 ivelichkovich

Will raise an MR soon

ivelichkovich avatar Jan 03 '25 19:01 ivelichkovich

so I was actually thinking the MR I linked was adding pagination to nfd-master for its informer cache so I raised an MR adding configurable pagination to the NFD master: https://github.com/kubernetes-sigs/node-feature-discovery/pull/2000

and then here's the MR for configurable pagination to NFD GC: https://github.com/kubernetes-sigs/node-feature-discovery/pull/2001

ivelichkovich avatar Jan 04 '25 19:01 ivelichkovich