Skip to content

[chassis] Enhance test_default_route.py for chassis#7024

Merged
abdosi merged 26 commits intosonic-net:masterfrom
abdosi:default_route
Dec 21, 2022
Merged

[chassis] Enhance test_default_route.py for chassis#7024
abdosi merged 26 commits intosonic-net:masterfrom
abdosi:default_route

Conversation

@abdosi
Copy link
Copy Markdown
Contributor

@abdosi abdosi commented Dec 14, 2022

What I did :

Enhance test_default_route.py for chassis.

How I did:

  • Concept of selecting upstream duthost and verify test case on that
  • Depends on this PR Everflow for Chassis/T2 topology #6945 for common utility function.
  • Some of Core Router even though T3 we might not expect Default Route based on Device Type.
  • For verifying Default route for downstream duthost in chassis via iBGP we will need to enhance test case further. Will create test gap for that.

How I verify:
Verified on both T1 and chassis topology.

abdosi added 26 commits February 5, 2021 17:00
@abdosi
Copy link
Copy Markdown
Contributor Author

abdosi commented Dec 14, 2022

cc @judyjoseph

@abdosi abdosi removed the request for review from arlakshm December 20, 2022 16:53
@abdosi
Copy link
Copy Markdown
Contributor Author

abdosi commented Dec 20, 2022

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@abdosi abdosi merged commit ef1a0d8 into sonic-net:master Dec 21, 2022
@abdosi abdosi deleted the default_route branch December 21, 2022 17:57
wangxin pushed a commit that referenced this pull request Dec 26, 2022
Enhance test_default_route.py for chassis.

How I did:

Concept of selecting upstream duthost and verify test case on that
Depends on this PR Everflow for Chassis/T2 topology #6945 for common utility function.
Some of Core Router even though T3 we might not expect Default Route based on Device Type.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants