# Copyright 2016 Mirantis, Inc.
#
#    Licensed under the Apache License, Version 2.0 (the "License"); you may
#    not use this file except in compliance with the License. You may obtain
#    a copy of the License at
#
#         http://www.apache.org/licenses/LICENSE-2.0
#
#    Unless required by applicable law or agreed to in writing, software
#    distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
#    WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
#    License for the specific language governing permissions and limitations
#    under the License.

import atexit
import contextlib
import functools
import hashlib
import queue
import uuid

import netmiko
from neutron_lib import constants as const
from oslo_config import cfg
from oslo_log import log as logging
from oslo_utils import strutils
from paramiko import PKey as _pkey  # noqa - This is for a monkeypatch
import paramiko  # noqa - Must load after the patch
import tenacity
from tooz import coordination

from networking_generic_switch._i18n import _
from networking_generic_switch import batching
from networking_generic_switch import devices
from networking_generic_switch.devices import utils as device_utils
from networking_generic_switch import exceptions as exc
from networking_generic_switch import locking as ngs_lock
from networking_generic_switch import utils as ngs_utils

# NOTE(TheJulia) monkey patch paramiko's get_fingerprint function
# to use sha256 instead of md5, since Paramiko's maintainer doesn't
# seem to be concerned about FIPS compliance.
_pkey.get_fingerprint = lambda x: hashlib.sha256(x.asbytes()).digest()

LOG = logging.getLogger(__name__)
CONF = cfg.CONF


def check_output(operation):
    """Returns a decorator that checks the output of an operation.

    :param operation: Operation being attempted. One of 'add network',
        'delete network', 'plug port', 'unplug port'.
    """
    def decorator(func):
        """The real decorator."""

        @functools.wraps(func)
        def wrapper(self, *args, **kwargs):
            """Wrapper that checks the output of an operation.

            :returns: The return value of the wrapped method.
            :raises: GenericSwitchNetmikoConfigError if the driver detects that
                an error has occurred.
            """
            output = func(self, *args, **kwargs)
            try:
                self.check_output(output, operation)
            except exc.GenericSwitchNetmikoConfigError:
                # Raised if ERROR_MSG_PATTERNS catches an output error
                self._drain_cached_connections()
                raise
            return output

        return wrapper

    return decorator


