Open voigt opened 4 months ago
this will be a great addition. would adding the shim version as part of the status field makes sense as well? it will be useful to have that info exposed as status field that other tools can then read. (e.g. spinkube-doctor)
Interesting thought @rajatjindal! Any idea where to get this info from?
If you consider the shim spec, there is no reliable source to derive the shim version from. Here an example:
apiVersion: runtime.kwasm.sh/v1alpha1
kind: Shim
metadata:
name: spin-v2
labels:
[...]
spec:
nodeSelector:
spin: "true"
fetchStrategy:
type: annonymousHttp
anonHttp:
location: "https://github.com/spinkube/containerd-shim-spin/releases/download/v0.14.1/containerd-shim-spin-v2-linux-aarch64.tar.gz"
runtimeClass:
name: spin-v2
handler: spin
rolloutStrategy:
type: recreate
We could extract it from the filename or the URL path. But this is not a universally applicable approach. Not sure if it is worth to adjust the spec and add a version...? This again would require manual maintenance...
As a shim operator, I would like to see correct status information per shim in the
.shim.status
field.Furthermore we'd like to display proper conditions, depending on the rollout state of the shim.
Possible example:
Please consider this as an idea for inspiration only. I welcome any suggestions for displaying the relevant status information!