Skip to content

[202012][dualtor] fixing test_downstream_ecmp_nexthops#8196

Closed
Ndancejic wants to merge 4 commits intosonic-net:202012from
Ndancejic:202012
Closed

[202012][dualtor] fixing test_downstream_ecmp_nexthops#8196
Ndancejic wants to merge 4 commits intosonic-net:202012from
Ndancejic:202012

Conversation

@Ndancejic
Copy link
Contributor

@Ndancejic Ndancejic commented May 2, 2023

What I did:
changed test_downstream_ecmp_nexthops to check if packets being sent are funneled towards a single mux neighbor or portchannel

Why I did it:
test_downstream_ecmp_nexthops fails due to swss change which programs a single active mux nexthop or a single tunnel route for a given route

How to test:
run sonic-mgmt dualtor/test_orchagent_active_tor_downstream.py test on t0 topo.

Summary:

Type of change

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

Back port request

  • 201911
  • 202012
  • 202205

What I did:
changed test_downstream_ecmp_nexthops to check if packets being sent are
funneled towards a single mux neighbor or portchannel

Why I did it:
test_downstream_ecmp_nexthops fails due to swss change which programs a
single active mux nexthop or a single tunnel route for a given route

How to test:
run sonic-mgmt dualtor/test_orchagent_active_tor_downstream.py test on t0 topo.

Signed-off-by: Nikola Dancejic <[email protected]>
@Ndancejic Ndancejic marked this pull request as ready for review May 2, 2023 21:49
@prsunny
Copy link
Contributor

prsunny commented May 23, 2023

Please create a PR to master

@prsunny
Copy link
Contributor

prsunny commented Jun 13, 2023

Please create a PR to master

@Ndancejic , can you please tag master PR here?

@wangxin
Copy link
Collaborator

wangxin commented Jul 10, 2023

@Ndancejic Is this PR a backport of a master branch PR? Can you mention the master PR number in PR description?

@Ndancejic
Copy link
Contributor Author

cancelled in favor of master PR: #8906 will open new 202012 PR if needed

@Ndancejic Ndancejic closed this Jul 11, 2023
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