-
Notifications
You must be signed in to change notification settings - Fork 631
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
ci: use aws codebuild #8498
ci: use aws codebuild #8498
Conversation
Thank you for contributing to the Autoware project! 🚧 If your pull request is in progress, switch it to draft mode. Please ensure:
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #8498 +/- ##
==========================================
- Coverage 23.91% 23.86% -0.05%
==========================================
Files 1381 1383 +2
Lines 101939 102005 +66
Branches 38822 38845 +23
==========================================
- Hits 24376 24346 -30
- Misses 75119 75187 +68
- Partials 2444 2472 +28
*This pull request uses carry forward flags. Click here to find out more. ☔ View full report in Codecov by Sentry. |
concurrency fixed the "old runners keep running for the older commits" problem. |
eb3c6ec
to
27868f8
Compare
the test errors were probably caused by somewhere here: |
5cf3650
to
a095d61
Compare
0a79402
to
a37e933
Compare
Signed-off-by: M. Fatih Cırıt <[email protected]>
8309e8a
to
96c5e1a
Compare
Signed-off-by: M. Fatih Cırıt <[email protected]>
sensing/autoware_pointcloud_preprocessor/include/autoware/pointcloud_preprocessor/filter.hpp
Outdated
Show resolved
Hide resolved
Signed-off-by: M. Fatih Cırıt <[email protected]>
Signed-off-by: M. Fatih Cırıt <[email protected]>
Signed-off-by: M. Fatih Cırıt <[email protected]>
@mitsudome-r I've merged it. If something goes wrong, feel free to revert this change. But before reverting, please link the failing CI reports under here and try to rerun them at least once. I will keep the old machines alive for 1 more week. And if things go smoothly, I will terminate them. |
@xmfcx @mitsudome-r cc: @yhisaki It seems that the README added in this PR overrides the README in the repository root.
|
Signed-off-by: M. Fatih Cırıt <[email protected]>
Description
Following:
Relevant machines:
total cpu count - 1
cores to build, to allow runner to function.taskset
from GNU coreutils for this jobbuild-and-test will usegeneral1.medium
since it is not time critical and it's ok for it to be slowarm1.large
arm1.large
general1.large
general1.xlarge
general1.medium
Related links
Private Links:
Tests
Old tests
https://docs.google.com/spreadsheets/d/1hO7rxgq_cYAoAEc7EWOFs-e_fwxeTwTJOBznAtBkPac/edit?usp=sharing
Notes for reviewers
None.
Interface changes
None.
Effects on system behavior
None.