-
Notifications
You must be signed in to change notification settings - Fork 351
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: support k8s native ingress and backend service in different… #1377
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1377 +/- ##
==========================================
- Coverage 40.73% 40.68% -0.06%
==========================================
Files 77 77
Lines 7030 7039 +9
==========================================
Hits 2864 2864
- Misses 3851 3860 +9
Partials 315 315
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
Thanks! Let me take a look. |
@tao12345666333 e2e test has passed in my local environment,how can i rerun this,would you help please |
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.
Sorry for the late review, code LGTM, but PR needs to resolve the conflict. Thanks!
@tao12345666333 PTAL |
yes! code LGTM. Please resolve conflicts. Thanks! |
166b392
to
8892ad2
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
… namespace
Type of change:
What this PR does / why we need it:
Resolves #1374
Pre-submission checklist: