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

[202012] Fix tagged VlanInterface if attached to multiple vlan as untagged member #10589

Merged
merged 3 commits into from
May 9, 2022

Conversation

qiluo-msft
Copy link
Collaborator

Backport #8927 to 202012 branch

commit 7b9800b
Merge: 211776c 12ebe3f
Author: Qi Luo <qiluo-msft@users.noreply.github.com>
Date:   Wed Apr 13 00:24:31 2022 +0000

    Merge branch 'master' into qiluo/tagged

commit 211776c
Author: Qi Luo <qiluo-msft@users.noreply.github.com>
Date:   Sun Apr 10 20:22:16 2022 +0000

    Add test case of vlan member tagged attribute for non-backend TOR

    Signed-off-by: Qi Luo <qiluo-msft@users.noreply.github.com>

commit 2ebb727
Merge: 4deb792 487a29a
Author: Qi Luo <qiluo-msft@users.noreply.github.com>
Date:   Sat Apr 9 00:20:40 2022 +0000

    Merge branch 'master' into qiluo/tagged

commit 4deb792
Author: Qi Luo <qiluo-msft@users.noreply.github.com>
Date:   Fri Oct 8 05:34:56 2021 +0000

    Fix bug: uninitialized vlantype_name

commit 6366538
Author: Qi Luo <qiluo-msft@users.noreply.github.com>
Date:   Fri Oct 8 03:52:58 2021 +0000

    [minigraph] Fix tagged VlanInterface if attached to multiple vlan as untagged member

    Signed-off-by: Qi Luo <qiluo-msft@users.noreply.github.com>
Copy link
Contributor

@kktheballer kktheballer left a comment

Choose a reason for hiding this comment

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

Validated AzD scenario using 202012 minigraph.py change

@qiluo-msft qiluo-msft merged commit be5eb80 into sonic-net:202012 May 9, 2022
@qiluo-msft qiluo-msft deleted the qiluo/tagged_202012 branch May 9, 2022 21:07
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