talos/internal
Jorik Jonker 1e19e3720f fix: prefer extraConfig over OVF env, skip empty config
Some users reported that OVF env takes precedence over extraConfig.
Combined with the fact that `talos.config` is mandatory in the OVA spec,
this causes two problems:
- when you don't supply the mandatory OVF env, it will cause an empty
  Talos config to be retrieved, which is unparsable;
- when you deploy a VM, you cannot override the built in Talos config,
  as OVF env takes precedence over extraConfig.

This patch fixes this in two ways:
- config from OVF env needs to be non-empty.
- the order in which OVF env/extraConfig are probed is reversed.

Signed-off-by: Jorik Jonker <jorik.jonker@eu.equinix.com>
(cherry picked from commit 5aede1a83313152bd83891d0cae4b388a54bd9c2)
2021-06-22 14:54:23 +03:00
..