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

[VS Test] Skip flaky tests #1875

Merged
merged 2 commits into from
Aug 17, 2021
Merged

[VS Test] Skip flaky tests #1875

merged 2 commits into from
Aug 17, 2021

Conversation

prsunny
Copy link
Collaborator

@prsunny prsunny commented Aug 14, 2021

What I did
Skip flaky tests for unblocking PRs, this is under investigation.

Why I did it

How I verified it

Details if related

@prsunny prsunny requested a review from lguohan August 14, 2021 02:28
@lguohan lguohan merged commit f501e1c into sonic-net:master Aug 17, 2021
@prsunny prsunny deleted the pytest branch August 17, 2021 16:15
qiluo-msft pushed a commit that referenced this pull request Aug 19, 2021
* Skip flaky test, under investigation
qiluo-msft pushed a commit to sonic-net/sonic-buildimage that referenced this pull request Aug 26, 2021
In order to include the following commits: 
d29a49a 2021-08-25 [ACL] Match TCP protocol while matching TCP_FLAG (sonic-net/sonic-swss#1854) 
2569ad9 2021-08-25 Fix sFlow sampling-rate and admin-state (sonic-net/sonic-swss#1728) 
8908a8f 2021-08-19 Change rif_rates.lua and port_rates.lua scripts to calculate rates correct (sonic-net/sonic-swss#1848) 
b42c2fb 2021-08-19 [VS Test] Skip flaky tests (sonic-net/sonic-swss#1875)
neethajohn pushed a commit that referenced this pull request Aug 27, 2021
Ref: #1875

The test expects buffer PGs created for all ports.
However, the vs image is deployed as cable length being 0m,
which provides the buffer PGs to be created and fails the test.

To fix the failure:
- Config the cable length of port under test to 300m if it was 0m
- Startup the port after cable length configured
  By doing that the buffer PG can be configured correctly
- Collect the orig_profiles after port is started up

Signed-off-by: Stephen Sun <stephens@nvidia.com>
qiluo-msft pushed a commit that referenced this pull request Aug 30, 2021
Ref: #1875

The test expects buffer PGs created for all ports.
However, the vs image is deployed as cable length being 0m,
which provides the buffer PGs to be created and fails the test.

To fix the failure:
- Config the cable length of port under test to 300m if it was 0m
- Startup the port after cable length configured
  By doing that the buffer PG can be configured correctly
- Collect the orig_profiles after port is started up

Signed-off-by: Stephen Sun <stephens@nvidia.com>
raphaelt-nvidia pushed a commit to raphaelt-nvidia/sonic-swss that referenced this pull request Oct 5, 2021
* Skip flaky test, under investigation
raphaelt-nvidia pushed a commit to raphaelt-nvidia/sonic-swss that referenced this pull request Oct 5, 2021
Ref: sonic-net#1875

The test expects buffer PGs created for all ports.
However, the vs image is deployed as cable length being 0m,
which provides the buffer PGs to be created and fails the test.

To fix the failure:
- Config the cable length of port under test to 300m if it was 0m
- Startup the port after cable length configured
  By doing that the buffer PG can be configured correctly
- Collect the orig_profiles after port is started up

Signed-off-by: Stephen Sun <stephens@nvidia.com>
EdenGri pushed a commit to EdenGri/sonic-swss that referenced this pull request Feb 28, 2022
What I did
Added EXP to TC map to tables affected by clear_qos command

How I did it
Found references to DSCP to TC map, added EXP to TC map in same place

How to verify it
Run clear_qos with EXP to TC map present
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants