v1beta2: Remove deprecated retryDelayMinutes field

[nerdeveloper]

What type of PR is this?

/kind cleanup
/kind api-change
/kind deprecation

What this PR does / why we need it:

This PR removes the long-deprecated retryDelayMinutes field from Kueue's v1beta2 AdmissionCheckSpec. The retryDelayMinutes field has been deprecated since v0.8 and was unused, providing no functionality.

Technical changes:

• Remove RetryDelayMinutes field from v1beta2 AdmissionCheckSpec
• Update conversion functions to ignore retryDelayMinutes field during v1beta1 to v1beta2 conversion
• Update test to remove retryDelayMinutes usage
• Regenerate deepcopy, conversion code, and CRD manifests

Migration note: The field remains available in v1beta1 for backward compatibility.

Which issue(s) this PR fixes:

Part of #7247

Does this PR introduce a user-facing change?

Remove deprecated retryDelayMinutes field from v1beta2 AdmissionCheckSpec. This field was deprecated since v0.8 and provided no functionality. Users using v1beta1 are unaffected.

[k8s-ci-robot]

Hi @nerdeveloper. Thanks for your PR.

I'm waiting for a github.com 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.

[netlify]

✅ Deploy Preview for kubernetes-sigs-kueue canceled.

Name	Link
🔨 Latest commit	8fb45ff
🔍 Latest deploy log	https://app.netlify.com/projects/kubernetes-sigs-kueue/deploys/69049ad4f11d3900081197a8

[mimowo]

/ok-to-test

[mbobrovskyi]

To fix linter you can use command make lint-fix

[nerdeveloper]

/retest

[PBundyra]

/retest

[mimowo]

@nerdeveloper please rebase, it looks almost ready to go

[nerdeveloper]

what is the lint issue, when i try the command, i get this

find . -path ./site -prune -false -o -name go.mod -exec dirname {} \; | xargs -I {} sh -c 'cd "{}" && /Users/odirionye/Documents/go/src/github.com/nerdeveloper/kueue/bin/golangci-lint run --fix --timeout 15m0s --config "/Users/odirionye/Documents/go/src/github.com/nerdeveloper/kueue/.golangci.yaml"'
Error: can't load config: the Go language version (go1.24) used to build golangci-lint is lower than the targeted Go version (1.25.0)
The command is terminated due to an error: can't load config: the Go language version (go1.24) used to build golangci-lint is lower than the targeted Go version (1.25.0)
Error: can't load config: the Go language version (go1.24) used to build golangci-lint is lower than the targeted Go version (1.25.0)
The command is terminated due to an error: can't load config: the Go language version (go1.24) used to build golangci-lint is lower than the targeted Go version (1.25.0)
Error: can't load config: the Go language version (go1.24) used to build golangci-lint is lower than the targeted Go version (1.25.0)
The command is terminated due to an error: can't load config: the Go language version (go1.24) used to build golangci-lint is lower than the targeted Go version (1.25.0)
make: *** [ci-lint] Error 1

[mbobrovskyi]

what is the lint issue, when i try the command, i get this

I think you need to upgrade golang version.

[mimowo]

@nerdeveloper please update the PR

[mimowo]

I think you need to rebase again. Recently there have been many PRs related to v1beta2 so it is moving fast in this area

[nerdeveloper]

i thkink i have messed up my git cos not sure why i am seeing 2018 changes

[mimowo]

yeah something looks odd, maybe redo from clean main branch

[mimowo]

/lgtm
/approve
/test pull-kueue-test-e2e-main-1-31
this is a known flake, currently being worked on under #7479

[k8s-ci-robot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: mimowo, nerdeveloper

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

The pull request process is described here

Needs approval from an approver in each of these files:

• OWNERS [mimowo]

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]

LGTM label has been added.

Git tree hash: 295c8b6904943ae867b6bb5f3300b017f4dc8467