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

Moveit2 build takes over 2.5 hours in GHA workflow #77

Closed
mkhansenbot opened this issue Aug 14, 2023 · 5 comments · Fixed by #95
Closed

Moveit2 build takes over 2.5 hours in GHA workflow #77

mkhansenbot opened this issue Aug 14, 2023 · 5 comments · Fixed by #95
Assignees

Comments

@mkhansenbot
Copy link
Contributor

This is a performance issue. The Moveit2 build takes 2.5 hours in the GHA workflow, which makes it very painful to submit PR's and get them integrated in a timely fashion.

The simplest thing to fix this might be to push the image to dockerhub, so that if nothing in the Moveit2 build has changed, it will simply pull the latest image. However, we probably still want to work on getting the build down to an hour or less if possible.

@mkhansenbot
Copy link
Contributor Author

@mkhansenbot
Copy link
Contributor Author

mkhansenbot commented Aug 16, 2023

We can change the build to only build moveit2 if the /moveit2 files are changed:
https://docs.github.com/en/actions/using-workflows/events-that-trigger-workflows#pull_request

Edit- I mean we can do that if we are pushing nightly builds to Dockerhub

@mkhansenbot
Copy link
Contributor Author

@EzraBrooks - what do you think of the idea of pushing the latest moveit2 build to docker and only re-building it if the /moveit2 files change?

@EzraBrooks
Copy link
Member

I think that makes sense as an immediate measure before we have time/energy to refactor our containers to take good advantage of Docker's layer caching so it handles that for us.

@Bckempa
Copy link
Contributor

Bckempa commented Dec 3, 2023

This might get partially resolved during this next release but I'm not adding the milestone tag because we don't want to depend on it

EzraBrooks added a commit that referenced this issue Dec 28, 2023
This is a common hack to avoid running out of memory while building a large C++ codebase. However, it's unnecessary in an era where users just need to be educated to configure `zram` on their system.
EzraBrooks added a commit that referenced this issue Dec 28, 2023
This is a common hack to avoid running out of memory while building a large C++ codebase. However, it's unnecessary in an era where users just need to be educated to configure `zram` on their system.
EzraBrooks added a commit that referenced this issue Dec 28, 2023
Don't limit parallel workers in moveit2 build (#77)
@Bckempa Bckempa added this to the humble-2024.01.0 milestone Dec 28, 2023
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 a pull request may close this issue.

3 participants