Skip to content

Commit

Permalink
[ycabled] remove some spurious logs (sonic-net#282)
Browse files Browse the repository at this point in the history
* [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 <vdahiya@microsoft.com>
  • Loading branch information
vdahiya12 authored Aug 10, 2022
1 parent c7b7543 commit a1d7257
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions sonic-ycabled/ycable/ycable_utilities/y_cable_helper.py
Original file line number Diff line number Diff line change
Expand Up @@ -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":
Expand Down Expand Up @@ -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")
Expand Down

0 comments on commit a1d7257

Please sign in to comment.