-
Notifications
You must be signed in to change notification settings - Fork 775
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: set leveledlogger for the pkg retryablehttp #2769
base: main
Are you sure you want to change the base?
Conversation
Codecov Report
@@ Coverage Diff @@
## main #2769 +/- ##
=======================================
Coverage 20.84% 20.84%
=======================================
Files 68 68
Lines 7344 7344
=======================================
Hits 1531 1531
Misses 5699 5699
Partials 114 114
Flags with carried forward coverage won't be shown. Click here to find out more. 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
@saltbo |
I have not found failed cause with this PR, can you help me retry to trigger it? |
Could you please rebase your branch from main ? |
3326d2a
to
baa4e29
Compare
I can't find why failed for the integration-test, the log is too long... can you help me to look at it? @shubham-bansal96 |
We typically export kubernetes logs in kind-logs. https://github.com/fission/fission/actions/runs/4932062313?pr=2769#artifacts All fission services logs can be noticed here. I have also written small utility to speedup analysis @shubham-bansal96 can you please check logs ? We may need to retrigger build. |
Can you please rebase with latest main ? |
Signed-off-by: saltbo <saltbo@foxmail.com>
Description
set leveledlogger for the pkg retryablehttp
Which issue(s) this PR fixes:
Fixes #2768
Testing
Checklist: