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

New module - meraki_management_interface #57539

Open
wants to merge 4 commits into
base: devel
from

Conversation

Projects
None yet
2 participants
@kbreit
Copy link
Contributor

commented Jun 7, 2019

SUMMARY

Meraki recently added support for configuring management interfaces. This module can do full configuration of these interfaces.

Fixes #57233

ISSUE TYPE
  • New Module Pull Request
COMPONENT NAME

meraki_management_interface

ADDITIONAL INFORMATION
TASK [meraki_management_interface : Set WAN2 as static IP] *********************
changed: [localhost -> localhost]

TASK [meraki_management_interface : debug] *************************************
ok: [localhost] =>
  set_wan2:
    changed: true
    data:
      wan1:
        usingStaticIp: false
        vlan: 1
        wanEnabled: enabled
      wan2:
        staticDns:
        - 1.1.1.1
        staticGatewayIp: 192.168.16.1
        staticIp: 192.168.16.195
        staticSubnetMask: 255.255.255.0
        usingStaticIp: true
        vlan: 1
        wanEnabled: enabled
    diff:
      after:
        wan2:
          staticDns:
          - 1.1.1.1
          staticGatewayIp: 192.168.16.1
          staticIp: 192.168.16.195
          staticSubnetMask: 255.255.255.0
          usingStaticIp: true
      before:
        wan2:
          usingStaticIp: false
    failed: false
    response: OK (unknown bytes)
    status: 200

TASK [meraki_management_interface : assert] ************************************
ok: [localhost] => changed=false
  msg: All assertions passed

TASK [meraki_management_interface : Test too many DNS servers] *****************
fatal: [localhost -> localhost]: FAILED! => changed=false
  msg: Maximum number of static DNS addresses is 2.
  response: null
  status: null
...ignoring

TASK [meraki_management_interface : debug] *************************************
ok: [localhost] =>
  too_many_dns:
    changed: false
    failed: true
    msg: Maximum number of static DNS addresses is 2.
    response: null
    status: null
    warnings:
    - The value 133277 (type int) in a string field was converted to '133277' (type string). If this does not look like what you expect, quote the entire value to ensure it does not change.

TASK [meraki_management_interface : assert] ************************************
ok: [localhost] => changed=false
  msg: All assertions passed

TASK [meraki_management_interface : Query management information] **************
ok: [localhost -> localhost]

TASK [meraki_management_interface : debug] *************************************
ok: [localhost] =>
  query_mx:
    changed: false
    data:
      wan1:
        usingStaticIp: false
        vlan: 1
        wanEnabled: enabled
      wan2:
        staticDns:
        - 1.1.1.1
        staticGatewayIp: 192.168.16.1
        staticIp: 192.168.16.195
        staticSubnetMask: 255.255.255.0
        usingStaticIp: true
        vlan: 1
        wanEnabled: enabled
    failed: false
    response: OK (unknown bytes)
    status: 200
@ansibot

This comment has been minimized.

Copy link
Contributor

commented Jun 7, 2019

The test ansible-test sanity --test ansible-doc --python 2.6 [explain] failed with 1 error:

lib/ansible/modules/network/meraki/meraki_management_interface.py:0:0: has a documentation error formatting or is missing documentation.

The test ansible-test sanity --test compile --python 2.6 [explain] failed with 1 error:

lib/ansible/modules/network/meraki/meraki_management_interface.py:3:1: SyntaxError:

The test ansible-test sanity --test line-endings [explain] failed with 1 error:

lib/ansible/modules/network/meraki/meraki_management_interface.py:0:0: use "\n" for line endings instead of "\r\n"

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.