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

add configurable pagination to nfd-gc

Open ivelichkovich opened this issue 11 months ago • 7 comments

adds command line flag to configure pagination for nfd-gc for issue: https://github.com/kubernetes-sigs/node-feature-discovery/issues/1998

ivelichkovich avatar Jan 04 '25 19:01 ivelichkovich

Hi @ivelichkovich. Thanks for your PR.

I'm waiting for a kubernetes-sigs member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

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 04 '25 19:01 k8s-ci-robot

Deploy Preview for kubernetes-sigs-nfd ready!

Name Link
Latest commit a7634030de78622db8863bc011f70cb6e696c40b
Latest deploy log https://app.netlify.com/sites/kubernetes-sigs-nfd/deploys/6779e65051012a0009dc2181
Deploy Preview https://deploy-preview-2001--kubernetes-sigs-nfd.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

netlify[bot] avatar Jan 04 '25 19:01 netlify[bot]

Let's tackle #2000 first and then align this PR and get it merged

marquiz avatar Jan 22 '25 10:01 marquiz

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: ivelichkovich, ozhuraki Once this PR has been reviewed and has the lgtm label, please assign arangogutierrez for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment Approvers can cancel approval by writing /approve cancel in a comment

k8s-ci-robot avatar Mar 26 '25 09:03 k8s-ci-robot

I'll update this one with the same stuff as the last PR

ivelichkovich avatar May 16 '25 06:05 ivelichkovich

I'll update this one with the same stuff as the last PR

Awesome @ivelichkovich, I was just about to comment on this :blush:

marquiz avatar May 16 '25 07:05 marquiz

@ivelichkovich any update on this? Would be cool to get in the next release

marquiz avatar May 28 '25 08:05 marquiz

LGTM

ffromani avatar Aug 05 '25 07:08 ffromani

PR needs rebase.

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 Aug 05 '25 07:08 k8s-ci-robot

@ivelichkovich would you rebase the PR, or let somebody else pick up this one?

marquiz avatar Sep 19 '25 16:09 marquiz

Superseded by #2294 /close

marquiz avatar Sep 22 '25 16:09 marquiz

@marquiz: Closed this PR.

In response to this:

Superseded by #2294 /close

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 Sep 22 '25 16:09 k8s-ci-robot