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

Re-template rmf_deployment_template #39

Merged
merged 108 commits into from
Oct 16, 2024

Conversation

akash-roboticist
Copy link
Collaborator

@akash-roboticist akash-roboticist commented Sep 5, 2024

New feature implementation

Implemented feature

Re-write of rmf deployment template for production use

  • primary goal is to streamline the deployment process using lessons from production deployments
  • helm charts, build dockerfiles and sources are in a single branch (down from 3 branches main, cloud_infra, build/rmf-site ), thus enabling easier feature iterations
  • also updates the dockerfiles to use current versions (eg. ROS 2 Jazzy, Keycloak 24, etc)

Tested deploying using steps described in the README to deploy Open-RMF in a kubernetes cluster and running locally using docker-compose.

akash-roboticist and others added 30 commits June 20, 2022 13:01
Signed-off-by: Akash Vibhute <[email protected]>
Signed-off-by: Akash Vibhute <[email protected]>
…n-rmf-img

Cloud infra/runtime command with common rmf docker img
Signed-off-by: youliang <[email protected]>
Cleanup rmf_deployment helm chart dir structure
…en-rmf#15)

* fix cyclonedds config and missing server-uri env

Signed-off-by: youliang <[email protected]>

* provide both cyclonedds config for humble and galactic

Signed-off-by: youliang <[email protected]>

* fix building map path

Signed-off-by: youliang <[email protected]>

Signed-off-by: youliang <[email protected]>
Signed-off-by: Akash Vibhute <[email protected]>
* WIP: update to keycloak

The pod is able to start and forward port displays the initial screen
fine. However the admin interface seems frozen in the process of
loading.

Signed-off-by: Jose Luis Rivero <[email protected]>

* Do not change hostName

Signed-off-by: Jose Luis Rivero <[email protected]>

* Do not run DEBUG mode by default

Signed-off-by: Jose Luis Rivero <[email protected]>

---------

Signed-off-by: Jose Luis Rivero <[email protected]>
Signed-off-by: Jose Luis Rivero <[email protected]>
Co-authored-by: Akash Vibhute <[email protected]>
Signed-off-by: Akash Vibhute <[email protected]>
Signed-off-by: Akash Vibhute <[email protected]>
Signed-off-by: Akash Vibhute <[email protected]>
Signed-off-by: Akash Vibhute <[email protected]>
Signed-off-by: Akash Vibhute <[email protected]>
Signed-off-by: Akash Vibhute <[email protected]>
Signed-off-by: Akash Vibhute <[email protected]>
Signed-off-by: Akash Vibhute <[email protected]>
Signed-off-by: Akash Vibhute <[email protected]>
Signed-off-by: Akash Vibhute <[email protected]>
Signed-off-by: Akash Vibhute <[email protected]>
Signed-off-by: Akash Vibhute <[email protected]>
…somethingwentwrong message on /auth

Signed-off-by: Akash Vibhute <[email protected]>
…g somethingwentwrong message on /auth

Signed-off-by: Akash Vibhute <[email protected]>
Signed-off-by: Akash Vibhute <[email protected]>
Signed-off-by: Akash Vibhute <[email protected]>
Signed-off-by: Akash Vibhute <[email protected]>
Signed-off-by: Akash Vibhute <[email protected]>
Signed-off-by: Akash Vibhute <[email protected]>
Signed-off-by: Akash Vibhute <[email protected]>
Signed-off-by: Akash Vibhute <[email protected]>
@akash-roboticist akash-roboticist marked this pull request as ready for review October 8, 2024 07:49
@akash-roboticist akash-roboticist changed the title Wip deploy Re-template rmf_deployment_template Oct 9, 2024
aaronchongth and others added 3 commits October 10, 2024 14:18
- point repo url back to open-rmf
- remove old commented lines
- remove merge conflit artefacts

Signed-off-by: Akash Vibhute <[email protected]>
Copy link
Member

@aaronchongth aaronchongth left a comment

Choose a reason for hiding this comment

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

Just did a quick first pass while trying to re-run the steps on a fresh machine, had some comments and questions

README.md Outdated Show resolved Hide resolved
charts/infrastructure/tools/helm_charts_build.bash Outdated Show resolved Hide resolved
charts/infrastructure/tools/helm_charts_build.bash Outdated Show resolved Hide resolved
charts/infrastructure/tools/helm_charts_build.bash Outdated Show resolved Hide resolved
README.md Show resolved Hide resolved
Copy link
Member

@aaronchongth aaronchongth left a comment

Choose a reason for hiding this comment

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

The cluster is working well for me!

One more note for other reviewers, the images that the deployment will pull are old images hosted upstream in open-rmf, the latest images that work are in this fork, so you'll need to pull them and tag them manually to test

docker pull ghcr.io/akash-roboticist/rmf_deployment_template/rmf-deployment/api-server
docker pull ghcr.io/akash-roboticist/rmf_deployment_template/rmf-deployment/dashboard
docker pull ghcr.io/akash-roboticist/rmf_deployment_template/rmf-deployment/keycloak-setup

docker tag ghcr.io/akash-roboticist/rmf_deployment_template/rmf-deployment/keycloak-setup ghcr.io/open-rmf/rmf_deployment_template/rmf-deployment/keycloak-setup
docker tag ghcr.io/akash-roboticist/rmf_deployment_template/rmf-deployment/dashboard ghcr.io/open-rmf/rmf_deployment_template/rmf-deployment/dashboard
docker tag ghcr.io/akash-roboticist/rmf_deployment_template/rmf-deployment/api-server ghcr.io/open-rmf/rmf_deployment_template/rmf-deployment/api-server

README.md Show resolved Hide resolved
Signed-off-by: Akash Vibhute <[email protected]>
@akash-roboticist akash-roboticist merged commit d9b3d36 into open-rmf:main Oct 16, 2024
1 check failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

4 participants