Skip to content

[show] clean up residue of multi-asic show runningconfig test#3198

Merged
gechiang merged 1 commit intosonic-net:masterfrom
wen587:multi_show_test_fix
Mar 8, 2024
Merged

[show] clean up residue of multi-asic show runningconfig test#3198
gechiang merged 1 commit intosonic-net:masterfrom
wen587:multi_show_test_fix

Conversation

@wen587
Copy link
Copy Markdown
Contributor

@wen587 wen587 commented Mar 7, 2024

ADO: 27108147

What I did

Clean up TOPO residue left for multi-asic test

How I did it

change topo to default

How to verify it

unit test in 202205-chassis

Previous command output (if the output of a command-line utility has changed)

New command output (if the output of a command-line utility has changed)

@wen587
Copy link
Copy Markdown
Contributor Author

wen587 commented Mar 8, 2024

Since the base PR #3148 is not merged, it will auto set cherry pick conflict for 202305 and 202311. Will add 2023XX flag after base PR merged.

Copy link
Copy Markdown
Contributor

@gechiang gechiang left a comment

Choose a reason for hiding this comment

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

Thanks for finding this issue and fixing it!

@mssonicbld
Copy link
Copy Markdown
Collaborator

Cherry-pick PR to 202305: #3261

@mssonicbld
Copy link
Copy Markdown
Collaborator

Cherry-pick PR to 202311: #3263

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.

5 participants