kne icon indicating copy to clipboard operation
kne copied to clipboard

Add check for topology create to validate that images exist before starting cluster

Open mhines01 opened this issue 3 years ago • 1 comments

Add at least a warning that says if images are present on the cluster for the images referenced in the topology. This should be added in the form of an API check to the node implementations. If the node implementation is provided by a controller the controller should response to a RPC from the cli on if the controller can serve that image version

mhines01 avatar Sep 04 '21 18:09 mhines01

Do you mean a per-node type implementation, for example during node.New() for ceos (https://github.com/google/kne/blob/e76ec3a7476a47cefbf762f2c1a0cb2fecc0b54e/topo/node/ceos/ceos.go#L42) so that way the defaults for the node type could be set first?

Or do you mean an addition to the base node.go New() fn that would validate the proto? https://github.com/google/kne/blob/e76ec3a7476a47cefbf762f2c1a0cb2fecc0b54e/topo/node/node.go#L114

alexmasi avatar Sep 08 '21 00:09 alexmasi

this has been fixed to fail early in the topology creation if the image cannot be pulled: https://github.com/openconfig/kne/pull/398

alexmasi avatar Aug 22 '23 23:08 alexmasi