unito-bot

@gmloose can you check if this behavior also happens in cwltool?

➤ Melaina Legaspi commented: Adam Novak :"Needs further investigation: is something actually going wrong with kubernetes set up or is the file just not available by the time we look...

➤ Melaina Legaspi commented: Adam Novak :"Need input from AGC folks, not just mentioned people above.”

➤ Melaina Legaspi commented: We should just increase the number of tries.

➤ Adam Novak commented: Since S3 is strongly consistent now, we’re planning to just use that and not DynamoDB.

➤ Denis Yuen commented: So a few things to figure out 1. Update to newest version (security updates and often fixed bugs) 2. Is it closed or not (saw it...

➤ Denis Yuen commented: Looks like they do support JSON: https://github.com/common-workflow-language/cwljava/blob/cwl-1.2.0/src/test/resources/org/w3id/cwl/cwl1_2/utils/525.cwl ( https://github.com/common-workflow-language/cwljava/blob/cwl-1.2.0/src/test/resources/org/w3id/cwl/cwl1_2/utils/525.cwl|smart-link )

➤ Charles Overbeck commented: The tools and DAG pages are returning different results for the second CWL workflow I looked at. Reopening. Use this workflow: https://qa.dockstore.org/workflows/github.com/sevenbridges-openworkflows/Broad-Best-Practice-Data-pre-processing-CWL1.0-workflow-GATK-4.1.0.0/GATK_4_1_0_0_data_pre_processing_workflow:master?tab=info ( https://qa.dockstore.org/workflows/github.com/sevenbridges-openworkflows/Broad-Best-Practice-Data-pre-processing-CWL1.0-workflow-GATK-4.1.0.0/GATK_4_1_0_0_data_pre_processing_workflow:master?tab=info|smart-link ) Notes:...

➤ Denis Yuen commented: Interesting, looks like the external viewer validates that workflow properly, but only displays a grey screen for the structure. https://view.commonwl.org/workflows/github.com/svonworl/Broad-Best-Practice-Data-pre-processing-CWL1.0-workflow/blob/fe43d13c174816704f1d7941dc8cb5fce358c0de/broad-best-practice-data-pre-processing-workflow-4-1-0-0_decomposed.cwl ( https://view.commonwl.org/workflows/github.com/svonworl/Broad-Best-Practice-Data-pre-processing-CWL1.0-workflow/blob/fe43d13c174816704f1d7941dc8cb5fce358c0de/broad-best-practice-data-pre-processing-workflow-4-1-0-0_decomposed.cwl )

➤ Steve Von Worley commented: The only non-test change in the PR is the regex, so one idea would be to revert it, build, and test. If reverting the regex...