nd_interface_ethernet_access module

[allenrobel]

Summary

• Add nd_interface_ethernet_access Ansible module for managing ethernet accessHost interfaces on Nexus Dashboard
• Fan-out playbook shape: interface_names list with shared config_data, expanded into flat items before the state machine
• Shared EthernetBaseOrchestrator base class for all future ethernet interface modules (trunk, routed, breakout, freeform)
• Port-channel membership enforcement with whitelisted modifiable fields (description, admin_state, extra_config)
• Bulk normalize-based delete via interfaceActions/normalize with the int_trunk_host template defaults
• InterfaceDefaultConfig model providing the full int_trunk_host normalize payload
• EpManageInterfacesDelete and EpManageInterfacesNormalize endpoint classes
• File-based logging integrated via setup_logging() in main(). Module-level nd.nd_interface_ethernet_access logger emits debug records at expand_config, manage_state begin/end, and on NDStateMachineError (via log.exception so tracebacks reach the log file regardless of output_level). Logging activates only when ND_LOGGING_CONFIG points at a logging.config.dictConfig JSON file; otherwise the calls are no-ops.
• Integration tests for all four states: merged, replaced, overridden, deleted. The test target's tasks/main.yaml wraps its include_tasks in a block that forwards nd_logging_config (set in inventory.networking [nd:vars]) into the module subprocess as ND_LOGGING_CONFIG, since ansible-test strips the controller's shell environment.

Delete mechanism for physical ethernet interfaces

/api/v1/manage/fabrics/{fabricName}/interfaceActions/normalize is used with the full int_trunk_host config template defaults (mode: "trunk", policyType: "trunkHost", etc). This resets the interface to fabric defaults and changes its policyType to trunkHost, which naturally removes it from the accessHost query_all() filter, enabling idempotent state: overridden.

Merge order

This PR must be merged AFTER nd_interface_loopback (PR #208). It is based on that branch and depends on the state machine, orchestrator base classes, fabric context, and endpoint infrastructure introduced there.

Merge & rebase note

This PR is part of a stack with a follow-on branch (nd_interface_ethernet_trunk). Because the team uses squash merges, after this PR is merged, @allenrobel will rebase the remaining stack branch onto develop before the next PR can be opened/merged. Please ping him once this is squashed.

Unit tests

Unit tests now cover the two branch-new source files with 105 tests and 100% real line coverage:

• tests/unit/module_utils/models/test_ethernet_access_interface.py exercises every nested Pydantic model (Policy, NetworkOS, ConfigData, Interface) with field validation, range and enum constraints, normalize_policy_type, serialize_policy_type, round-trip to_payload / to_config / from_response / from_config, composite identifier, diff, merge, and argument_spec shape.
• tests/unit/module_utils/orchestrators/test_ethernet_access_interface.py wires a real RestSend with the file-based Sender from tests/unit/module_utils/sender_file.py and ResponseHandler to verify model_class, supports_bulk_* inheritance, and _managed_policy_types returning {"accessHost"}. A query_all smoke test exercises the inherited base-class logic end-to-end across two switches, confirming non-accessHost interfaces are filtered out and switchIp is injected on the kept interfaces.

Parametrize collapses all value-table tests (range checks, enum choice checks, serializer modes, normalizer inputs) into single functions with descriptive ids= labels so pytest output still names each case.

Test plan

• Integration tests pass for state: merged (create, fan-out, idempotency, update, deploy:false)
• Integration tests pass for state: replaced (full replace, fan-out, idempotency)
• Integration tests pass for state: overridden (reduce, idempotency, swap, fan-out, non-accessHost filtering)
• Integration tests pass for state: deleted (single, fan-out, idempotency, non-existent)
• Unit tests pass: python -m pytest tests/unit/module_utils/models/test_ethernet_access_interface.py tests/unit/module_utils/orchestrators/test_ethernet_access_interface.py (105 tests, 100% line coverage of the two new files)
• To enable file-based logging during an integration run, add nd_logging_config=/absolute/path/to/logging_config.json to [nd:vars] in tests/integration/inventory.networking. An example config conforming to logging.config.dictConfig is documented in plugins/module_utils/common/log.py.

🤖 Generated with Claude Code