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

ACI CDP Interface Policy Module Creation #54818

Open
wants to merge 6 commits into
base: devel
from

Conversation

Projects
None yet
3 participants
@tknipper11
Copy link

tknipper11 commented Apr 3, 2019

SUMMARY

This module creates a fabric interface policy for CDP enable and CDP disable. This is based on the aci_interface_policy_lldp module already in version 2.7. cc @kbreit

ISSUE TYPE
  • New Module Pull Request
COMPONENT NAME

aci_interface_policy_cdp

ADDITIONAL INFORMATION
TASK [aci_interface_policy_cdp : debug] ************************************************************************************************************************
ok: [localhost] => {
    "cdp_disable": {
        "changed": true,
        "current": [
            {
                "cdpIfPol": {
                    "attributes": {
                        "adminSt": "disabled",
                        "annotation": "",
                        "descr": "",
                        "dn": "uni/infra/cdpIfP-Ansible_CDP_Test_Policy",
                        "name": "Ansible_CDP_Test_Policy",
                        "nameAlias": "",
                        "ownerKey": "",
                        "ownerTag": ""
                    }
                }
            }
        ],
        "failed": false
    }
}


@kbreit

This comment has been minimized.

Copy link
Contributor

kbreit commented Apr 3, 2019

@dagwieers I assisted @tknipper11 with developing this module. Please take a look and provide review. But in the background, I'll possibly be helping him with any changes.

Is it possible for this to get into 2.8? I know we're short on time but the module is relatively simple.

@ansibot

This comment has been minimized.

@ansibot

This comment has been minimized.

Copy link
Contributor

ansibot commented Apr 3, 2019

@brunocalogero @jmcgill298 @mtorelli @rost-d @rsmeyers @schunduri @sgerhart @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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.