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

CASMTRIAGE-5647: Update cray-keycloak-users-localize #2555

Merged
merged 2 commits into from
Jul 11, 2023

Conversation

ndavidson-hpe
Copy link
Contributor

Summary and Scope

This changes the TTL for deletion of the keycloak-users-localize job so it no longer gets deleted automatically so the test for this job being done never fails, as well as changes the initial setup of LDAP on keycloak so the LDAP federation on keycloak is enabled at startup which allows all users to be seen right after sync instead of having an error.

Issues and Related PRs

Testing

Tested on:

  • Fanta

Test description:

Ran the new version on fanta to check the ttl was correct in the new job and that the admin UI had users available after sync without disabling and enabling.

  • Were the install/upgrade-based validation checks/tests run (goss tests/install-validation doc)?y
  • Were continuous integration tests run? If not, why?y
  • Was upgrade tested? If not, why?y
  • Was downgrade tested? If not, why?y
  • Were new tests (or test issues/Jiras) created for this change?y

Risks and Mitigations

No risks or mitigations

Pull Request Checklist

  • Version number(s) incremented, if applicable
  • Copyrights updated
  • License file intact
  • Target branch correct
  • CHANGELOG.md updated
  • Testing is appropriate and complete, if applicable
  • HPC Product Announcement prepared, if applicable

@ndavidson-hpe ndavidson-hpe requested a review from a team as a code owner July 11, 2023 17:19
@ndavidson-hpe
Copy link
Contributor Author

/backport stable/1.5

@github-actions
Copy link

Backporting into branch stable/1.5 was successful. New PR: #2556

@denniswalker denniswalker merged commit 7f5dd91 into main Jul 11, 2023
1 check passed
@denniswalker denniswalker deleted the CASMTRIAGE-5647 branch July 11, 2023 19:53
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