This PR has a number of changes, mostly tied to operator behaviors and bug fixes around failure logging. Included:
URI Encoding of client ID on deletion/updates - when we call updates/deletions on KC clients it gets appended to our URL for our request and must be encoded
Moves around the try/catch to only wrap the Keycloak API call so that errors are surfaced more accurately in events and modifies the thrown error from the Keycloak response to include the keycloak response status and data (see https://github.com/defenseunicorns/uds-core/issues/448)
Adds a new Phase to Package for PendingRetry - this differentiates from a Pending package that is already being reconciled to allow retries to function as expected (see https://github.com/defenseunicorns/uds-core/issues/492)
Moves uidSeen addition to original location (before starting reconcile) to handle infinite failure -> pending -> failure ... loop on first apply
Description
This PR has a number of changes, mostly tied to operator behaviors and bug fixes around failure logging. Included:
PendingRetry
- this differentiates from aPending
package that is already being reconciled to allow retries to function as expected (see https://github.com/defenseunicorns/uds-core/issues/492)Related Issue
Fixes https://github.com/defenseunicorns/uds-core/issues/492
Fixes https://github.com/defenseunicorns/uds-core/issues/448
Type of change
Checklist before merging