Skip to content

Use conditional for keycloak-operator CR proxies injection

Bogdan-Adrian Burciu requested to merge keycloak-operator-proxy into main

What does this MR do and why?

Move keycloak-operator CR Kustomize patching for proxies logic to conditional injection, so that when .Values.proxies.https_proxy is not set, we don't attempt to define null values the Keycloak operator dislikes.

Related reference(s)

Possibly a fix to #376 (closed), while it was recently experienced on a CAPM3 deployment where no proxies were used.

Test coverage

It does not bring any drawback to the current CAPO CI tests (with .Values.proxies.https_proxy), while no test could be yet done for an env where such proxies are not needed.

Edited by Bogdan-Adrian Burciu

Merge request reports