Skip to content

Pfcwd: copy ptftests dir to ptf#1854

Merged
neethajohn merged 1 commit intosonic-net:masterfrom
neethajohn:fix_pfcwd_copy
Jul 7, 2020
Merged

Pfcwd: copy ptftests dir to ptf#1854
neethajohn merged 1 commit intosonic-net:masterfrom
neethajohn:fix_pfcwd_copy

Conversation

@neethajohn
Copy link
Contributor

Signed-off-by: Neetha John [email protected]

Description of PR

Fix error seen in pytest nightly runs:
"usage: usage: ptf [options] --test-dir TEST_DIR [tests]\nptf: error: invalid value for --test-dir: directory ptftests does not exist",

Type of change

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

How did you verify/test it?

Removed the existing ptftest dir from the ptf docker. Ran the test with the changes and it passed

@lgtm-com
Copy link

lgtm-com bot commented Jul 6, 2020

This pull request introduces 1 alert when merging 582ce82 into de77268 - view on LGTM.com

new alerts:

  • 1 for Unused import

@neethajohn neethajohn merged commit f9049ea into sonic-net:master Jul 7, 2020
@neethajohn neethajohn deleted the fix_pfcwd_copy branch July 7, 2020 00:36
kazinator-arista pushed a commit to kazinator-arista/sonic-mgmt that referenced this pull request Mar 4, 2026
d29a49a [ACL] Match TCP protocol while matching TCP_FLAG (sonic-net#1854)
2569ad9 Fix sFlow sampling-rate and admin-state (sonic-net#1728)
8908a8f Change rif_rates.lua and port_rates.lua scripts to calculate rates correct (sonic-net#1848)
b42c2fb [VS Test] Skip flaky tests (sonic-net#1875)

Signed-off-by: bingwang <[email protected]>
kazinator-arista pushed a commit to kazinator-arista/sonic-mgmt that referenced this pull request Mar 4, 2026
Submodule update for sonic-utilties

```
48035d75 [202012] [techsupport] Techsupport Error Reporting pending fixes (sonic-net#1854)
8b2ec09a Fix log_ssd_health hang issue (sonic-net#1904)
ac9c4254 Fix the option missing in kernel config issue (sonic-net#1888)
5cc9417a disk_check: Script updated to run good in 201811 & 201911 (sonic-net#1747)
```
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.

2 participants