-
Notifications
You must be signed in to change notification settings - Fork 449
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
Export the k8s logs of graphscope helm test and upload to artifact when the helm test failed #2683
Conversation
0bd708f
to
789d5c9
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.
Export logs for k8s-ci as well.
Don't touch the graphlearn submodule.
I'm sorry we can't dump the logs of the k8s-ci test at present, and current |
40893d8
to
ed814b1
Compare
gss.yaml also has a helm test. |
b55946d
to
6ca6621
Compare
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## main #2683 +/- ##
===========================================
- Coverage 73.61% 41.48% -32.14%
===========================================
Files 98 98
Lines 10317 10306 -11
===========================================
- Hits 7595 4275 -3320
- Misses 2722 6031 +3309 see 55 files with indirect coverage changes Continue to review full report in Codecov by Sentry.
|
Added. @sighingnow, @siyuan0322 could you please take a look at this? |
…to the artifact. Signed-off-by: Ye Cao <[email protected]>
Signed-off-by: Ye Cao <[email protected]>
Signed-off-by: Ye Cao <[email protected]>
6ca6621
to
932ba03
Compare
… during a period of time such as pytest (#2706) ## Related issue number An optimization for #2683 Signed-off-by: Ye Cao <[email protected]>
What do these changes do?
As titled.