Skip to content
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

[201911] Create Vxlan and Vnet default configs #13403

Merged
merged 4 commits into from
Feb 2, 2023

Conversation

prsunny
Copy link
Contributor

@prsunny prsunny commented Jan 17, 2023

Why I did it

Create Vxlan and Vnet default configs as below for certain cases:

    "VNET": {
        "Vnet-default": {
            "scope": "default",
            "vni": "8000",
            "vxlan_tunnel": "Tunnel-default"
        }
    },
    "VXLAN_TUNNEL": {
        "Tunnel-default": {
            "src_ip": "10.1.0.32"
        }
    },

How I did it

Master PR - #13409

How to verify it

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205
  • 202211

Description for the changelog

Ensure to add label/tag for the feature raised. example - PR#2174 under sonic-utilities repo. where, Generic Config and Update feature has been labelled as GCU.

Link to config_db schema for YANG module changes

A picture of a cute animal (not mandatory but encouraged)

@prsunny prsunny marked this pull request as ready for review January 18, 2023 19:53
@prsunny prsunny requested a review from lguohan January 25, 2023 00:24
@prsunny prsunny merged commit 5bd7761 into sonic-net:201911 Feb 2, 2023
@prsunny prsunny deleted the vxlan_ha_default branch February 2, 2023 02:36
@abdosi
Copy link
Contributor

abdosi commented Mar 1, 2023

@prsunny can you created PR for 201911 PR checker failure after this PR. Looks like PR checker were bypassed as part of this PR merge

@prsunny prsunny mentioned this pull request Mar 1, 2023
8 tasks
prsunny added a commit that referenced this pull request Mar 3, 2023
*Fix a typo introduced as part of #13403
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.

3 participants