Skip to content

Conversation

@ccoueffe
Copy link
Collaborator

@ccoueffe ccoueffe commented Dec 17, 2025

…wercase for interfaces template

Related Issue(s)

Related Collection Role

  • cisco.nac_dc_vxlan.validate
  • cisco.nac_dc_vxlan.dtc.create
  • cisco.nac_dc_vxlan.dtc.deploy
  • cisco.nac_dc_vxlan.dtc.remove
  • other

Related Data Model Element

  • vxlan.fabric
  • vxlan.global
  • vxlan.topology
  • vxlan.underlay
  • vxlan.overlay
  • vxlan.overlay_extensions
  • vxlan.policy
  • vxlan.multisite
  • defaults.vxlan
  • other

Proposed Changes

add flag monitor for interface access, trunk, po access and po trunk

Test Notes

Cisco Nexus Dashboard Version

Checklist

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

…wercase for interfaces template

Signed-off-by: ccoueffe <ccoueffe@cisco.com>
Signed-off-by: ccoueffe <ccoueffe@cisco.com>
@ccoueffe ccoueffe self-assigned this Dec 17, 2025
@ccoueffe ccoueffe added the enhancement New feature or request label Dec 17, 2025
@ccoueffe ccoueffe marked this pull request as ready for review December 18, 2025 18:17
@ccoueffe ccoueffe requested a review from a team as a code owner December 18, 2025 18:17
deploy: false
profile:
admin_state: {{ interface['enabled'] | default(defaults.vxlan.topology.switches.interfaces.topology_switch_access_interface.enabled) }}
admin_state: {{ interface['enabled'] | default(defaults.vxlan.topology.switches.interfaces.topology_switch_access_interface.enabled) | lower }}
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why is the lower filter needed?

Copy link
Collaborator Author

@ccoueffe ccoueffe Dec 23, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's not mandatory. It was only to fix YAML boolean in lowercase.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants