Closed stepanblyschak closed 1 year ago
@sumanbrcm Could you please handle this issue?
@stepanblyschak Sure, I will check and work on the fix accordingly . As per the previous discussion in PR (https://github.com/sonic-net/sonic-swss/pull/2199) , the proposal was to move cleanup code to syncd . As per your suggestion >>the cleanup of COUNTERS RIF tables should be done in syncd while cleanup of the mapping should still be done in orchagent << Once we have converged on the use case will fix on that line. Specific to the issue which you reported the likely reason is fetching in_octets has failed and but we are still using in_octets in calculation in below line : local rx_bps_new = (in_octets - in_octets_last) / delta * 1000 This is because table (counters_table_name) might not be existing for the rif. Similar issue is taken care in port_rates.lua script in the following way : if not in_ucast_pkts or not in_non_ucast_pkts or not out_ucast_pkts or not out_non_ucast_pkts or not in_octets or not out_octets then logit("Not found some counters on " .. port) return end We can fix this fix in similar lines along with moving the cleanup code to syncd.
@sumanbrcm is actively working on this issue.
@sumanbrcm could you please provide ETA when the fix can be in 202205?
@liat-grozovik ETA for this fix is 10th Oct.
Fix merged
Description
Errors in the log comming from RIF rate lua script due to orchagent removes RIF rates from COUNTERS DB.
Steps to reproduce the issue:
Describe the results you received:
Errors in the logs:
Describe the results you expected:
No errors
Output of
show version
:Output of
show techsupport
:Additional information you deem important (e.g. issue happens only occasionally):