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

Make necessary changes for TPV 2.0 #405

Merged
merged 1 commit into from
Jan 10, 2023
Merged

Make necessary changes for TPV 2.0 #405

merged 1 commit into from
Jan 10, 2023

Conversation

nuwang
Copy link
Member

@nuwang nuwang commented Dec 24, 2022

No description provided.

@nuwang nuwang changed the title Make changed necessary for TPV 2.0 Make changes necessary for TPV 2.0 Dec 24, 2022
@nuwang nuwang changed the title Make changes necessary for TPV 2.0 Make necessary changes for TPV 2.0 Dec 24, 2022
@afgane
Copy link
Contributor

afgane commented Jan 9, 2023

Will this work with with older Galaxy versions? I cannot find where TPV version is pinned so wondering how does an admin update from <2.0.

@nuwang
Copy link
Member Author

nuwang commented Jan 10, 2023

Yes. The runner parameter will be ignored in older versions. The TPV version is constrained in Galaxy (https://github.com/galaxyproject/galaxy/blob/c54ec2f18d7a8d5fa7a73710c550ad24ecda594b/lib/galaxy/dependencies/conditional-requirements.txt#L16), but unconstrained in the container (https://github.com/galaxyproject/galaxy-docker-k8s/blob/3d0ccac59107841ff28acc2eb60e433e967fc7eb/files/k8s-requirements.txt#L3).

@afgane afgane merged commit 25eb6ce into master Jan 10, 2023
@afgane afgane deleted the upgrade_to_tpv_2 branch January 10, 2023 14:05
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