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

Refs/heads/test/keycloak26 from main #1104

Merged
merged 5 commits into from
Oct 30, 2024
Merged

Conversation

clean-coder
Copy link
Collaborator

No description provided.

@clean-coder clean-coder mentioned this pull request Oct 29, 2024
@clean-coder
Copy link
Collaborator Author

Kurzanalyse

Habe versucht das Problem etwas zu analysieren.

  • der Import des realm-pitc.json schlägt fehl
  • ich bin (neben SQL errors) auf folgende Error Message gestossen: KC-SERVICES0010: Failed to add user 'admin' to realm 'master': user with username exists
  • es scheint so zu sein, dass beim Import versucht wird ein User admin zu erstellen. Und dieser existiert bereits im json Dump. Man kann nun den Namen (im docker-compose.yml) von admin auf etwas andere anpassen, und Keycloak startet wieder und lokal kann ich sogar server und client starten und sich erfolgreich im OKR anmelden ... alles läuft wie gehabt. Intern wird in Keycloak dann einfach ein zusätzlicher User admin26 erstellt:
  keycloak-pitc:
    image: quay.io/keycloak/keycloak:26.0.1
    environment:
      - KC_BOOTSTRAP_ADMIN_USERNAME=admin26
      - KC_BOOTSTRAP_ADMIN_PASSWORD=keycloak26
  • leider funktioniert aber cypress nicht mehr, weder lokal noch auf dem Build Server

@clean-coder
Copy link
Collaborator Author

Fix Problem 1

Das Login bei KeyCloak schlägt fehl. Grund: bei KeyCloak v.26 hat sich die das Html der Login Form etwas geändert. Der Submit Button war früher ein input und ist nun ein button geworden. Anpassung in commands.ts hat das gefixt.

Fix Problem 2

Diverse Test in teammanagement.cy.ts schlagen fehl. Es hat sich als Timing Problem herausgestellt. Der Fix sind Aufruf(e) von cy.intercept() Calls. Danke an Yanick Minder für die Unterstützung

@clean-coder clean-coder linked an issue Oct 29, 2024 that may be closed by this pull request
@peggimann peggimann force-pushed the test/keycloak26_from_main branch from 451eac8 to e0489b3 Compare October 30, 2024 09:26
@peggimann peggimann merged commit bc1b429 into main Oct 30, 2024
5 checks passed
@peggimann peggimann deleted the test/keycloak26_from_main branch October 30, 2024 10:15
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.

Update keycloak Docker tag to v26
2 participants