-
Notifications
You must be signed in to change notification settings - Fork 79
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
cli: add debug mode to upload-bin
#3672
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #3672 +/- ##
==========================================
+ Coverage 83.04% 83.07% +0.03%
==========================================
Files 334 334
Lines 46613 46623 +10
==========================================
+ Hits 38708 38732 +24
+ Misses 6333 6313 -20
- Partials 1572 1578 +6 ☔ View full report in Codecov by Sentry. |
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.
Should work, let's see the logs after testing, but let's not merge it into master. Binary can be taken from GA job anyway.
cli/util/upload_bin.go
Outdated
if errUpload != nil { | ||
return errUpload | ||
} | ||
fmt.Fprintf(ctx.App.Writer, "Uploaded block %d with object ID: %s\n", blockIndex, resOid.String()) |
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.
Let's move it under --debug, d
flag, these logs are useful. By default we don't want these logs to be enabled, but it's a nice opportunity to have them in debug mode.
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.
only for blocks?
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.
Removed index files OIDs logging.
Signed-off-by: Ekaterina Pavlova <[email protected]>
067af5b
to
0fd9717
Compare
upload-bin
for testing purposesupload-bin
Refs. #3658 (comment)