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

Fix keycloak_realm module #3231

Merged
merged 6 commits into from
Aug 27, 2021

Conversation

laurpaum
Copy link
Contributor

SUMMARY

Fixed bad element type for events_listeners parameter (string array in Keycloak realm representation).
Added events_enabled parameter to allow activation or deactivation of login events.

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

keycloak_realm

add events_enabled parameter
@ansibullbot
Copy link
Collaborator

@ansibullbot ansibullbot added affects_2.10 bug This issue/PR relates to a bug community_review identity module module needs_ci This PR requires CI testing to be performed. Please close and re-open this PR to trigger CI needs_triage plugins plugin (any type) stale_ci CI is older than 7 days, rerun before merging labels Aug 17, 2021
@kris2kris
Copy link
Contributor

LGTM

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.

Please add a changelog fragment.

@felixfontein felixfontein added backport-3 check-before-release PR will be looked at again shortly before release and merged if possible. labels Aug 18, 2021
@felixfontein
Copy link
Collaborator

Looks good to me. @kris2kris are you (still) happy as well?

@felixfontein
Copy link
Collaborator

I assume that is a yes.

@felixfontein felixfontein removed the check-before-release PR will be looked at again shortly before release and merged if possible. label Aug 27, 2021
@felixfontein felixfontein merged commit 825e17c into ansible-collections:main Aug 27, 2021
@patchback
Copy link

patchback bot commented Aug 27, 2021

Backport to stable-3: 💚 backport PR created

✅ Backport PR branch: patchback/backports/stable-3/825e17c1cfc33571b273984f195166f268b0850c/pr-3231

Backported as #3277

🤖 @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 Aug 27, 2021
* fix events_listeners element type
add events_enabled parameter

* Update plugins/modules/identity/keycloak/keycloak_realm.py

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

* add changelog

* Update changelogs/fragments/3231-fix-keycloak-realm-events.yml

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

* Update changelogs/fragments/3231-fix-keycloak-realm-events.yml

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

* Update plugins/modules/identity/keycloak/keycloak_realm.py

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

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

@laurpaum thanks for fixing this!
@kris2kris thanks for reviewing!

felixfontein pushed a commit that referenced this pull request Aug 27, 2021
* fix events_listeners element type
add events_enabled parameter

* Update plugins/modules/identity/keycloak/keycloak_realm.py

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

* add changelog

* Update changelogs/fragments/3231-fix-keycloak-realm-events.yml

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

* Update changelogs/fragments/3231-fix-keycloak-realm-events.yml

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

* Update plugins/modules/identity/keycloak/keycloak_realm.py

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

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

Co-authored-by: Laurent Paumier <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This issue/PR relates to a bug community_review identity module module needs_ci This PR requires CI testing to be performed. Please close and re-open this PR to trigger CI plugins plugin (any type) stale_ci CI is older than 7 days, rerun before merging
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants