.github/workflows: Migrate workflows to Blacksmith runners#2340
Open
blacksmith-sh[bot] wants to merge 11 commits intomainfrom
Open
.github/workflows: Migrate workflows to Blacksmith runners#2340blacksmith-sh[bot] wants to merge 11 commits intomainfrom
blacksmith-sh[bot] wants to merge 11 commits intomainfrom
Conversation
blacksmith-sh
bot
requested review from
ValentaTomas,
dobrac and
jakubno
as code owners
![claude[bot]](https://avatars.githubusercontent.com/in/1236702?s=60&v=4){kind=small}
Increased size
Blacksmith runners ship nbd as a kernel built-in, so modprobe fails with "module not found". Check /sys/module/nbd first and skip modprobe if nbd is already present. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
tomassrnka
force-pushed
the
blacksmith-migration-bba0313
branch
from
f9f6889 to
8c428b1
Compare
Blacksmith runners may have a swapfile already in use. Check with swapon --show before attempting fallocate. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This comment has been minimized.
This comment has been minimized.
The NFS proxy test mounts NFS inside a Docker container which requires nfs kernel modules on the host. Load nbd, nfs, nfsv3 — skip if built-in. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Blacksmith runners have nbd/nfs built into kernel (6.5.13) and don't ship linux-modules-extra. Apply the same built-in check pattern to: - pr-tests-arm64.yml (remove apt-get install linux-modules-extra) - init-client.sh (integration tests) Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
NBD modprobe needs nbds_max parameter (4096 for integration tests, 256 for unit tests). Separate nbd from the generic module loop. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Blacksmith ARM64 kernel (6.5.13) lacks nbd module. Make it non-fatal so CI proceeds — NBD-dependent tests will fail individually. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
The fsconfig/fsopen API requires kernel 6.8+. Blacksmith runners use kernel 6.5.13 which doesn't support it. Add fallback to traditional mount syscall with lowerdir= option (limited to ~4096 chars but sufficient for CI). Production nodes run kernel 6.8+ and continue using the fsconfig path. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This reverts commit 1532202.
Author
|
Found 33 test failures on Blacksmith runners: Failures
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
To whomever may be reviewing this PR,
Blacksmith is the fastest way to run your GitHub Actions.
What does this PR change?
This PR has been automatically generated by a team member in your GitHub organization using Blacksmith's Migration Wizard, or MigWiz for short. This PR changes the following:
runs-on: blacksmith-4vcpu-ubuntu-2204). Learn more about the different instances available to choose from.FAQ