Skip to content
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

Jazzy CI and build status #2851

Closed
wants to merge 32 commits into from
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
32 commits
Select commit Hold shift + click to select a range
47f1293
Test CI on ROS 2 Jazzy
henningkayser May 24, 2024
2861a53
Add Jazzy build badges
henningkayser May 24, 2024
a37b670
Only source generate_parameter_library for Humble and Iron
henningkayser May 24, 2024
d0b8299
Switch testspace subdomain to moveit
henningkayser May 24, 2024
9c5bbd4
Temporarily disable testspace in CI
henningkayser May 24, 2024
0ce5d48
Update README Rolling badges to noble
henningkayser May 24, 2024
8bdc819
Remove octomap, geometric_shapes from rolling repos
henningkayser May 24, 2024
7d9aacf
Remove unused variable in test, fix clang-tidy
henningkayser May 24, 2024
dac7638
TEMP: Remove ROS' octomap after 'rosdep install'
rhaschke May 26, 2024
97528ce
Silence clang warnings
henningkayser Jun 6, 2024
889c112
Source geometric_shapes
henningkayser Jun 6, 2024
48f60c4
Fix double-free in Pilz unit tests (#3561)
rhaschke Feb 6, 2024
7f691cd
Fix clang-format
henningkayser Jun 7, 2024
fb5421c
Revert "Fix double-free in Pilz unit tests (#3561)"
henningkayser Jun 7, 2024
578572f
Temporarily disable mock testing in PILZ
henningkayser Jun 7, 2024
4082252
Enforce liboctomap-dev by using a cmake version range
henningkayser Jun 7, 2024
d2270ef
Test geometric_shapes with fixed octomap version
henningkayser Jun 7, 2024
ee147c7
Fix clang-format
henningkayser Jun 7, 2024
14bc36e
Fix warning suppression in ikfast
henningkayser Jun 7, 2024
f54c718
Revert "Simplify controller manager namespacing (#2210)"
henningkayser Jun 7, 2024
d3a11e0
Revert "TEMP: Remove ROS' octomap after 'rosdep install'"
rhaschke Jun 8, 2024
4df4ea4
Revert "Test geometric_shapes with fixed octomap version"
rhaschke Jun 8, 2024
b9de8bc
Silence clang warnings
rhaschke Jun 8, 2024
16f7878
CI: Drop jazzy-ci-testing
rhaschke Jun 8, 2024
944ac8d
Fix -Wmaybe-uninitialized for clang
rhaschke Jun 8, 2024
a7ce893
Fix clang-tidy config
rhaschke Jun 8, 2024
2aa76e3
Apply clang-tidy fixes
rhaschke Jun 8, 2024
46fe5be
Disable clang-tidy for ikfast plugin in moveit_planners/test_configs
rhaschke Jun 9, 2024
9edfd66
Remove Testspace from Sonarcloud
henningkayser Jun 10, 2024
efe14bd
Suppress "mismatch" action
henningkayser Jun 10, 2024
1f61935
Merge branch 'main' into jazzy_support
sjahr Jun 12, 2024
eee39e4
Merge branch 'main' into jazzy_support
sjahr Jun 13, 2024
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion .clang-tidy
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
---
Checks: '-*,
performance-*,
-performance-enum-size,
llvm-namespace-comment,
modernize-redundant-void-arg,
modernize-use-nullptr,
Expand All @@ -21,7 +22,6 @@ Checks: '-*,
readability-static-definition-in-anonymous-namespace,
'
HeaderFilterRegex: ''
AnalyzeTemporaryDtors: false
CheckOptions:
- key: llvm-namespace-comment.ShortNamespaceLines
value: '10'
Expand Down
27 changes: 17 additions & 10 deletions .github/workflows/ci.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -20,14 +20,16 @@ jobs:
- IMAGE: rolling-ci
CCOV: true
ROS_DISTRO: rolling
- IMAGE: rolling-ci-testing
- IMAGE: rolling-ci
ROS_DISTRO: rolling
IKFAST_TEST: true
CLANG_TIDY: pedantic
- IMAGE: humble-ci
ROS_DISTRO: humble
- IMAGE: humble-ci-testing
ROS_DISTRO: humble
- IMAGE: jazzy-ci
ROS_DISTRO: jazzy
env:
CXXFLAGS: >-
-Wall -Wextra -Wwrite-strings -Wunreachable-code -Wpointer-arith -Wredundant-decls
Expand Down Expand Up @@ -61,6 +63,7 @@ jobs:
(cd $TARGET_REPO_PATH; clang-tidy --list-checks)
# Disable clang-tidy for ikfast plugins as we cannot fix the generated code
find $BASEDIR/target_ws/build -iwholename "*_ikfast_plugin/compile_commands.json" -exec rm {} \;
find $BASEDIR/target_ws/build -iwholename "*_ikfast_manipulator_plugin/compile_commands.json" -exec rm {} \;
CC: ${{ matrix.env.CLANG_TIDY && 'clang' }}
CXX: ${{ matrix.env.CLANG_TIDY && 'clang++' }}
ADDITIONAL_DEBS: lld
Expand All @@ -83,10 +86,12 @@ jobs:
sudo rm -rf /usr/local
df -h
- uses: actions/checkout@v4
- uses: testspace-com/setup-testspace@v1
if: github.repository == 'moveit/moveit2'
with:
domain: ros-planning
# NOTE: Testspace is temporarily disabled and needs to be installed for the MoveIt org
# See: https://github.com/moveit/moveit2/issues/2852
# - uses: testspace-com/setup-testspace@v1
# if: github.repository == 'moveit/moveit2'
# with:
# domain: moveit
- name: Get latest release date for rosdistro
id: rosdistro_release_date
uses: JafarAbdi/latest-rosdistro-release-date-action@main
Expand Down Expand Up @@ -140,18 +145,20 @@ jobs:
name: Run industrial_ci
uses: ros-industrial/industrial_ci@master
env: ${{ matrix.env }}
- name: Push result to Testspace
if: always() && (github.repository == 'moveit/moveit2')
run: |
testspace "[ ${{ matrix.env.IMAGE }} ]${{ env.BASEDIR }}/target_ws/**/test_results/**/*.xml"
# NOTE: Testspace is temporarily disabled and needs to be installed for the MoveIt org
# See: https://github.com/moveit/moveit2/issues/2852
# - name: Push result to Testspace
# if: always() && (github.repository == 'moveit/moveit2')
# run: |
# testspace "[ ${{ matrix.env.IMAGE }} ]${{ env.BASEDIR }}/target_ws/**/test_results/**/*.xml"
- name: Upload test artifacts (on failure)
uses: actions/upload-artifact@v4
if: failure() && (steps.ici.outputs.run_target_test || steps.ici.outputs.target_test_results)
with:
name: test-results-${{ matrix.env.IMAGE }}
path: ${{ env.BASEDIR }}/target_ws/**/test_results/**/*.xml
- name: Generate codecov report
uses: rhaschke/lcov-action@main
uses: henningkayser/lcov-action@main
if: always() && matrix.env.CCOV && steps.ici.outputs.target_test_results == '0'
with:
docker: $DOCKER_IMAGE
Expand Down
6 changes: 3 additions & 3 deletions .github/workflows/docker.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ jobs:
strategy:
fail-fast: false
matrix:
ROS_DISTRO: [rolling]
ROS_DISTRO: [rolling, jazzy]
runs-on: ubuntu-latest
permissions:
packages: write
Expand Down Expand Up @@ -72,7 +72,7 @@ jobs:
strategy:
fail-fast: false
matrix:
ROS_DISTRO: [rolling]
ROS_DISTRO: [rolling, jazzy]
runs-on: ubuntu-latest
permissions:
packages: write
Expand Down Expand Up @@ -128,7 +128,7 @@ jobs:
strategy:
fail-fast: false
matrix:
ROS_DISTRO: [rolling]
ROS_DISTRO: [rolling, jazzy]
runs-on: ubuntu-latest
permissions:
packages: write
Expand Down
9 changes: 5 additions & 4 deletions .github/workflows/sonar.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -56,10 +56,11 @@ jobs:
sudo rm -rf /usr/local
df -h
- uses: actions/checkout@v4
- uses: testspace-com/setup-testspace@v1
if: github.repository == 'moveit/moveit2'
with:
domain: ros-planning
# Testspace disabled temporarily: https://github.com/moveit/moveit2/issues/2852
# - uses: testspace-com/setup-testspace@v1
# if: github.repository == 'moveit/moveit2'
# with:
# domain: ros-planning
- name: Get latest release date for rosdistro
id: rosdistro_release_date
uses: JafarAbdi/latest-rosdistro-release-date-action@main
Expand Down
Loading
Loading