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.

Sign up for GitHub

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

Jump to bottom

msc_user: Manage users on ACI MultiSite #47754

Merged
merged 1 commit into from
Nov 2, 2018
Merged

msc_user: Manage users on ACI MultiSite #47754

merged 1 commit into from
Nov 2, 2018

Conversation

dagwieers
Copy link
Contributor

@dagwieers dagwieers commented Oct 29, 2018 •
edited
Loading

SUMMARY

This includes the new msc_user module and integration tests.

This PR requires #46116 to be merged.

ISSUE TYPE
  • New Module Pull Request
COMPONENT NAME

msc_user

ANSIBLE VERSION

v2.8

@ansibot
Copy link
Contributor

ansibot commented Oct 29, 2018

Hi @dagwieers, thank you for submitting this pull-request!

click here for bot help

@ansibot
Copy link
Contributor

ansibot commented Oct 29, 2018

cc @brunocalogero @jmcgill298 @schunduri
click here for bot help

@ansibot ansibot added aci Cisco ACI community affects_2.8 This issue/PR affects Ansible v2.8 community_review In order to be merged, this PR must follow the community review workflow. module This issue/PR relates to a module. needs_triage Needs a first human triage before being processed. networking Network category new_module This PR includes a new module. new_plugin This PR includes a new plugin. support:community This issue/PR relates to code supported by the Ansible community. labels Oct 29, 2018
@dagwieers
Copy link
Contributor Author

cc @kbreit

@ansibot

This comment has been minimized.

Sign in to view
@ansibot ansibot added needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR. and removed community_review In order to be merged, this PR must follow the community review workflow. labels Oct 29, 2018
@ansibot

This comment has been minimized.

Sign in to view
@ansibot

This comment has been minimized.

Sign in to view
@ansibot

This comment has been minimized.

Sign in to view
@ansibot

This comment has been minimized.

Sign in to view
@ansibot ansibot added the ci_verified Changes made in this PR are causing tests to fail. label Oct 29, 2018
@ansibot ansibot removed the ci_verified Changes made in this PR are causing tests to fail. label Oct 29, 2018
@ansibot

This comment has been minimized.

Sign in to view
@ansibot ansibot added the ci_verified Changes made in this PR are causing tests to fail. label Oct 29, 2018
@ansibot ansibot removed the ci_verified Changes made in this PR are causing tests to fail. label Oct 30, 2018
@ansibot

This comment has been minimized.

Sign in to view
@ansibot ansibot added the ci_verified Changes made in this PR are causing tests to fail. label Oct 30, 2018
@dagwieers
msc_user: Manage users on ACI MultiSite
91f86f5 
This includes the new msc_user module and integration tests.
@ansibot ansibot removed the ci_verified Changes made in this PR are causing tests to fail. label Oct 30, 2018
@ansibot
Copy link
Contributor

ansibot commented Oct 30, 2018

@brunocalogero @jmcgill298 @rost-d @schunduri @smnmtzgr

As a maintainer of a module in the same namespace this new module has been submitted to, your vote counts for shipits. Please review this module and add shipit if you would like to see it merged.

click here for bot help

@ansibot ansibot added community_review In order to be merged, this PR must follow the community review workflow. and removed needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR. labels Oct 30, 2018
@jborean93 jborean93 removed the needs_triage Needs a first human triage before being processed. label Nov 1, 2018
@dagwieers dagwieers merged commit 7142715 into ansible:devel Nov 2, 2018
mjmayer pushed a commit to mjmayer/ansible that referenced this pull request Nov 30, 2018
@dagwieers @mjmayer
msc_user: Manage users on ACI MultiSite (ansible#47754)
7b925cc 
This includes the new msc_user module and integration tests.
Tomorrow9 pushed a commit to Tomorrow9/ansible that referenced this pull request Dec 4, 2018
@dagwieers @Tomorrow9
msc_user: Manage users on ACI MultiSite (ansible#47754)
9792c78 
This includes the new msc_user module and integration tests.
@dagwieers dagwieers added the cisco Cisco technologies label Feb 22, 2019
@ansible ansible locked and limited conversation to collaborators Jul 22, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@kbreit kbreit kbreit left review comments

Assignees
No one assigned
Labels
aci Cisco ACI community affects_2.8 This issue/PR affects Ansible v2.8 cisco Cisco technologies community_review In order to be merged, this PR must follow the community review workflow. module This issue/PR relates to a module. networking Network category new_module This PR includes a new module. new_plugin This PR includes a new plugin. support:community This issue/PR relates to code supported by the Ansible community.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@dagwieers @ansibot @kbreit @jborean93