Skip to content

[UX2.0] Bump Transport WAN VPN Interface Ethernet Feature to 20.15 #604

Open
tzarski0 wants to merge 4 commits intoCiscoDevNet:mainfrom
tzarski0:bump_transport_wan_vpn_ethernet_interface
Open

[UX2.0] Bump Transport WAN VPN Interface Ethernet Feature to 20.15 #604
tzarski0 wants to merge 4 commits intoCiscoDevNet:mainfrom
tzarski0:bump_transport_wan_vpn_ethernet_interface

Conversation

@tzarski0
Copy link
Collaborator

Description

Bumps the minimum supported version of sdwan_transport_wan_vpn_interface_ethernet_feature UX2.0 feature to 20.15

Types of Changes

  • New feature (non-breaking change which adds functionality)
  • Bug fix (non-breaking change which fixes an issue)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Build/CI change
  • Code quality improvement/refactoring/documentation (no functional changes)

Checklist

  • My code follows the code style of this project
  • I have added tests to cover my changes
  • All new and existing tests pass locally

… with AND/OR logic

Redesign the conditional_attribute schema and implementation to support:
- Multiple conditions in a single conditional_attribute block
- Logical operators (AND/OR) between conditions
- Negation to check when conditions are NOT met
- Explicit null checks using empty value + negate flag

Breaking Changes:
- Old conditional_attribute format is no longer supported
- All 37 definition files migrated to new format

Changes:
- Update schema.yaml with new conditional_attribute structure
- Add YamlConfigCondition struct and update YamlConfigConditionalAttribute
- Implement helper functions: BuildConditionalLogic, BuildConditionCheck, BuildConditionalDescription
- Update all templates: profile_parcels, generic, feature_templates
- Migrate all 37 definition files to new format
- Fix cisco_vpn negation logic for empty value conditionals

The new format improves clarity and enables complex conditional logic
while maintaining backward compatibility in generated code behavior.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant