Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

ci: increase retention for logs of tests to 144 hours #4677

Merged
merged 2 commits into from
Jan 20, 2020

Conversation

gabreal
Copy link
Contributor

@gabreal gabreal commented Jan 20, 2020

increase test log file retention to 144h

cannot debug https://gitlab.parity.io/parity/substrate/-/jobs/356316/artifacts/file/cc589213_int_failure.log anymore

@gabreal gabreal added A0-please_review Pull request needs code review. M1-ci labels Jan 20, 2020
@TriplEight
Copy link
Contributor

Why 144 hours instead of 6 days? And why 6 when we use 7 for build artifacts?

Copy link
Contributor

@TriplEight TriplEight left a comment

Choose a reason for hiding this comment

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

change to days please

@gabreal
Copy link
Contributor Author

gabreal commented Jan 20, 2020

I changed it to 3 days, just being cautious. Afaics the size of the files is quite small in recent runs.

@TriplEight TriplEight self-requested a review January 20, 2020 16:18
Copy link
Contributor

@TriplEight TriplEight left a comment

Choose a reason for hiding this comment

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

OK, let's see how it goes and maybe will add archiving here

@gavofyork gavofyork merged commit 0995bb3 into master Jan 20, 2020
@gavofyork gavofyork deleted the ci-increase-log-retention branch January 20, 2020 16:26
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A0-please_review Pull request needs code review.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants