Skip to content

[202205] fix keyerror of PR #10382#11729

Merged
wangxin merged 1 commit intosonic-net:202205from
XuChen-MSFT:xuchen3/202205/keyerror-pr10382
Mar 4, 2024
Merged

[202205] fix keyerror of PR #10382#11729
wangxin merged 1 commit intosonic-net:202205from
XuChen-MSFT:xuchen3/202205/keyerror-pr10382

Conversation

@XuChen-MSFT
Copy link
Contributor

Description of PR

Summary:
Fixes # (issue)

Type of change

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

Back port request

  • 201911
  • 202012
  • 202205
  • 202305
  • 202311

Approach

What is the motivation for this PR?

double commit PR #11722

PR #10382 caused below two KeyError:

    @pytest.fixture(scope="function", autouse=False)
    def skip_pacific_dst_asic(self, dutConfig):
>       if dutConfig['dstDutAsic'] == "pac":
E       KeyError: 'dstDutAsic'
        if "wm_pg_shared_lossless" in pgProfile:
            pktsNumFillShared = qosConfig[pgProfile]["pkts_num_trig_pfc"]
        elif "wm_pg_shared_lossy" in pgProfile:
>           if dutConfig['dstDutAsic'] == "pac":
E           KeyError: 'dstDutAsic'

How did you do it?

fix keyerror

How did you verify/test it?

Any platform specific information?

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

Documentation

@wangxin wangxin merged commit 2e58059 into sonic-net:202205 Mar 4, 2024
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