-
Notifications
You must be signed in to change notification settings - Fork 799
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(experiments): add ability to set service port mappings #3544
base: master
Are you sure you want to change the base?
Conversation
Go Published Test Results2 160 tests 2 160 ✅ 2m 53s ⏱️ Results for commit 4a99969. ♻️ This comment has been updated with latest results. |
E2E Tests Published Test Results 4 files 4 suites 3h 29m 2s ⏱️ For more details on these failures, see this check. Results for commit 4a99969. ♻️ This comment has been updated with latest results. |
9f31bbd
to
ed8aa29
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #3544 +/- ##
==========================================
- Coverage 81.83% 78.18% -3.66%
==========================================
Files 135 158 +23
Lines 20688 18414 -2274
==========================================
- Hits 16931 14397 -2534
- Misses 2883 3108 +225
- Partials 874 909 +35 ☔ View full report in Codecov by Sentry. |
Signed-off-by: Yohan Belval <ybelval@turo.com>
Signed-off-by: Yohan Belval <ybelval@turo.com>
ed8aa29
to
74a6051
Compare
👋 @zachaller not sure why the code coverage dropped. I added an e2e test that covers the new code paths 🤔 |
Quality Gate passedIssues Measures |
@zachaller ping |
Adds enhancement #3428
Checklist:
"fix(controller): Updates such and such. Fixes #1234"
.