Skip to content

[Mellanox] Extend mellanox platform API to report SFP error event#4365

Merged
jleveque merged 6 commits intosonic-net:masterfrom
keboliu:sfp-error-event
Apr 14, 2020
Merged

[Mellanox] Extend mellanox platform API to report SFP error event#4365
jleveque merged 6 commits intosonic-net:masterfrom
keboliu:sfp-error-event

Conversation

@keboliu
Copy link
Collaborator

@keboliu keboliu commented Apr 3, 2020

- What I did
Extend SFP event API to report SFP error to the upper layer app.

- How I did it
handle SFP event which reported from SDK, log all the detail error type and pass the error event that will block EEPROM accessing to the upper layer.

- How to verify it
test on Mellanox platform

- Description for the changelog

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

This PR has a dependency on sonic-net/sonic-platform-common#80

@qiluo-msft
Copy link
Collaborator

qiluo-msft commented Apr 3, 2020

            if not port_dict == {}:

If not bool(port_dict): #Closed


Refers to: platform/mellanox/mlnx-platform-api/sonic_platform/chassis.py:451 in 3cef25c. [](commit_id = 3cef25c, deletion_comment = False)

@qiluo-msft
Copy link
Collaborator

qiluo-msft commented Apr 3, 2020

        return True

return found != 0; #Closed


Refers to: platform/mellanox/mlnx-platform-api/sonic_platform/sfp_event.py:268 in 3cef25c. [](commit_id = 3cef25c, deletion_comment = False)

@keboliu
Copy link
Collaborator Author

keboliu commented Apr 10, 2020

        return True

return found != 0;

Refers to: platform/mellanox/mlnx-platform-api/sonic_platform/sfp_event.py:268 in 3cef25c. [](commit_id = 3cef25c, deletion_comment = False)

revised, thanks.

@keboliu
Copy link
Collaborator Author

keboliu commented Apr 10, 2020

            if not port_dict == {}:

If not bool(port_dict):

Refers to: platform/mellanox/mlnx-platform-api/sonic_platform/chassis.py:451 in 3cef25c. [](commit_id = 3cef25c, deletion_comment = False)

revised, thanks.

@liat-grozovik
Copy link
Collaborator

retest mellanox please

@jleveque jleveque merged commit cfa112a into sonic-net:master Apr 14, 2020
abdosi pushed a commit that referenced this pull request Apr 15, 2020
)

* extend mellanox platform API to report SFP error event
* remove unnecessary loop code
* install enum34 to pmon to support using Enum
tiantianlv pushed a commit to SONIC-DEV/sonic-buildimage that referenced this pull request Apr 24, 2020
…nic-net#4365)

* extend mellanox platform API to report SFP error event
* remove unnecessary loop code
* install enum34 to pmon to support using Enum
mssonicbld added a commit that referenced this pull request Mar 21, 2026
…atically (#26306)

#### Why I did it
src/sonic-utilities
```
* fb3d73db - (HEAD -> master, origin/master, origin/HEAD) Fix migrate_sonic_packages() crash on symlink resolv.conf (#4365) (6 hours ago) [william8545]
```
#### 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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants