Skip to content

Commit

Permalink
Merge branch 'main' into obstacle_stop_planner
Browse files Browse the repository at this point in the history
  • Loading branch information
mitsudome-r authored Aug 22, 2024
2 parents c2c3061 + 60cc9c7 commit 4f548b8
Show file tree
Hide file tree
Showing 207 changed files with 29,316 additions and 2,767 deletions.
2 changes: 0 additions & 2 deletions .cppcheck_suppressions
Original file line number Diff line number Diff line change
Expand Up @@ -10,5 +10,3 @@ unreadVariable
unusedFunction
useInitializationList
useStlAlgorithm
variableScope
virtualCallInConstructor
4 changes: 2 additions & 2 deletions .github/CODEOWNERS
Validating CODEOWNERS rules …
Original file line number Diff line number Diff line change
Expand Up @@ -88,7 +88,7 @@ localization/autoware_twist2accel/** [email protected] kento.yabuuchi.2@tier
localization/ekf_localizer/** [email protected] [email protected] [email protected] [email protected] [email protected] [email protected] [email protected] [email protected] [email protected]
localization/autoware_geo_pose_projector/** [email protected] [email protected] [email protected] [email protected] [email protected] [email protected] [email protected]
localization/gyro_odometer/** [email protected] [email protected] [email protected] [email protected] [email protected] [email protected] [email protected]
localization/localization_error_monitor/** [email protected] [email protected] [email protected] [email protected] [email protected] [email protected] [email protected]
localization/autoware_localization_error_monitor/** [email protected] [email protected] [email protected] [email protected] [email protected] [email protected] [email protected]
localization/localization_util/** [email protected] [email protected] [email protected] [email protected] [email protected] [email protected] [email protected]
localization/ndt_scan_matcher/** [email protected] [email protected] [email protected] [email protected] [email protected] [email protected] [email protected]
localization/pose_estimator_arbiter/** [email protected] [email protected] [email protected] [email protected] [email protected] [email protected] [email protected]
Expand Down Expand Up @@ -201,6 +201,7 @@ planning/sampling_based_planner/autoware_path_sampler/** [email protected]
planning/sampling_based_planner/autoware_sampler_common/** [email protected]
sensing/autoware_image_diagnostics/** [email protected] [email protected]
sensing/autoware_image_transport_decompressor/** [email protected] [email protected]
sensing/autoware_pcl_extensions/** [email protected] [email protected] [email protected]
sensing/autoware_pointcloud_preprocessor/** [email protected] [email protected] [email protected] [email protected] [email protected] [email protected] [email protected] [email protected] [email protected] [email protected]
sensing/autoware_radar_scan_to_pointcloud2/** [email protected] [email protected] [email protected] [email protected]
sensing/autoware_radar_static_pointcloud_filter/** [email protected] [email protected] [email protected] [email protected]
Expand All @@ -209,7 +210,6 @@ sensing/autoware_radar_tracks_noise_filter/** [email protected] shunsuke.m
sensing/gnss_poser/** [email protected] [email protected] [email protected] [email protected] [email protected] [email protected] [email protected]
sensing/imu_corrector/** [email protected] [email protected]
sensing/livox/autoware_livox_tag_filter/** [email protected] [email protected]
sensing/tier4_pcl_extensions/** [email protected] [email protected] [email protected]
sensing/vehicle_velocity_converter/** [email protected]
simulator/autoware_carla_interface/** [email protected] [email protected]
simulator/dummy_perception_publisher/** [email protected]
Expand Down
55 changes: 55 additions & 0 deletions .github/README.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,55 @@
# Overall CI Infrastructure

## Machine Types

### Standard GitHub-hosted runners

- [Documentation](https://docs.github.com/en/actions/using-github-hosted-runners/using-github-hosted-runners/about-github-hosted-runners#standard-github-hosted-runners-for-public-repositories)

These runners are utilized by the majority of the workflows.
They are free to use for public repositories, with a concurrency limit of 20 jobs per organization.

**Listed specs:**

| vCPU | RAM | Storage (SSD) |
| ---- | ----- | ------------- |
| 4 | 16 GB | 14 GB |

**Note:** While the official documentation lists 14 GB of storage, the actual available storage is approximately 73 GB.

### AWS CodeBuild runners

- [AWS CodeBuild Documentation](https://docs.aws.amazon.com/codebuild/latest/userguide/action-runner.html)

These runners are employed for workflows that require more resources and are funded by the Autoware Foundation budget.

**Relevant machine types:**

| Instance Type | Memory | vCPUs | Price per Minute |
| --------------- | ------ | ----- | ---------------- |
| arm1.large | 16 GiB | 8 | $0.015 |
| general1.medium | 7 GB | 4 | $0.01 |
| general1.large | 15 GB | 8 | $0.02 |
| general1.xlarge | 72 GiB | 36 | $0.0798 |
| gpu1.small | 15 GB | 4 | $0.05 |

**Sources:**

- [Compute images supported with the CodeBuild-hosted GitHub Actions runner](https://docs.aws.amazon.com/codebuild/latest/userguide/sample-github-action-runners-update-yaml.images.html)
- [AWS CodeBuild pricing](https://aws.amazon.com/codebuild/pricing/)

## Key workflows and their runners

| Workflow | Trigger | Runner |
| ---------------------------------- | --------------------- | -------------- |
| build-and-test (cuda) | merge to main | general1.large |
| build-and-test-daily | daily on main | github-std |
| build-and-test-daily-arm64 | daily on main | arm1.large |
| build-and-test-differential | PR update | github-std |
| build-and-test-differential (cuda) | PR update | general1.large |
| build-and-test-differential-arm64 | PR update (arm label) | arm1.large |

## Additional notes

- We use [`taskset`](https://manpages.ubuntu.com/manpages/jammy/man1/taskset.1.html) from GNU Coreutils to limit the number of cores utilized by build processes. This is done to prevent overloading the self-hosted runners.
- The number of cores is limited to `vCPU count - 1`.
109 changes: 109 additions & 0 deletions .github/actions/build-and-test-differential/action.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,109 @@
name: build-and-test-differential
description: ""

inputs:
rosdistro:
description: ""
required: true
container:
description: ""
required: true
container-suffix:
description: ""
required: true
runner:
description: ""
required: true
build-depends-repos:
description: ""
required: true
build-pre-command:
description: ""
required: true
codecov-token:
description: ""
required: true

runs:
using: composite
steps:
- name: Show disk space before the tasks
run: df -h
shell: bash

- name: Show machine specs
run: lscpu && free -h
shell: bash

- name: Remove exec_depend
uses: autowarefoundation/autoware-github-actions/remove-exec-depend@v1

- name: Get modified packages
id: get-modified-packages
uses: autowarefoundation/autoware-github-actions/get-modified-packages@v1

- name: Create ccache directory
run: |
mkdir -p ${CCACHE_DIR}
du -sh ${CCACHE_DIR} && ccache -s
shell: bash

- name: Attempt to restore ccache
uses: actions/cache/restore@v4
with:
path: |
/root/.ccache
key: ccache-main-${{ runner.arch }}-${{ inputs.rosdistro }}-${{ github.event.pull_request.base.sha }}
restore-keys: |
ccache-main-${{ runner.arch }}-${{ inputs.rosdistro }}-
- name: Show ccache stats before build
run: du -sh ${CCACHE_DIR} && ccache -s
shell: bash

- name: Export CUDA state as a variable for adding to cache key
run: |
build_type_cuda_state=nocuda
if [[ "${{ inputs.container-suffix }}" == "-cuda" ]]; then
build_type_cuda_state=cuda
fi
echo "BUILD_TYPE_CUDA_STATE=$build_type_cuda_state" >> "${GITHUB_ENV}"
echo "::notice::BUILD_TYPE_CUDA_STATE=$build_type_cuda_state"
shell: bash

- name: Build
if: ${{ steps.get-modified-packages.outputs.modified-packages != '' }}
uses: autowarefoundation/autoware-github-actions/colcon-build@v1
with:
rosdistro: ${{ inputs.rosdistro }}
target-packages: ${{ steps.get-modified-packages.outputs.modified-packages }}
build-depends-repos: ${{ inputs.build-depends-repos }}
cache-key-element: ${{ env.BUILD_TYPE_CUDA_STATE }}
build-pre-command: ${{ inputs.build-pre-command }}

- name: Show ccache stats after build
run: du -sh ${CCACHE_DIR} && ccache -s
shell: bash

- name: Test
id: test
if: ${{ steps.get-modified-packages.outputs.modified-packages != '' }}
uses: autowarefoundation/autoware-github-actions/colcon-test@v1
with:
rosdistro: ${{ inputs.rosdistro }}
target-packages: ${{ steps.get-modified-packages.outputs.modified-packages }}
build-depends-repos: ${{ inputs.build-depends-repos }}

- name: Upload coverage to CodeCov
if: ${{ steps.test.outputs.coverage-report-files != '' }}
uses: codecov/codecov-action@v4
with:
files: ${{ steps.test.outputs.coverage-report-files }}
fail_ci_if_error: false
verbose: true
flags: differential
token: ${{ inputs.codecov-token }}

- name: Show disk space after the tasks
run: df -h
shell: bash
5 changes: 5 additions & 0 deletions .github/labeler.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -39,3 +39,8 @@
- tools/**/*
"component:vehicle":
- vehicle/**/*
"tag:require-cuda-build-and-test":
- perception/**/*
- sensing/**/*
- common/cuda_utils/**/*
- common/tensorrt_common/**/*
10 changes: 9 additions & 1 deletion .github/workflows/build-and-test-daily-arm64.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -5,9 +5,13 @@ on:
- cron: 0 0 * * *
workflow_dispatch:

