Skip to content

[link state] look up topology name until the separator char#857

Merged
yxieca merged 1 commit intosonic-net:masterfrom
yxieca:linkstate
Apr 5, 2019
Merged

[link state] look up topology name until the separator char#857
yxieca merged 1 commit intosonic-net:masterfrom
yxieca:linkstate

Conversation

@yxieca
Copy link
Collaborator

@yxieca yxieca commented Apr 5, 2019

Type of change

  • Bug fix
  • [] Testbed and Framework(new/improvement)
  • [] Test case(new/improvement)

Approach

How did you do it?

Otherwise, if 2 systems have names where one is prefix of the other one, parsing of the
shorter name will come up with 2 lines.

Signed-off-by: Ying Xie [email protected]

How did you verify/test it?

both fast/warm reboot tests requires linkstate change before and after.

Otherwise, if 2 systems have names where one is prefix of the other one, parsing of the
shorter name will come up with 2 lines.

Signed-off-by: Ying Xie <[email protected]>
@yxieca yxieca merged commit 8ac8651 into sonic-net:master Apr 5, 2019
@yxieca yxieca deleted the linkstate branch April 5, 2019 17:56
yxieca added a commit that referenced this pull request Apr 5, 2019
Otherwise, if 2 systems have names where one is prefix of the other one, parsing of the
shorter name will come up with 2 lines.

Signed-off-by: Ying Xie <[email protected]>
deerao02 pushed a commit to deerao02/sonic-mgmt that referenced this pull request Dec 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants