(CAT-2193): Fixed kubernetes environment setup for Debian.

[span786]

Summary

Fixed kubernetes environment setup for Debian. This allows spec tests to run successfully for debian platform

Checklist

• 🟢 Spec tests.
• 🟢 Acceptance tests.
• Manually verified. (For example puppet apply)

[span786]

puppetlabs/provision#281 required to be merged before merging this.

[TheMeier]

Can you extract the version to a parameter. Also why set the default to v1.28 latests is v1.32

[span786]

Updated the default values to use latest v1.32 for kubernetes, also version is now extracted to $kubernetes_core_version.

[deric]

Can we move this to params instead of hardcoded string? Maybe call it minor_version (according to semantic versioning)?
It might be derived from kubernetes::version

$minor_version = $kubernetes::version.split('\.')[0, 2].join('.')

[span786]

This Kubernetes version is intended for core packages, which is why I've named it accordingly. Additionally, I've created a new params.pp file to consolidate all the default values.

[deric]

It doesn't make sense to use kubernetes::version = 1.28.10 and k8s_core_package_version = 1.32, right?. It doesn't make sense to me to define the same information on two places.

params.pp are considered anti-pattern, could you please move it to data/common.yaml resp. data/os/RedHat.yaml etc.?

[span786]

@deric

I was confused between kubernetes::version and k8s_core_package_version. Thank you for clarifying and helping me make sense of it. I've updated the code accordingly.