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

add a new module ce_mdn_interface. #58255

Open
wants to merge 6 commits into
base: devel
from

Conversation

Projects
None yet
2 participants
@xuxiaowei0512
Copy link
Contributor

commented Jun 23, 2019

SUMMARY

add a new module ce_mdn_interface.

ISSUE TYPE
  • New Module Pull Request
COMPONENT NAME

lib/ansible/modules/network/cloudengine/ce_mdn_interface.py

ADDITIONAL INFORMATION

Manages MDN configuration on HUAWEI CloudEngine switches.
Set interface MDN enable state.


EXAMPLES = '''

  • name: LLDP mdn test
    hosts: cloudengine
    connection: local
    gather_facts: no

    tasks:

    • name: "Configure global LLDP enable state"
      ce_mdn_interface:
      lldpenable: enabled

    • name: "Configure interface MDN enable state"
      ce_mdn_interface:
      ifname: 10GE1/0/1
      mdnstatus: rxOnly
      '''

@xuxiaowei0512 xuxiaowei0512 force-pushed the xuxiaowei0512:dev/new/ce_mdn_interface branch from 546fa1a to 7581440 Jun 23, 2019

@ansibot

This comment has been minimized.

Copy link
Contributor

commented Jun 23, 2019

The test ansible-test sanity --test validate-modules [explain] failed with 2 errors:

lib/ansible/modules/network/cloudengine/ce_mdn_interface.py:0:0: E319 RETURN.changed.type: not a valid value for dictionary value @ data['changed']['type']. Got 'boolean'
lib/ansible/modules/network/cloudengine/ce_mdn_interface.py:0:0: E338 Argument 'state' in argument_spec uses default type ('str') but documentation doesn't define type

click here for bot help

@ansibot

This comment has been minimized.

xuxiaowei0512 added some commits Jun 23, 2019

@xuxiaowei0512 xuxiaowei0512 marked this pull request as ready for review Jun 23, 2019

@ansibot

This comment has been minimized.

Copy link
Contributor

commented Jun 23, 2019

@CloudEngine-Ansible @QijunPan @TommyLike @edisonxiang @freesky-edward @niuzhenguo @numone213 @zengchen1024 @zhongjun2

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

This comment has been minimized.

Copy link
Contributor

commented Jul 4, 2019

@ansibot ansibot added the stale_ci label Jul 4, 2019

@xuxiaowei0512 xuxiaowei0512 reopened this Jul 10, 2019

@ansibot ansibot removed the stale_ci label Jul 10, 2019

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.