Double commit #8546 add overflow_egress for PGDropTest multi-asic

[zhixzhu]

Description of PR

Double commit #8546
testQosSaiPGDrop is cisco only case, make it same as 202205 branch.

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?

How did you do it?

How did you verify/test it?

Any platform specific information?

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

Documentation

[mssonicbld]

The pre-commit check detected issues in the files touched by this pull request.
The pre-commit check is a mandatory check, please fix detected issues.

Detailed pre-commit check results:
trim trailing whitespace.................................................Passed
fix end of files.........................................................Passed
check yaml...........................................(no files to check)Skipped
check for added large files..............................................Passed
check python ast.........................................................Passed
flake8...................................................................Failed
- hook id: flake8
- exit code: 1

tests/saitests/py3/sai_qos_tests.py:255:18: E225 missing whitespace around operator
tests/saitests/py3/sai_qos_tests.py:255:19: F821 undefined name 'sai_thrift_read_pg_occupancy'
tests/saitests/py3/sai_qos_tests.py:260:17: E225 missing whitespace around operator
tests/saitests/py3/sai_qos_tests.py:260:18: F821 undefined name 'sai_thrift_read_pg_occupancy'
tests/saitests/py3/sai_qos_tests.py:264:17: E128 continuation line under-indented for visual indent
tests/saitests/py3/sai_qos_tests.py:265:17: E122 continuation line missing indentation or outdented
tests/saitests/py3/sai_qos_tests.py:3880:13: E225 missing whitespace around operator
tests/saitests/py3/sai_qos_tests.py:3881:12: E225 missing whitespace around operator
tests/saitests/py3/sai_qos_tests.py:3882:19: E225 missing whitespace around operator
tests/saitests/py3/sai_qos_tests.py:3883:11: E225 missing whitespace around operator
tests/saitests/py3/sai_qos_tests.py:3884:14: E225 missing whitespace around operator
...
[truncated extra lines, please run pre-commit locally to view full check results]

To run the pre-commit checks locally, you can follow below steps:

1. Ensure that default python is python3. In sonic-mgmt docker container, default python is python2. You can run
   the check by activating the python3 virtual environment in sonic-mgmt docker container or outside of sonic-mgmt
   docker container.
2. Ensure that the pre-commit package is installed:

sudo pip install pre-commit

3. Go to repository root folder
4. Install the pre-commit hooks:

pre-commit install

5. Use pre-commit to check staged file:

pre-commit

6. Alternatively, you can check committed files using:

pre-commit run --from-ref <commit_id> --to-ref <commit_id>

[mssonicbld]

The pre-commit check detected issues in the files touched by this pull request.
The pre-commit check is a mandatory check, please fix detected issues.

Detailed pre-commit check results:
trim trailing whitespace.................................................Passed
fix end of files.........................................................Passed
check yaml...........................................(no files to check)Skipped
check for added large files..............................................Passed
check python ast.........................................................Passed
flake8...................................................................Failed
- hook id: flake8
- exit code: 1

tests/saitests/py3/sai_qos_tests.py:255:18: E225 missing whitespace around operator
tests/saitests/py3/sai_qos_tests.py:255:19: F821 undefined name 'sai_thrift_read_pg_occupancy'
tests/saitests/py3/sai_qos_tests.py:260:17: E225 missing whitespace around operator
tests/saitests/py3/sai_qos_tests.py:260:18: F821 undefined name 'sai_thrift_read_pg_occupancy'
tests/saitests/py3/sai_qos_tests.py:264:17: E128 continuation line under-indented for visual indent
tests/saitests/py3/sai_qos_tests.py:265:17: E122 continuation line missing indentation or outdented
tests/saitests/py3/sai_qos_tests.py:3937:53: E128 continuation line under-indented for visual indent

flake8...............................................(no files to check)Skipped
check conditional mark sort..........................(no files to check)Skipped

To run the pre-commit checks locally, you can follow below steps:

1. Ensure that default python is python3. In sonic-mgmt docker container, default python is python2. You can run
   the check by activating the python3 virtual environment in sonic-mgmt docker container or outside of sonic-mgmt
   docker container.
2. Ensure that the pre-commit package is installed:

sudo pip install pre-commit

3. Go to repository root folder
4. Install the pre-commit hooks:

pre-commit install

5. Use pre-commit to check staged file:

pre-commit

6. Alternatively, you can check committed files using:

pre-commit run --from-ref <commit_id> --to-ref <commit_id>

[mssonicbld]

The pre-commit check detected issues in the files touched by this pull request.
The pre-commit check is a mandatory check, please fix detected issues.

Detailed pre-commit check results:
trim trailing whitespace.................................................Passed
fix end of files.........................................................Passed
check yaml...........................................(no files to check)Skipped
check for added large files..............................................Passed
check python ast.........................................................Passed
flake8...................................................................Failed
- hook id: flake8
- exit code: 1

tests/saitests/py3/sai_qos_tests.py:266:19: E122 continuation line missing indentation or outdented
tests/saitests/py3/sai_qos_tests.py:267:19: E122 continuation line missing indentation or outdented
tests/saitests/py3/switch.py:890:9: F405 'SAI_INGRESS_PRIORITY_GROUP_STAT_CURR_OCCUPANCY_BYTES' may be undefined, or defined from star imports: ptf.thriftutils, switch_sai_thrift.sai_headers, switch_sai_thrift.ttypes

flake8...............................................(no files to check)Skipped
check conditional mark sort..........................(no files to check)Skipped

To run the pre-commit checks locally, you can follow below steps:

1. Ensure that default python is python3. In sonic-mgmt docker container, default python is python2. You can run
   the check by activating the python3 virtual environment in sonic-mgmt docker container or outside of sonic-mgmt
   docker container.
2. Ensure that the pre-commit package is installed:

sudo pip install pre-commit

3. Go to repository root folder
4. Install the pre-commit hooks:

pre-commit install

5. Use pre-commit to check staged file:

pre-commit

6. Alternatively, you can check committed files using:

pre-commit run --from-ref <commit_id> --to-ref <commit_id>

[abdosi]

@zhixzhu : please check if this PR is their in 202305

[zhixzhu]

@abdosi Yes, it got in 202305 via #10725