ignite icon indicating copy to clipboard operation
ignite copied to clipboard

Pass vm.Labels to runtime containers

Open networkop opened this issue 4 years ago • 2 comments

Right now VM labels only exist on a VM itself. The only label that's propagated to the containers is ignite.name https://github.com/weaveworks/ignite/blob/main/pkg/operations/start.go#L77

It'd be nice to propagate the full set of labels to the underlying container as well. wdyt?

networkop avatar Jul 02 '21 13:07 networkop

Sounds good to me. We'll have to add some validation in the labels to prevent the user provided VM labels from overwriting the ignite set labels like ignite.name.

darkowlzz avatar Jul 04 '21 19:07 darkowlzz

Notable that k8s api machinery dictates labels be dns-1123 subdomains. That means that labels should be small and in a fairly restricted schema that should be compatible with most or all container runtimes. (currently, docker and containerd are supported by ignite.)

Should be okay to just pass down all VM labels to container runtime, with the exception of ignite runtime necessities like ignite.name.

This lets ignite propagate user-created partitions in their VM workloads into the container runtime, which can make cleanup and debugging more comprehensible.

stealthybox avatar Jul 12 '21 15:07 stealthybox