-
Notifications
You must be signed in to change notification settings - Fork 265
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
chore(CI): remove end-to-end/Earthfile #9364
Merged
Merged
Changes from all commits
Commits
Show all changes
27 commits
Select commit
Hold shift + click to select a range
fd0786a
chore(CI): remove end-to-end/Earthfile
spypsy 098dc27
add test_config.yml, remove logic from ci.yml
spypsy 93987b0
fix select_runner usage
spypsy 7a772e3
Merge branch 'master' into spy/remove-e2e-earthfile
spypsy fa323b2
remove test logging
spypsy 5426dfd
debug log
spypsy 8b2c02b
setup yq
spypsy 04770ae
install yq in e2e_test
spypsy 1e67318
fix typo
spypsy 76eae4a
fix chmod
spypsy 169fabe
merge master conflicts
spypsy 5fe4d17
try in ensure-tester action
spypsy c945428
chmod -> then move
spypsy 485ac9c
fix test config loc
spypsy 2f88819
rm unused script
spypsy d41eb49
handle tests in subdirectories
spypsy 8284205
updates for some cases
spypsy 4e698df
no test.ts postfix
spypsy cc41f5b
add ignore_failures option
spypsy 8e637e0
use old compose script
spypsy 94e59d0
merge with master
spypsy 6da5ceb
fix dir
spypsy 5534268
fix some more test dirs
spypsy f9c6f20
use test_path, not $TEST
spypsy ca8ace5
Merge branch 'master' into spy/remove-e2e-earthfile
spypsy 84c6379
rm comment
spypsy 30ee025
use select_runner for bench as well
spypsy File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains 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
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -35,6 +35,10 @@ runs: | |
# TODO reconsider how jq gets into image | ||
- name: Setup jq | ||
uses: dcarbone/[email protected] | ||
- name: Setup yq | ||
uses: dcarbone/[email protected] | ||
with: | ||
version: v4.44.3 | ||
|
||
- name: Setup Earthly | ||
uses: earthly/actions-setup@v1 | ||
|
This file contains 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
This file contains 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
This file contains 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
This file contains 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
This file contains 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
This file contains 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
This file contains 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
This file contains 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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,19 @@ | ||
#!/bin/bash | ||
|
||
# Check if the test requires a prover | ||
requires_prover() { | ||
local test_name=$1 | ||
[[ $test_name == *"prover"* ]] | ||
} | ||
|
||
test_name=$1 | ||
|
||
if requires_prover "$test_name"; then | ||
echo "64core-tester-x86" | ||
else | ||
if [[ $test_name == *"bench"* ]]; then | ||
echo "16core-tester-x86" | ||
else | ||
echo "8core-tester-x86" | ||
fi | ||
fi |
Oops, something went wrong.
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this ideally would be encapsulated in the same shell script that runs the things, but nbd
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wasn't sure on how to do this. The result of
select_runner
is used to start the machine on whiche2e_test.sh
is then run on.