Use kubeVersion to constrain kubernetes version when using helm

[hhk7734]

Proposed Changes

I moved the Kubernetes version restriction from values.yaml to Chart.yaml because I thought it would be nice to be limited by Helm.

[knative-prow]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: hhk7734
Once this PR has been reviewed and has the lgtm label, please assign psschwei for approval. For more information see the Kubernetes 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:

• OWNERS

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

[codecov]

Codecov Report

Patch and project coverage have no change.

Comparison is base (8129f02) 79.15% compared to head (12db174) 79.15%.
Report is 13 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #1503   +/-   ##
=======================================
  Coverage   79.15%   79.15%           
=======================================
  Files          40       40           
  Lines        1794     1794           
=======================================
  Hits         1420     1420           
  Misses        272      272           
  Partials      102      102           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[knative-prow]

@hhk7734: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name	Commit	Details	Required	Rerun command
eventing-upgrade-tests_operator_main	12db174	link	true	/test eventing-upgrade-tests

Your PR dashboard.

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/test-infra repository. I understand the commands that are listed here.

[houshengbo]

@hhk7734 The reason I put it in values is to make it configurable by different platforms. Like in my clusters, I need to override this value.

[hhk7734]

@houshengbo Is it better to leave both kubeVersion and kubernetes_min_version? Or should I close this PR?

As an additional question, is there a reason for using snake case? Most of the helm charts I've seen so far are in camel case, so it feels out of place. Also, I'm wondering if there's a reason why the apiVersion is v1. I know v1 is recognized by helm3, but I don't see the need to support older versions.

https://helm.sh/docs/topics/version_skew/

[knative-prow-robot]

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/test-infra repository.