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: wrong endpoint for LoadBalancer type service #4729
Conversation
Codecov ReportBase: 45.93% // Head: 60.56% // Increases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## master #4729 +/- ##
===========================================
+ Coverage 45.93% 60.56% +14.62%
===========================================
Files 318 321 +3
Lines 31291 31614 +323
===========================================
+ Hits 14375 19147 +4772
+ Misses 14640 9974 -4666
- Partials 2276 2493 +217
Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
49cc7ee
to
1ecfea3
Compare
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: Qiaozp <qiaozhongpei.qzp@alibaba-inc.com> Fix test Signed-off-by: Qiaozp <qiaozhongpei.qzp@alibaba-inc.com> fix test Signed-off-by: Qiaozp <qiaozhongpei.qzp@alibaba-inc.com>
e82c3aa
to
2d511b7
Compare
Signed-off-by: Qiaozp <qiaozhongpei.qzp@alibaba-inc.com>
Successfully created backport PR #4738 for |
) * Chore: wrong endpoint for LoadBalancer type service(revert #4729) Signed-off-by: barnettZQG <barnett.zqg@gmail.com> * Fix: change the unit test Signed-off-by: barnettZQG <barnett.zqg@gmail.com> Signed-off-by: barnettZQG <barnett.zqg@gmail.com>
Signed-off-by: barnettZQG <barnett.zqg@gmail.com> (cherry picked from commit 92ed75c)
…rvice(revert #4729) (#4906) * Chore: wrong endpoint for LoadBalancer type service(revert #4729) Signed-off-by: barnettZQG <barnett.zqg@gmail.com> (cherry picked from commit 92ed75c) * Fix: change the unit test Signed-off-by: barnettZQG <barnett.zqg@gmail.com> (cherry picked from commit 7985353) Co-authored-by: barnettZQG <barnett.zqg@gmail.com>
) (kubevela#4902) * Chore: wrong endpoint for LoadBalancer type service(revert kubevela#4729) Signed-off-by: barnettZQG <barnett.zqg@gmail.com> * Fix: change the unit test Signed-off-by: barnettZQG <barnett.zqg@gmail.com> Signed-off-by: barnettZQG <barnett.zqg@gmail.com> Signed-off-by: tangoliu <ltg1019@163.com>
Signed-off-by: Qiaozp qiaozhongpei.qzp@alibaba-inc.com
Description of your changes
When service type is LoadBalancer, the actual exposed port is
port.nodePort
in status. per https://kubernetes.io/docs/reference/kubernetes-api/service-resources/service-v1/#ServiceSpecTest:
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