This means the controller will see it as unchanged when it fires up.

It's awkward having to do this every time the controller does anything.

so it can be used by other packages

Signed-off-by: Dennis Marttinen <dennis@weave.works>

Almost everywhere uses Public.Address or Private.Address,
so these other ones are just a source of confusion.

Blank means use the default, but when ImageRepository is set the
current code is failing.

To fit with upstream requirements:
https://cluster-api.sigs.k8s.io/developer/architecture/controllers/cluster.html#infrastructure-provider

v1alpha3 won't run on anything earlier.

Allow wks-controller role to write status subresources
but not to write to capi objects.

Also do it on a separate client because the manager isn't started yet.