-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
[help]Questions about some pull requests #1899
Comments
5 tasks
judyjoseph
added a commit
that referenced
this issue
Sep 14, 2021
sonic-swss commits 5fbd113 [Flex-counters] Fix the delay of flex counters flow to prevent infinite loop (#1899) dc685b6 [portsorch] Add an extra check before setting oper speed to APPL_DB (#1885) ceef728 Update port_rates & rif_rates lua scripts to convert poll_interval to MS (#1855) sonic-utilities commits 3160753 [ci]: Support PR coverage (#1806) 3316fdb fix wrong code indent in sfputil (#1808) c33e3a8 [config reload] Fix config reload failure due to sonic.target job cancellation (#1814) 4f7e107 [portconfig] Validate duplicate speed value and interface type value (#1745) 59817e2 [warm-reboot] Add new preboot health check: verify database integrity (#1785) bf2ff3c [portstat, intfstat] added rates and utilization (#1750) 3bf962c [show][platform] Revise chassis info fallback to only fall back on pmon crash (#1751)
theasianpianist
pushed a commit
to theasianpianist/sonic-buildimage
that referenced
this issue
Feb 5, 2022
…te loop (sonic-net#1899) * Fix delay flex counters flow to prevent infinite loop Signed-off-by: Shlomi Bitton <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Description
have opened some pull requests on sonic-buildimage two months ago:
#1702
#1703
these two PRs has been opened over two months and not reviewed yet
is there something wrong in these PRs or what i was missing
what should i do next?
thanks
Steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Additional information you deem important (e.g. issue happens only occasionally):
The text was updated successfully, but these errors were encountered: