An unrelated logic change appears to have been accidentally included
with #2851
This PR reverts the update to use EqualFold, and the discarding of the
ToLower calls so the function reads as it was when it was approved here:
8abaa39f97
Signed-off-by: Kingdon Barrett <kingdon@weave.works>
This sets the container to `manager` which is used by all Flux
controllers as the container name.
The other options I thought about were selecting the first, or doing
something with image detection. But both can be sensitive to either
users adding their patch as a first entry, or e.g. mirroring the image
to a different name.
Signed-off-by: Hidde Beydals <hello@hidde.co>
Use `app.kubernetes.io/part-of: flux` label instead of `app.kubernetes.io/instance` to select the in-cluster objects used in flux version, check, logs and uninstall commands.
Signed-off-by: Stefan Prodan <stefan.prodan@gmail.com>