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.

Sign up for GitHub

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

Jump to bottom

[ci] Add GitHub actions for runtime tests #1

Merged
merged 29 commits into from
May 10, 2024
Merged

[ci] Add GitHub actions for runtime tests #1

merged 29 commits into from
May 10, 2024

Conversation

zero9178
Copy link
Member

@zero9178 zero9178 commented May 1, 2024 •
edited
Loading

This PR adds a GitHub action which automatically:

  • Builds iree-compile for use in the runtime
  • Builds the IREE runtime with test and samples
  • Executes the tests

The installation flow as to how to build, run and test the runtime is also implicitly established

zero9178 added 28 commits May 1, 2024 22:54
@zero9178
[WIP] Add GitHub actions doing basic compile test for now
6327b13 
Draft whose changes will get split out one by one until just CI things remain.
Only does build right now since we don't have tests
@zero9178
make checkout recurse into submodules
50b2bbb
@zero9178
fix cmake version
7b3ab86
@zero9178
bump minimum cmake to match IREE
21b6e5c
@zero9178
fix cmake build command
7c5201c
@zero9178
don't forget about installing bender
8f943a5
@zero9178
make the iree-compile build leaner, test out newer ubuntu
77be1fb
@zero9178
actually properly delete object files
7b3bd97
@zero9178
actually perform runtime build as well, increase ccache size
d97f580
@zero9178
correct path where iree-compile should be downloaded
f9b405a
@zero9178
try using tar to preserve permissions as recommended in https://git…
e0b0ad2 
…hub.com/actions/download-artifact?tab=readme-ov-file#permission-loss
@zero9178
fix tar typo
e5a08a7
@zero9178
test without path in download
8890f63
@zero9178
don't attempt to untar with gzip
3d73319
@zero9178
I really should have used tar --help earlier
c28facb
@zero9178
adjust path used to invoke tar to avoid absolute paths in artifact
c189856
@zero9178
undo typo
231bbf6
@zero9178
test out using a custom toolchain in a docker image, add cmake emulat…
b61586a 
…or for running sanity tests, add known good baseline sample using local_sync.c
@zero9178
upgrade action versions
10f7c3c
@zero9178
also run docker build on workflow file changes
6ad2386
@zero9178
fix version typo
6fe2eca
@zero9178
reduce image size, update github actions version, fix ref for runtime…
80467df 
… build to pull toolchain
@zero9178
use multistage build which drastically reduces image size
344e292
@zero9178
remove redundant command, add toolchain file to the toolchain
8f73a77
@zero9178
minor fixes, use runtime as context
528eb47
@zero9178
Merge branch 'refs/heads/main' into runtime-github-actions
2c38d88 
# Conflicts:
#	runtime/toolchain/Dockerfile
#	runtime/toolchain/README.md
@zero9178
Merge branch 'refs/heads/main' into runtime-github-actions
a37e589 
# Conflicts:
#	runtime/CMakeLists.txt
#	runtime/samples/CMakeLists.txt
#	runtime/snitch_cluster/CMakeLists.txt
#	runtime/tests/CMakeLists.txt
@zero9178
revert local sync baseline changes for now
90aece4
@zero9178 zero9178 changed the title [WIP] Add GitHub actions doing basic compile test for now [ci] Add GitHub actions for runtime tests May 10, 2024
@zero9178 zero9178 marked this pull request as ready for review May 10, 2024 09:28
@zero9178 zero9178 merged commit 6a5dfb1 into main May 10, 2024
2 checks passed
@zero9178 zero9178 deleted the runtime-github-actions branch June 27, 2024 20:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@zero9178