Skip to content

Conversation

@Sjors
Copy link
Collaborator

@Sjors Sjors commented Oct 8, 2025

Mirrors bitcoin/bitcoin#33514

The first commits catch up on https://github.com/bitcoin/bitcoin/commits/master/ci, minus the windows changes.

Assessing impact on runtimes...

@Sjors
Copy link
Collaborator Author

Sjors commented Oct 8, 2025

Typical CI runtime for comparison:

Scherm­afbeelding 2025-10-08 om 11 55 11

@Sjors
Copy link
Collaborator Author

Sjors commented Oct 8, 2025

Test failure is unrelated.

@Sjors
Copy link
Collaborator Author

Sjors commented Oct 8, 2025

No difference in run time, but I'm also not sure if it runs this new task at all.

Scherm­afbeelding 2025-10-08 om 12 08 53

@Sjors Sjors force-pushed the 2025/10/clear-ci-space branch from 42a69f6 to 3a4a088 Compare October 8, 2025 10:58
@Sjors
Copy link
Collaborator Author

Sjors commented Oct 8, 2025

Needed to include f79a6e1 so the "gha" string gets set.

@Sjors
Copy link
Collaborator Author

Sjors commented Oct 8, 2025

Ok, I see the "Clear unnecessary files" step now:

Scherm­afbeelding 2025-10-08 om 13 10 16

On most jobs it takes just 1 minute. But on the ASan + LSan + UBSan + integer, no depends jobs is takes almost 6 minutes, about half the total runtime. On depends DEBUG it takes 3.5 minutes, fuzzer,address,undefined,integer, no depends 5 minutes, centOS 6 minutes.

@Sjors
Copy link
Collaborator Author

Sjors commented Oct 8, 2025

Pushed the first 4 cherry-picks directly to master. Will wait to see what happens upstream.

@Sjors Sjors force-pushed the 2025/10/clear-ci-space branch from 8556a5a to d9f128a Compare October 8, 2025 11:21
Clear out space on jobs running on GHA by deleteing unnecessary files.

Only runs when cache provider (runner) is `gha`.

Based on: bitcoin/bitcoin#33514
@Sjors Sjors force-pushed the 2025/10/clear-ci-space branch from d9f128a to 6c2dba6 Compare October 8, 2025 11:23
@Sjors Sjors marked this pull request as draft October 24, 2025 13:35
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.

3 participants