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

Get behavior of gitlab_project_members to the one of gitlab_group_members #3319

Conversation

suukit
Copy link
Contributor

@suukit suukit commented Aug 31, 2021

SUMMARY

gitlab_group_members has been enhanced to accept list of users/access levels and purge users not in list. This change does the same changes to gitlab_project_members

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

gitlab_project_members

@felixfontein @zanssa

@ansibullbot
Copy link
Collaborator

@ansibullbot ansibullbot added affects_2.10 community_review feature This issue/PR relates to a feature request gitlab integration tests/integration 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) source_control stale_ci CI is older than 7 days, rerun before merging tests tests labels Aug 31, 2021
@felixfontein felixfontein added backport-3 check-before-release PR will be looked at again shortly before release and merged if possible. labels Sep 1, 2021
@felixfontein
Copy link
Collaborator

Follow-up to #3047.

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.

Looks good to me. I would still avoid some of the unnecessary reformatting of existing code.

@suukit
Copy link
Contributor Author

suukit commented Sep 8, 2021

Looks good to me. I would still avoid some of the unnecessary reformatting of existing code.

Hey, sorry about that. Had to disable an editor plugin to prevent those changes. Hope I found all of them.

@felixfontein
Copy link
Collaborator

If nobody complains, I'll merge this in ~a week.

@ansibullbot ansibullbot removed needs_ci This PR requires CI testing to be performed. Please close and re-open this PR to trigger CI stale_ci CI is older than 7 days, rerun before merging labels Sep 13, 2021
@felixfontein felixfontein removed the check-before-release PR will be looked at again shortly before release and merged if possible. label Sep 16, 2021
@felixfontein felixfontein merged commit 7a2efb4 into ansible-collections:main Sep 16, 2021
@patchback
Copy link

patchback bot commented Sep 16, 2021

Backport to stable-3: 💚 backport PR created

✅ Backport PR branch: patchback/backports/stable-3/7a2efb4775af4296bf4dfe640cbc0ae52c87d5dd/pr-3319

Backported as #3383

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

@felixfontein
Copy link
Collaborator

@suukit thanks for implementing this!

patchback bot pushed a commit that referenced this pull request Sep 16, 2021
…bers (#3319)

* Initial change to get behaviour of gitlab_project_members like the new gitlab_group_members

* added changelog

* linter: removed trainling whitespaces

* Update plugins/modules/source_control/gitlab/gitlab_project_members.py

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

* Update plugins/modules/source_control/gitlab/gitlab_project_members.py

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

* Update plugins/modules/source_control/gitlab/gitlab_project_members.py

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

* Update plugins/modules/source_control/gitlab/gitlab_project_members.py

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

* Update plugins/modules/source_control/gitlab/gitlab_project_members.py

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

* requested changes

* linter fixes

* undoing formatting changes to existing code

Co-authored-by: Max-Florian Bidlingmaier <[email protected]>
Co-authored-by: Felix Fontein <[email protected]>
Co-authored-by: Max Bidlingmaier <[email protected]>
(cherry picked from commit 7a2efb4)
felixfontein pushed a commit that referenced this pull request Sep 17, 2021
…bers (#3319) (#3383)

* Initial change to get behaviour of gitlab_project_members like the new gitlab_group_members

* added changelog

* linter: removed trainling whitespaces

* Update plugins/modules/source_control/gitlab/gitlab_project_members.py

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

* Update plugins/modules/source_control/gitlab/gitlab_project_members.py

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

* Update plugins/modules/source_control/gitlab/gitlab_project_members.py

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

* Update plugins/modules/source_control/gitlab/gitlab_project_members.py

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

* Update plugins/modules/source_control/gitlab/gitlab_project_members.py

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

* requested changes

* linter fixes

* undoing formatting changes to existing code

Co-authored-by: Max-Florian Bidlingmaier <[email protected]>
Co-authored-by: Felix Fontein <[email protected]>
Co-authored-by: Max Bidlingmaier <[email protected]>
(cherry picked from commit 7a2efb4)

Co-authored-by: Max Bidlingmaier <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community_review feature This issue/PR relates to a feature request gitlab integration tests/integration module module plugins plugin (any type) source_control tests tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants