Skip to content

[8101] Fix bug that confuses sonic name with alias#18744

Merged
Blueve merged 1 commit intosonic-net:masterfrom
Xichen96:dev/xichenlin/fix-8101-fanout-bug
Jun 3, 2025
Merged

[8101] Fix bug that confuses sonic name with alias#18744
Blueve merged 1 commit intosonic-net:masterfrom
Xichen96:dev/xichenlin/fix-8101-fanout-bug

Conversation

@Xichen96
Copy link
Contributor

@Xichen96 Xichen96 commented Jun 2, 2025

Description of PR

The previous ansible template does not distinguish between sonic name and alias in the links file, which causes sonic unable to understand vlan members which should always be sonic name

Summary:
Fixes # (issue)

Type of change

  • Bug fix
  • Testbed and Framework(new/improvement)
  • New Test case
    • Skipped for non-supported platforms
  • Test case improvement

Back port request

  • 202205
  • 202305
  • 202311
  • 202405
  • 202411
  • 202505

Approach

What is the motivation for this PR?

Fix 8101 fanout deployment bug

How did you do it?

Fix bug

How did you verify/test it?

Deployed fanout

Any platform specific information?

Cisco-8101 only

Supported testbed topology if it's a new test case?

Documentation

@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Copy link
Contributor

@yyynini yyynini left a comment

Choose a reason for hiding this comment

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

LGTM

@Blueve Blueve enabled auto-merge (squash) June 3, 2025 02:05
@Blueve Blueve merged commit c5b20e6 into sonic-net:master Jun 3, 2025
18 checks passed
opcoder0 pushed a commit to opcoder0/sonic-mgmt that referenced this pull request Dec 8, 2025
What is the motivation for this PR?
Fix 8101 fanout deployment bug

How did you do it?
The previous ansible template does not distinguish between sonic name and alias in the links file, which causes sonic unable to understand vlan members which should always be sonic name

How did you verify/test it?
Deployed fanout

Any platform specific information?
Cisco-8101 only

Signed-off-by: opcoder0 <[email protected]>
AharonMalkin pushed a commit to AharonMalkin/sonic-mgmt that referenced this pull request Dec 16, 2025
What is the motivation for this PR?
Fix 8101 fanout deployment bug

How did you do it?
The previous ansible template does not distinguish between sonic name and alias in the links file, which causes sonic unable to understand vlan members which should always be sonic name

How did you verify/test it?
Deployed fanout

Any platform specific information?
Cisco-8101 only

Signed-off-by: Aharon Malkin <[email protected]>
gshemesh2 pushed a commit to gshemesh2/sonic-mgmt that referenced this pull request Dec 21, 2025
What is the motivation for this PR?
Fix 8101 fanout deployment bug

How did you do it?
The previous ansible template does not distinguish between sonic name and alias in the links file, which causes sonic unable to understand vlan members which should always be sonic name

How did you verify/test it?
Deployed fanout

Any platform specific information?
Cisco-8101 only

Signed-off-by: Guy Shemesh <[email protected]>
venu-nexthop pushed a commit to venu-nexthop/sonic-mgmt that referenced this pull request Jan 13, 2026
What is the motivation for this PR?
Fix 8101 fanout deployment bug

How did you do it?
The previous ansible template does not distinguish between sonic name and alias in the links file, which causes sonic unable to understand vlan members which should always be sonic name

How did you verify/test it?
Deployed fanout

Any platform specific information?
Cisco-8101 only
gshemesh2 pushed a commit to gshemesh2/sonic-mgmt that referenced this pull request Jan 26, 2026
What is the motivation for this PR?
Fix 8101 fanout deployment bug

How did you do it?
The previous ansible template does not distinguish between sonic name and alias in the links file, which causes sonic unable to understand vlan members which should always be sonic name

How did you verify/test it?
Deployed fanout

Any platform specific information?
Cisco-8101 only

Signed-off-by: Guy Shemesh <[email protected]>
kazinator-arista pushed a commit to kazinator-arista/sonic-mgmt that referenced this pull request Mar 4, 2026
…D automatically (sonic-net#18744)

src/sonic-platform-daemons

* dee5310 - (HEAD -> 202205, origin/202205) [ci] Fix 202205 pipeline issue. (sonic-net#474) (3 months ago) [Liu Shilong]
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.

4 participants