Skip to content

Integration of Adgangskomponent - #34

Merged
hypesystem merged 5 commits into
developfrom
feature/adgangskomponent
Jul 1, 2026
Merged

Integration of Adgangskomponent#34
hypesystem merged 5 commits into
developfrom
feature/adgangskomponent

Conversation

@hypesystem

Copy link
Copy Markdown
Contributor

This PR integrates the OS2ai Adgangskomponent chart to provide authentication via Fælleskommunal Adgangsstyring (FKA).

The chart relies on the presence of a keycloak-operator already installed to manage the component.

The component itself is a configured Keycloak instance that on first setup (if configured correctly) creates a realm that integrates the openwebui instance to FKA.

Once the integration is set up, the Adgangskomponent must be registered as an IT-system in FKA for the connection to work.

@hypesystem
hypesystem requested a review from cableman June 11, 2026 07:28
@snapq
snapq requested a review from lasseborly June 17, 2026 08:14

@cableman cableman left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

You also need to update https://os2ai.github.io/documentation/technical/installation/deployment.html with information about installation and what placeholder should be replaces and with what.

project: searxng
namespace: searxng
automated: false
automated: true

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The changes here with automated is not related to adgangskomponent. If it is changed, https://os2ai.github.io/documentation/technical/installation/deployment.html, needs to be updated as well.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Right - will revert.

Let's talk about what the default policy should be at the next core meeting. I know the deployments running in Computerome all use automated: true for these settings, hence the suggested change.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Not to begin with @hypesystem. When bootstrapping a new setup automated is "false" to make room for creating the required sealed-secrets, before we set everything to automated: true and start rolling out the rest of the infra.

This uses the https://github.com/os2ai/adgangskomponent chart to run an OS2ai Adgangskomponent deployment in the cluster, providing a connection to Fælleskommunal Adgangsstyring.
@hypesystem
hypesystem force-pushed the feature/adgangskomponent branch from d798448 to 71ff269 Compare June 29, 2026 15:25
Instead use automated: false for workloads that may require evaluation before update
@hypesystem
hypesystem requested a review from cableman June 29, 2026 15:31
Comment thread applications/openwebui/values.yaml Outdated
- name: OAUTH_CLIENT_ID
value: os2ai
- name: OAUTH_CLIENT_SECRET
value: en3tKNYg1yKKAwfve9gtFG6DtsjKqq #Adgangskomponent TODO: regen secret on startup, place in secret, make available here.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This still feels weird to include?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Fixed

@hypesystem
hypesystem merged commit b06ccee into develop Jul 1, 2026
1 check passed
@hypesystem
hypesystem deleted the feature/adgangskomponent branch July 1, 2026 11:57
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.

3 participants