Skip to content

meraki_appliance: Traffic_shaping & SD WAN Internet Policies Custom Performance Class and Depends On #67

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

Conversation

jon-humphries
Copy link
Collaborator

@jon-humphries jon-humphries commented May 28, 2025

  • Add depends on logic to all traffic shaping modules.

    The VLANs and Appliance must be added prior to configuring traffic shaping.

    Having the VLAN applied first ensures that if it is referenced in the rules or definitions it is there prior to applying the rules.

    Explicitly implies that a VLAN or Single VLAN must be configured first (which is a precursor to any Appliance deployment) and assumes the appliance must also have been added. You cannot add a VLAN or Single LAN address without adding an appliance to the inventory.

  • Add mapping from custom_performance_class_name to ID in:

    • networks_appliance_traffic_shaping_uplink_selection;
    • networks_appliance_sdwan_internet_policies.

@jon-humphries jon-humphries requested a review from vozhyk- May 28, 2025 15:24
@jon-humphries jon-humphries self-assigned this May 28, 2025
Co-authored-by: Witaut Bajaryn <[email protected]>
@jon-humphries jon-humphries force-pushed the feature-traffic-shaping-depends branch from 716db9e to 091cec7 Compare June 2, 2025 12:21
@jon-humphries jon-humphries changed the title meraki_appliance: Add depends_on for traffic shaping resources meraki_appliance: Traffic_shaping & SD WAN Internet Policies Custom Performance Class and Depends On Jun 2, 2025
@jon-humphries jon-humphries merged commit 4b099e1 into feature-0.3.0-release Jun 6, 2025
1 check passed
@jon-humphries jon-humphries deleted the feature-traffic-shaping-depends branch June 6, 2025 09:10
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.

2 participants