Skip to content

Commit

Permalink
Reusable asan invokes reusable build with asan mixin
Browse files Browse the repository at this point in the history
Signed-off-by: Yadunund <yadunund@openrobotics.org>
  • Loading branch information
Yadunund committed Mar 17, 2024
1 parent b28c7ec commit d0cf8ea
Show file tree
Hide file tree
Showing 2 changed files with 22 additions and 67 deletions.
17 changes: 17 additions & 0 deletions .github/workflows/asan.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,17 @@
name: asan
on:
pull_request:
schedule:
- cron: '0 0 * * *'
workflow_dispatch:

jobs:
build_and_test_asan:
name: test_ci_asan
uses: ./.github/workflows/reusable_asan.yaml
with:
# NOTE: Avoid adding comments in the packages lines, this can break some of the called scripts in github actions
# NOTE: Build upto rmf_fleet_adapter_python and rmf_visualization to cover all core open-rmf packages.
packages: |
rmf_fleet_adapter_python
rmf_visualization
72 changes: 5 additions & 67 deletions .github/workflows/reusable_asan.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -17,73 +17,11 @@ on:
"ubuntu_distribution": "jammy"},
{"ros_distribution": "rolling",
"ubuntu_distribution": "noble"}]
defaults:
run:
shell: bash
concurrency:
group: ${{ github.head_ref || github.ref_name }}
cancel-in-progress: true
jobs:
asan:
strategy:
fail-fast: false
matrix:
include: ${{ fromJson(inputs.dist-matrix) }}
name: asan
runs-on: ubuntu-20.04
container:
image: osrf/ros:${{ matrix.ros_distribution }}-desktop-${{ matrix.ubuntu_distribution }}
steps:
- name: install_clang_and_tools
run: sudo apt update && sudo apt install -y clang clang-tools lld wget python3-pip python3-colcon-coveragepy-result python3-colcon-lcov-result lcov
# TODO: fix for cryptography>2.8 failing in galactic https://github.com/open-rmf/rmf/pull/202#pullrequestreview-1100045417
- name: fix cryptography==2.8 # https://github.com/open-rmf/rmf/pull/202#pullrequestreview-1100045417
run: pip3 install cryptography==2.8
if: ${{ matrix.ros_distribution == 'galactic' || matrix.ros_distribution == 'foxy' }}
# TODO: remove this when uncrustify 0.72 is fixed https://github.com/uncrustify/uncrustify/issues/3191
- name: hack for uncrustify 0.72 problems
run: wget http://mirrors.kernel.org/ubuntu/pool/universe/u/uncrustify/uncrustify_0.69.0+dfsg1-1build1_amd64.deb && dpkg -i uncrustify_0.69.0+dfsg1-1build1_amd64.deb && apt install -f -y
if: ${{ matrix.ubuntu_distribution == 'jammy'}}
# TODO: Remove this step when the incompatibility between libunwind14 and libundind dissapears https://github.com/open-rmf/rmf_traffic_editor/issues/439
- name: Horrible hack for libceres
run: |
apt-get remove -y --purge libc++-dev || true
apt-get remove -y --purge libc++abi-dev || true
apt-get remove -y --purge libunwind-14-dev || true
apt-get remove -y --purge libunwind-14-dev || true
apt-get remove -y --purge libunwind-dev || true
apt -y autoremove
if: ${{ matrix.ubuntu_distribution == 'jammy'}}
- name: create_blacklist
run: |
mkdir -p ${{ github.workspace }}/
touch ${{ github.workspace }}/blacklist.txt
echo "fun:*Eigen*" > ${{ github.workspace }}/blacklist.txt
- name: asan_build_and_test
uses: ros-tooling/action-ros-ci@v0.3
env:
LANG: en_US.UTF-8
CC: clang -fsanitize-blacklist=${{ github.workspace }}/blacklist.txt
CXX: clang++ -fsanitize-blacklist=${{ github.workspace }}/blacklist.txt
QT_QPA_PLATFORM: offscreen
with:
target-ros2-distro: ${{ matrix.ros_distribution }}
# build all packages listed in the meta package
package-name: ${{ inputs.packages }}
vcs-repo-file-url: |
https://github.com/open-rmf/rmf/${{ matrix.ros_distribution }}/rmf.repos
colcon-defaults: |
{
"build": {
"mixin": ["asan-gcc", "lld"],
"cmake-args": ["-DCMAKE_BUILD_TYPE=Debug"],
"executor": "sequential"
}
}
colcon-mixin-repository: https://github.com/colcon/colcon-mixin-repository/master/index.yaml
- name: upload_test_stream
uses: actions/upload-artifact@v2
with:
name: colcon-asan-logs
path: ${{ steps.build_and_test.outputs.ros-workspace-directory-name }}/log
if: always()
uses: ./.github/workflows/reusable_build.yaml
with:
dist-matrix: ${{inputs.dist-matrix}}
packages: ${{inputs.packages}}
mixin: asan

0 comments on commit d0cf8ea

Please sign in to comment.