-
Notifications
You must be signed in to change notification settings - Fork 6
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
Upgrade terraform-provider-keycloak to v4.1.0 #162
Upgrade terraform-provider-keycloak to v4.1.0 #162
Conversation
424f4ee
to
dbcda5f
Compare
This is a major version upgrade.
dbcda5f
to
d5f6b0b
Compare
3d2e60b
to
f531530
Compare
f531530
to
68afc06
Compare
Does the PR have any schema changes?Looking good! No breaking changes found. New resources:
New functions:
|
68afc06
to
6ba39a0
Compare
Does the PR have any schema changes?Looking good! No breaking changes found. New resources:
New functions:
|
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.
LGTM
That sounds good thank you. You'll need to edit ci-mgmt and re-sync it back here, I'd appreciate that, because that sets us up for success in the near future for automated PRs from ci-mgmt that mass-update providers not to wipe the changes here. |
Fixes #158
Fixes #155
Fixes #153
To get the tests working, I changed the value of
KEYCLOAK_URL
fromhttp://localhost:8080
tohttp://localhost:8080/auth
. After this PR merges, I'm going to follow up with a PR to ci-mgmt.