sonic-net / sonic-buildimage

Scripts which perform an installable binary image build for SONiC
Other
743 stars 1.44k forks source link

CLI support for displaying DOM threshold fields #12911

Open mihirpat1 opened 2 years ago

mihirpat1 commented 2 years ago

Description

This issue will track the progress for enabling support for dumping the required fields for TRANSCEIVER_DOM_THRESHOLD through 'show int transceiver eeprom threshold' CLI Sample output of the CLI (the "show interfaces Ethernet4/21/1 transceiver detail" CLI will be changed to something else to be more specific to display DOM threshold values):

YQB20-0201-0100-03T2(s1)#show interfaces Ethernet4/21/1 transceiver detail 
mA: milliamperes, dBm: decibels (milliwatts), NA or N/A: not applicable.
A2D readouts (if they differ), are reported in parentheses.
The threshold values are calibrated.
                         High Alarm  High Warn   Low Alarm   Low Warn    
           Temperature   Threshold   Threshold   Threshold   Threshold   
Port       (Celsius)     (Celsius)   (Celsius)   (Celsius)   (Celsius)   
-------    ------------  ----------  ----------  ----------  ----------  
Et4/21/1   25.00         85.00       80.00       -15.00      -10.00    
                         High Alarm  High Warn   Low Alarm   Low Warn    
           Voltage       Threshold   Threshold   Threshold   Threshold   
Port       (Volts)       (Volts)     (Volts)     (Volts)     (Volts)     
-------    ------------  ----------  ----------  ----------  ----------  
Et4/21/1   3.26          3.60        3.50        2.90        3.00      
                         High Alarm  High Warn   Low Alarm   Low Warn    
           Current       Threshold   Threshold   Threshold   Threshold   
Port       (mA)          (mA)        (mA)        (mA)        (mA)        
-------    ------------  ----------  ----------  ----------  ----------  
Et4/21/1   23.27         54.37       53.42       0.00        0.00      
                         High Alarm  High Warn   Low Alarm   Low Warn    
           Tx Power      Threshold   Threshold   Threshold   Threshold   
Port       (dBm)         (dBm)       (dBm)       (dBm)       (dBm)       
-------    ------------  ----------  ----------  ----------  ----------  
Et4/21/1    N/A           N/A         N/A         N/A         N/A      
                         High Alarm  High Warn   Low Alarm   Low Warn    
           Rx Power      Threshold   Threshold   Threshold   Threshold   
Port       (dBm)         (dBm)       (dBm)       (dBm)       (dBm)       
-------    ------------  ----------  ----------  ----------  ----------  
Et4/21/1   -3.07         0.00        -0.46       -16.02      -13.01    
YQB20-0201-0100-03T2(s1)#

Steps to reproduce the issue:

1. 2. 3.

Describe the results you received:

Describe the results you expected:

Output of show version:

(paste your output here)

Output of show techsupport:

(paste your output here or download and attach the file here )

Additional information you deem important (e.g. issue happens only occasionally):

keboliu commented 1 year ago

@mihirpat1 shall this issue be closed since all PRs have been merged?

mihirpat1 commented 1 year ago

@mihirpat1 shall this issue be closed since all PRs have been merged?

No since transceiver DOM sensor and threshold related changes are still pending. So far, we have completed CLI for transceiver info and PM.