-
Notifications
You must be signed in to change notification settings - Fork 362
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 the AddressGroup duplicated deleted bug #3136
Conversation
e155616
to
60c12ec
Compare
/test-all |
Codecov Report
@@ Coverage Diff @@
## main #3136 +/- ##
===========================================
+ Coverage 40.35% 58.59% +18.24%
===========================================
Files 167 295 +128
Lines 20879 24907 +4028
===========================================
+ Hits 8426 14595 +6169
+ Misses 11632 8733 -2899
- Partials 821 1579 +758
Flags with carried forward coverage won't be shown. Click here to find out more.
|
60c12ec
to
852a641
Compare
/test-integration |
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
@Jexf thanks for fixing it. I believe I saw error logs caused by it before and forgot to follow up.
nit: in the commit message "when generating AddressGroup keys from NetworkPolicy rules." |
Use sets instead of slice to resolve duplicated AddressGroup key problem when generating AddressGroup keys from NetworkPolicy rules. Signed-off-by: Wu zhengdong <zhengdong.wu@transwarp.io>
852a641
to
a1ea0c3
Compare
|
/test-all |
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
@Jexf there are some issues with the CI this week, so all PRs are pending for merging. I will take care of merging it once it's back to normal. |
/skip-all I have validated the required checks manually. |
Use sets instead of slice to resolve duplicated AddressGroup key problem when generating AddressGroup keys from NetworkPolicy rules.
Fixes: #3135
Signed-off-by: Wu zhengdong zhengdong.wu@transwarp.io