Add SSO checks before using auth information

General MR

Summary

Fixes an issue that appears when overriding the cr to include the auth type for openid and not turning on sso.

kiali:
  enabled: true
  values:
    cr:
      spec:
        auth:
          strategy: "openid"

This caused downstream resources to assume that values existed, when they did not due to sso being disabled.

Relevant logs/screenshots

Log Error

message: 'Helm install failed for release kiali/kiali-kiali with chart kiali@1.89.0-bb.0: template: kiali/templates/bigbang/ssoServiceEntry.yaml:7:50: executing "kiali/templates/bigbang/ssoServiceEntry.yaml" at <.Values.cr.spec.auth.openid.issuer_uri>: nil pointer evaluating interface {}.issuer_uri'

Linked Issue

No issue linked. Found as part of testing changes related to this issue in Nexus Repository Manager.

Upgrade Notices

N/A

assigned to @kylegalloway-seed

added 1 commit

• 5041ca98 - Update other instances of openid

Compare with previous version

added 1 commit

• b9f86845 - Surround and with parentheses

Compare with previous version

added 1 commit

• 8029d2bf - Fix and in egress-sso

Compare with previous version

added 1 commit

• b73b36b8 - Use correct value path

Compare with previous version

added 1 commit

• 46db88d0 - Spacing

Compare with previous version

added 1 commit

• 6e258f5d - Force sso check

Compare with previous version

added 1 commit

• e981ff77 - Update docs/versions

Compare with previous version

marked this merge request as ready

changed title from Draft: Add sso check for service entry to Add SSO checks before using auth information

changed the description

added statusreview label

requested review from @kipten, @kylegalloway-seed, @lgomez2, @ronwerth, @stephen.galamb, and @wyatt.fry

requested review from @dpritchettrm and @michaelmartin

added skip-bb-mr label

added 1 commit

• 9bc55fad - Update README.md fix markdown linting error

Compare with previous version

added 1 commit

• 1299e883 - Fix markdownlint issues

Compare with previous version

approved this merge request

mentioned in merge request big-bang/bigbang!5045 (merged)

mentioned in commit dbe12c05