-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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 for data race in IP release features #18217
Fix for data race in IP release features #18217
Conversation
deb336e
to
643f0e5
Compare
Fixes: cilium#18204 Signed-off-by: Hemanth Malla <hemanth.malla@datadoghq.com>
643f0e5
to
55e1482
Compare
/test-race Edit: Looks like we hit a few data races and they are known #17914. |
/test Edit: |
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.
Looks good to me. Note that this fixes an issue in the Travis pipeline, not the Jenkins race pipeline.
Also marking for backport as the underlying code is scheduled for a v1.11 backport too.
Marking this as ready to merge, as this is impacting the stability of our Travis pipeline and all test failures are unrelated as per Chris' investigation above. |
@hemanthmalla Thanks for the PR! Merging. |
Fixes: #18204
Signed-off-by: Hemanth Malla hemanth.malla@datadoghq.com