concurrency:
group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.run_id }}
cancel-in-progress: true

jobs:
build-and-test-daily-arm64:
runs-on: [self-hosted, linux, ARM64]
runs-on: codebuild-autoware-us-east-1-${{ github.run_id }}-${{ github.run_attempt }}-arm-3.0-large
container: ${{ matrix.container }}${{ matrix.container-suffix }}
strategy:
fail-fast: false
Expand All @@ -30,6 +34,9 @@ jobs:
- name: Show disk space before the tasks
run: df -h

- name: Show machine specs
run: lscpu && free -h

- name: Remove exec_depend
uses: autowarefoundation/autoware-github-actions/remove-exec-depend@v1

Expand All @@ -55,6 +62,7 @@ jobs:
target-packages: ${{ steps.get-self-packages.outputs.self-packages }}
build-depends-repos: ${{ matrix.build-depends-repos }}
cache-key-element: ${{ env.BUILD_TYPE_CUDA_STATE }}
build-pre-command: taskset --cpu-list 0-6

- name: Test
if: ${{ steps.get-self-packages.outputs.self-packages != '' }}
Expand Down
10 changes: 9 additions & 1 deletion .github/workflows/build-and-test-differential-arm64.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,10 @@ on:
- reopened
- labeled

concurrency:
group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.run_id }}
cancel-in-progress: true

