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

build(ROS distribution)!: transition to ROS 2 Humble #2692

Merged
merged 13 commits into from
Dec 1, 2022
Merged

Conversation

kenji-miyake
Copy link
Contributor

@kenji-miyake kenji-miyake commented Jun 28, 2022

Description

Moved from autowarefoundation/autoware_core_universe_prototype#330.
Resolves #2443.

I won't merge this PR to main until the Galactic support by the AWF ends.
Users have to switch branches if they want to use Humble before that.
I've split the branches main(galactic) and humble because it would be too complex if we support both distros in one branch.

Related links

https://github.com/orgs/autowarefoundation/discussions/2622

Tests performed

  • TIER IV has tested this in simulations and real vehicles.

Notes for reviewers

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.

  • The PR follows the pull request guidelines.
  • The PR has been properly tested.
  • The PR has been reviewed by the code owners.

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.
  • The PR is ready for merge.

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

autoware.repos Outdated Show resolved Hide resolved
@kenji-miyake kenji-miyake marked this pull request as ready for review September 1, 2022 07:04
@kenji-miyake kenji-miyake marked this pull request as draft September 1, 2022 07:04
@kenji-miyake kenji-miyake mentioned this pull request Nov 30, 2022
32 tasks
@kenji-miyake kenji-miyake changed the title feat(ROS distribution)!: transition to ROS 2 Humble build(ROS distribution)!: transition to ROS 2 Humble Nov 30, 2022
@kenji-miyake kenji-miyake mentioned this pull request Nov 30, 2022
4 tasks
@kenji-miyake kenji-miyake marked this pull request as ready for review November 30, 2022 15:23
@kenji-miyake
Copy link
Contributor Author

If #3089 is merged to main, the version 20.04should be replaced by 22.04.

amd64.env Show resolved Hide resolved
sudo apt update
rosdistro=galactic
rosdistro=humble
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Probably we can remove this since amd64.env is sourced.
@xmfcx What do you think?

Suggested change
rosdistro=humble

Copy link
Contributor

Choose a reason for hiding this comment

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

If it works, we can do it.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Okay, I'll do it in another PR to keep the diff simple.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Copy link
Contributor

@xmfcx xmfcx left a comment

Choose a reason for hiding this comment

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

I think we can merge this if CI can pass. (Since it installs the ansible scripts and builds the autoware)

@kenji-miyake
Copy link
Contributor Author

@xmfcx Regarding build, we should manually trigger workflows because it's not triggered by the pull_request event due to the execution costs. So I'll check it tomorrow.

But before that, can I merge #3092 and #3093? They can reduce the diff of this PR.

Kenji Miyake and others added 5 commits December 1, 2022 02:22
Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp>

docs(ansible/pre_commit): remove adding golang repository for humble (#2837)

fix(docs): remove adding golang repository for humble

Signed-off-by: M. Fatih Cırıt <mfc@leodrive.ai>

Signed-off-by: M. Fatih Cırıt <mfc@leodrive.ai>
Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp>
Signed-off-by: M. Fatih Cırıt <mfc@leodrive.ai>
Signed-off-by: M. Fatih Cırıt <xmfcx@users.noreply.github.com>
Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp>
wep21 and others added 6 commits December 1, 2022 02:23
Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp>
… documentation (#3007)

Signed-off-by: Shin-kyoto <58775300+Shin-kyoto@users.noreply.github.com>

Signed-off-by: Shin-kyoto <58775300+Shin-kyoto@users.noreply.github.com>

build(ansible/ros2_dev_tools):  follow updated humble documentation (#3020)

Signed-off-by: Shin-kyoto <58775300+Shin-kyoto@users.noreply.github.com>

Signed-off-by: Shin-kyoto <58775300+Shin-kyoto@users.noreply.github.com>
Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp>
Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp>
Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp>
Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp>
@mitsudome-r mitsudome-r merged commit 9e35fd1 into main Dec 1, 2022
@mitsudome-r mitsudome-r deleted the humble branch December 1, 2022 12:11
1222-takeshi pushed a commit to 1222-takeshi/autoware that referenced this pull request Dec 2, 2022
…ion#2692)

* remove Go PPA

Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp>

docs(ansible/pre_commit): remove adding golang repository for humble (autowarefoundation#2837)

fix(docs): remove adding golang repository for humble

Signed-off-by: M. Fatih Cırıt <mfc@leodrive.ai>

Signed-off-by: M. Fatih Cırıt <mfc@leodrive.ai>

* rename galactic to humble

Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp>

* feat: add ament_cmake fork (autowarefoundation#370)

Signed-off-by: M. Fatih Cırıt <mfc@leodrive.ai>
Signed-off-by: M. Fatih Cırıt <xmfcx@users.noreply.github.com>

* revert(autoware.repos): revert "feat: add ament_cmake fork (autowarefoundation#370)" (autowarefoundation#3054)

This reverts commit 0d06dd3.

* update hash of grid_map

Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp>

* chore: remove grid_map from repos (autowarefoundation#2864)

* use Ubuntu 22.04 and CUDA 11.8

Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp>

* build(ansible/ros2_dev_tools): change ros2_dev_tools to follow humble documentation (autowarefoundation#3007)

Signed-off-by: Shin-kyoto <58775300+Shin-kyoto@users.noreply.github.com>

Signed-off-by: Shin-kyoto <58775300+Shin-kyoto@users.noreply.github.com>

build(ansible/ros2_dev_tools):  follow updated humble documentation (autowarefoundation#3020)

Signed-off-by: Shin-kyoto <58775300+Shin-kyoto@users.noreply.github.com>

Signed-off-by: Shin-kyoto <58775300+Shin-kyoto@users.noreply.github.com>

* docs(ansible/cuda): update manual installation steps

Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp>

* tmp: use ubuntu:22.04 for cuda_base_image

Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp>

* tmp: add liburcu6 repository for CUDA 20.04 repository

Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp>

* feat(ansible/playbooks): update Ubuntu version

Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp>

Signed-off-by: M. Fatih Cırıt <mfc@leodrive.ai>
Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp>
Signed-off-by: M. Fatih Cırıt <xmfcx@users.noreply.github.com>
Signed-off-by: Shin-kyoto <58775300+Shin-kyoto@users.noreply.github.com>
Co-authored-by: M. Fatih Cırıt <xmfcx@users.noreply.github.com>
Co-authored-by: Daisuke Nishimatsu <42202095+wep21@users.noreply.github.com>
Co-authored-by: Shintaro Tomie <58775300+Shin-kyoto@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support ROS 2 Humble
5 participants