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

feat(docker): separation from autoware-universe stage to autoware-universe-sensing-perception stage #5164

Merged
merged 8 commits into from
Sep 4, 2024

Conversation

youtalk
Copy link
Member

@youtalk youtalk commented Sep 4, 2024

Description

Resolved #5079

This PR separates from the autoware-universe stage to the autoware-universe-sensing-perception stage, which builds only the sensing and perception components.

364197115-ba2cf660-c9f0-4644-a97b-025ee2ab0d37

Subsequent PRs will similarly add stages divided by components.

Tests 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.

  • There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

Signed-off-by: Yutaka Kondo <[email protected]>
Signed-off-by: Yutaka Kondo <[email protected]>
Signed-off-by: Yutaka Kondo <[email protected]>
Signed-off-by: Yutaka Kondo <[email protected]>
Signed-off-by: Yutaka Kondo <[email protected]>
Signed-off-by: Yutaka Kondo <[email protected]>
This reverts commit 789ee21.
Signed-off-by: Yutaka Kondo <[email protected]>
@youtalk youtalk self-assigned this Sep 4, 2024
@youtalk youtalk added the tag:run-health-check Run health-check label Sep 4, 2024
@youtalk youtalk marked this pull request as ready for review September 4, 2024 02:39
@youtalk youtalk changed the title feat(docker): Separation from autoware-universe stage to autoware-universe-sensing-perception stage feat(docker): separation from autoware-universe stage to autoware-universe-sensing-perception stage Sep 4, 2024
Copy link
Contributor

@oguzkaganozt oguzkaganozt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, but we should find a way to simplify and clarify the dockerfile after completing all the intermediary stages. Since it is getting harder to read on each iteration.

@youtalk youtalk merged commit 42cba8b into main Sep 4, 2024
19 of 22 checks passed
@youtalk youtalk deleted the sensing-perception-stage branch September 4, 2024 22:54
@youtalk
Copy link
Member Author

youtalk commented Sep 5, 2024

I know. That is why I proposed the base image to split the Dockerfiles.
https://github.com/orgs/autowarefoundation/discussions/5110
https://github.com/youtalk/autoware/pull/92/files

However you said this is also complex...

https://github.com/orgs/autowarefoundation/discussions/5110#discussioncomment-10468337

While I agree that locking versions to ensure stable builds is a valid approach, the added redundancy of creating multiple base images and the increased complexity it introduces to the overall build pipeline do not, in my opinion, justify the trade-off.

youtalk added a commit to youtalk/autoware that referenced this pull request Sep 5, 2024
…universe-sensing-perception` stage (autowarefoundation#5164)

* split depend stage

Signed-off-by: Yutaka Kondo <[email protected]>

* add s/p stage

Signed-off-by: Yutaka Kondo <[email protected]>

* fix

Signed-off-by: Yutaka Kondo <[email protected]>

* rename

Signed-off-by: Yutaka Kondo <[email protected]>

* fix

Signed-off-by: Yutaka Kondo <[email protected]>

* revert rename

Signed-off-by: Yutaka Kondo <[email protected]>

* Revert "revert rename"

This reverts commit 789ee21.

* simplify rosdep-depend

Signed-off-by: Yutaka Kondo <[email protected]>

---------

Signed-off-by: Yutaka Kondo <[email protected]>
youtalk added a commit that referenced this pull request Sep 5, 2024
…universe-sensing-perception` stage (#5164)

* split depend stage

Signed-off-by: Yutaka Kondo <[email protected]>

* add s/p stage

Signed-off-by: Yutaka Kondo <[email protected]>

* fix

Signed-off-by: Yutaka Kondo <[email protected]>

* rename

Signed-off-by: Yutaka Kondo <[email protected]>

* fix

Signed-off-by: Yutaka Kondo <[email protected]>

* revert rename

Signed-off-by: Yutaka Kondo <[email protected]>

* Revert "revert rename"

This reverts commit 789ee21.

* simplify rosdep-depend

Signed-off-by: Yutaka Kondo <[email protected]>

---------

Signed-off-by: Yutaka Kondo <[email protected]>
@youtalk youtalk mentioned this pull request Sep 5, 2024
4 tasks
youtalk added a commit that referenced this pull request Sep 6, 2024
* rename stages

Signed-off-by: Yutaka Kondo <[email protected]>

* update docker-bake

Signed-off-by: Yutaka Kondo <[email protected]>

* update actions

Signed-off-by: Yutaka Kondo <[email protected]>

* fix target

Signed-off-by: Yutaka Kondo <[email protected]>

* fix inherits

Signed-off-by: Yutaka Kondo <[email protected]>

* feat(docker): separation from `autoware-universe` stage to `autoware-universe-sensing-perception` stage (#5164)

* split depend stage

Signed-off-by: Yutaka Kondo <[email protected]>

* add s/p stage

Signed-off-by: Yutaka Kondo <[email protected]>

* fix

Signed-off-by: Yutaka Kondo <[email protected]>

* rename

Signed-off-by: Yutaka Kondo <[email protected]>

* fix

Signed-off-by: Yutaka Kondo <[email protected]>

* revert rename

Signed-off-by: Yutaka Kondo <[email protected]>

* Revert "revert rename"

This reverts commit 789ee21.

* simplify rosdep-depend

Signed-off-by: Yutaka Kondo <[email protected]>

---------

Signed-off-by: Yutaka Kondo <[email protected]>

* fix rebase

Signed-off-by: Yutaka Kondo <[email protected]>

* rename stages

Signed-off-by: Yutaka Kondo <[email protected]>

* fix bake

Signed-off-by: Yutaka Kondo <[email protected]>

* fix dockerfile

Signed-off-by: Yutaka Kondo <[email protected]>

* fix merge

Signed-off-by: Yutaka Kondo <[email protected]>

* add universe-sensing-perception stage

Signed-off-by: Yutaka Kondo <[email protected]>

* update config

Signed-off-by: Yutaka Kondo <[email protected]>

---------

Signed-off-by: Yutaka Kondo <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tag:run-health-check Run health-check
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Split the sensing/perception packages into a separate container
2 participants