-
Notifications
You must be signed in to change notification settings - Fork 26
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
Fix state machine handler race conditions introduced by strand::wrap
#257
Conversation
e37ec72
to
abc122f
Compare
cannot repro the UT failure on local device, will investigate soon. |
strand::wrap
strand::wrap
@lolyu please look into the checker failures. |
Sure, it is a pretty tricky regression due to the change, will fix soon. |
Signed-off-by: Longxiang Lyu <lolv@microsoft.com>
28c057a
to
34ac1e0
Compare
a74a321
to
b16de0f
Compare
strand::wrap
strand::wrap
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
When will the
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add PR description.
the PR description is there |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
sonic-net#257) Approach What is the motivation for this PR? As the subject. Work item tracking Microsoft ADO (number only): 28585413 How did you do it? Post the handler directly to handler instead of calling it with strand::wrap How did you verify/test it? UT Any platform specific information? Documentation
Cherry-pick PR to 202405: #272 |
#257) Approach What is the motivation for this PR? As the subject. Work item tracking Microsoft ADO (number only): 28585413 How did you do it? Post the handler directly to handler instead of calling it with strand::wrap How did you verify/test it? UT Any platform specific information? Documentation
sonic-net#257) Approach What is the motivation for this PR? As the subject. Work item tracking Microsoft ADO (number only): 28585413 How did you do it? Post the handler directly to handler instead of calling it with strand::wrap How did you verify/test it? UT Any platform specific information? Documentation
Cherry-pick PR to 202311: #274 |
#257) Approach What is the motivation for this PR? As the subject. Work item tracking Microsoft ADO (number only): 28585413 How did you do it? Post the handler directly to handler instead of calling it with strand::wrap How did you verify/test it? UT Any platform specific information? Documentation
Description of PR
Summary:
Fixes # (issue)
Type of change
Approach
What is the motivation for this PR?
As the subject.
Work item tracking
How did you do it?
Post the handler directly to handler instead of calling it with
strand::wrap
How did you verify/test it?
UT
Any platform specific information?
Documentation