Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix KeyError: 'members' for test_inner_hashing #25

Closed
wants to merge 1 commit into from

Conversation

JibinBao
Copy link
Owner

@JibinBao JibinBao commented Mar 16, 2023

Description of PR

KeyError: 'members' is introduced by this PR sonic-net/sonic-buildimage#13660.
In original config, we use PORTCHANNEL, now it is changed to PORTCHANNEL_MEMBER. So, the relevant test should be updated accordingly.

Test log:

oports = []
for ifname in ifnames:
if po.has_key(ifname):
oports.append([str(mg_facts['minigraph_ptf_indices'][x]) for x in po[ifname]['members']])
E KeyError: 'members'

Summary:
Fixes # (issue)

Type of change

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

Back port request

  • 201911
  • 202012
  • 202205

Approach

What is the motivation for this PR?

Fix test_inner_harhing issue

How did you do it?

PORTCHANNEL is changed to PORTCHANNEL_MEMBER

How did you verify/test it?

Run test_inner_hashing

Any platform specific information?

Any

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

Documentation

@JibinBao JibinBao closed this Mar 16, 2023
@JibinBao
Copy link
Owner Author

open new

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.

1 participant