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.

Sign up for GitHub

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

Jump to bottom

[sonic-sairedis] Submodule update #3236

Merged
merged 1 commit into from
Jul 30, 2019
Merged

[sonic-sairedis] Submodule update #3236

merged 1 commit into from
Jul 30, 2019

Conversation

msosyak
Copy link
Contributor

@msosyak msosyak commented Jul 30, 2019

Signed-off-by: Myron Sosyak [email protected]

- What I did
Updated sonic-sairedis submodule with following changes:

  • acquire lock before accessing shared redis context (#489)
  • Fix bug: Restore sai log levels from LOGLEVEL_DB after saiDiscover (#491)
  • Fixed p4 profile change for BFN platform (#492)

- How I did it
Updated sonic-sairedis submodule

- How to verify it

  • Deploy image and verify all is up and running
  • Run ansbile tests

- Description for the changelog

[sonic-sairedis] Submodule updated

sonic-sairedis submodule update
747a92d 
Signed-off-by: Myron Sosyak <[email protected]>
@msosyak
Copy link
Contributor Author

msosyak commented Jul 30, 2019

@lguohan , @jleveque , could you please approve and merge? Thanks.

@lguohan lguohan merged commit 0175d14 into sonic-net:master Jul 30, 2019
mssonicbld added a commit that referenced this pull request May 17, 2024
@mssonicbld
[submodule] Update submodule sonic-utilities to the latest HEAD autom…
e53d48c 
…atically (#18977)

#### Why I did it
src/sonic-utilities
```
* 2d557ff9 - (HEAD -> 202311, origin/202311) Update sonic-utilities to support new SKU Mellanox-SN5600-O128 (#3236) (31 hours ago) [Stephen Sun]
* 60a0db88 - [chassis][midplane] Add notification to Supervisor when LC is graceful reboot (#3292) (31 hours ago) [Marty Y. Lok]
```
#### 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
Reviewers

@akokhan akokhan akokhan approved these changes

@lguohan lguohan lguohan approved these changes

Assignees
No one assigned
Labels
Submodule Update ⏫
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@msosyak @lguohan @akokhan @jleveque