Skip to content

Disable github actions to choose jumbo runners / increase build timeout#5756

Merged
rene merged 2 commits intolf-edge:masterfrom
christoph-zededa:disable_gh_jumbo_timeout
Apr 8, 2026
Merged

Disable github actions to choose jumbo runners / increase build timeout#5756
rene merged 2 commits intolf-edge:masterfrom
christoph-zededa:disable_gh_jumbo_timeout

Conversation

@christoph-zededa
Copy link
Copy Markdown
Contributor

@christoph-zededa christoph-zededa commented Apr 7, 2026

Description

For #5650 some temporary changes were made to make it possible to compile it:

  • increase build timeout
  • choose bigger runner

as the packages are now built successfully and put into the dockerhub cache, these changes are not needed anymore.

PR dependencies

#5650
#5749

How to test and validate this PR

  • Not only the jumbo runners are used
  • default timeout is used

Changelog notes

Internal changes - github build pipeline

PR Backports

For all current LTS branches, please state explicitly if this PR should be
backported or not. This section is used by our scripts to track the backports,
so, please, do not omit it.

Here is the list of current LTS branches (it should be always up to date):

  • 16.0-stable: no, it is automatically backported as actions are used from master
  • 14.5-stable: no
  • 13.4-stable: no

Checklist

  • I've provided a proper description
  • I've added the proper documentation
  • I've tested my PR on amd64 device
  • I've tested my PR on arm64 device
  • I've written the test verification instructions
  • I've set the proper labels to this PR

And the last but not least:

  • I've checked the boxes above, or I've provided a good reason why I didn't
    check them.

Please, check the boxes above after submitting the PR in interactive mode.

This reverts commit 95fb762.

Signed-off-by: Christoph Ostarek <christoph@zededa.com>
This reverts commit fb5a266.

Signed-off-by: Christoph Ostarek <christoph@zededa.com>
@rene
Copy link
Copy Markdown
Contributor

rene commented Apr 7, 2026

@christoph-zededa looks good to me, can we mark it as ready for review?

@christoph-zededa christoph-zededa marked this pull request as ready for review April 7, 2026 09:26
@christoph-zededa
Copy link
Copy Markdown
Contributor Author

@christoph-zededa looks good to me, can we mark it as ready for review?

Yes, done; I just wanted to read over the description again and check that the checks passed.

@christoph-zededa
Copy link
Copy Markdown
Contributor Author

@rene perhaps let's wait for #5749

@eriknordmark
Copy link
Copy Markdown
Contributor

@rene perhaps let's wait for #5749

Or #5746 which is the same running in parallel.

But I don't know how many days that will take. Once the riscv64 packages have been built once (currently been running for 20 hours!) then the cache will have timed out, but I hope that the next re-run will use the riscv64 packages but maybe that hope is misplaced?

@rene
Copy link
Copy Markdown
Contributor

rene commented Apr 7, 2026

This is just changing workflows, we don't need to wait, and this blocking several other runners that don't use jumbo label

@rene rene merged commit b8e95d0 into lf-edge:master Apr 8, 2026
16 of 17 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.

3 participants