Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: upgrade RH-SSO from 7.5 to 7.6 #1058

Merged

Conversation

iam-veeramalla
Copy link
Collaborator

What type of PR is this?

/kind enhancement

What does this PR do / why we need it:
Shipped version of RH-SSO in current release is obsolete and out of support. This PR updates the RH-SSO to the latest version and updates the required changes.

Have you updated the necessary documentation?

  • Documentation update is required by this PR.
  • Documentation has been updated.

How to test changes / Special notes to the reviewer:
Run operator locally using make install run
Create an Argo CD instance and add Keycloak using the below spec:

.....
spec:
 sso:
  provider: keycloak
  keycloak:
   rootCA: "=== BEGIN END ==="
......

Verify the Keycloak pod is up and running.

* upgrade RH-SSO from 7.5 to 7.6

Signed-off-by: iam-veeramalla <[email protected]>

* fix: failing tests

Signed-off-by: iam-veeramalla <[email protected]>

* fix: failing tests

Signed-off-by: iam-veeramalla <[email protected]>

---------

Signed-off-by: iam-veeramalla <[email protected]>
…goproj-labs#1008)

* fix: keycloak probes failure results in pod crash

Signed-off-by: iam-veeramalla <[email protected]>

* fix: use latest keycloak image to handle performance issue

Signed-off-by: iam-veeramalla <[email protected]>

---------

Signed-off-by: iam-veeramalla <[email protected]>
Copy link
Collaborator

@jaideepr97 jaideepr97 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm
thanks @iam-veeramalla

@iam-veeramalla iam-veeramalla merged commit 5482a8c into argoproj-labs:release-0.7 Nov 7, 2023
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants