Skip to content

[Mellanox] Provide default implementation for sfp error description when CMIS host management is enabled#203

Closed
Junchao-Mellanox wants to merge 1 commit intomasterfrom
master-error-desc
Closed

[Mellanox] Provide default implementation for sfp error description when CMIS host management is enabled#203
Junchao-Mellanox wants to merge 1 commit intomasterfrom
master-error-desc

Conversation

@Junchao-Mellanox
Copy link
Copy Markdown
Owner

@Junchao-Mellanox Junchao-Mellanox commented Nov 22, 2023

Why I did it

Provide a dummy implementation for SFP error description when CMIS host management is enabled. A future feature shall be raised to implement SFP error description for such mode.

Work item tracking
  • Microsoft ADO (number only):

How I did it

  1. if SFP is under software control, provide "Not supported" as error description
  2. if SFP is under initialization, provide "Initializing" as error description

How to verify it

unit test

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205
  • 202211
  • 202305

Tested branch (Please provide the tested image version)

Description for the changelog

Link to config_db schema for YANG module changes

A picture of a cute animal (not mandatory but encouraged)

@Junchao-Mellanox
Copy link
Copy Markdown
Owner Author

ci 3218 passed with known issue

Junchao-Mellanox pushed a commit that referenced this pull request Apr 2, 2024
…lly (sonic-net#18503)

#### Why I did it
src/sonic-gnmi
```
* feaa1df - (HEAD -> master, origin/master, origin/HEAD) Add inotify monitoring for certificates (#203) (11 hours ago) [Zain Budhwani]
```
#### How I did it
#### How to verify it
#### Description for the changelog
Junchao-Mellanox pushed a commit that referenced this pull request Mar 31, 2026
…omatically (sonic-net#26185)

#### Why I did it
src/sonic-mgmt-common
```
* 61d8b07 - (HEAD -> master, origin/master, origin/HEAD) Implement translib backend logic for the gNSI Credentialz (#203) (11 hours ago) [jayaragini-hcl]
```
#### How I did it
#### How to verify it
#### Description for the changelog
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants