-
Notifications
You must be signed in to change notification settings - Fork 867
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 AnalysisRunMetadata and Dryrun for experiments via Rollout #3213
Conversation
a18c8e9
to
34badf8
Compare
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## master #3213 +/- ##
==========================================
+ Coverage 81.82% 81.86% +0.03%
==========================================
Files 135 135
Lines 20688 20712 +24
==========================================
+ Hits 16928 16955 +27
+ Misses 2886 2883 -3
Partials 874 874 ☔ View full report in Codecov by Sentry. |
Go Published Test Results2 109 tests 2 109 ✅ 2m 50s ⏱️ Results for commit a13bc53. ♻️ This comment has been updated with latest results. |
34badf8
to
6eb0681
Compare
E2E Tests Published Test Results 4 files 4 suites 3h 27m 8s ⏱️ For more details on these failures, see this check. Results for commit a13bc53. ♻️ This comment has been updated with latest results. |
6eb0681
to
623abbd
Compare
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
Hi @zachaller , |
Hi @zachaller , could I get a review on this one ? |
18dc5a6
to
4eaf7ff
Compare
Signed-off-by: Guillaume Doussin <[email protected]>
4eaf7ff
to
a13bc53
Compare
Quality Gate passedThe SonarCloud Quality Gate passed, but some issues were introduced. 4 New issues |
Fixes #3209
Checklist:
"fix(controller): Updates such and such. Fixes #1234"
.