add cicero CI actions #4489
Merged
add cicero CI actions #4489
+548
−13
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
dermetfan
force-pushed
the
cicero
branch
2 times, most recently
from
0f8ed2e to
e9e338b
Compare
dermetfan
requested review from
deepfire,
fmaste and
mgmeier
as code owners
dermetfan
force-pushed
the
cicero
branch
2 times, most recently
from
9f48cff to
ea57a13
Compare
angerman
requested changes
Sep 28, 2022
Pacman99
reviewed
Sep 28, 2022
nix/tullia.nix
Outdated
Comment on lines
30
to
32
| if config.actionRun.facts != {} | ||
| then with config.preset.github-ci; "github:${repo}/${sha}" | ||
| else "." |
There was a problem hiding this comment.
How come having a fact means you can refer to the flake with .?
Contributor
Author
There was a problem hiding this comment.
Tullia tasks can be run locally or on Cicero. When no facts are present we know that we are running locally and vice versa. When running locally, the current directory is already bind-mounted into the container, so we don't need to fetch the source from GitHub.
dermetfan
force-pushed
the
cicero
branch
17 times, most recently
from
95d249a to
feaaa6c
Compare
dermetfan
force-pushed
the
cicero
branch
3 times, most recently
from
60becc3 to
633f88f
Compare
dermetfan
force-pushed
the
cicero
branch
3 times, most recently
from
5a2f74a to
991e182
Compare
dermetfan
force-pushed
the
cicero
branch
2 times, most recently
from
6383c69 to
2125590
Compare
Contributor
@dermetfan can you show where the jobs are translated over? It's not clear to me. |
Contributor
Author
|
@Jimbo4350 Here: The |
deepfire
approved these changes
Oct 27, 2022
dermetfan
force-pushed
the
cicero
branch
7 times, most recently
from
7490e62 to
7348613
Compare
add nix cache to flake config
add tullia flake input
add cardano-node/ci/{push,pr} cicero actions
generate one tullia task for each hydra job
run tullia tasks sequentially in CI actions
to reduce load at one time
also build os-agnostic hydra jobs
build everything in one task
instead of dynamically creating tasks for every hydra job
Contributor
|
bors r+ |
Contributor
|
Build succeeded: |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This adds a tullia task
for every hydra joband cicero actions that run them for every push and PR.Edit: There's only one tullia task now that builds everything.