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

Simulation image based on Nvidia L4T #308

Open
wants to merge 23 commits into
base: master
Choose a base branch
from

Conversation

random82
Copy link

@random82 random82 commented Aug 3, 2021

No description provided.

@random82
Copy link
Author

random82 commented Aug 3, 2021

Hi @dagar
I pushed some fixed since you approved it, could you kick off the workflow to check if it builds?

@random82 random82 requested a review from dagar August 3, 2021 23:31
@dagar
Copy link
Member

dagar commented Aug 4, 2021

Hi @dagar
I pushed some fixed since you approved it, could you kick off the workflow to check if it builds?

Thanks, it's running now.

@dagar
Copy link
Member

dagar commented Aug 17, 2021

@random82 did you see the build failure?

@random82
Copy link
Author

@random82 did you see the build failure?

Yes, I'm trying to figure out how to build an ARM image with Github Actions and will come back to you shortly

@random82
Copy link
Author

@dagar - I managed to build those images using BuildKit and ARM64 emulator but they take 1 hour each at GitHub (3-5m on the bare metal...) and there is a dependency between them.

I'm going to submit two separate pull requests for base and simulation images to prevent the build from failing when the base is not in a Docker hub yet.

There is a problem with long build time and I was thinking to have separate workflows for ARM images CI to avoid 2 hours build every time someones modify other images.

Thoughts?

@random82
Copy link
Author

@dagar

The ARM64 builds are super slow on Github so I split x86 and ARM64 build workflows to avoid unnecessary builds and save time

@random82 random82 closed this Oct 27, 2021
@random82 random82 reopened this Oct 27, 2021
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.

2 participants