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

ci: avoid installing the toolchain from scratch in each job #257

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

umarcor
Copy link
Contributor

@umarcor umarcor commented Feb 21, 2022

Instead of installing the toolchain in each job, this PR splits CI in two stages. First, the 18 variants of the toolchain are installed in containers, which are pushed to GitHub Packages. Then, example jobs pull the containers/packages and use them straightaway.

BTW, a container step is used instead of a container job, so that actions/checkout is executed on the "host".

@umarcor umarcor marked this pull request as draft February 21, 2022 22:01
@umarcor umarcor marked this pull request as ready for review February 21, 2022 22:13
@umarcor
Copy link
Contributor Author

umarcor commented Feb 21, 2022

CI is failing because the default token for PRs does not allow pushing to the registry. See a run in my fork: https://github.com/umarcor/f4pga-examples/actions/runs/1878523142 (which uses ghcr.io/umarcor/f4pga/dev/conda instead of ghcr.io/chipsalliance/f4pga/dev/conda).

umarcor and others added 2 commits August 3, 2022 03:49
* Use GitHub Packages
* Use container in a step, instead of a container job

F4PGA_INSTALL_DIR changed to an absolute location due to actions/runner#1525

Signed-off-by: Unai Martinez-Corral <umartinezcorral@antmicro.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.

1 participant