Mgmt kubeconfig flag

[daimaxiaxie]

+--------+                         +-----------+             +----------+
|  mgmt  |                         |           |             | workload |
|        |  cluster-api-kubeconfig | --------- | kubeconfig  |          |
|        |<------------------------+ karpenter +------------>|          |
+--------+                         +-----------+             +----------+

ClusterAPINodeClass in work cluster, Machine/MachineDeployment in manager cluster.

[elmiko]

thanks @daimaxiaxie , i will need some time to study this, but in general i would like to defer on adding the extra clients until we have a clearer picture of how deployment will work. i'm also not sure about the client we will pass to the karpenter core for reading the nodes and pods.

i think this is a good start though.

[elmiko]

@daimaxiaxie the one thing i would like to see in this PR is a default mechanism so that if the user does not specify the --cluster-api-kubeconfig it uses the original --kubeconfig (or service account credentials).

[daimaxiaxie]

It should be so.

[elmiko]

this looks great, thanks for the update. i'm going to test it locally and if everything works as expected i will merge this week.

[elmiko]

i have a minor request to change the flag text, but otherwise i think this is working as expected.

i am continuing to run some local tests.

[elmiko]

thanks for the updates!