kwok icon indicating copy to clipboard operation
kwok copied to clipboard

🐛reuse kubectl found in path

Open Huang-Wei opened this issue 3 years ago • 4 comments
trafficstars

In my M1 env, I'd like to reuse the kubectl; otherwise you cannot download a darwin/arm64 kubectl binary:

w/o this PR, the following command won't work in M1:

KWOK_KUBE_VERSION=v1.20.15 kwokctl create cluster --runtime binary --kube-scheduler-binary /usr/local/bin/[email protected] --kube-controller-manager-binary /usr/local/bin/[email protected]  --kube-apiserver-binary /usr/local/bin/[email protected] --kwok-controller-binary /usr/local/bin/kwok --etcd-binary /opt/homebrew/bin/etcd --name 120-binary

Huang-Wei avatar Sep 06 '22 20:09 Huang-Wei

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: Huang-Wei

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:

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 avatar Sep 06 '22 20:09 k8s-ci-robot

/kind bug

Huang-Wei avatar Sep 06 '22 20:09 Huang-Wei

/retest

Huang-Wei avatar Sep 07 '22 00:09 Huang-Wei

Ooops seems to have triggered another bug, I've fixed it, #44 merge first and then rebase this RP

wzshiming avatar Sep 07 '22 09:09 wzshiming

Do we need to provide a way to disable kubectl in the path? Otherwise it will prompt

WARNING: version difference between client (1.24) and server (1.22) exceeds the supported minor version skew of +/-1

wzshiming avatar Sep 07 '22 10:09 wzshiming

Do we need to provide a way to disable kubectl in the path? Otherwise it will prompt

It doesn't bother me as kubectl version skew is more robust than the skew among other core components.

Huang-Wei avatar Sep 07 '22 18:09 Huang-Wei

/retest

Huang-Wei avatar Sep 14 '22 17:09 Huang-Wei

/lgtm

wzshiming avatar Sep 15 '22 02:09 wzshiming