-
Notifications
You must be signed in to change notification settings - Fork 339
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: fix sync delete svc error #1576
Conversation
@tao12345666333 please take a look at this PR, thanks |
thanks |
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.
Please add e2e test cases to cover this
Codecov Report
@@ Coverage Diff @@
## master #1576 +/- ##
==========================================
- Coverage 41.44% 41.36% -0.08%
==========================================
Files 87 87
Lines 7420 7434 +14
==========================================
Hits 3075 3075
- Misses 3990 4004 +14
Partials 355 355
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
how can i run this e2e test in local? cc @tao12345666333 |
Please check Makefile |
done, added e2e test |
Could you give some hint about this error: The push refers to repository [localhost:5000/apisix]
Get "http://localhost:5000/v2/": net/http: request canceled (Client.Timeout exceeded while awaiting headers)
make: *** [push-images] Error 1 Run this command |
@tao12345666333 please take a look at this PR, i had fixed the e2e test error, thanks |
@AlinsRan please take a look at this PR, thanks |
@tao12345666333 @AlinsRan please take a look at this PR, thanks |
Sorry for delay, let me take a look. |
@tao12345666333 please take a look at this PR, thanks |
…e#1576) Co-authored-by: Fabriceli <li842162578@gmail.com>
#1755) Co-authored-by: fabriceli <yihao.li.cn@hotmail.com> Co-authored-by: Fabriceli <li842162578@gmail.com>
Type of change:
What this PR does / why we need it:
#1528
Pre-submission checklist: