feat: add Gateway UDPRoute
Type of change:
- [ ] Bugfix
- [x] New feature provided
- [ ] Improve performance
- [ ] Backport patches
What this PR does / why we need it:
Just implement basic usage. Need other PR to support UDPRoute weight feature after Upstream can create nodes by weight.
@tao12345666333 Could you please help me rerun CI again? The unit-test passed in my local env.
sure
Codecov Report
Merging #1278 (496442c) into master (40f1372) will decrease coverage by
0.26%. The diff coverage is0.00%.
@@ Coverage Diff @@
## master #1278 +/- ##
==========================================
- Coverage 40.73% 40.47% -0.27%
==========================================
Files 77 78 +1
Lines 7030 7076 +46
==========================================
Hits 2864 2864
- Misses 3851 3897 +46
Partials 315 315
| Impacted Files | Coverage Δ | |
|---|---|---|
| pkg/providers/gateway/translation/gateway.go | 0.00% <0.00%> (ø) |
|
| .../providers/gateway/translation/gateway_udproute.go | 0.00% <0.00%> (ø) |
|
| pkg/providers/gateway/translation/translator.go | 0.00% <ø> (ø) |
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more
@stillfox-lee could you please resolve the conflicts? thanks!
Some CI tasks failed, do you have time to deal with them? Because some logic changes were made in some recently merged PRs A breaking change occurred. #1251
Some CI tasks failed, do you have time to deal with them? Because some logic changes were made in some recently merged PRs A breaking change occurred. #1251
I will fix it ASAP.
@tao12345666333 I think this PR is ready for review.
Except for the comments above, everything else is LGTM. Also, could you merge master branch to resolve conflicts?
Then can merge this one.
@tao12345666333 It looks like CI needs your approval to run.
First-time contributors need a maintainer to approve running workflows. Learn more.
@tao12345666333, Please approve the CI. It's weird that GitHub always identifies me as first time contributor.
This is a known issue and I'm already looking for a solution
@tao12345666333 Please trigger CI again...
The failed test case seems not related to this PR.
all passed!