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

Allow custom providerId in keycloak_user_federation #7789

Merged
merged 5 commits into from
Dec 31, 2023

Conversation

clifff
Copy link
Contributor

@clifff clifff commented Dec 28, 2023

SUMMARY

Removes the restriction from the keycloak_user_federation module that providerId can only be ldap, kerberos, or sssd.

ISSUE TYPE
  • Refactoring Pull Request
COMPONENT NAME

keycloak_user_federation

ADDITIONAL INFORMATION

Restricting the options for this fields prevents being able to manage custom federation adapters.

@ansibullbot
Copy link
Collaborator

@ansibullbot ansibullbot added module module needs_ci This PR requires CI testing to be performed. Please close and re-open this PR to trigger CI needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR new_contributor Help guide this first time contributor plugins plugin (any type) labels Dec 28, 2023
@ansibullbot

This comment was marked as outdated.

@ansibullbot ansibullbot added ci_verified Push fixes to PR branch to re-run CI and removed needs_ci This PR requires CI testing to be performed. Please close and re-open this PR to trigger CI labels Dec 28, 2023
@ansibullbot ansibullbot removed ci_verified Push fixes to PR branch to re-run CI needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR labels Dec 28, 2023
Copy link
Collaborator

@felixfontein felixfontein left a comment

Choose a reason for hiding this comment

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

Thanks for your contribution!

@felixfontein felixfontein added check-before-release PR will be looked at again shortly before release and merged if possible. backport-8 Automatically create a backport for the stable-8 branch labels Dec 28, 2023
@felixfontein
Copy link
Collaborator

Looks good to me! If nobody objects, I'll merge this in a few days.

@felixfontein felixfontein removed the check-before-release PR will be looked at again shortly before release and merged if possible. label Dec 31, 2023
@felixfontein felixfontein merged commit 08ece2e into ansible-collections:main Dec 31, 2023
121 checks passed
Copy link

patchback bot commented Dec 31, 2023

Backport to stable-8: 💚 backport PR created

✅ Backport PR branch: patchback/backports/stable-8/08ece2e0fabdcc698e003bcb480b1f0f17360f53/pr-7789

Backported as #7804

🤖 @patchback
I'm built with octomachinery and
my source is open — https://github.com/sanitizers/patchback-github-app.

patchback bot pushed a commit that referenced this pull request Dec 31, 2023
* Update keycloak_user_federation.py

* stub out changelog

* update PR reference

* remove 'choices' from documentation

* Apply suggestions from code review

Co-authored-by: Felix Fontein <[email protected]>

---------

Co-authored-by: Felix Fontein <[email protected]>
(cherry picked from commit 08ece2e)
@felixfontein
Copy link
Collaborator

@clifff thanks for your contribution!

felixfontein pushed a commit that referenced this pull request Dec 31, 2023
…cloak_user_federation (#7804)

Allow custom providerId in keycloak_user_federation (#7789)

* Update keycloak_user_federation.py

* stub out changelog

* update PR reference

* remove 'choices' from documentation

* Apply suggestions from code review

Co-authored-by: Felix Fontein <[email protected]>

---------

Co-authored-by: Felix Fontein <[email protected]>
(cherry picked from commit 08ece2e)

Co-authored-by: Clif Reeder <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-8 Automatically create a backport for the stable-8 branch module module new_contributor Help guide this first time contributor plugins plugin (any type)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants