nd_interface_loopback: align deploy control to config_actions.deploy

[allenrobel]

⚠️ Merge Ordering — land before #403's post-#422 rebase

This PR and #403 (loopback policy_type discriminated union) both modify plugins/modules/nd_interface_loopback.py in the same regions (options block, EXAMPLES, main()), so textual conflicts between them are guaranteed. #403 is already held until #422 merges and owes a substantial rebase at that point (per the merge-ordering section in its body). Merging this small PR first lets #403 absorb the config_actions.deploy change in that one rebase it already owes. The reverse order works but is more expensive: this PR would have to re-express deploy control against the union-era module (which by then documents nine policy templates across two OSes).

Translation:

1. Merge Detect field removals in replaced/overridden states (one-way subset diff) #422
2. Merge this PR
3. Rebase nd_interface_loopback: policy_type discriminated union #403 onto develop after 1 and 2 above

Related Issue(s)

Closes #364

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_loopback 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.
• version_added corrected to 2.0.0 (review feedback).

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: 10.00/10; ndmypy: only the pre-existing environmental ansible_collections.*
  import-resolution error on the unchanged import block; no new findings from this change.
• ndtest --test validate-modules plugins/modules/nd_interface_loopback.py: passes (exit 0) —
  confirms DOCUMENTATION suboptions match the fragment-built argspec options.
• ndpytest tests/unit/module_utils/orchestrators/test_loopback_interface.py tests/unit/module_utils/test_nd_argument_specs.py: 28 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

Aligns nd_interface_loopback’s “deploy after mutation” toggle with the interface-module convention by nesting it under config_actions.deploy instead of a flat top-level deploy option, and updates the loopback integration scenario accordingly.

Changes:

• Replace module option deploy: bool with config_actions: { deploy: bool } in DOCUMENTATION/EXAMPLES and in argument_spec.
• Update the module runtime read site to set model_orchestrator.deploy from config_actions.deploy (defaulting to true).
• Update the loopback integration “no-deploy” task to use the new config_actions.deploy shape.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File	Description
plugins/modules/nd_interface_loopback.py	Moves deploy control under config_actions.deploy in docs, examples, argspec, and orchestration setup.
tests/integration/targets/nd_interface_loopback/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.

[Copilot]

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.