-
Notifications
You must be signed in to change notification settings - Fork 25
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
run-vmtest@v2 and build-selftests@v2 #150
Changes from all commits
52da70f
48f4efb
9ccd9cc
439276d
dc9c625
b6d0593
5502345
af375af
fe0bd8d
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,34 +1,27 @@ | ||
name: 'build selftests' | ||
description: 'Build BPF selftests' | ||
inputs: | ||
kernel: | ||
description: 'kernel version' | ||
default: 'LATEST' | ||
toolchain: | ||
description: 'what toolchain to use' | ||
default: 'gcc' | ||
kbuild-output: | ||
description: 'relative or absolute path to use for storing build artifacts' | ||
arch: | ||
description: 'arch' | ||
required: true | ||
kernel-root: | ||
description: 'Path to the root of the kernel source tree' | ||
required: true | ||
max-make-jobs: | ||
description: 'Maximum number of jobs to use when running make (e.g argument to -j). Default: 4*nproc' | ||
default: '' | ||
llvm-version: | ||
description: 'llvm version' | ||
required: false | ||
default: '16' | ||
arch: | ||
description: 'arch' | ||
required: true | ||
default: '18' | ||
toolchain: | ||
description: 'gcc or llvm' | ||
default: 'gcc' | ||
|
||
runs: | ||
using: "composite" | ||
steps: | ||
- name: build selftests | ||
shell: bash | ||
run: | | ||
kbuild_output="$(realpath ${{ inputs.kbuild-output }})" | ||
export LLVM_VERSION=${{ inputs.llvm-version }} | ||
${GITHUB_ACTION_PATH}/build_selftests.sh "${{ inputs.arch }}" "${{ inputs.kernel }}" "${{ inputs.toolchain }}" "${kbuild_output}" | ||
env: | ||
MAX_MAKE_JOBS: ${{ inputs.max-make-jobs }} | ||
LLVM_VERSION: ${{ inputs.llvm-version }} | ||
run: | | ||
${GITHUB_ACTION_PATH}/build_selftests.sh "${{ inputs.arch }}" "${{ inputs.toolchain }}" "${{ inputs.kernel-root }}" | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -6,7 +6,7 @@ DIFF_DIR=$1 | |
for ext in diff patch; do | ||
if ls ${DIFF_DIR}/*.${ext} 1>/dev/null 2>&1; then | ||
for file in ${DIFF_DIR}/*.${ext}; do | ||
if patch --dry-run -p1 -s < "${file}" 2>/dev/null; then | ||
if patch --dry-run -N --silent -p1 -s < "${file}" 2>/dev/null; then | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. any reason for the -N here? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. From man:
This is to avoid prints like this in the job logs:
IIRC |
||
patch -s -p1 < "${file}" 2>/dev/null | ||
echo "Successfully applied ${file}!" | ||
else | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this used to be used to control how many cpus were used when compiling. On bare metal, with 96 CPUs, if all containers are building at the same time, they will all use that many CPUs, bringing the machine to a crawl.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's still used, it's just not an action input anymore. A caller can set it like here.
The default is 4*nproc, controlled here.