Skip to content
This repository has been archived by the owner on Oct 22, 2024. It is now read-only.

CI: no logging by default #445

Merged
merged 1 commit into from
Oct 21, 2019
Merged

CI: no logging by default #445

merged 1 commit into from
Oct 21, 2019

Conversation

pohly
Copy link
Contributor

@pohly pohly commented Oct 19, 2019

The recently added increased logging made the resulting
log files rather large, which is a problem when downloading
them and exhausted available space in the Jenkins master.

Now that extra logging is turned off by default respectively
done less frequently (top and df run once a minute). To enable
logging, edit the pipeline in a PR or in the Jenkins UI.

The recently added increased logging made the resulting
log files rather large, which is a problem when downloading
them and exhausted available space in the Jenkins master.

Now that extra logging is turned off by default respectively
done less frequently (top and df run once a minute). To enable
logging, edit the pipeline in a PR or in the Jenkins UI.
@pohly
Copy link
Contributor Author

pohly commented Oct 19, 2019

This worked. Here's a PR test run with the new default settings: https://cloudnative-k8sci.southcentralus.cloudapp.azure.com/view/pmem-csi/job/pmem-csi/view/change-requests/job/PR-445/3 => 7MB log file.

Here's one with all output enabled and 30s delay, done by using the "replay" action for the previous job and editing the script of the pipeline: https://cloudnative-k8sci.southcentralus.cloudapp.azure.com/view/pmem-csi/job/pmem-csi/view/change-requests/job/PR-445/4/ => 15.6MB log file.

I can't reproduce the figure of several GB for a log file that @okartau mentioned. Perhaps that was for a run with severe errors and retries where simply the long runtime with continuous output from the Kubernetes pods caused the log to balloon. That worst-case scenario now cannot happen anymore.

@avalluri, @okartau: Note that triggering a replay with edited script is also a fast way to retry just the failing configuration - simply remove all other steps.

@okartau
Copy link
Contributor

okartau commented Oct 19, 2019

I can't reproduce the figure of several GB for a log file that @okartau mentioned

If you refer to my recent statement in chat, I did not mean "several GB" in context of one build, but overall in CI system. I haven't made measurements myself, I can see that one e2e run locally produces 13..15 MB text file. We have multiple such targets built in every PR. And in case of failure, we have retries. So it does not take so many PR jobs to get to multi GB level.

@okartau okartau merged commit eb727bd into intel:devel Oct 21, 2019
@pohly
Copy link
Contributor Author

pohly commented Oct 21, 2019

If you refer to my recent statement in chat, I did not mean "several GB" in context of one build, but overall in CI system. I haven't made measurements myself, I can see that one e2e run locally produces 13..15 MB text file.

I it actually was Chuy who mentioned PR-383 in his email. The largest log for a single job in that PR consumed 6.7G.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants