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

Reduce log size and free up disk space on Linux #1717

Merged
merged 2 commits into from
Aug 16, 2023

Conversation

cherylking
Copy link
Member

@cherylking cherylking commented Aug 15, 2023

Recently we began running out of disk space on at least half the Linux jobs in each GHA build.

  1. Changed streamLogs to streamLogsOnFailures to reduce log size.
  2. Used GHA action jlumbroso/free-disk-space@main to free up some disk space before each job runs on Ubuntu.

I checked one of the Linux jobs and saw that the free-disk-space action freed up 18 GB. It started with 21 GB free and had 39 GB free after the action ran.

Copy link
Member

@scottkurz scottkurz left a comment

Choose a reason for hiding this comment

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

Guess an option would be to only include the GHA disk free action and keep streamLogs=true...

Personally, I agree we could make both changes ... If we ever wanted to revert the streamLogs we could do that whenever.

@cherylking
Copy link
Member Author

Guess an option would be to only include the GHA disk free action and keep streamLogs=true...

Yeah, I think we should give this a try and see if we like it. At least locally, I love having streamLogsOnFailures=true instead of streamLogs. On the builds, it may be preferred to still have streamLogs=true. The yaml can override what is in the pom.xml. We can change the setting back in the yaml if we decide it is more useful later.

@cherylking cherylking merged commit 0e99501 into OpenLiberty:main Aug 16, 2023
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants