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

Add Dockerfile for ARM #572

Merged
merged 11 commits into from
Feb 13, 2025
Merged

Conversation

mschweig
Copy link
Contributor

@mschweig mschweig commented Feb 6, 2025

Change Overview

Created a Dockerfile.arm for ARM deployment as the current Dockerfile does not support ARM architecture. Useful for deployment and decoupled setup with several machines. Thought it could be useful for RAI.

Testing Done

Tested on NVIDIA Jetson ORIN (attached to robot via ethernet) with docker-compose (also included for demo purpose)

Copy link
Collaborator

@mhidalgo-bdai mhidalgo-bdai left a comment

Choose a reason for hiding this comment

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

First pass

docker-compose.yaml Outdated Show resolved Hide resolved
docker-compose.yaml Outdated Show resolved Hide resolved
docker-compose.yaml Outdated Show resolved Hide resolved
Dockerfile.arm Outdated Show resolved Hide resolved
Signed-off-by: Manuel Schweiger <[email protected]>
Signed-off-by: Manuel Schweiger <[email protected]>
Signed-off-by: Manuel Schweiger <[email protected]>
Signed-off-by: Manuel Schweiger <[email protected]>
Signed-off-by: Manuel Schweiger <[email protected]>
Signed-off-by: Manuel Schweiger <[email protected]>
Signed-off-by: Manuel Schweiger <[email protected]>
Signed-off-by: Manuel Schweiger <[email protected]>
Signed-off-by: Manuel Schweiger <[email protected]>
@mschweig mschweig force-pushed the feat/add-arm-docker branch from b3eeb05 to df316c5 Compare February 11, 2025 14:21
Dockerfile Outdated Show resolved Hide resolved
Dockerfile Show resolved Hide resolved
Dockerfile Outdated Show resolved Hide resolved
docker-compose.yaml Outdated Show resolved Hide resolved
Signed-off-by: Manuel Schweiger <[email protected]>
Copy link
Collaborator

@mhidalgo-bdai mhidalgo-bdai left a comment

Choose a reason for hiding this comment

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

LGTM though I don't have an arm64 box at hand.

@khughes-bdai khughes-bdai merged commit e882afc into bdaiinstitute:main Feb 13, 2025
4 of 5 checks passed
@mschweig mschweig deleted the feat/add-arm-docker branch February 13, 2025 19:07
@mschweig
Copy link
Contributor Author

@khughes-bdai awesome. :)

@tcappellari-bdai as discussed I am now writing a disclaimer in the readme for rmw_zenoh. Do you want me to add a launch parameter to start the zenoh router with the driver? Its a bit more convenient.

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.

4 participants