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.

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

allow docker image pinning to be skipped with #skip-global-version-pin annotation #493

Merged
merged 1 commit into from
Oct 17, 2023

Conversation

tomkinsc
Copy link
Member

@tomkinsc tomkinsc commented Oct 17, 2023

The script github_actions_ci/version-wdl-runtimes.sh is used to replace docker image tags in WDL task files with the tags listed in the requirements-modules.txt file. These replacements were previously applied globally. With this PR, image references on lines ending with the comment #skip-global-version-pin are exempt from these replacements. This allows tasks to have image versions held back; this is useful in the event that a task has commands that are incompatible with newer image versions.

The script github_actions_ci/check-wdl-runtimes.sh also honors these annotations.

The #skip-global-version-pin annotation is also applied to two tasks which use krakenuniq, as newer versions of the viral-classify image will not provide krakenuniq until an updated conda rebuild exists for the fork https://github.com/yesimon/krakenuniq and its associated dependencies.

…n annotation

The script `github_actions_ci/version-wdl-runtimes.sh` is used to replace docker image tags in WDL task files with the tags listed in the `requirements-modules.txt` file. These replacements were previously applied globally. With this commit, image references on lines ending with the comment `#skip-global-version-pin` are exempt from these replacements. This allows tasks to have image versions held back; this is useful in the event that a task has commands that are incompatible with newer image versions.
@tomkinsc tomkinsc merged commit 3f5be63 into master Oct 17, 2023
12 checks passed
@tomkinsc tomkinsc deleted the ct-task-docker-pinning-disable branch October 17, 2023 18:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants