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

win_domain_group_membership module: Bug Fix #57104

Closed
wants to merge 1 commit into from

Conversation

AravindBalajiS
Copy link
Contributor

@AravindBalajiS AravindBalajiS commented May 29, 2019

SUMMARY

Bug fixes while Adding and Deleting a group from another group #56953

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

win_domain_group_membership.ps1

ADDITIONAL INFORMATION

I was trying to add DEF group to ABC group after executing the ansible script I checked the members that are present in ABC groups, the DEF group was not added to it. As DEF group was not present in ABC group after executing the ansible script then I tried to add DEF group to ABC group using the ps1 script, after execution of the ps1 script I found that DEF is a universal group and ABC is a global group, as a global group cannot have a universal group as a member. So the ps1 script failed while executing. But while executing the ansible script it got passed, it didn't show any error while execting it.

* Bug fixes while Adding and Deleting a group from another group
* Changelog fragment
@ansibot ansibot added affects_2.9 This issue/PR affects Ansible v2.9 bug This issue/PR relates to a bug. community_review In order to be merged, this PR must follow the community review workflow. needs_triage Needs a first human triage before being processed. support:community This issue/PR relates to code supported by the Ansible community. labels May 29, 2019
@ansibot ansibot added the stale_ci This PR has been tested by CI more than one week ago. Close and re-open this PR to get it retested. label Jun 6, 2019
@ansibot ansibot added the owner_pr This PR is made by the module's maintainer. label Jun 30, 2019
@jborean93
Copy link
Contributor

Not sure how this was slipped through sorry but the win_domain_group_membership module now lives in the community.windows collection. Any bug fixes or issues should be submitted to that repo.

@jborean93 jborean93 closed this May 20, 2020
@sivel sivel removed the needs_triage Needs a first human triage before being processed. label Jun 2, 2020
@ansible ansible locked and limited conversation to collaborators Jun 17, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
affects_2.9 This issue/PR affects Ansible v2.9 bug This issue/PR relates to a bug. community_review In order to be merged, this PR must follow the community review workflow. owner_pr This PR is made by the module's maintainer. stale_ci This PR has been tested by CI more than one week ago. Close and re-open this PR to get it retested. support:community This issue/PR relates to code supported by the Ansible community.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants