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

ci/cd: tune ext4 for speed #2568

Closed
wants to merge 2 commits into from

Conversation

pachulo
Copy link
Member

@pachulo pachulo commented Oct 17, 2023

I want to test if builds are quicker with this change.

Inspired by canonical/lxd#12385

@pachulo
Copy link
Member Author

pachulo commented Oct 18, 2023

Well, it seems like there was some improvement in build tme: from 1:58 hours to 1:38 or so.

I would say it's worth the change, WDYT @kyrofa ?

@kyrofa
Copy link
Member

kyrofa commented Oct 18, 2023

That's a small enough change I wouldn't be surprised if it was due to less of a runner load at the time or something. I recommend calculating averages: run this PR's integration suite 5 times, and run master's 5 times, and take the average of each. Each of those 5 triggers should happen at roughly the same time for each branch (e.g. both master and this PR should be triggered at 1400, then 1600, and so on); that will enable us to ignore load differences.

Copy link

This pull request is stale because it has been without activity for 60 days. It will be closed after 7 more days of inactivity.

@github-actions github-actions bot added the Stale label Dec 19, 2023
@github-actions github-actions bot closed this Dec 27, 2023
@pachulo pachulo added feature New feature or request and removed Stale labels Jan 14, 2024
@pachulo pachulo reopened this Jan 14, 2024
Copy link

This pull request is stale because it has been without activity for 60 days. It will be closed after 7 more days of inactivity.

@github-actions github-actions bot added the Stale label Mar 15, 2024
@github-actions github-actions bot closed this Mar 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or request Stale
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants