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

[sairedis] Update sairedis submodule #5677

Closed
wants to merge 1 commit into from

Conversation

daall
Copy link
Contributor

@daall daall commented Oct 20, 2020

[vs] Add CRM SAI attributes to virtual switch interface (#673)
Fix boot type for fast boot (#674)
Add support for saiplayer bulk API and add performance timers (#666)
Add ZeroMQ communication channel between sairedis and syncd (#659)
Support System ports config (#657)
Enable fabric counter for syncd's FlexCounter (#669)

Signed-off-by: Danny Allen [email protected]

@daall
Copy link
Contributor Author

daall commented Oct 21, 2020

retest broadcom please

@daall
Copy link
Contributor Author

daall commented Oct 21, 2020

retest mellanox please

@daall
Copy link
Contributor Author

daall commented Oct 21, 2020

retest vsimage please

@lguohan
Copy link
Collaborator

lguohan commented Oct 21, 2020

retest vsimage please

1 similar comment
@lguohan
Copy link
Collaborator

lguohan commented Oct 22, 2020

retest vsimage please

@lguohan
Copy link
Collaborator

lguohan commented Oct 23, 2020

@daall , there seems some geninue problem of this pr. the vsimage tests continuously failing.

@daall
Copy link
Contributor Author

daall commented Oct 23, 2020

@daall , there seems some geninue problem of this pr. the vsimage tests continuously failing.

ack, I'll take a look

@lguohan
Copy link
Collaborator

lguohan commented Oct 25, 2020

depends on #5715

@daall
Copy link
Contributor Author

daall commented Oct 25, 2020

depends on #5715

Thanks, @lguohan! Do you want me to update this PR to grab the other bug fixes from sairedis as well?

@lguohan
Copy link
Collaborator

lguohan commented Oct 26, 2020

retest vsimage please

@lguohan
Copy link
Collaborator

lguohan commented Oct 26, 2020

let's just do your current commits, i haven't tested other commits from sairedis yet.

@daall
Copy link
Contributor Author

daall commented Oct 26, 2020

#5703 includes all of these commits, closing

@daall daall closed this Oct 26, 2020
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