-
-
Notifications
You must be signed in to change notification settings - Fork 28
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
Upgrade cwltool version to something from 2021 #306
Comments
I built a new docker image that upgraded the cwltool to version 3.0 and started a docker container with it. I tested a random URL from https://view.commonwl.org/workflows and was able to parse the workflow from my local without encountering any problem. |
We should run the tests before upgrading production
`mvn clean cobertura:cobertura coveralls:report `
…--
Michael R. Crusoe
On Fri, Apr 9, 2021, 20:22 oceenachi ***@***.***> wrote:
I built a new docker image that upgraded the cwltool to version 3.0 and
started a docker container with it. I tested a random URL from
https://view.commonwl.org/workflows and was able to parse the workflow
from my local without encountering any problem.
Attached here is a screenshot of the parsed workflow
[image: image]
<https://user-images.githubusercontent.com/44432472/114224277-ca066f80-9968-11eb-8fb0-319bf15c32cf.png>
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#306 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAKE4CALHWMMW7ZYDGR7SVDTH5AXHANCNFSM42E6BSFA>
.
|
OKay. Do I run this? |
I think you have done everything you need to. One of the other team members should run that command ( or fix our continuous integration to do it for us) |
Okay great. I guess I'll pick another ticket then. |
After looking into this I think the issue is that production uses the docker image rather than the master branch of the codebase. Right now only the test pipeline gets run on GitHub but it never actually gets updated in prod. I believe to solve this issue the docker-compose file has to be set to build rather than an image on the docker hub |
@cure What's the status on improving the deployment of CWL viewer? |
This was done |
Currently the container being used seems to be stuck in 2018 (!!)
Verified with cwltool version 1.0.20180525185854
The text was updated successfully, but these errors were encountered: