Closed youtalk closed 1 month ago
@mitsudome-r Please merge it because I have no longer the write access anyhow.
How was this tested?
I can merge it but I'm worried it might break the containers since I didn't test it.
@xmfcx How can I test the container? I'm sorry but I don't know the test function since I'm new.
Description
This PR adds the
src-imported
stage to the top stage of theDockerfile
. This allows thebase
andsrc-imported
stages to be processed in parallel, reducing the download latency of thevcs import
command. It also eliminates the need to run thevcs import
command again in theruntime
stage. The trial in my local environment speeded up thedocker build
by about 5 minutes.This PR does not cut out
rosdep install
, based on the reflection of the previous PR: https://github.com/autowarefoundation/autoware/pull/4656#issuecomment-2082095239Tests performed
Not applicable.
Effects on system behavior
Not applicable.
Interface changes
Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.
In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.
Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.
After all checkboxes are checked, anyone who has write access can merge the PR.