Skip to content

feat: add Keycloak details in CI run#57

Open
edivalentinitu wants to merge 2 commits intomainfrom
edit-kc-ci
Open

feat: add Keycloak details in CI run#57
edivalentinitu wants to merge 2 commits intomainfrom
edit-kc-ci

Conversation

@edivalentinitu
Copy link
Contributor

@edivalentinitu edivalentinitu commented Jan 12, 2026

Add posibility to inject arguments for KeycloakSettingsHelper configuration class from a yaml file to the invenio.cfg in CI via python script. This helps to choose between different keycloak providers without changing the invenio.cfg.

@edivalentinitu edivalentinitu force-pushed the edit-kc-ci branch 2 times, most recently from 072cb0b to 3fe4058 Compare January 13, 2026 08:12
@edivalentinitu edivalentinitu marked this pull request as ready for review January 13, 2026 08:13
@edivalentinitu edivalentinitu linked an issue Jan 13, 2026 that may be closed by this pull request
the placeholder this instance.
"""

OAUTHCLIENT_REMOTE_APPS = {
Copy link
Contributor

Choose a reason for hiding this comment

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

what about this?

Copy link
Contributor Author

@edivalentinitu edivalentinitu Jan 14, 2026

Choose a reason for hiding this comment

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

we will have only one app which will have the configuration injected at CI runtime. maybe as a future task we could extend the template to add multiple configs/apps.

@edivalentinitu edivalentinitu force-pushed the edit-kc-ci branch 4 times, most recently from d39d067 to 87e70e0 Compare February 23, 2026 10:02
run: |
source .venv/bin/activate
if [[ ${{ !startsWith( github.ref, 'refs/tags') }} ]]; then
KEYCLOAK_NODE="cyverse"
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
KEYCLOAK_NODE="cyverse"
KEYCLOAK_NODE="rdm"

title: "Cyverse SSO"
description: Cyverse SSO
base_url: https://keycloak.cyverse.at
realm: CyVerse
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
realm: CyVerse
realm: rdm

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.

Keycloak auth providers configurable via GitHub Actions

2 participants