[YUNIKORN-3174] Read go mod to get the correct kubectl version

[pohanhuang]

What is this PR for?

• read go mod to get the correct kubectl version
• change the endpoint of kubectl downloading. (from docs https://kubernetes.io/docs/tasks/tools/install-kubectl-linux/)

What type of PR is it?

• - Bug Fix
• - Improvement
• - Feature
• - Documentation
• - Hot Fix
• - Refactoring

Todos

• - Task

What is the Jira issue?

• https://issues.apache.org/jira/browse/YUNIKORN-3174

How should this be tested?

Screenshots (if appropriate)

Questions:

• - The licenses files need update.
• - There is breaking changes for older versions.
• - It needs documentation.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 66.19%. Comparing base (0406fa7) to head (f95df04).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #998      +/-   ##
==========================================
+ Coverage   66.14%   66.19%   +0.05%     
==========================================
  Files          72       72              
  Lines        7723     7723              
==========================================
+ Hits         5108     5112       +4     
+ Misses       2401     2398       -3     
+ Partials      214      213       -1     

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

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[pohanhuang]

@chenyulin0719 should we update the default dev cluster with same setting as well?

yunikorn-k8shim/Makefile

Line 74 in e93bc42

K8S_VERSION := v1.34.0

to

K8S_VERSION :=$KUBECTL_VERSION
or
KUBECTL_VERSION=$(shell go list -m 'k8s.io/kubernetes' | cut -d' ' -f 2)

[chenyulin0719]

Hi @pohanhuang, thanks for submitting this PR.

Regarding changing K8S_VERSION := v1.34.0 to match the Kubernetes version in mod.go, my only concern is that there is a small time gap between the Kubernetes release and the corresponding kind release. However, we usually upgrade the Kubernetes version only after a new kind version has been released. So I think it should be fine.

[pohanhuang]

Hi @chenyulin0719 ,
I've update the with KUBECTL_VERSION=$(shell go list -m 'k8s.io/kubernetes' | cut -d' ' -f 2)

Please help review when you are avialable, thanks

[chenyulin0719]

+1 LGTM