Skip to content

Fix pfcwd error setting poll interval#1965

Merged
neethajohn merged 1 commit intosonic-net:masterfrom
neethajohn:pfcwd_interval
Jul 27, 2020
Merged

Fix pfcwd error setting poll interval#1965
neethajohn merged 1 commit intosonic-net:masterfrom
neethajohn:pfcwd_interval

Conversation

@neethajohn
Copy link
Copy Markdown
Contributor

Signed-off-by: Neetha John nejo@microsoft.com

Description of PR

Fixes the error seen in test_pfc_config.py run on T1
"stderr": "unable to use polling interval = 400ms, value is bigger than one of the configured detection time values, please choose a smaller polling_interval",

Type of change

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

How did you verify/test it?

Ran the test with the change and it passed

platform linux2 -- Python 2.7.12, pytest-4.6.9, py-1.8.1, pluggy-0.13.1
ansible: 2.8.7
rootdir: /var/nejo/Networking-acs-sonic-mgmt/tests, inifile: pytest.ini
plugins: ansible-2.2.2
collected 8 items

pfcwd/test_pfc_config.py ........ [100%]

====================================================================================================== 8 passed in 129.54 seconds =======================================================================================================

Signed-off-by: Neetha John <nejo@microsoft.com>
@neethajohn neethajohn merged commit d12b351 into sonic-net:master Jul 27, 2020
@neethajohn neethajohn deleted the pfcwd_interval branch July 27, 2020 15:28
kazinator-arista pushed a commit to kazinator-arista/sonic-mgmt that referenced this pull request Mar 4, 2026
dd71848 [GCU] Show default option for '--format' (sonic-net#2003)
f296e76 [GCU] Disallowing DeleteInsteadOfReplaceMoveExtender from generating delete whole config move (sonic-net#2006)
731d643 [flow counter] Fix issue: should not compare str with int (sonic-net#2001)
e628f01 Support CLI for buffer queue configuration (sonic-net#1965)
585fd40 Fix show ip bgp nei command rw required issue (sonic-net#2011)
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