-
Notifications
You must be signed in to change notification settings - Fork 427
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
Conversation
Integration tests failure for 28f1140f30eed5a99df6fbb51b25e3ea1592ae50 |
Integration tests failure for 8bdb93cf0f601bf95b4ba47f55cc438db405262d |
|
||
The following Terraform configuration can be used to authenticate using the OAuth access token | ||
|
||
``` |
There was a problem hiding this comment.
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: |
There was a problem hiding this comment.
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: |
There was a problem hiding this comment.
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 @@ | |||
{ |
There was a problem hiding this comment.
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?
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. |
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.