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

chore: add keycloak for acc tests #2286

Closed
wants to merge 3 commits into from
Closed

chore: add keycloak for acc tests #2286

wants to merge 3 commits into from

Conversation

sfc-gh-swinkler
Copy link
Collaborator

This adds documentation for running a keycloak server that can be used to test SAML/OAuth and Browser based workflows. More work will need to be done to automate the process. Perhaps with https://github.com/compose-spec/compose-go to start up container automatically.

Copy link

Integration tests failure for 28f1140f30eed5a99df6fbb51b25e3ea1592ae50

Copy link

Integration tests failure for 8bdb93cf0f601bf95b4ba47f55cc438db405262d


The following Terraform configuration can be used to authenticate using the OAuth access token

```
Copy link
Collaborator

Choose a reason for hiding this comment

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

double ```

- ./realm-export.json:/opt/keycloak/data/import/realm-export.json

networks:
webnet:
Copy link
Collaborator

Choose a reason for hiding this comment

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

empty options ?

webnet:

volumes:
realm-data:
Copy link
Collaborator

Choose a reason for hiding this comment

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

empty options ?

@@ -0,0 +1,2450 @@
{
Copy link
Collaborator

Choose a reason for hiding this comment

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

this is a really long json. I have a few questions:

  • Do we need all of it?
  • Isn't there any sensitive data leaked here?

@sfc-gh-asawicki
Copy link
Collaborator

This adds documentation for running a keycloak server that can be used to test SAML/OAuth and Browser based workflows. More work will need to be done to automate the process. Perhaps with https://github.com/compose-spec/compose-go to start up container automatically.

I like it. I think we can merge it without adding any automation for it now. We can add it as a part of SNOW-999050.

I did not try to run it myself yet, but the description looks solid. I left just one question for the realm json.

@sfc-gh-asawicki sfc-gh-asawicki deleted the keycloak branch July 3, 2024 09:59
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