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: First Docker setup with slang pipeline #5

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

ci: First Docker setup with slang pipeline #5

wants to merge 35 commits into from

Conversation

paulsc96
Copy link
Collaborator

@paulsc96 paulsc96 commented Feb 7, 2023

This is a continuation of #2.

@joennlae joennlae marked this pull request as ready for review February 15, 2023 13:30
@joennlae
ci: add python env to docker container
8e6fe3f
@joennlae
ci: unholy commit to run workflow
0944ee0
@joennlae
ci: end of the line fix
c6dccea
@joennlae
ci: use correct pip package name on ubuntu
e5f8924
@joennlae
ci: conditional container build + build ci with container
1a5e448
@joennlae
ci: remove safe directory warning for container
cc46200
@joennlae
ci: github action add safe directory workaround
0cfc4ac
@joennlae
ci: cleanup
fac9957
@joennlae
ci: add morty pickle + slang parsing test
cd2e28b
@joennlae
ci: add wget to docker container
0e22e2e
@joennlae
ci: source cargo in container
c957adf
@joennlae
fix(ci): fix docker build
13f086d
@joennlae
feat(cva6): update to correct includes for cva6
68271bc 
pulp-platform/cva6#4
@joennlae
feat(slang): add slang check of cva6 using bender + morty
45de7fe 
not yet working due to morty package import failure
@joennlae
ci: add true to slang check to pass actions atm
47f3b9f
@joennlae
ci: fix slang check action
c5fa813
@joennlae
ci: add unzip to container
7efac83
@joennlae
ci: docker update env
f1074a0
@joennlae
ips: update cva6 commit
9718c13
@joennlae
ci: update slang check
d8100bb
@paulsc96 paulsc96 marked this pull request as draft February 17, 2023 12:28
@joennlae joennlae marked this pull request as ready for review February 17, 2023 13:23
@joennlae
Copy link
Contributor

Probably not wise to merge yet as two changes for CVA6 would not be in here, specifically these:
pulp-platform/cva6#6
pulp-platform/cva6#8
This pull request uses pulp-platform/cva6#9

  1. Adds docker ci.
  2. add slang check of cva6 (had to change bender file; that is why another branch is used, but it will be changed when nils merge that).

I don't know what the gitlab-ci fails, as I should not have changed anything regarding that. But I also have no right to see the log, so I cannot investigate it :-)

@joennlae joennlae force-pushed the docker-ci branch 2 times, most recently from d0f00b5 to d62c7b7 Compare February 24, 2023 18:23
@joennlae
bender: update lock with newer cva6 rev
c7b9239
@joennlae
ci: add slang check for cheshire (currently failing)
9f8eaa7 
due to multiple reasons:
1) morty not importing REGBUS package
2) strange always_ff in `spi_host_fsm`
3) one conversion that is not allowed in `axi_llc_tag_store`
@joennlae
bender: update bender with new cva6 rev
60031a8
@paulsc96 paulsc96 marked this pull request as draft February 23, 2024 22:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@joennlae joennlae

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@paulsc96 @joennlae