Skip to content

Conversation

@pditommaso
Copy link
Member

@pditommaso pditommaso commented May 29, 2022

This PR adds the ability to partially run the script execution, without submitting any job execution.

To some extent, it may be comparable to a dry-run, even though real dry-run capability simulates the full graph of execution, task by task. This feature instead computes the DAG of process dependencies and terminates the script bypassing the jobs submission, which remains unknown.

This can be useful to render the pipeline DAG without running any task. It could be also used to evaluate the pipeline inputs and similar tasks.

Currently, it's activated via the CLI option -dry. However, it would be preferable to use a different name for the reason explained above.

Feedback is welcome.

Signed-off-by: Paolo Di Tommaso <[email protected]>
@pditommaso pditommaso changed the title Add dry-run like capability [ci skip] Add dry-run like capability May 29, 2022
@drpatelh
Copy link
Contributor

Given the functionality is specific to the DAG generation, would it make sense to make this option specific to the current -with-dag parameter or adding something like -with-dag-dry?

@pditommaso
Copy link
Member Author

I was thinking the same origin, but potentially it could be useful for other purposes.

@jorgeaguileraseqera is also suggesting -preview that sounds good.

Signed-off-by: Paolo Di Tommaso <[email protected]>
Signed-off-by: Paolo Di Tommaso <[email protected]>
@pditommaso
Copy link
Member Author

So, preview option was merged!

@pditommaso pditommaso merged this pull request into master May 31, 2022
@pditommaso pditommaso deleted the dry-run branch May 31, 2022 18:58
pditommaso added a commit that referenced this pull request May 31, 2022
This commit adds a new CLI run option `-preview`.

When running a workflow script with preview all process executions are skipped.
This is useful to evaluate the pipeline execution without actually running.

Also when used along with `-with-dag` option it allows the rendering of the
execution DAG without running the pipeline.

Signed-off-by: Paolo Di Tommaso <[email protected]>
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.

3 participants