From a1d7257a75ac152963ccc807baf97da76093f352 Mon Sep 17 00:00:00 2001 From: vdahiya12 <67608553+vdahiya12@users.noreply.github.com> Date: Wed, 10 Aug 2022 12:59:06 -0700 Subject: [PATCH] [ycabled] remove some spurious logs (#282) * [ycabled] remove some spurious logs For Ports which do not belong to active-active or active-standby type, we should not try to post muxcable telemetry information for them. This PR remove the warning message for such a posting. Also fixes the PR pipeline by changing the ConsumerTable to SubscriberStateTable Signed-off-by: vaibhav-dahiya --- sonic-ycabled/ycable/ycable_utilities/y_cable_helper.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/sonic-ycabled/ycable/ycable_utilities/y_cable_helper.py b/sonic-ycabled/ycable/ycable_utilities/y_cable_helper.py index f0e4aad8ce48..245932d233ef 100644 --- a/sonic-ycabled/ycable/ycable_utilities/y_cable_helper.py +++ b/sonic-ycabled/ycable/ycable_utilities/y_cable_helper.py @@ -1523,7 +1523,7 @@ def check_identifier_presence_and_update_mux_info_entry(state_db, mux_tbl, asic_ (cable_status, cable_type) = check_mux_cable_port_type(logical_port_name, port_tbl, asic_index) if status is False: - helper_logger.log_warning("Could not retreive fieldvalue pairs for {}, inside config_db table {}".format(logical_port_name, port_tbl[asic_index].getTableName())) + helper_logger.log_info("Could not retreive fieldvalue pairs for {}, inside config_db table {}".format(logical_port_name, port_tbl[asic_index].getTableName())) return elif cable_status and cable_type == "active-standby": @@ -3400,7 +3400,7 @@ def task_worker(self): hw_mux_cable_tbl[asic_id] = swsscommon.Table( state_db[asic_id], swsscommon.STATE_HW_MUX_CABLE_TABLE_NAME) # TODO add definition inside app DB - status_tbl_peer[asic_id] = swsscommon.ConsumerStateTable( + status_tbl_peer[asic_id] = swsscommon.SubscriberStateTable( appl_db[asic_id], "HW_FORWARDING_STATE_PEER") fwd_state_command_tbl[asic_id] = swsscommon.SubscriberStateTable( appl_db[asic_id], "FORWARDING_STATE_COMMAND")