Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Spell check test #1

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Spell check test #1

wants to merge 1 commit into from

Conversation

AntonHryshchuk
Copy link
Owner

dummy PR

blabla

@AntonHryshchuk
Dummy commit - blabla
6e9c214 
Signed-off-by: AntonHryshchuk <antonh@nvidia.com>
AntonHryshchuk pushed a commit that referenced this pull request Apr 2, 2023
@DavidZagury
[Mellanox] Improve FW upgrade logging (sonic-net#13465)
4cc84c6 
- Why I did it
To improve ASIC FW upgrade logging and have information about the cause of FW update failure in the log.

- How I did it
Added syslog logger support

In case the FW update has failed the update tool will give the cause of the failure in the output in the last line, starting with "Fail".
When running the tool, in case of a failed update, we will parse the output to retrieve the cause and log it.

Device #1:
 ----------
 
 Device Type:      ConnectX6DX
   Part Number:      MCX623106AN-CDA_Ax
   Description:      ConnectX-6 Dx EN adapter card; 100GbE; Dual-port QSFP56; PCIe 4.0/3.0 x16;
   PSID:             MT_0000000359
   PCI Device Name:  /dev/mst/mt4125_pciconf0
   Base GUID:        0c42a103007d22d4
   Base MAC:         0c42a17d22d4
   Versions:         Current        Available     
      FW             22.32.0498     22.32.0498    
      PXE            3.6.0500       3.6.0500      
      UEFI           14.25.0015     14.25.0015    
 
 Status:           Forced update required
 
---------
 Found 1 device(s) requiring firmware update...
 
Device #1: Updating FW ...     
 FSMST_INITIALIZE -   OK          
 Writing Boot image component -   OK          
 Fail : The Digest in the signature is wrong

- How to verify it
mlnx-fw-upgrade.sh --upgrade
@AntonHryshchuk AntonHryshchuk changed the base branch from master to 202111 April 2, 2023 15:05
@AntonHryshchuk AntonHryshchuk changed the base branch from 202111 to master April 2, 2023 15:05
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@AntonHryshchuk