Skip to content

Commit a1d7257

Browse files
authored
[ycabled] remove some spurious logs (sonic-net#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 <[email protected]>
1 parent c7b7543 commit a1d7257

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

sonic-ycabled/ycable/ycable_utilities/y_cable_helper.py

+2-2
Original file line numberDiff line numberDiff line change
@@ -1523,7 +1523,7 @@ def check_identifier_presence_and_update_mux_info_entry(state_db, mux_tbl, asic_
15231523
(cable_status, cable_type) = check_mux_cable_port_type(logical_port_name, port_tbl, asic_index)
15241524

15251525
if status is False:
1526-
helper_logger.log_warning("Could not retreive fieldvalue pairs for {}, inside config_db table {}".format(logical_port_name, port_tbl[asic_index].getTableName()))
1526+
helper_logger.log_info("Could not retreive fieldvalue pairs for {}, inside config_db table {}".format(logical_port_name, port_tbl[asic_index].getTableName()))
15271527
return
15281528

15291529
elif cable_status and cable_type == "active-standby":
@@ -3400,7 +3400,7 @@ def task_worker(self):
34003400
hw_mux_cable_tbl[asic_id] = swsscommon.Table(
34013401
state_db[asic_id], swsscommon.STATE_HW_MUX_CABLE_TABLE_NAME)
34023402
# TODO add definition inside app DB
3403-
status_tbl_peer[asic_id] = swsscommon.ConsumerStateTable(
3403+
status_tbl_peer[asic_id] = swsscommon.SubscriberStateTable(
34043404
appl_db[asic_id], "HW_FORWARDING_STATE_PEER")
34053405
fwd_state_command_tbl[asic_id] = swsscommon.SubscriberStateTable(
34063406
appl_db[asic_id], "FORWARDING_STATE_COMMAND")

0 commit comments

Comments
 (0)