Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

[Arista] Update platform library submodules #16112

Merged
merged 1 commit into from
Aug 23, 2023

Conversation

Staphylo
Copy link
Collaborator

Work item tracking
  • Microsoft ADO (number only):

How I did it

  • Ignore intermittent IO errors during get_change_event in the Platform API
  • Fix tunings for some ports on CatalinaDD
  • Fix kernel module build for 6.1 kernel in preparation of bookworm upgrade

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)

  • 202205
  • 202211
  • 202305
  • master

Description for the changelog

Update Arista platform library submodules

@Staphylo Staphylo requested a review from lguohan as a code owner August 11, 2023 14:48
@Staphylo
Copy link
Collaborator Author

@prgeor @StormLiangMS this addresses the xcvrd crash issue.
A backport to 202205, 202211 and 202305 could be attempted but IIRC backporting submodule updates doesn't work on release branches.
If that doesn't do it, I'll create individual PRs for the other branches.

@StormLiangMS
Copy link
Contributor

@prgeor could you help to review? this is the fix for 7050qx xcvrd issue.

@Staphylo
Copy link
Collaborator Author

@prgeor could we get some attention on this PR ?

@prgeor
Copy link
Contributor

prgeor commented Aug 23, 2023

@yxieca @lguohan please merge

@prgeor
Copy link
Contributor

prgeor commented Aug 23, 2023

@Staphylo for Ignore intermittent IO errors during get_change_event in the Platform API please raise separate PR for 202012, 202205, 202305

@yxieca yxieca merged commit c1c1054 into sonic-net:master Aug 23, 2023
qiluo-msft pushed a commit that referenced this pull request Aug 28, 2023
#### Why I did it

When an exception happens during `get_change_event` it brings the process invoking it down.

##### Work item tracking
- Microsoft ADO **(number only)**: 24599154

#### How I did it

This is a backport of #16112
Handle exceptions gracefully within `get_change_event`

#### Description for the changelog

Ignore poll errors during get_event_change on Arista devices
yxieca pushed a commit that referenced this pull request Aug 29, 2023
This is a backport of #16112
Handle exceptions gracefully within get_change_event
sonic-otn pushed a commit to sonic-otn/sonic-buildimage that referenced this pull request Sep 20, 2023
Ignore intermittent IO errors during get_change_event in the Platform API
Fix tunings for some ports on CatalinaDD
Fix kernel module build for 6.1 kernel in preparation of bookworm upgrade
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.

4 participants