Skip to content

Commit

Permalink
Clean up unused imports for nxos related code (#61856)
Browse files Browse the repository at this point in the history
Signed-off-by: Paul Belanger <pabelanger@redhat.com>
  • Loading branch information
pabelanger committed Sep 9, 2019
1 parent 6ccb245 commit a680ff2
Show file tree
Hide file tree
Showing 30 changed files with 27 additions and 57 deletions.
Expand Up @@ -14,9 +14,9 @@

import re
from ansible.module_utils.network.common.cfg.base import ConfigBase
from ansible.module_utils.network.common.utils import dict_diff, to_list, remove_empties
from ansible.module_utils.network.common.utils import dict_diff, to_list
from ansible.module_utils.network.nxos.facts.facts import Facts
from ansible.module_utils.network.nxos.utils.utils import flatten_dict, get_interface_type, normalize_interface, search_obj_in_list, vlan_range_to_list
from ansible.module_utils.network.nxos.utils.utils import flatten_dict, search_obj_in_list


class Bfd_interfaces(ConfigBase):
Expand Down
Expand Up @@ -17,7 +17,7 @@
from ansible.module_utils.network.common.cfg.base import ConfigBase
from ansible.module_utils.network.common.utils import dict_diff, to_list, remove_empties
from ansible.module_utils.network.nxos.facts.facts import Facts
from ansible.module_utils.network.nxos.utils.utils import get_interface_type, normalize_interface, search_obj_in_list
from ansible.module_utils.network.nxos.utils.utils import normalize_interface, search_obj_in_list


class Interfaces(ConfigBase):
Expand Down
Expand Up @@ -17,7 +17,7 @@
from ansible.module_utils.network.common.cfg.base import ConfigBase
from ansible.module_utils.network.common.utils import dict_diff, to_list, remove_empties
from ansible.module_utils.network.nxos.facts.facts import Facts
from ansible.module_utils.network.nxos.utils.utils import flatten_dict, get_interface_type, normalize_interface, search_obj_in_list, vlan_range_to_list
from ansible.module_utils.network.nxos.utils.utils import flatten_dict, normalize_interface, search_obj_in_list, vlan_range_to_list


class L2_interfaces(ConfigBase):
Expand Down
Expand Up @@ -15,9 +15,9 @@
__metaclass__ = type

from ansible.module_utils.network.common.cfg.base import ConfigBase
from ansible.module_utils.network.common.utils import dict_diff, to_list, remove_empties
from ansible.module_utils.network.common.utils import to_list, remove_empties
from ansible.module_utils.network.nxos.facts.facts import Facts
from ansible.module_utils.network.nxos.utils.utils import get_interface_type, normalize_interface, search_obj_in_list, validate_ipv4_addr, validate_ipv6_addr
from ansible.module_utils.network.nxos.utils.utils import normalize_interface, search_obj_in_list


class L3_interfaces(ConfigBase):
Expand Down
Expand Up @@ -16,7 +16,7 @@
from ansible.module_utils.network.common.cfg.base import ConfigBase
from ansible.module_utils.network.common.utils import to_list, remove_empties
from ansible.module_utils.network.nxos.facts.facts import Facts
from ansible.module_utils.network.nxos.utils.utils import get_interface_type, normalize_interface, search_obj_in_list
from ansible.module_utils.network.nxos.utils.utils import normalize_interface, search_obj_in_list


class Lag_interfaces(ConfigBase):
Expand Down
Expand Up @@ -14,7 +14,7 @@
__metaclass__ = type

from ansible.module_utils.network.common.cfg.base import ConfigBase
from ansible.module_utils.network.common.utils import to_list, remove_empties, dict_diff, dict_merge
from ansible.module_utils.network.common.utils import remove_empties, dict_diff
from ansible.module_utils.network.nxos.facts.facts import Facts


Expand Down
Expand Up @@ -13,9 +13,6 @@
from __future__ import absolute_import, division, print_function
__metaclass__ = type

import re
from copy import deepcopy

from ansible.module_utils.network.common.cfg.base import ConfigBase
from ansible.module_utils.network.common.utils import to_list
from ansible.module_utils.network.nxos.facts.facts import Facts
Expand Down
Expand Up @@ -17,7 +17,7 @@
from ansible.module_utils.network.common.cfg.base import ConfigBase
from ansible.module_utils.network.common.utils import dict_diff, to_list, remove_empties
from ansible.module_utils.network.nxos.facts.facts import Facts
from ansible.module_utils.network.nxos.utils.utils import get_interface_type, normalize_interface, search_obj_in_list
from ansible.module_utils.network.nxos.utils.utils import search_obj_in_list


class Vlans(ConfigBase):
Expand Down
1 change: 0 additions & 1 deletion lib/ansible/module_utils/network/nxos/facts/facts.py
Expand Up @@ -9,7 +9,6 @@
calls the appropriate facts gathering function
"""

from ansible.module_utils.network.nxos.argspec.facts.facts import FactsArgs
from ansible.module_utils.network.common.facts.facts import FactsBase
from ansible.module_utils.network.nxos.facts.legacy.base import Default, Legacy, Hardware, Config, Interfaces, Features
from ansible.module_utils.network.nxos.facts.bfd_interfaces.bfd_interfaces import Bfd_interfacesFacts
Expand Down
Expand Up @@ -20,7 +20,7 @@
from ansible.module_utils.network.nxos.cmdref.telemetry.telemetry import TMS_GLOBAL, TMS_DESTGROUP, TMS_SENSORGROUP, TMS_SUBSCRIPTION
from ansible.module_utils.network.nxos.utils.telemetry.telemetry import get_instance_data, cr_key_lookup
from ansible.module_utils.network.nxos.utils.telemetry.telemetry import normalize_data
from ansible.module_utils.network.nxos.nxos import NxosCmdRef, normalize_interface
from ansible.module_utils.network.nxos.nxos import NxosCmdRef


class TelemetryFacts(object):
Expand Down
2 changes: 1 addition & 1 deletion lib/ansible/module_utils/network/nxos/nxos.py
Expand Up @@ -41,7 +41,7 @@
from ansible.module_utils.common._collections_compat import Mapping
from ansible.module_utils.network.common.config import NetworkConfig, dumps
from ansible.module_utils.network.common.config import CustomNetworkConfig
from ansible.module_utils.six import iteritems, string_types, PY2, PY3
from ansible.module_utils.six import iteritems, PY2, PY3
from ansible.module_utils.urls import fetch_url

try:
Expand Down
3 changes: 1 addition & 2 deletions lib/ansible/modules/network/nxos/_nxos_l2_interface.py
Expand Up @@ -120,10 +120,9 @@
- switchport access vlan 20
"""

import re
from copy import deepcopy

from ansible.module_utils.network.nxos.nxos import get_config, load_config, run_commands
from ansible.module_utils.network.nxos.nxos import load_config, run_commands
from ansible.module_utils.network.nxos.nxos import nxos_argument_spec, get_interface_type
from ansible.module_utils.basic import AnsibleModule
from ansible.module_utils.network.common.utils import remove_default_spec
Expand Down
2 changes: 1 addition & 1 deletion lib/ansible/modules/network/nxos/_nxos_linkagg.py
Expand Up @@ -137,7 +137,7 @@
from copy import deepcopy

from ansible.module_utils.network.nxos.nxos import get_config, load_config, run_commands
from ansible.module_utils.network.nxos.nxos import get_capabilities, nxos_argument_spec
from ansible.module_utils.network.nxos.nxos import nxos_argument_spec
from ansible.module_utils.network.nxos.nxos import normalize_interface
from ansible.module_utils.basic import AnsibleModule
from ansible.module_utils.network.common.utils import remove_default_spec
Expand Down
1 change: 0 additions & 1 deletion lib/ansible/modules/network/nxos/_nxos_vlan.py
Expand Up @@ -179,7 +179,6 @@
from ansible.module_utils.network.nxos.nxos import get_config, load_config, run_commands
from ansible.module_utils.network.nxos.nxos import normalize_interface, nxos_argument_spec
from ansible.module_utils.basic import AnsibleModule
from ansible.module_utils.connection import ConnectionError
from ansible.module_utils.network.common.config import CustomNetworkConfig
from ansible.module_utils.network.common.utils import remove_default_spec

Expand Down
4 changes: 1 addition & 3 deletions lib/ansible/modules/network/nxos/nxos_bfd_global.py
Expand Up @@ -139,12 +139,10 @@
'''


import re
from ansible.module_utils.network.nxos.nxos import NxosCmdRef
from ansible.module_utils.network.nxos.nxos import nxos_argument_spec, check_args
from ansible.module_utils.network.nxos.nxos import load_config, run_commands
from ansible.module_utils.network.nxos.nxos import load_config
from ansible.module_utils.basic import AnsibleModule
from ansible.module_utils.network.common.config import CustomNetworkConfig

BFD_CMD_REF = """
# The cmd_ref is a yaml formatted list of module commands.
Expand Down
2 changes: 1 addition & 1 deletion lib/ansible/modules/network/nxos/nxos_evpn_vni.py
Expand Up @@ -95,7 +95,7 @@
'''

import re
from ansible.module_utils.network.nxos.nxos import get_config, load_config, run_commands
from ansible.module_utils.network.nxos.nxos import get_config, load_config
from ansible.module_utils.network.nxos.nxos import nxos_argument_spec, check_args
from ansible.module_utils.basic import AnsibleModule
from ansible.module_utils.network.common.config import CustomNetworkConfig
Expand Down
5 changes: 2 additions & 3 deletions lib/ansible/modules/network/nxos/nxos_gir.py
Expand Up @@ -158,9 +158,8 @@
sample: true
'''

import re
from ansible.module_utils.network.nxos.nxos import get_config, load_config, run_commands
from ansible.module_utils.network.nxos.nxos import get_capabilities, nxos_argument_spec
from ansible.module_utils.network.nxos.nxos import load_config, run_commands
from ansible.module_utils.network.nxos.nxos import nxos_argument_spec
from ansible.module_utils.basic import AnsibleModule


Expand Down
Expand Up @@ -98,8 +98,7 @@
'''


import re
from ansible.module_utils.network.nxos.nxos import get_config, load_config, run_commands
from ansible.module_utils.network.nxos.nxos import get_config, load_config
from ansible.module_utils.network.nxos.nxos import nxos_argument_spec
from ansible.module_utils.basic import AnsibleModule
from ansible.module_utils.network.common.config import CustomNetworkConfig
Expand Down
2 changes: 1 addition & 1 deletion lib/ansible/modules/network/nxos/nxos_igmp_interface.py
Expand Up @@ -189,7 +189,7 @@
sample: true
'''

from ansible.module_utils.network.nxos.nxos import get_config, load_config, run_commands
from ansible.module_utils.network.nxos.nxos import load_config, run_commands
from ansible.module_utils.network.nxos.nxos import nxos_argument_spec, check_args
from ansible.module_utils.network.nxos.nxos import get_interface_type
from ansible.module_utils.basic import AnsibleModule
Expand Down
2 changes: 1 addition & 1 deletion lib/ansible/modules/network/nxos/nxos_igmp_snooping.py
Expand Up @@ -95,7 +95,7 @@

import re

from ansible.module_utils.network.nxos.nxos import get_config, load_config, run_commands
from ansible.module_utils.network.nxos.nxos import load_config, run_commands
from ansible.module_utils.network.nxos.nxos import nxos_argument_spec, check_args
from ansible.module_utils.basic import AnsibleModule

Expand Down
2 changes: 1 addition & 1 deletion lib/ansible/modules/network/nxos/nxos_ntp_auth.py
Expand Up @@ -86,7 +86,7 @@

import re

from ansible.module_utils.network.nxos.nxos import get_config, load_config, run_commands
from ansible.module_utils.network.nxos.nxos import load_config, run_commands
from ansible.module_utils.network.nxos.nxos import nxos_argument_spec, check_args
from ansible.module_utils.basic import AnsibleModule

Expand Down
2 changes: 1 addition & 1 deletion lib/ansible/modules/network/nxos/nxos_ntp_options.py
Expand Up @@ -75,7 +75,7 @@
'''
import re

from ansible.module_utils.network.nxos.nxos import get_config, load_config, run_commands
from ansible.module_utils.network.nxos.nxos import load_config, run_commands
from ansible.module_utils.network.nxos.nxos import nxos_argument_spec, check_args
from ansible.module_utils.basic import AnsibleModule

Expand Down
1 change: 0 additions & 1 deletion lib/ansible/modules/network/nxos/nxos_nxapi.py
Expand Up @@ -166,7 +166,6 @@
from ansible.module_utils.network.nxos.nxos import nxos_argument_spec
from ansible.module_utils.network.nxos.nxos import get_capabilities
from ansible.module_utils.basic import AnsibleModule
from ansible.module_utils.six import iteritems


def check_args(module, warnings, capabilities):
Expand Down
1 change: 0 additions & 1 deletion lib/ansible/modules/network/nxos/nxos_pim_interface.py
Expand Up @@ -148,7 +148,6 @@
from ansible.module_utils.network.nxos.nxos import get_config, load_config, run_commands
from ansible.module_utils.network.nxos.nxos import nxos_argument_spec, check_args
from ansible.module_utils.network.nxos.nxos import get_interface_type
from ansible.module_utils.six import string_types


PARAM_TO_COMMAND_KEYMAP = {
Expand Down
2 changes: 1 addition & 1 deletion lib/ansible/modules/network/nxos/nxos_rpm.py
Expand Up @@ -82,7 +82,7 @@
from copy import deepcopy

from ansible.module_utils.network.nxos.nxos import load_config, run_commands
from ansible.module_utils.network.nxos.nxos import nxos_argument_spec, check_args
from ansible.module_utils.network.nxos.nxos import nxos_argument_spec
from ansible.module_utils.basic import AnsibleModule
from ansible.module_utils.network.common.utils import remove_default_spec

Expand Down
4 changes: 1 addition & 3 deletions lib/ansible/modules/network/nxos/nxos_smu.py
Expand Up @@ -63,11 +63,9 @@
'''


import collections
import re
import time

from ansible.module_utils.network.nxos.nxos import get_config, load_config, run_commands
from ansible.module_utils.network.nxos.nxos import load_config, run_commands
from ansible.module_utils.network.nxos.nxos import nxos_argument_spec, check_args
from ansible.module_utils.basic import AnsibleModule

Expand Down
6 changes: 2 additions & 4 deletions lib/ansible/modules/network/nxos/nxos_udld.py
Expand Up @@ -103,10 +103,8 @@
sample: true
'''

import re

from ansible.module_utils.network.nxos.nxos import get_config, load_config, run_commands
from ansible.module_utils.network.nxos.nxos import get_capabilities, nxos_argument_spec
from ansible.module_utils.network.nxos.nxos import load_config, run_commands
from ansible.module_utils.network.nxos.nxos import nxos_argument_spec
from ansible.module_utils.basic import AnsibleModule


Expand Down
4 changes: 1 addition & 3 deletions lib/ansible/modules/network/nxos/nxos_user.py
Expand Up @@ -137,15 +137,13 @@
type: str
sample: "0:00:10.469466"
"""
import re

from copy import deepcopy
from functools import partial

from ansible.module_utils.network.nxos.nxos import run_commands, load_config
from ansible.module_utils.network.nxos.nxos import nxos_argument_spec
from ansible.module_utils.basic import AnsibleModule
from ansible.module_utils.six import string_types, iteritems
from ansible.module_utils.six import iteritems
from ansible.module_utils.network.common.utils import remove_default_spec, to_list

VALID_ROLES = ['network-admin', 'network-operator', 'vdc-admin', 'vdc-operator',
Expand Down
8 changes: 0 additions & 8 deletions lib/ansible/plugins/action/nxos_file_copy.py
Expand Up @@ -21,21 +21,13 @@
import hashlib
import os
import re
import sys
import time
import traceback
import uuid

from ansible.errors import AnsibleError
from ansible.module_utils._text import to_text, to_bytes
from ansible.module_utils.connection import Connection
from ansible.plugins.action import ActionBase
from ansible.module_utils.six.moves.urllib.parse import urlsplit
from ansible.utils.display import Display
from ansible.module_utils.compat.paramiko import paramiko
from ansible.module_utils.network.nxos.nxos import run_commands
from ansible.module_utils._text import to_native, to_text, to_bytes
from ansible.module_utils.basic import AnsibleModule
from ansible.module_utils import six

try:
Expand Down
4 changes: 0 additions & 4 deletions test/units/modules/network/nxos/test_nxos_evpn_vni.py
Expand Up @@ -31,9 +31,6 @@ class TestNxosEvpnVniModule(TestNxosModule):
def setUp(self):
super(TestNxosEvpnVniModule, self).setUp()

self.mock_run_commands = patch('ansible.modules.network.nxos.nxos_evpn_vni.run_commands')
self.run_commands = self.mock_run_commands.start()

self.mock_load_config = patch('ansible.modules.network.nxos.nxos_evpn_vni.load_config')
self.load_config = self.mock_load_config.start()

Expand All @@ -42,7 +39,6 @@ def setUp(self):

def tearDown(self):
super(TestNxosEvpnVniModule, self).tearDown()
self.mock_run_commands.stop()
self.mock_load_config.stop()
self.mock_get_config.stop()

Expand Down

0 comments on commit a680ff2

Please sign in to comment.