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

In which field of the wes interface should the logs of the Cromwell engine and the tes server be reflected? #211

Open
zhou-jianwen opened this issue Sep 13, 2023 · 7 comments

Comments

@zhou-jianwen
Copy link

While attempting to implement a WES server, we founded that WES only provides stdout and stderr fields within the run and task dimensions. We would like to clarify whether, according to the standard design of WES, how should TES server's own log of a specific task be retained? Similarly, we seek to determine if the logs generated by the workflow execution engine, such as Cromwell, should be represented in a specific field within the WES interface.

@patmagee
Copy link
Contributor

@zhou-jianwen at the moment this is not well supported. The WES 1.1.0 Release candidate (which I am hoping to merge and tag tomorrow) does address this by introducing a new field in the TaskLog schema: https://github.com/ga4gh/workflow-execution-service-schemas/pull/205/files#diff-1f171071387eb98ede480c7beb336dec6aa0a03e48a4c7a726fc19830121d804R907

@uniqueg
Copy link
Contributor

uniqueg commented Sep 14, 2023

To add on to what @patmagee said, we (and I think others) are storing the workflow engine (e.g., Cromwell) logs in stderr and stdout.

Are you willing to explain a bit about more about what WES/TES you are building? I'm curious. Perhaps it would be useful to schedule a call :)

@zhou-jianwen
Copy link
Author

zhou-jianwen commented Sep 15, 2023

@zhou-jianwen at the moment this is not well supported. The WES 1.1.0 Release candidate (which I am hoping to merge and tag tomorrow) does address this by introducing a new field in the TaskLog schema: https://github.com/ga4gh/workflow-execution-service-schemas/pull/205/files#diff-1f171071387eb98ede480c7beb336dec6aa0a03e48a4c7a726fc19830121d804R907

Thank you for your enthusiastic help. I apologize for not keeping up with the those developments.
I'm considering whether it would be beneficial to record more properties , allowing wes instance to add additional information about TES task,like TaskState maybe .

@patmagee
Copy link
Contributor

@zhou-jianwen for what it's worth, I view compliance with the spec as meaning you "must have what is in the spec, but not limited to that"

Imo if you have an api that has custom keys and properties, it would still be a valid WES api if it implements all Wes endpoints and dat models

@zhou-jianwen
Copy link
Author

zhou-jianwen commented Sep 15, 2023

To add on to what @patmagee said, we (and I think others) are storing the workflow engine (e.g., Cromwell) logs in stderr and stdout.

Are you willing to explain a bit about more about what WES/TES you are building? I'm curious. Perhaps it would be useful to schedule a call :)

In line with your perspective, I also believe that the workflow execution engine log serves as the standard output within the WES dimension.

We are considering coupling AAI and accounting functions with WES and TES servers. It would be delighted to engage in a more in-depth conversation.

@uniqueg
Copy link
Contributor

uniqueg commented Sep 16, 2023

Great! Can you see the email address in my GitHub profile? Alternatively, you could also join the GA4GH Slack board with this invite link and contact me there. Note that there are dedicated #wes and #tes channels on that board that could be useful to you.

@patmagee
Copy link
Contributor

@zhou-jianwen fyi Wes 1.1 is now released!

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

No branches or pull requests

3 participants