-
Notifications
You must be signed in to change notification settings - Fork 857
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
Feat: velaux support override policy #4352
Feat: velaux support override policy #4352
Conversation
Signed-off-by: 楚岳 <wangyike.wyk@alibaba-inc.com> add comments Signed-off-by: 楚岳 <wangyike.wyk@alibaba-inc.com>
65fc032
to
b447b13
Compare
Codecov Report
@@ Coverage Diff @@
## master #4352 +/- ##
==========================================
+ Coverage 60.49% 61.09% +0.60%
==========================================
Files 343 345 +2
Lines 33832 33990 +158
==========================================
+ Hits 20467 20767 +300
+ Misses 10679 10498 -181
- Partials 2686 2725 +39
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
Signed-off-by: 楚岳 <wangyike.wyk@alibaba-inc.com> fix in the middle of work Signed-off-by: 楚岳 <wangyike.wyk@alibaba-inc.com> add test Signed-off-by: 楚岳 <wangyike.wyk@alibaba-inc.com>
24ecaea
to
3f28fcc
Compare
9b264e4
to
8d97ca2
Compare
Signed-off-by: 楚岳 <wangyike.wyk@alibaba-inc.com> reverse flacky test Signed-off-by: 楚岳 <wangyike.wyk@alibaba-inc.com> fix goling Signed-off-by: 楚岳 <wangyike.wyk@alibaba-inc.com>
8d97ca2
to
195b9d7
Compare
} | ||
} | ||
// the target policy isn't exist yet, put it in. | ||
return res[:i], len(c) != i |
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.
len(c) != i
it will only be true?
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.
if len(c) != i, that's mean target policy exist in the list, this function has delete it from returned result, and outer caller should update with it.
Signed-off-by: 楚岳 <wangyike.wyk@alibaba-inc.com>
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
Signed-off-by: 楚岳 wangyike.wyk@alibaba-inc.com
Description of your changes
Fixes #
I have:
make reviewable
to ensure this PR is ready for review.backport release-x.y
labels to auto-backport this PR if necessary.How has this code been tested
Special notes for your reviewer