class NetmikoSwitch(devices.GenericSwitchDevice):

    NETMIKO_DEVICE_TYPE = None

    ADD_NETWORK = None

    DELETE_NETWORK = None

    PLUG_SWITCH_TO_NETWORK = None

    UNPLUG_SWITCH_FROM_NETWORK = None

    SHOW_VLAN_PORTS = None

    SHOW_VLAN_VNI = None

    PLUG_PORT_TO_NETWORK = None

    DELETE_PORT = None

    PLUG_BOND_TO_NETWORK = None

    UNPLUG_BOND_FROM_NETWORK = None

    ADD_NETWORK_TO_TRUNK = None

    REMOVE_NETWORK_FROM_TRUNK = None

    ENABLE_PORT = None

    DISABLE_PORT = None

    ENABLE_BOND = None

    DISABLE_BOND = None

    SAVE_CONFIGURATION = None

    SET_NATIVE_VLAN = None

    DELETE_NATIVE_VLAN = None

    SET_NATIVE_VLAN_BOND = None

    DELETE_NATIVE_VLAN_BOND = None

    ADD_NETWORK_TO_BOND_TRUNK = None

    DELETE_NETWORK_ON_BOND_TRUNK = None

    ADD_SECURITY_GROUP = None

    ADD_SECURITY_GROUP_COMPLETE = None

    REMOVE_SECURITY_GROUP = None

    ADD_SECURITY_GROUP_RULE_INGRESS = None

    ADD_SECURITY_GROUP_RULE_EGRESS = None

    BIND_SECURITY_GROUP = None

    UNBIND_SECURITY_GROUP = None

    SUPPORT_SG_PORT_RANGE = True
    """Whether a security group rule supports a port range on this switch

    Set to False for switches which support only a single port per rule.
    """

    ERROR_MSG_PATTERNS = ()
    """Sequence of error message patterns.

    Sequence of re.RegexObject objects representing patterns to check for in
    device output that indicate a failure to apply configuration.
    """

    def __init__(self, device_cfg, *args, **kwargs):
        super(NetmikoSwitch, self).__init__(device_cfg, *args, **kwargs)
        if self.NETMIKO_DEVICE_TYPE:
            device_type = self.NETMIKO_DEVICE_TYPE
        else:
            device_type = self.config.get('device_type', '')
            # use part that is after 'netmiko_'
            device_type = device_type.partition('netmiko_')[2]
        if device_type not in netmiko.platforms:
            raise exc.GenericSwitchNetmikoNotSupported(
                device_type=device_type)
        self.config['device_type'] = device_type
        # Don't pass disabled_algorithms by default to keep compatibility
        # with older versions of Netmiko.
        disabled_algorithms = self._get_ssh_disabled_algorithms()
        if disabled_algorithms:
            self.config['disabled_algorithms'] = disabled_algorithms
        if CONF.ngs.session_log_file:
            self.config['session_log'] = CONF.ngs.session_log_file
            self.config['session_log_record_writes'] = True
            self.config['session_log_file_mode'] = 'append'

        _NETMIKO_CAST = {
            "port": int,
            "verbose": bool,
            "global_delay_factor": float,
            "global_cmd_verify": bool,
            "use_keys": bool,
            "disable_sha2_fix": bool,
            "allow_agent": bool,
            "ssh_strict": bool,
            "system_host_keys": bool,
            "alt_host_keys": bool,
            "conn_timeout": float,
            "auth_timeout": float,
            "banner_timeout": float,
            "blocking_timeout": float,
            "timeout": float,
            "session_timeout": float,
            "read_timeout_override": float,
            "keepalive": float,
            "fast_cli": bool,
            "_legacy_mode": bool,
            "session_log_record_writes": bool,
            "allow_auto_change": bool,
            "auto_connect": bool,
            "delay_factor_compat": bool,
            "disable_lf_normalization": bool,
        }

        for key, expected_type in _NETMIKO_CAST.items():
            value = self.config.get(key)
            if isinstance(value, str):
                try:
                    if expected_type == bool:
                        self.config[key] = strutils.bool_from_string(value)
                    else:
                        self.config[key] = expected_type(value)
                except ValueError:
                    LOG.error(
                        "Invalid value %s for %s; expected %s",
                        value, key, expected_type.__name__,
                    )
                    raise exc.GenericSwitchNetmikoConfigError()

        self.lock_kwargs = {
            'locks_pool_size': int(self.ngs_config['ngs_max_connections']),
            'locks_prefix': self.config.get(
                'host', '') or self.config.get('ip', ''),
            'timeout': CONF.ngs_coordination.acquire_timeout}

        self.locker = None
        self.batch_cmds = None
        self._connection_pool = None
        if self._batch_requests():
            if not CONF.ngs_coordination.backend_url:
                error = ("ngs_batch_requests is true but [ngs_coordination] "
                         "backend_url is not provided")
                LOG.error(
                    _("%(device)s, error: %(error)s"),
                    {'device': device_utils.sanitise_config(self.config),
                     'error': error})
                raise exc.GenericSwitchNetmikoConfigError()
            # NOTE: we skip the lock if we are batching requests
            self.locker = None
            switch_name = self.lock_kwargs['locks_prefix']
            self.batch_cmds = batching.SwitchBatch(
                switch_name, CONF.ngs_coordination.backend_url)
        elif CONF.ngs_coordination.backend_url:
            self.locker = coordination.get_coordinator(
                CONF.ngs_coordination.backend_url,
                ('ngs-' + device_utils.get_hostname()).encode('ascii'))
            self.locker.start()
            atexit.register(self.locker.stop)
        else:
            LOG.warning(
                "Switch %s: [ngs_coordination] backend_url is not "
                "configured. The ngs_max_connections is ignored.",
                self.lock_kwargs['locks_prefix'])

        if self._reuse_connection():
            max_connections = int(self.ngs_config['ngs_max_connections'])
            self._connection_pool = queue.LifoQueue(maxsize=max_connections)
            # Connections are returned to pool rather than closed;
            # register cleanup on exit
            atexit.register(self._drain_cached_connections)

    @property
    def support_trunk_on_ports(self):
        return bool(self.ADD_NETWORK_TO_TRUNK)

    @property
    def support_trunk_on_bond_ports(self):
        return bool(self.ADD_NETWORK_TO_BOND_TRUNK)

    def _format_commands(self, commands, **kwargs):
        if not commands:
            return []
        if not all(kwargs.values()):
            raise exc.GenericSwitchNetmikoMethodError(cmds=commands,
                                                      args=kwargs)
        try:
            cmd_set = [cmd.format(**kwargs) for cmd in commands]
        except (KeyError, TypeError):
            raise exc.GenericSwitchNetmikoMethodError(cmds=commands,
                                                      args=kwargs)
        return cmd_set

    def _get_pooled_connection(self):
        """Pop a live connection from the pool, or return None."""
        while not self._connection_pool.empty():
            try:
                conn = self._connection_pool.get_nowait()
            except queue.Empty:
                return None

            if conn.is_alive():
                return conn
            conn.disconnect()

        return None

    @contextlib.contextmanager
    def _get_connection(self):
        """Context manager providing a netmiko SSH connection object.

        This function hides the complexities of gracefully handling retrying
        failed connection attempts.
        """
        if not self._reuse_connection():
            net_connect = self._connect_with_retry()
            with net_connect:
                yield net_connect
            return

        net_connect = (
            self._get_pooled_connection()
            or self._connect_with_retry()
        )

        try:
            yield net_connect
        except Exception:
            # If the caller raises an exception assume the connection
            # is no longer alive and clean it up.
            net_connect.disconnect()
            raise
        else:
            # If the caller completes without an exception assume the
            # connection is still alive and put it back in the pool.
            try:
                self._connection_pool.put_nowait(net_connect)
            except queue.Full:
                net_connect.disconnect()

    def _connect_with_retry(self):
        """Create a new netmiko connection with retries."""
        retry_exc_types = (paramiko.SSHException, EOFError)

        # Use tenacity to handle retrying.
        @tenacity.retry(
            # Log a message after each failed attempt.
            after=tenacity.after_log(LOG, logging.DEBUG),
            # Reraise exceptions if our final attempt fails.
            reraise=True,
            # Retry on SSH connection errors.
            retry=tenacity.retry_if_exception_type(retry_exc_types),
            # Stop after the configured timeout.
            stop=tenacity.stop_after_delay(
                int(self.ngs_config['ngs_ssh_connect_timeout'])),
            # Wait for the configured interval between attempts.
            wait=tenacity.wait_fixed(
                int(self.ngs_config['ngs_ssh_connect_interval'])),
        )
        def _create_connection():
            return netmiko.ConnectHandler(**self.config)

        # First, create a connection.
        try:
            return _create_connection()
        except tenacity.RetryError as e:
            LOG.error(
                _("Reached maximum SSH connection attempts, not retrying "
                  "for device: %(device)s, error: %(error)s"), {
                      'device': device_utils.sanitise_config(self.config),
                      'error': e})
            raise exc.GenericSwitchNetmikoConnectError()
        except Exception as e:
            LOG.error(
                _("Unexpected exception during SSH connection "
                  "to device: %(device)s, error: %(error)s"), {
                      'device': device_utils.sanitise_config(self.config),
                      'error': e})
            raise exc.GenericSwitchNetmikoConnectError()

    def _drain_cached_connections(self):
        if not self._connection_pool:
            return

        while not self._connection_pool.empty():
            try:
                net_connect = self._connection_pool.get_nowait()
                net_connect.disconnect()
            except queue.Empty:
                break
            except Exception:
                LOG.debug(
                    "Failed to close cached SSH connection during drain",
                    exc_info=True,
                )

    def send_commands_to_device(self, cmd_set):
        if not cmd_set:
            LOG.debug("Nothing to execute")
            return

        # If configured, batch up requests to the switch
        if self.batch_cmds is not None:
            return self.batch_cmds.do_batch(self, cmd_set)

        try:
            return self._send_commands_to_device(cmd_set)
        except Exception as error:
            if self._reuse_connection():
                LOG.warning(
                    "Retrying switch command execution with "
                    "a clean SSH connection for device: "
                    "%(device)s, error: %(error)s", {
                        'device': device_utils.sanitise_config(self.config),
                        'error': error})
                return self._send_commands_to_device(cmd_set)
            raise

    def _send_commands_to_device(self, cmd_set):
        try:
            with ngs_lock.PoolLock(self.locker, **self.lock_kwargs):
                with self._get_connection() as net_connect:
                    output = self.send_config_set(net_connect, cmd_set)
                    if self._get_save_configuration():
                        # Save configuration only if enabled in settings
                        # and when configuration is applied successfully.
                        self.save_configuration(net_connect)
        except exc.GenericSwitchException:
            # Reraise without modification exceptions originating from this
            # module.
            raise
        except Exception as e:
            LOG.error(
                _("Error sending commands to device: %(device)s, "
                  "error: %(error)s"), {
                      'device': device_utils.sanitise_config(self.config),
                      'error': e})
            raise exc.GenericSwitchNetmikoConnectError()

        LOG.debug(output)
        return output

    @check_output('add network')
    def add_network(self, segmentation_id, network_id, physnet_vlans=None):
        if physnet_vlans is not None:
            raise exc.GenericSwitchException(
                method="add_network: physnet_vlans is not supported by "
                       "Netmiko drivers")
        if not self._do_vlan_management():
            LOG.info(f"Skipping add network for {segmentation_id}")
            return ""

        # NOTE(zhenguo): Remove dashes from uuid as on most devices 32 chars
        # is the max length of vlan name.
        network_id = uuid.UUID(network_id).hex
        network_name = self._get_network_name(network_id, segmentation_id)
        # NOTE(mgoddard): Pass network_id and segmentation_id for drivers not
        # yet using network_name.
        cmds = self._format_commands(self.ADD_NETWORK,
                                     segmentation_id=segmentation_id,
                                     network_id=network_id,
                                     network_name=network_name)
        for port in self.get_trunk_ports():
            cmds += self._format_commands(self.ADD_NETWORK_TO_TRUNK,
                                          port=port,
                                          segmentation_id=segmentation_id)
        return self.send_commands_to_device(cmds)

    @check_output('delete network')
    def del_network(self, segmentation_id, network_id, physnet_vlans=None):
        if physnet_vlans is not None:
            raise exc.GenericSwitchException(
                method="del_network: physnet_vlans is not supported by "
                       "Netmiko drivers")
        if not self._do_vlan_management():
            LOG.info(f"Skipping delete network for {segmentation_id}")
            return ""

        # NOTE(zhenguo): Remove dashes from uuid as on most devices 32 chars
        # is the max length of vlan name.
        network_id = uuid.UUID(network_id).hex
        cmds = []
        for port in self.get_trunk_ports():
            cmds += self._format_commands(self.REMOVE_NETWORK_FROM_TRUNK,
                                          port=port,
                                          segmentation_id=segmentation_id)
        network_name = self._get_network_name(network_id, segmentation_id)
        # NOTE(mgoddard): Pass network_id and segmentation_id for drivers not
        # yet using network_name.
        cmds += self._format_commands(self.DELETE_NETWORK,
                                      segmentation_id=segmentation_id,
                                      network_id=network_id,
                                      network_name=network_name)
        return self.send_commands_to_device(cmds)

    @check_output('plug vni')
    def plug_switch_to_network(self, vni: int, segmentation_id: int,
                               physnet: str = None):
        # NOTE: PLUG_SWITCH_TO_NETWORK commands typically start with
        # 'vlan {segmentation_id}' which creates the VLAN if it doesn't
        # exist. This is necessary for L2VNI scenarios where the VLAN
        # segment may be dynamically allocated and not created via
        # add_network().
        #
        # physnet parameter added for per-physnet mcast-group resolution
        # in vendor-specific implementations (e.g., Cisco NX-OS).
        # Devices that don't need physnet can ignore this parameter.
        cmds = [
            self._format_commands(
                self.PLUG_SWITCH_TO_NETWORK,
                vni=vni,
                segmentation_id=segmentation_id)
        ]

        return self.send_commands_to_device(cmds)

    @check_output('unplug vni')
    def unplug_switch_from_network(self, vni: int, segmentation_id: int,
                                   physnet: str = None):
        # physnet parameter added for signature consistency with
        # plug_switch_to_network. Most implementations won't need it.
        cmds = [
            self._format_commands(
                self.UNPLUG_SWITCH_FROM_NETWORK,
                vni=vni,
                segmentation_id=segmentation_id)
        ]

        return self.send_commands_to_device(cmds)

    def vlan_has_ports(self, segmentation_id: int) -> bool:
        """Check if a VLAN has any ports attached.

        :param segmentation_id: VLAN identifier to check
        :returns: True if VLAN has ports, False otherwise.
        """
        if not self.SHOW_VLAN_PORTS:
            # If no command defined, conservatively assume ports exist
            LOG.warning("SHOW_VLAN_PORTS not implemented for %s, "
                        "assuming VLAN %s has ports",
                        self.device_name, segmentation_id)
            return True

        cmd = self._format_commands(
            self.SHOW_VLAN_PORTS,
            segmentation_id=segmentation_id)

        try:
            with self._get_connection() as net_connect:
                output = net_connect.send_command(cmd[0])
                return self._parse_vlan_ports(output, segmentation_id)
        except Exception as e:
            LOG.error("Failed to check VLAN %s ports on %s: %s",
                      segmentation_id, self.device_name, e)
            # On error, conservatively assume ports exist
            return True

    def _parse_vlan_ports(self, output: str, segmentation_id: int) -> bool:
        """Parse output of SHOW_VLAN_PORTS command.

        Subclasses should override this for vendor-specific parsing.

        :param output: Command output from switch
        :param segmentation_id: VLAN identifier being checked
        :returns: True if VLAN has ports, False otherwise
        """
        # Default implementation: conservatively assume ports exist
        return True

    def vlan_has_vni(self, segmentation_id: int, vni: int) -> bool:
        """Check if a VLAN has a specific VNI configured.

        :param segmentation_id: VLAN identifier to check
        :param vni: VNI to check for
        :returns: True if VLAN has this VNI configured, False otherwise
        """
        if not self.SHOW_VLAN_VNI:
            # If no command defined, conservatively return False
            # (assume VNI not configured, will attempt to configure)
            LOG.warning("SHOW_VLAN_VNI not implemented for %s, "
                        "cannot check if VNI %s is on VLAN %s",
                        self.device_name, vni, segmentation_id)
            return False

        cmd = self._format_commands(
            self.SHOW_VLAN_VNI,
            segmentation_id=segmentation_id,
            vni=vni,
            nve_interface=getattr(self, 'nve_interface', 'nve1'),
            vxlan_interface=getattr(self, 'vxlan_interface', 'Vxlan1'))

        try:
            with self._get_connection() as net_connect:
                output = net_connect.send_command(cmd[0])
                return self._parse_vlan_vni(output, segmentation_id, vni)
        except Exception as e:
            LOG.error("Failed to check VLAN %s VNI %s on %s: %s",
                      segmentation_id, vni, self.device_name, e)
            # On error, conservatively return False (will attempt to configure)
            return False

    def _parse_vlan_vni(self, output: str, segmentation_id: int,
                        vni: int) -> bool:
        """Parse output to check if VLAN has VNI configured.

        Subclasses should override this for vendor-specific parsing.

        :param output: Command output from switch
        :param segmentation_id: VLAN identifier being checked
        :param vni: VNI to check for
        :returns: True if VLAN has this VNI, False otherwise
        """
        # Default implementation: return False (assume not configured)
        return False

    @check_output('plug port')
    def plug_port_to_network(self, port, segmentation_id, trunk_details=None,
                             default_vlan=None):
        cmds = []
        if self._disable_inactive_ports() and self.ENABLE_PORT:
            cmds += self._format_commands(self.ENABLE_PORT, port=port)
        port_default_vlan = default_vlan or self._get_port_default_vlan()
        if port_default_vlan:
            cmds += self._format_commands(
                self.DELETE_PORT,
                port=port,
                segmentation_id=port_default_vlan)

        if trunk_details:
            cmds += self._format_commands(self.SET_NATIVE_VLAN,
                                          port=port,
                                          segmentation_id=segmentation_id)
            for sub_port in trunk_details.get('sub_ports', []):
                cmds += self._format_commands(
                    self.ADD_NETWORK_TO_TRUNK, port=port,
                    segmentation_id=sub_port['segmentation_id'])
        else:
            cmds += self._format_commands(
                self.PLUG_PORT_TO_NETWORK,
                port=port,
                segmentation_id=segmentation_id)

        return self.send_commands_to_device(cmds)

    @check_output('unplug port')
    def delete_port(self, port, segmentation_id, trunk_details=None,
                    default_vlan=None):
        cmds = self._format_commands(self.DELETE_PORT,
                                     port=port,
                                     segmentation_id=segmentation_id)
        port_default_vlan = default_vlan or self._get_port_default_vlan()
        if trunk_details:
            cmds += self._format_commands(self.DELETE_NATIVE_VLAN,
                                          port=port,
                                          segmentation_id=segmentation_id)
            for sub_port in trunk_details.get('sub_ports', []):
                cmds += self._format_commands(
                    self.REMOVE_NETWORK_FROM_TRUNK, port=port,
                    segmentation_id=sub_port['segmentation_id'])

        if port_default_vlan:
            # NOTE(mgoddard): Pass network_id and segmentation_id for drivers
            # not yet using network_name.
            network_name = self._get_network_name(port_default_vlan,
                                                  port_default_vlan)
            cmds += self._format_commands(
                self.ADD_NETWORK,
                segmentation_id=port_default_vlan,
                network_id=port_default_vlan,
                network_name=network_name)
            cmds += self._format_commands(
                self.PLUG_PORT_TO_NETWORK,
                port=port,
                segmentation_id=port_default_vlan)
        if self._disable_inactive_ports() and self.DISABLE_PORT:
            cmds += self._format_commands(self.DISABLE_PORT, port=port)

        return self.send_commands_to_device(cmds)

    @check_output('plug bond')
    def plug_bond_to_network(self, bond, segmentation_id, trunk_details=None,
                             default_vlan=None):
        # Fallback to regular plug port if no specialist PLUG_BOND_TO_NETWORK
        # commands set
        if not self.PLUG_BOND_TO_NETWORK:
            return self.plug_port_to_network(bond, segmentation_id,
                                             trunk_details=trunk_details)
        cmds = []
        if self._disable_inactive_ports() and self.ENABLE_BOND:
            cmds += self._format_commands(self.ENABLE_BOND, bond=bond)
        port_default_vlan = default_vlan or self._get_port_default_vlan()
        if port_default_vlan:
            cmds += self._format_commands(
                self.UNPLUG_BOND_FROM_NETWORK,
                bond=bond,
                segmentation_id=port_default_vlan)

        if trunk_details:
            cmds += self._format_commands(self.SET_NATIVE_VLAN_BOND,
                                          bond=bond,
                                          segmentation_id=segmentation_id)
            for sub_port in trunk_details.get('sub_ports', []):
                cmds += self._format_commands(
                    self.ADD_NETWORK_TO_BOND_TRUNK, bond=bond,
                    segmentation_id=sub_port['segmentation_id'])
        else:
            cmds += self._format_commands(
                self.PLUG_BOND_TO_NETWORK,
                bond=bond,
                segmentation_id=segmentation_id)

        return self.send_commands_to_device(cmds)

    @check_output('unplug bond')
    def unplug_bond_from_network(self, bond, segmentation_id,
                                 trunk_details=None, default_vlan=None):
        # Fallback to regular port delete if no specialist
        # UNPLUG_BOND_FROM_NETWORK commands set
        if not self.UNPLUG_BOND_FROM_NETWORK:
            return self.delete_port(bond, segmentation_id,
                                    trunk_details=trunk_details)
        cmds = self._format_commands(self.UNPLUG_BOND_FROM_NETWORK,
                                     bond=bond,
                                     segmentation_id=segmentation_id)
        port_default_vlan = default_vlan or self._get_port_default_vlan()
        if trunk_details:
            cmds += self._format_commands(self.DELETE_NATIVE_VLAN_BOND,
                                          bond=bond,
                                          segmentation_id=segmentation_id)
            for sub_port in trunk_details.get('sub_ports', []):
                cmds += self._format_commands(
                    self.DELETE_NETWORK_ON_BOND_TRUNK, bond=bond,
                    segmentation_id=sub_port['segmentation_id'])

        if port_default_vlan:
            # NOTE(mgoddard): Pass network_id and segmentation_id for drivers
            # not yet using network_name.
            network_name = self._get_network_name(port_default_vlan,
                                                  port_default_vlan)
            cmds += self._format_commands(
                self.ADD_NETWORK,
                segmentation_id=port_default_vlan,
                network_id=port_default_vlan,
                network_name=network_name)
            cmds += self._format_commands(
                self.PLUG_BOND_TO_NETWORK,
                bond=bond,
                segmentation_id=port_default_vlan)
        if self._disable_inactive_ports() and self.DISABLE_BOND:
            cmds += self._format_commands(self.DISABLE_BOND, bond=bond)

        return self.send_commands_to_device(cmds)

    def send_config_set(self, net_connect, cmd_set):
        """Send a set of configuration lines to the device.

        :param net_connect: a netmiko connection object.
        :param cmd_set: a list of configuration lines to send.
        :returns: The output of the configuration commands.
        """
        net_connect.enable()
        return net_connect.send_config_set(config_commands=cmd_set,
                                           cmd_verify=False)

    def save_configuration(self, net_connect):
        """Try to save the device's configuration.

        :param net_connect: a netmiko connection object.
        """
        try:
            net_connect.save_config()
        except NotImplementedError:
            if self.SAVE_CONFIGURATION:
                for cmd in self.SAVE_CONFIGURATION:
                    net_connect.send_command(cmd)
            else:
                LOG.warning("Saving config is not supported for %s,"
                            " all changes will be lost after switch"
                            " reboot", self.config['device_type'])

    @check_output('add trunk subports')
    def add_subports_on_trunk(self, binding_profile, port_id, subports,
                              trunk_details=None):
        """Allow subports on trunk

        :param binding_profile: Binding profile of parent port
        :param port_id: The name of the switch port from
               Local Link Information
        :param subports: List with subports objects.
        :param trunk_details: Unused by Netmiko drivers.
        """
        cmds = []
        is_802_3ad = ngs_utils.is_802_3ad(binding_profile)

        for sub_port in subports:
            if is_802_3ad:
                cmds += self._format_commands(
                    self.ADD_NETWORK_TO_BOND_TRUNK, bond=port_id,
                    segmentation_id=sub_port['segmentation_id'])
            else:
                cmds += self._format_commands(
                    self.ADD_NETWORK_TO_TRUNK, port=port_id,
                    segmentation_id=sub_port['segmentation_id'])
        return self.send_commands_to_device(cmds)

    @check_output('delete trunk subports')
    def del_subports_on_trunk(self, binding_profile, port_id, subports,
                              trunk_details=None):
        """Remove subports from trunk

        :param binding_profile: Binding profile of parent port
        :param port_id: The name of the switch port from
               Local Link Information
        :param subports: List with subports objects.
        :param trunk_details: Unused by Netmiko drivers.
        """

        cmds = []
        is_802_3ad = ngs_utils.is_802_3ad(binding_profile)

        for sub_port in subports:
            if is_802_3ad:
                cmds += self._format_commands(
                    self.DELETE_NETWORK_ON_BOND_TRUNK, bond=port_id,
                    segmentation_id=sub_port['segmentation_id'])
            else:
                cmds += self._format_commands(
                    self.REMOVE_NETWORK_FROM_TRUNK, port=port_id,
                    segmentation_id=sub_port['segmentation_id'])
        return self.send_commands_to_device(cmds)

    def _get_acl_names(self, sg_id):
        """Get the ACL names for a security group.

        Subclasses should override this method to provide multiple ACL names
        on devices which require distinct ACLs for ipv4 vs ipv6 or ingress vs
        egress.

        :param sg_id: Security group ID
        :returns: A dict of ACL names
        """
        return {'security_group': f"ngs-{sg_id}"}

    def _prepare_security_group_rule(self, sg_id, rule):
        """Prepare a security group rule for use in a command.

        Subclasses should override this method when the rule dict does not
        contain the information required to run the rule commands.
        Transformations of remote_ip_prefix will be common.

        :param sg_id: Security group ID
        :param rule: A dict containing security group rule information
        :returns: A dict containing required substitution values
        """
        # reduce neutron object to a dict with only the required items
        rule_dict = {k: getattr(rule, k) for k in (
            'ethertype',
            'direction',
            'protocol',
            'port_range_min',
            'port_range_max',
            'remote_ip_prefix',
            'normalized_cidr'
        ) if getattr(rule, k, None) is not None}
        rule_dict.update(self._get_acl_names(sg_id))
        return rule_dict

    def _security_group_rule_commands(self, sg_id, rule):
        rule = self._prepare_security_group_rule(sg_id, rule)
        if rule['direction'] == 'ingress':
            cmd_template = self.ADD_SECURITY_GROUP_RULE_INGRESS
        else:
            cmd_template = self.ADD_SECURITY_GROUP_RULE_EGRESS
        if not cmd_template:
            # This could be empty because some switches don't support egress
            # rules. This error should propagate back to the user
            raise NotImplementedError()

        return self._format_commands(
            cmd_template,
            **rule)

    def _validate_rule(self, rule):
        """Validate a security group rule.

        :param rule: A security group rule.
        :returns: True if the rule should be created,
                  False if the rule should be silently ignored
        :raises: GenericSwitchSecurityGroupRuleNotSupported for rules not
                 supported by this switch.
        """
        LOG.debug("Validating rule: %s", rule)

        # handle switches which don't support port ranges
        min_port = rule.port_range_min
        max_port = rule.port_range_max
        if rule.protocol in (const.PROTO_NAME_TCP,
                             const.PROTO_NAME_UDP):
            if not self.SUPPORT_SG_PORT_RANGE:
                if min_port and max_port and min_port != max_port:
                    raise exc.GenericSwitchSecurityGroupRuleNotSupported(
                        message='Only one port per rule is supported '
                        'on this switch')

        # Ignore deny-all rule. Drivers need to be aware if this is implicit
        # when an ACL is created or if they need to create the deny-all rule
        # explicitly.
        elif not rule.protocol:
            return False
        return True

    def _security_group_commands(self, sg, delete_first=False):
        names = self._get_acl_names(sg.id)
        cmds = []
        if delete_first:
            cmds += self._format_commands(self.REMOVE_SECURITY_GROUP,
                                          **names)

        cmds += self._format_commands(self.ADD_SECURITY_GROUP,
                                      **names)
        for rule in sg.rules:
            if self._validate_rule(rule):
                cmds += self._security_group_rule_commands(sg.id, rule)

        cmds += self._format_commands(self.ADD_SECURITY_GROUP_COMPLETE)
        return cmds

    @check_output('add security group')
    def add_security_group(self, sg):
        """Add a security group to a switch

        :param sg: Security group object including rules
        """
        cmds = self._security_group_commands(sg)
        return self.send_commands_to_device(cmds)

    @check_output('update security group')
    def update_security_group(self, sg):
        """Updates an existing a security group on a switch

        Rules may have been added or deleted so the driver
        needs to update the switch state to accurately reflect
        the provided security group.

        :param sg: Security group object including rules
        """
        cmds = self._security_group_commands(sg, delete_first=True)
        return self.send_commands_to_device(cmds)

    @check_output('delete security group')
    def del_security_group(self, sg_id):
        """Delete a security group

        :param sg_id: Security group ID
        """
        names = self._get_acl_names(sg_id)
        cmds = self._format_commands(self.REMOVE_SECURITY_GROUP,
                                     **names)
        return self.send_commands_to_device(cmds)

    @check_output('bind security group')
    def bind_security_group(self, sg, port_id, port_ids):
        """Apply security group to all ports

        The rules in the provided security group will also be
        used to assert the state with the switch.

        :param sg: Security group object including rules
        :param port_id: Name of switch port to bind group to
        :param port_ids: Names of all switch ports currently
                         bound to this group
        """
        names = self._get_acl_names(sg.id)
        # Recreate the security group based on the passed object
        # before binding to an interface
        cmds = self._security_group_commands(sg, delete_first=True)
        # Bind to all ports provided by port_ids
        for port in port_ids:
            cmds += self._format_commands(self.BIND_SECURITY_GROUP,
                                          port=port, **names)
        return self.send_commands_to_device(cmds)

    @check_output('unbind security group')
    def unbind_security_group(self, sg_id, port_id, port_ids):
        """Remove a bound security group from a port

        All existing bindings will also be asserted with the switch.

        :param sg_id: ID of security group to unbind
        :param port_id: Name of switch port to unbind group from
        :param port_ids: Names of all switch ports currently
                         bound to this group
        """
        names = self._get_acl_names(sg_id)
        # Unbind the one port specified
        cmds = self._format_commands(self.UNBIND_SECURITY_GROUP,
                                     port=port_id,
                                     **names)
        # Bind to all ports provided by port_ids
        for port in port_ids:
            cmds += self._format_commands(self.BIND_SECURITY_GROUP,
                                          port=port, **names)
        return self.send_commands_to_device(cmds)

    # NOTE(stevebaker) methods decorated with @check_output need to be
    # above this method
    def check_output(self, output, operation):
        """Check the output from the device following an operation.

        Drivers should implement this method to handle output from devices and
        perform any checks necessary to validate that the configuration was
        applied successfully.

        :param output: Output from the device.
        :param operation: Operation being attempted. One of 'add network',
            'delete network', 'plug port', 'unplug port'.
        :raises: GenericSwitchNetmikoConfigError if the driver detects that an
            error has occurred.
        """
        if not output:
            return

        for pattern in self.ERROR_MSG_PATTERNS:
            if pattern.search(output):
                LOG.error(
                    _("Found invalid configuration in device response. "
                      "Operation: %(operation)s. Output: %(output)s. "
                      "Device: %(device)s"), {
                          'operation': operation, 'output': output,
                          'device': device_utils.sanitise_config(self.config)
                    })
                raise exc.GenericSwitchNetmikoConfigError()
