Skip to content

Allow group members handling from any auth provider for local groups - #212

Open
fpfuetsch wants to merge 2 commits into
dell:mainfrom
fpfuetsch:feature-210
Open

Allow group members handling from any auth provider for local groups#212
fpfuetsch wants to merge 2 commits into
dell:mainfrom
fpfuetsch:feature-210

Conversation

@fpfuetsch

@fpfuetsch fpfuetsch commented Oct 22, 2025

Copy link
Copy Markdown
Contributor

Description

This PR allows getting / adding / deleting members from any auth provider to local groups by removing the provider (="local") filter.

GitHub Issues

Fixes #210

Checklist:

  • I have performed a self-review of my own code to ensure there are no formatting, pep8, linting, or security issues
  • I have performed Ansible Sanity test using --docker default
  • I have verified that new and existing unit tests pass locally with my changes
  • I have not allowed coverage numbers to degenerate
  • I have maintained at least 90% code coverage
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have added tests that prove my fix is effective or that my feature works
  • Backward compatibility is not broken

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Please also list any relevant details for your test configuration

  • Tested using local module build

@fpfuetsch

Copy link
Copy Markdown
Contributor Author

@anupamaloke @ShrinidhiRao15 @gokul-srivathsan anybody there? 😶‍🌫️

@gokul-srivathsan

Copy link
Copy Markdown

@fpfuetsch , this PR merge requires further functional tests on Group module. Please expect some delay in further action.

anupamaloke
anupamaloke previously approved these changes Nov 5, 2025

@anupamaloke anupamaloke left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

Comment thread plugins/modules/group.py
@fpfuetsch fpfuetsch changed the title Allow adding group members from any auth provider to local groups Allow group members handling from any auth provider for local groups Nov 13, 2025
@fpfuetsch

Copy link
Copy Markdown
Contributor Author

@ShrinidhiRao15 any progress updates here? Will this be included in the next release?

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.

[FEATURE]: Allow adding users from different providers (LDAP) to local groups

3 participants