-
Notifications
You must be signed in to change notification settings - Fork 4
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
Compatible with (but not mandatory) Java 21 #293
Changes from all commits
71f0cab
6315860
69e685e
392259b
b1d29bc
798b253
c1548de
2fdfd24
a0d8b0e
5a40563
1a9f299
9080cd9
7067b6f
4de8628
c84e75e
a5a64b6
cdfa850
6c8d7cf
149b426
a8c2523
1469ef7
dfded62
6d6fa4b
8da34b3
1fa891f
9de2336
dddc979
fa84e68
1364f7f
e095d09
e628a28
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -10,7 +10,7 @@ doc: "Mutect 1.1.5" | |
|
||
hints: | ||
- class: DockerRequirement | ||
dockerPull: quay.io/jeltje/mutect | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Some Docker repos are forked, some can be downloaded and reuploaded in place (by simply pulling and pushing with an older client, which I had handy on my laptop) This is an example of forking, the pull/push updates don't leave a trace. |
||
dockerPull: quay.io/dockstore-testing/mutect | ||
|
||
requirements: | ||
- class: InlineJavascriptRequirement | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -14,7 +14,15 @@ fi | |
if [ "${TESTING_PROFILE}" = "toil-integration-tests" ]; then | ||
pip3 install --user toil[cwl]==7.0.0 | ||
else | ||
pip3 install --user -r https://raw.githubusercontent.com/dockstore/dockstore/develop/dockstore-webservice/src/main/resources/requirements/1.15.0/requirements3.txt | ||
sudo apt-get update | ||
# https://stackoverflow.com/questions/44331836/apt-get-install-tzdata-noninteractive needed by cwltool | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This matches 1.16 quickstart with pipx, but had to do a bit of extra work to get schema_salad and cwlref-runner (may be using pipx runpip wrong) |
||
DEBIAN_FRONTEND=noninteractive sudo apt-get -qq --yes --force-yes install tzdata pipx curl | ||
curl -o requirements.txt "https://dockstore.org/api/metadata/runner_dependencies?client_version=1.16.0&python_version=3" | ||
pipx install cwltool==3.1.20240708091337 | ||
pipx install schema_salad==8.7.20240718183047 | ||
pipx install cwlref-runner | ||
pipx runpip cwltool install -r requirements.txt # this ensures that your version of cwltool and its dependencies matches what we test with | ||
pipx ensurepath | ||
fi | ||
|
||
if [ "${TESTING_PROFILE}" = "singularity-tests" ]; then | ||
|
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.
Why is the tag different for this and the
toil_wes_test_executor
below?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.
A sad reason
While experimenting, I bumped to a newer version to try to get a newer Java version on the machine executor. It turned out to be unnecessary, but since we ran the tests anyway I kept the newer versions that worked (
toil_wes_test_executor
used forwes-toil-test
).Something about
ubuntu-2204:2024.11.1
breaks the toil install silently (but not all of toil, the part used for regular integration tests includingintegration-tests-toil-integration-tests
, not a typo)