Don't consider units with additional reconciliation annotation as Ready

Francois Eleouetrequested to merge
handle-addt-reconciliation into main

When 2 step upgrade is happening, we should not consider units with additional reconciliation annotation (essentially sylva-units-status kustomization) as Ready since another reconciliation is about to be triggered by the operator (see sylva-units-operator!566 (merged))

Note that 2 slightly unrelated changes have been added to this MR as distinct commits:

  • a first one to remove -a option from build command, since it was useless and causing local rebuilds to take a while
  • a second one to please the linter and adapt to the newer controller-runtime package version (required to use the newer suo version) that deprecates the use of client.Patch for SSA.

Related epic: sylva-projects#73 (closed) Related issue: sylva-projects/sylva-core#3782 (closed)

Edited by Francois Eleouet

Merge request reports