jobs:
make-sure-label-is-present:
uses: autowarefoundation/autoware-github-actions/.github/workflows/make-sure-label-is-present.yaml@v1
Expand All @@ -17,7 +21,7 @@ jobs:
build-and-test-differential-arm64:
needs: make-sure-label-is-present
if: ${{ needs.make-sure-label-is-present.outputs.result == 'true' }}
runs-on: [self-hosted, linux, ARM64]
runs-on: codebuild-autoware-us-east-1-${{ github.run_id }}-${{ github.run_attempt }}-arm-3.0-large
container: ${{ matrix.container }}${{ matrix.container-suffix }}
strategy:
fail-fast: false
Expand All @@ -44,6 +48,9 @@ jobs:
- name: Show disk space before the tasks
run: df -h

- name: Show machine specs
run: lscpu && free -h

- name: Remove exec_depend
uses: autowarefoundation/autoware-github-actions/remove-exec-depend@v1

Expand All @@ -69,6 +76,7 @@ jobs:
target-packages: ${{ steps.get-modified-packages.outputs.modified-packages }}
build-depends-repos: ${{ matrix.build-depends-repos }}
cache-key-element: ${{ env.BUILD_TYPE_CUDA_STATE }}
build-pre-command: taskset --cpu-list 0-6

- name: Test
id: test
Expand Down
Loading

0 comments on commit 4f548b8

Please sign in to comment.