Skip to content

GNU ci: compress logs before upload #3267

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.

Sign up for GitHub

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

Jump to bottom
Merged
sylvestre merged 1 commit into from
Mar 19, 2022
Merged

GNU ci: compress logs before upload #3267

sylvestre merged 1 commit into from
Mar 19, 2022

Conversation

@sylvestre
Copy link
Contributor

@sylvestre sylvestre commented Mar 18, 2022

Failed with

A 500 status code has been received, will attempt to retry the upload
Exponential backoff for retry #1. Waiting for 5416 milliseconds before continuing the upload at offset 0
An error has been caught http-client index 0, retrying the upload
Error: Client has already been disposed.
    at HttpClient.request (/home/runner/work/_actions/actions/upload-artifact/v2/dist/index.js:6081:19)
    at HttpClient.sendStream (/home/runner/work/_actions/actions/upload-artifact/v2/dist/index.js:6042:21)
    at UploadHttpClient.<anonymous> (/home/runner/work/_actions/actions/upload-artifact/v2/dist/index.js:7208:37)
    at Generator.next (<anonymous>)
    at /home/runner/work/_actions/actions/upload-artifact/v2/dist/index.js:6923:71
    at new Promise (<anonymous>)
    at module.exports.608.__awaiter (/home/runner/work/_actions/actions/upload-artifact/v2/dist/index.js:6919:12)
    at uploadChunkRequest (/home/runner/work/_actions/actions/upload-artifact/v2/dist/index.js:7206:46)
    at UploadHttpClient.<anonymous> (/home/runner/work/_actions/actions/upload-artifact/v2/dist/index.js:7243:38)
    at Generator.next (<anonymous>)
Exponential backoff for retry #1. Waiting for 5138 milliseconds before continuing the upload at offset 0

@sylvestre sylvestre mentioned this pull request Mar 18, 2022
Closed
@sylvestre sylvestre requested a review from tertsdiepraam March 18, 2022 19:43
@sylvestre
Copy link
Contributor Author

sylvestre commented Mar 18, 2022

#3269 is the real fix but we can still compress logs :)

tertsdiepraam
tertsdiepraam approved these changes Mar 18, 2022
View reviewed changes
Copy link
Member

@tertsdiepraam tertsdiepraam left a comment

Choose a reason for hiding this comment

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

Looks good. Do we have to decompress this in coreutils-tracking?

@sylvestre
Copy link
Contributor Author

sylvestre commented Mar 18, 2022

the tracking only relies on the json file

@sylvestre sylvestre merged commit ad237f8 into uutils:main Mar 19, 2022
@sylvestre sylvestre deleted the gnu-size branch March 19, 2022 08:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@tertsdiepraam tertsdiepraam tertsdiepraam approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@sylvestre @tertsdiepraam