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
Tests fix #406
Tests fix #406
Conversation
deniszh
commented
Mar 5, 2021
- pining ubuntu
- removing go 1.13
@@ -837,7 +837,7 @@ func (listener *CarbonserverListener) updateFileList(dir string, cacheMetricName | |||
|
|||
var freeSpace uint64 | |||
// diskspace can be negative and Bavail is therefore int64 | |||
if stat.Bavail >= 0 { // nolint:staticcheck skipcq: SCC-SA4003 | |||
if stat.Bavail >= 0 { // nolint:staticcheck // skipcq: SCC-SA4003 |
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.
Nice catch!
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.
That's not me, but golangci-lint 1.38
But still dunno why "TestThrottleChan" failing, it's working on my laptop :D
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.
The test seems to be a bit unstable. Sometimes it passed, sometimes it failed.
I think we can try modify the 0.95 to 0.9.
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.
Or I could re-phase it as the current newThrottleTicker can't meet the [0.95, 1.05] range 100% of the time. :P
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.
Strange, for me it's failing 100% on GHA and passing 100% on my laptop. Trying 90%-110% range now.
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.
Ok, working now, merging