Skip to content
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

Error running 1000s of tasks: "etcdserver: request is too large" #1186 #1264

Merged
merged 16 commits into from
Mar 18, 2019
Prev Previous commit
Next Next commit
Fixed Testcase
  • Loading branch information
sarabala1979 committed Mar 15, 2019
commit 1195457006d2dd53205cb8fa2d92c18ad4608d95
3 changes: 2 additions & 1 deletion workflow/util/file/fileutil.go
Original file line number Diff line number Diff line change
Expand Up @@ -91,11 +91,12 @@ func DecodeDecompressString(content string) (string, error) {
func CompressContent(content []byte) []byte {
var buf bytes.Buffer
zipWriter := gzip.NewWriter(&buf)
defer close(zipWriter)

_, err := zipWriter.Write(content)
if err != nil {
log.Warn("Error in compressing. v%", err)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should be log.Warnf also it should be %v not v%.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed

}
close(zipWriter)
return buf.Bytes()
}

Expand Down