Ensure compatibility with default Kubernetes v1.34 clusters

[enoodle]

Bump the API version from v1beta1 to v1 in all relevant files, including Go imports, Helm charts, and test cases. This ensures compatibility with the latest Kubernetes resource APIs.

[linux-foundation-easycla]

The committers listed above are authorized under a signed CLA.

• ✅ login: enoodle / name: Erez Freiberger (f5336c9)

[k8s-ci-robot]

Welcome @enoodle!

It looks like this is your first PR to kubernetes-sigs/dra-example-driver 🎉. Please refer to our pull request process documentation to help your PR have a smooth ride to approval.

You will be prompted by a bot to use commands during the review process. Do not be afraid to follow the prompts! It is okay to experiment. Here is the bot commands documentation.

You can also check if kubernetes-sigs/dra-example-driver has its own contribution guidelines.

You may want to refer to our testing guide if you run into trouble with your tests not passing.

If you are having difficulty getting your pull request seen, please follow the recommended escalation practices. Also, for tips and tricks in the contribution process you may want to read the Kubernetes contributor cheat sheet. We want to make sure your contribution gets all the attention it needs!

Thank you, and welcome to Kubernetes. 😃

[nojnhuh]

For context, some of the versions were kept at v1beta1 in #116 to maintain compatibility back to Kubernetes v1.32. There are a couple of changes though that will make things a bit smoother for v1.34+ users:

• More intelligently pick an apiVersion for the DeviceClass in the Helm chart, like how the NVIDIA driver does it:
  • https://github.com/NVIDIA/k8s-dra-driver-gpu/blob/8b17bb27a245b7389ec99e9e39a3a201d85f762a/deployments/helm/nvidia-dra-driver-gpu/templates/deviceclass-gpu.yaml#L3
  • https://github.com/NVIDIA/k8s-dra-driver-gpu/blob/8b17bb27a245b7389ec99e9e39a3a201d85f762a/deployments/helm/nvidia-dra-driver-gpu/templates/_helpers.tpl#L177
• Process v1 (in addition to v1beta1) natively in the webhooks

@enoodle Would you be willing to reformulate this PR to address those points?

[enoodle]

Thanks @nojnhuh , I'll look into updating the PR with these changes, hopefully next week.

[enoodle]

@nojnhuh I updated the PR with the requested changes from github.com/NVIDIA/k8s-dra-drive-gpu. I also added v1beta2 because it seemed weird to leave it out.

[nojnhuh]

Thanks @enoodle! I promise I haven't lost track of this, it's still on my list to review.

[nojnhuh]

I tested that this doesn't obviously break upgrading from a previous version of the driver and that things work with a 1.34 kind cluster without any config overrides aside from adding a worker node.

Just left one minor cleanup comment.

[nojnhuh]

@bart0sh If you have perms to invoke CI for this PR that would be good to run. Thanks!

[nojnhuh]

@enoodle This looks good, thanks! Could you please squash your changes into one commit?

I think this more accurately reflects the new scope of the PR:

/retitle Ensure compatibility with default Kubernetes v1.34 clusters

@bart0sh @pohly @klueska I think this PR is good to go, but would like to get a positive signal from CI to make sure I'm not missing anything. I don't have permission to kick off the jobs, could one of you please do that?

[nojnhuh]

/lgtm
/approve

Thanks @enoodle!

[k8s-ci-robot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: enoodle, nojnhuh

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 [nojnhuh]

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