nd_interface_ethernet_access: align deploy control to config_actions.deploy

[allenrobel]

Related Issue(s)

Closes #365

Also checks off the #384 task-1 box for this module: its argspec imports now come from nd_argument_specs.py instead of the legacy nd.py.

Proposed Changes

Align nd_interface_ethernet_access deploy control with the convention used by the unmerged
interface modules: replace the flat top-level deploy boolean with a config_actions dict whose
single key is deploy.

• DOCUMENTATION: deploy: option replaced by config_actions: (dict) with a deploy suboption (type: bool, default true).
• EXAMPLES: deploy: false → config_actions: { deploy: false }.
• argument_spec: deploy=dict(...) → config_actions_spec(include=("deploy",)), the shared fragment added by Relocate nd_argument_spec to nd_argument_specs.py; add config_actions_spec fragment #387
  (reproduces the deploy-only block hand-written in the sibling interface modules byte-for-byte).
• nd_argument_spec import re-pointed from legacy nd.py to nd_argument_specs.py (Tracking: Gen-3 argspec imports via nd_argument_specs.py #384 task 1 for this module).
• Read site: reads config_actions.deploy (default true) and sets orchestrator.deploy.

Deploy scope and batching are unchanged (interface-scope interfaceActions/deploy, one bulk call per run);
only the toggle moves under config_actions.

No deprecation/changelog fragment: the module is unreleased (released changelog stops at 1.2.0;
galaxy 1.5.0), so nothing shipped is broken.

Test Notes

• ndblack --check / ndisort --check-only: clean.
• ndpylint / ndmypy: no new findings from this change. Remaining messages are pre-existing baseline
  in untouched code (missing-module-docstring, import-position, existing TODOs, and the environmental
  ansible_collections.* import-resolution errors).
• ndtest --test validate-modules plugins/modules/nd_interface_ethernet_access.py: passes (exit 0) —
  confirms DOCUMENTATION suboptions match the fragment-built argspec options.
• ndpytest tests/unit/module_utils/orchestrators/test_ethernet_access_interface.py tests/unit/module_utils/test_nd_argument_specs.py: 39 passed.
• Integration merged.yaml no-deploy task updated to the new param shape.

Cisco Nexus Dashboard Version

4.2.1

Related ND API Resource Category

• analyze
• infra
• manage
• onemanage
• other

Checklist

• Latest commit is rebased from develop with merge conflicts resolved
• New or updates to documentation has been made accordingly
• Assigned the proper reviewers

🤖 Generated with Claude Code

[Copilot]

Pull request overview

This PR aligns the nd_interface_ethernet_access module’s deploy toggle with the convention used by the other interface modules by moving the deploy boolean under a config_actions.deploy suboption, while keeping deploy scope/batching behavior unchanged (still interface-scope interfaceActions/deploy and a single bulk deploy call per run).

Changes:

• Replaced the top-level deploy module option with config_actions: { deploy: <bool> } in DOCUMENTATION/EXAMPLES and the module argument spec.
• Updated the module runtime to read config_actions.deploy (defaulting to true) and wire it to model_orchestrator.deploy.
• Updated the integration “no deploy” task to use the new parameter shape.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File	Description
plugins/modules/nd_interface_ethernet_access.py	Moves deploy control under config_actions.deploy across docs, examples, argspec, and runtime read site.
tests/integration/targets/nd_interface_ethernet_access/tasks/merged.yaml	Updates the no-deploy integration task to pass config_actions.deploy: false.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.