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

Add example for using SPIRE for mTLS with Keycloak #248

Merged
merged 8 commits into from
Feb 19, 2024

Conversation

moritzschmitz-oviva
Copy link
Contributor

No description provided.

@kfox1111
Copy link
Collaborator

Very interesting. Thanks for working on this. :)

@kfox1111
Copy link
Collaborator

For setting up the initial values, would it make sense to use the charts built in bootstrapping capabilities?
https://artifacthub.io/packages/helm/bitnami/keycloak#keycloak-config-cli-parameters

can use it like:

keycloakConfigCli:
  enabled: true
  configuration:
    test.json: |
      {
        "realm": "test",
        "enabled": true,
      ...

@moritzschmitz-oviva
Copy link
Contributor Author

For setting up the initial values, would it make sense to use the charts built in bootstrapping capabilities? https://artifacthub.io/packages/helm/bitnami/keycloak#keycloak-config-cli-parameters

can use it like:

keycloakConfigCli:
  enabled: true
  configuration:
    test.json: |
      {
        "realm": "test",
        "enabled": true,
      ...

Yes, I will check. Probably better way to go.

@moritzschmitz-oviva moritzschmitz-oviva force-pushed the example/keycloak-mtls branch 3 times, most recently from ec88470 to 47ef967 Compare February 16, 2024 09:06
@moritzschmitz-oviva moritzschmitz-oviva marked this pull request as ready for review February 16, 2024 09:06
@kfox1111 kfox1111 changed the title Add example for using Spire for mTLS with Keycloak Add example for using SPIRE for mTLS with Keycloak Feb 18, 2024
moritzschmitz-oviva and others added 2 commits February 19, 2024 08:33
Co-authored-by: kfox1111 <[email protected]>
Signed-off-by: Moritz Schmitz von Hülst <[email protected]>
…hart and pin node version to it has a matching rancher/kubectl image

Signed-off-by: Moritz Schmitz von Hülst <[email protected]>
Copy link
Collaborator

@kfox1111 kfox1111 left a comment

Choose a reason for hiding this comment

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

This looks great. Thanks! :)

@kfox1111 kfox1111 merged commit 43a72a2 into spiffe:main Feb 19, 2024
55 checks passed
@v0lkan
Copy link

v0lkan commented Feb 28, 2024

+1 for this contribution 🤘.

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