Develop

.github/PULL_REQUEST_TEMPLATE.md

@@ -5,15 +5,33 @@
 - [ ] Documentation update
 
 ## Description
-Please include a summary of the changes and the related issue. Also, include relevant motivation and context.
 
-Bug Fix: [Brief description of the bug fixed]
-Root Cause (if applicable): [Explain what caused the bug]
-Fix Implemented: [Describe the fix applied]
+Summary:
+A brief description of the issue or bug.
 
-Enhancement: [Brief description of the improvement/enhancement made]
-Enhancement Description: [Explain what was enhanced, why, and how]
-Impact Area: [Mention which part of the system/codebase is affected]
+Steps to Reproduce:
+Detailed steps that developers can follow to reproduce the issue.
+
+Observed Behavior:
+What is currently happening, including error messages or symptoms.
+
+Expected Behavior:
+What should happen under normal conditions.
+
+Root Cause Analysis:
+Technical explanation of the underlying cause, if known.
+
+Workaround:
+Any temporary solutions or mitigations available.
+
+Fix Details:
+Description of the fix or code changes made.
+
+Impact:
+Information on the scope and severity of the issue.
+
+Additional Information:
+Logs, configuration snippets, environment details, or references to related bugs.
 
 Testing Done:
 - [] Manual testing

.gitignore

@@ -25,3 +25,4 @@ test.yaml
 sanity_fixes.py
 .yamlfmt
 .github/copilot-instructions.md
+ai-assistant-catalyst-center-ansible-iac/

CONTRIBUTING.md

@@ -0,0 +1,154 @@
+# Contributing
+
+This document applies to the entire `cisco.dnac` collection and defines
+the coding and code review standards for Ansible modules.
+
+## Core Principles
+
+- Follow Ansible core module development standards.
+- Follow PEP 8 with a maximum of 100 characters per line.
+- Keep YAML `DOCUMENTATION` blocks to a maximum of 80 characters per
+  line.
+- Code must be idempotent.
+- Avoid unnecessary complexity.
+- Use early returns for failure cases.
+- Keep logic linear and easy to follow.
+
+## Ansible Module Structure
+
+- Use `AnsibleModule` properly with `argument_spec`.
+- Validate required parameters via `argument_spec`, not manual checks.
+- Use `supports_check_mode=True` where applicable.
+- Respect `check_mode` and do not make changes in check mode.
+- Always return:
+  - `changed` (`bool`)
+  - `failed` (`bool`) when applicable
+  - meaningful result data
+- Use `module.exit_json()` and `module.fail_json()`.
+- Do not raise raw exceptions. Wrap errors using `fail_json()`.
+- Ensure idempotency:
+  - compare current state with desired state
+  - set `changed=True` only when an actual change occurs
+
+## Early Returns and Flow
+
+- Use early return for:
+  - validation failures
+  - unsupported states
+  - no-op and idempotent cases
+- Avoid nested `if`/`else` blocks where possible.
+- Keep execution flow sequential.
+
+## Logging Standards
+
+- Use `module.log()` for logging.
+- Do not use `print()` or the Python `logging` module.
+- Add an entry log describing the action.
+- Log input parameters, excluding sensitive data.
+- Do not log the function name directly.
+- Logs must describe the action.
+  - Example: `Validating VLAN configuration for vlan_id=10`
+- Log major decision points.
+- Log external API calls and why they are made.
+- Never log passwords, tokens, or sensitive values.
+- Logs must explain why an action is happening.
+
+## Documentation Requirements
+
+### `DOCUMENTATION` Block
+
+The `DOCUMENTATION` block must include:
+
+- `module`
+- `short_description`
+- `description`
+- `version_added`
+- `options` with `type`, `required`, `default`, and `choices` where
+  applicable
+- `author`
+
+### `EXAMPLES` Block
+
+- Keep examples minimal but meaningful.
+- Demonstrate idempotent usage.
+
+### `RETURN` Block
+
+- Document returned fields.
+- Include type and sample values.
+- Keep lines at 80 characters or fewer.
+
+### Documentation Rules
+
+- Be concise and accurate.
+- Keep descriptions clear and action-oriented.
+- Avoid unnecessary verbosity.
+- Follow consistent formatting across modules.
+
+## API Review Instructions
+
+- Review only the API logic.
+- In function docstrings include only:
+  - short description
+  - long description
+  - `Args`
+  - `Returns`
+- Do not include extra explanation.
+
+## Code Quality and Maintainability
+
+- Avoid redundant key lookups.
+- Avoid repeated validations.
+- Use helper functions only if they are reusable.
+- Do not split logic unnecessarily.
+- Keep state comparison clean and explicit.
+- Suggest better naming if unclear, but do not modify directly.
+- Ensure consistent naming patterns such as `success_count` and
+  `failed_count`.
+
+## Error Handling
+
+- Use `fail_json(msg=..., details=...)` where helpful.
+- Provide actionable error messages.
+- Include relevant identifiers in errors.
+- Avoid generic error messages.
+
+## Security Considerations
+
+- Mark sensitive parameters with `no_log=True`.
+- Never log sensitive data.
+- Avoid exposing secrets in return values.
+- Validate input types strictly.
+
+## Performance and Scalability
+
+- Avoid unnecessary API calls.
+- Cache fetched state when possible.
+- Minimize repeated lookups.
+- Prefer bulk operations when supported.
+
+## Suggestions Policy
+
+When suggesting improvements during review, suggestions may cover:
+
+- refactoring
+- optimization
+- documentation
+- tests
+- security
+- performance
+- readability
+- maintainability
+- scalability
+- compatibility
+- best practices
+- design patterns
+- architecture
+- conventions
+- anti-patterns
+
+Suggestions must:
+
+- be clear and actionable
+- not directly modify original code
+- preserve structure unless reuse justifies refactoring

