Skip to content

Conversation

@whiskeyjimbo
Copy link

info: this stacked off of #100 so when that is merged this will need to be synced.

since it adds extra logic to get the kubernetes version i figured it would be better off in a separate PR.

this PR adds a KubernetesVersion field to the VClusterStatus which is set from the client getting the server's version (git version) (which i think should distinguish between k8s/k3s/k0s(truthfully i havent tested k0s yet))

k get vcluster -n vcluster
NAME        VERSION   KUBERNETESVERSION   READY   PHASE      MESSAGE                                                                                                                                                            AGE
vcluster1   0.22.1    v1.31.1+k3s1        true    Deployed                                                                                                                                                                      3h
vcluster3   0.22.1    v1.32.0             true    Deployed                                                                                                                                                                      3h
vcluster4   0.22.1    v1.32.0             true    Deployed                                                                                                                                                                      3h
vcluster5   0.22.3    v1.32.0             true    Failed     error installing / upgrading vcluster: error executing helm upgrade: Error: chart "vcluster" version "0.22.33" not found in https://charts.loft.sh repository...   3h

@johannesfrey johannesfrey force-pushed the add-kubernetes-version-to-status branch from bc8a644 to 7bc688a Compare February 3, 2025 09:58
@johannesfrey johannesfrey force-pushed the add-kubernetes-version-to-status branch from 7bc688a to d360641 Compare February 6, 2025 07:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant