mirror of https://github.com/ArduPilot/ardupilot
CI: remove test_size for stable releases
this makes no sense as tests against master size
This commit is contained in:
Andrew Tridgell
parent
6ddb813fba
commit
0f3e33e766
|
|
@ -1,180 +0,0 @@
|
|||
name: test size
|
||||
|
||||
on: [push, pull_request, workflow_dispatch]
|
||||
# paths:
|
||||
# - "*"
|
||||
# - "!README.md" <-- don't rebuild on doc change
|
||||
concurrency:
|
||||
group: ci-${{github.workflow}}-${{ github.ref }}
|
||||
cancel-in-progress: true
|
||||
|
||||
jobs:
|
||||
build:
|
||||
runs-on: ubuntu-20.04
|
||||
container: ardupilot/ardupilot-dev-chibios:latest
|
||||
strategy:
|
||||
fail-fast: false # don't cancel if a job from the matrix fails
|
||||
matrix:
|
||||
toolchain: [
|
||||
base, # GCC
|
||||
]
|
||||
config: [
|
||||
Durandal,
|
||||
MatekF405,
|
||||
Pixhawk1-1M,
|
||||
Hitec-Airspeed, # see special code for Periph below (3 places!)
|
||||
f103-GPS # see special code for Periph below (3 places!)
|
||||
]
|
||||
steps:
|
||||
- uses: actions/checkout@v2
|
||||
with:
|
||||
ref: 'master'
|
||||
path: 'master'
|
||||
submodules: 'recursive'
|
||||
# Put ccache into github cache for faster build
|
||||
- name: Prepare ccache timestamp
|
||||
id: ccache_cache_timestamp
|
||||
run: |
|
||||
NOW=$(date -u +"%F-%T")
|
||||
echo "::set-output name=timestamp::${NOW}"
|
||||
- name: ccache cache files
|
||||
uses: actions/cache@v2
|
||||
with:
|
||||
path: ~/.ccache
|
||||
key: ${{github.workflow}}-ccache-${{ matrix.toolchain }}-${{steps.ccache_cache_timestamp.outputs.timestamp}}
|
||||
restore-keys: ${{github.workflow}}-ccache-${{ matrix.toolchain }}- # restore ccache from either previous build on this branch or on master
|
||||
- name: setup ccache
|
||||
run: |
|
||||
mkdir -p ~/.ccache
|
||||
echo "base_dir = ${GITHUB_WORKSPACE}" > ~/.ccache/ccache.conf
|
||||
echo "compression = true" >> ~/.ccache/ccache.conf
|
||||
echo "compression_level = 6" >> ~/.ccache/ccache.conf
|
||||
echo "max_size = 400M" >> ~/.ccache/ccache.conf
|
||||
ccache -s
|
||||
ccache -z
|
||||
|
||||
- name: Build master ${{matrix.config}} ${{ matrix.toolchain }}
|
||||
env:
|
||||
CI_BUILD_TARGET: ${{matrix.config}}
|
||||
shell: bash
|
||||
run: |
|
||||
PATH="/github/home/.local/bin:$PATH"
|
||||
cd master
|
||||
./waf configure --board ${{matrix.config}}
|
||||
if [ "${{matrix.config}}" = "Hitec-Airspeed" ] ||
|
||||
[ "${{matrix.config}}" = "f103-GPS" ]; then
|
||||
./waf AP_Periph
|
||||
else
|
||||
./waf
|
||||
fi
|
||||
mkdir -p $GITHUB_WORKSPACE/master_bin
|
||||
cp -r build/${{matrix.config}}/bin/* $GITHUB_WORKSPACE/master_bin/
|
||||
|
||||
# build a set of binaries without symbols so we can check if
|
||||
# the binaries have changed.
|
||||
echo [`date`] Building master with no versions
|
||||
|
||||
NO_VERSIONS_DIR="$GITHUB_WORKSPACE/master_bin_no_versions"
|
||||
mkdir "$NO_VERSIONS_DIR"
|
||||
|
||||
if [ "${{matrix.config}}" = "Hitec-Airspeed" ] ||
|
||||
[ "${{matrix.config}}" = "f103-GPS" ]; then
|
||||
CHIBIOS_GIT_VERSION="12345678" GIT_VERSION="abcdef" ./waf AP_Periph
|
||||
else
|
||||
CHIBIOS_GIT_VERSION="12345678" GIT_VERSION="abcdef" ./waf
|
||||
fi
|
||||
cp -r build/${{matrix.config}}/bin/* "$NO_VERSIONS_DIR"
|
||||
|
||||
echo [`date`] Built master with no versions
|
||||
|
||||
- uses: actions/checkout@v2
|
||||
with:
|
||||
fetch-depth: 0
|
||||
path: 'pr'
|
||||
|
||||
- name: Build PR rebased ${{matrix.config}} ${{ matrix.toolchain }}
|
||||
env:
|
||||
CI_BUILD_TARGET: ${{matrix.config}}
|
||||
shell: bash
|
||||
run: |
|
||||
PATH="/github/home/.local/bin:$PATH"
|
||||
cd pr/
|
||||
git config user.email "ardupilot-ci@ardupilot.org"
|
||||
git config user.name "ArduPilot CI"
|
||||
git remote add ardupilot https://github.com/ArduPilot/ardupilot.git
|
||||
git fetch --no-tags --prune --progress ardupilot master
|
||||
git rebase ardupilot/master
|
||||
git submodule update --init --recursive --depth=1
|
||||
./waf configure --board ${{matrix.config}}
|
||||
if [ "${{matrix.config}}" = "Hitec-Airspeed" ] ||
|
||||
[ "${{matrix.config}}" = "f103-GPS" ]; then
|
||||
./waf AP_Periph
|
||||
else
|
||||
./waf
|
||||
fi
|
||||
mkdir $GITHUB_WORKSPACE/pr_bin
|
||||
cp -r build/${{matrix.config}}/bin/* $GITHUB_WORKSPACE/pr_bin/
|
||||
|
||||
# build a set of binaries without symbols so we can check if
|
||||
# the binaries have changed.
|
||||
echo [`date`] Building PR with no versions
|
||||
|
||||
NO_VERSIONS_DIR="$GITHUB_WORKSPACE/pr_bin_no_versions"
|
||||
mkdir "$NO_VERSIONS_DIR"
|
||||
|
||||
if [ "${{matrix.config}}" = "Hitec-Airspeed" ] ||
|
||||
[ "${{matrix.config}}" = "f103-GPS" ]; then
|
||||
CHIBIOS_GIT_VERSION="12345678" GIT_VERSION="abcdef" ./waf AP_Periph
|
||||
else
|
||||
CHIBIOS_GIT_VERSION="12345678" GIT_VERSION="abcdef" ./waf
|
||||
fi
|
||||
cp -r build/${{matrix.config}}/bin/* "$NO_VERSIONS_DIR"
|
||||
|
||||
echo [`date`] Built PR with no versions
|
||||
|
||||
# build MatekF405 Plane without quadplane
|
||||
if [ "${{matrix.config}}" = "MatekF405" ]; then
|
||||
PLANE_BINARY="build/MatekF405/bin/arduplane.bin"
|
||||
echo "normal size"
|
||||
ls -l "$PLANE_BINARY"
|
||||
EXTRA_HWDEF="/tmp/extra-options.def"
|
||||
echo "define HAL_QUADPLANE_ENABLED 0" >"$EXTRA_HWDEF"
|
||||
./waf configure --board ${{matrix.config}} --extra-hwdef="$EXTRA_HWDEF"
|
||||
./waf plane
|
||||
rm "$EXTRA_HWDEF"
|
||||
echo "non-quadplane size:"
|
||||
ls -l "$PLANE_BINARY"
|
||||
fi
|
||||
|
||||
- name: Full size compare with Master
|
||||
shell: bash
|
||||
run: |
|
||||
cd pr/
|
||||
python3 -m pip install -U tabulate
|
||||
Tools/scripts/pretty_diff_size.py -m $GITHUB_WORKSPACE/master_bin -s $GITHUB_WORKSPACE/pr_bin
|
||||
|
||||
- name: Checksum compare with Master
|
||||
shell: bash
|
||||
run: |
|
||||
diff -r $GITHUB_WORKSPACE/master_bin_no_versions $GITHUB_WORKSPACE/pr_bin_no_versions --exclude=*.elf --exclude=*.apj || true
|
||||
|
||||
- name: elf_diff compare with Master
|
||||
shell: bash
|
||||
run: |
|
||||
python3 -m pip install -U weasyprint elf_diff anytree
|
||||
mkdir elf_diff
|
||||
if [ "${{matrix.config}}" = "Hitec-Airspeed" ] ||
|
||||
[ "${{matrix.config}}" = "f103-GPS" ]; then
|
||||
python3 -m elf_diff --bin_prefix=arm-none-eabi- --html_dir=elf_diff/AP_Periph $GITHUB_WORKSPACE/master_bin/AP_Periph $GITHUB_WORKSPACE/pr_bin/AP_Periph
|
||||
else
|
||||
python3 -m elf_diff --bin_prefix=arm-none-eabi- --html_dir=elf_diff/plane $GITHUB_WORKSPACE/master_bin/arduplane $GITHUB_WORKSPACE/pr_bin/arduplane
|
||||
python3 -m elf_diff --bin_prefix=arm-none-eabi- --html_dir=elf_diff/copter $GITHUB_WORKSPACE/master_bin/arducopter $GITHUB_WORKSPACE/pr_bin/arducopter
|
||||
fi
|
||||
zip -r elf_diff.zip elf_diff
|
||||
|
||||
- name: Archive elf_diff output
|
||||
uses: actions/upload-artifact@v2
|
||||
with:
|
||||
name: ELF_DIFF_${{matrix.config}}
|
||||
path: elf_diff.zip
|
||||
retention-days: 14
|
||||
Loading…
Reference in New Issue