changelogs/changelog.yaml

@@ -1985,3 +1985,23 @@ releases:
         - Changes in template_workflow_manager module
         - Changes in wired_campus_automation_workflow_manager module
         - Changes in wireless_design_workflow_manager module
+
+  6.49.0:
+    release_date: "2026-03-26"
+    changes:
+      release_summary: Added playbook config generator modules and shared helper
+        enhancements for workflow manager modules
+      minor_changes:
+        - Added playbook config generator modules for access point, application
+          policy, assurance, backup and restore, device credential, discovery,
+          events and notifications, inventory, ISE radius integration, network
+          profile, network settings, PnP, provision, and RMA workflows.
+        - Added playbook config generator modules for SDA fabric, SDA
+          extranet policies, SDA host port onboarding, site, tags, template,
+          user role, wired campus automation, and wireless design workflows.
+        - Added shared brownfield helper support, config validation, and
+          component filter deduplication for playbook config generator modules.
+        - Added device_ips, serial_numbers, and hostnames filters to
+          sda_host_port_onboarding_playbook_config_generator.
+        - Updated related workflow manager modules to align with the new
+          playbook config generator design.

galaxy.yml

@@ -1,7 +1,7 @@
 ---
 namespace: cisco
 name: dnac
-version: 6.48.2
+version: 6.49.0
 readme: README.md
 authors:
   - Rafael Campos <rcampos@cloverhound.com>
@@ -31,6 +31,9 @@ authors:
   - Karthick S N <kasn@cisco.com>
   - Sunil Shatagopa <sshatago@cisco.com>
   - Vivek Raj Penke <vpenke@cisco.com>
+  - Vidhya Rathinam <virathin@cisco.com>
+  - Mridul Saurabh <msaurabh@cisco.com>
+  - Jeet Ram <jeeram@cisco.com>
 
 description: "DEPRECATED - Use cisco.catalystcenter instead. Ansible Modules for Cisco DNA Center."
 license_file: "LICENSE"

playbooks/accesspoint_location_playbook_config_generator.yml

@@ -0,0 +1,140 @@
+---
+- name: Generate access point location playbook configuration from Cisco Catalyst Center
+  hosts: localhost
+  connection: local
+  gather_facts: false
+  vars_files:
+    - "credentials.yml"
+  tasks:
+    # Example 1: Generate all access point location configurations
+    - name: Generate complete access point location configuration
+      cisco.dnac.accesspoint_location_playbook_config_generator:
+        dnac_host: "{{ dnac_host }}"
+        dnac_username: "{{ dnac_username }}"
+        dnac_password: "{{ dnac_password }}"
+        dnac_verify: "{{ dnac_verify }}"
+        dnac_port: "{{ dnac_port }}"
+        dnac_version: "{{ dnac_version }}"
+        dnac_debug: "{{ dnac_debug }}"
+        dnac_log: true
+        dnac_log_level: DEBUG
+        dnac_api_task_timeout: 1000
+        dnac_task_poll_interval: 1
+        state: gathered
+        file_path: "tmp/accesspoint_location_workflow_playbook.yml"
+        file_mode: append
+
+    # Example 2: Generate access point location configurations by sites
+    - name: Generate access point location configurations by sites
+      cisco.dnac.accesspoint_location_playbook_config_generator:
+        dnac_host: "{{ dnac_host }}"
+        dnac_username: "{{ dnac_username }}"
+        dnac_password: "{{ dnac_password }}"
+        dnac_verify: "{{ dnac_verify }}"
+        dnac_port: "{{ dnac_port }}"
+        dnac_version: "{{ dnac_version }}"
+        dnac_debug: "{{ dnac_debug }}"
+        dnac_log: true
+        dnac_log_level: DEBUG
+        dnac_api_task_timeout: 1000
+        dnac_task_poll_interval: 1
+        state: gathered
+        file_path: "tmp/accesspoint_location_workflow_playbook_site_base.yml"
+        file_mode: "append"
+        config:
+          global_filters:
+            site_list:
+              - Global/USA/SAN JOSE/SJ_BLD23/FLOOR1
+              - Global/USA/SAN JOSE/SJ_BLD23/FLOOR2
+              - Global/USA/SAN JOSE/SJ_BLD23/FLOOR4
+
+    # Example 3: Generate access point location configurations by planned APs
+    - name: Generate access point location configurations by planned APs
+      cisco.dnac.accesspoint_location_playbook_config_generator:
+        dnac_host: "{{ dnac_host }}"
+        dnac_username: "{{ dnac_username }}"
+        dnac_password: "{{ dnac_password }}"
+        dnac_verify: "{{ dnac_verify }}"
+        dnac_port: "{{ dnac_port }}"
+        dnac_version: "{{ dnac_version }}"
+        dnac_debug: "{{ dnac_debug }}"
+        dnac_log: true
+        dnac_log_level: DEBUG
+        dnac_api_task_timeout: 1000
+        dnac_task_poll_interval: 1
+        state: gathered
+        file_path: "tmp/accesspoint_location_workflow_playbook_PAP_base.yml"
+        file_mode: "append"
+        config:
+          global_filters:
+            planned_accesspoint_list:
+              - ap_test_auto-1
+              - ap_test_auto-3
+
+    # Example 4: Generate access point location configurations by real APs
+    - name: Generate access point location configurations by real APs
+      cisco.dnac.accesspoint_location_playbook_config_generator:
+        dnac_host: "{{ dnac_host }}"
+        dnac_username: "{{ dnac_username }}"
+        dnac_password: "{{ dnac_password }}"
+        dnac_verify: "{{ dnac_verify }}"
+        dnac_port: "{{ dnac_port }}"
+        dnac_version: "{{ dnac_version }}"
+        dnac_debug: "{{ dnac_debug }}"
+        dnac_log: true
+        dnac_log_level: DEBUG
+        dnac_api_task_timeout: 1000
+        dnac_task_poll_interval: 1
+        state: gathered
+        file_path: "tmp/accesspoint_location_workflow_playbook_real_ap_base.yml"
+        file_mode: "append"
+        config:
+          global_filters:
+            real_accesspoint_list:
+              - AP687D.B402.1614-AP-Test6
+              - Cisco_9120AXE_IP4-01-Test2
+
+    # Example 5: Generate access point location configurations by access point models
+    - name: Generate access point location configurations by AP models
+      cisco.dnac.accesspoint_location_playbook_config_generator:
+        dnac_host: "{{ dnac_host }}"
+        dnac_username: "{{ dnac_username }}"
+        dnac_password: "{{ dnac_password }}"
+        dnac_verify: "{{ dnac_verify }}"
+        dnac_port: "{{ dnac_port }}"
+        dnac_version: "{{ dnac_version }}"
+        dnac_debug: "{{ dnac_debug }}"
+        dnac_log: true
+        dnac_log_level: DEBUG
+        dnac_api_task_timeout: 1000
+        dnac_task_poll_interval: 1
+        state: gathered
+        file_path: "tmp/accesspoint_location_workflow_playbook_accesspoint_model_base.yml"
+        file_mode: "overwrite"
+        config:
+          global_filters:
+            accesspoint_model_list:
+              - AP9120E
+              - CW9172I
+
+    # Example 6: Generate access point location configurations by MAC addresses
+    - name: Generate access point location configurations by MAC addresses
+      cisco.dnac.accesspoint_location_playbook_config_generator:
+        dnac_host: "{{ dnac_host }}"
+        dnac_username: "{{ dnac_username }}"
+        dnac_password: "{{ dnac_password }}"
+        dnac_verify: "{{ dnac_verify }}"
+        dnac_port: "{{ dnac_port }}"
+        dnac_version: "{{ dnac_version }}"
+        dnac_debug: "{{ dnac_debug }}"
+        dnac_log: true
+        dnac_log_level: DEBUG
+        dnac_api_task_timeout: 1000
+        dnac_task_poll_interval: 1
+        state: gathered
+        file_path: "tmp/accesspoint_location_workflow_playbook_mac_address_base.yml"
+        file_mode: "overwrite"
+        config:
+          global_filters:
+            mac_address_list:
+              - cc:6e:2a:e1:02:40