nv-morpheus / Morpheus

Morpheus SDK
Apache License 2.0
344 stars 128 forks source link

[FEA]: Make the `Project Automation` workflows reusable #1224

Open jarmak-nv opened 1 year ago

jarmak-nv commented 1 year ago

Is this a new feature, an improvement, or a change to existing functionality?

Improvement

How would you describe the priority of this feature request

Medium

Please provide a clear description of problem this feature solves

Right now only the nv-morpheus/morpheus repo benefits from the project automations. Rather than copy-pasting the same automations into each repo and needing to maintain them separately, we should create a reusable workflow and then call it in each repo.

Describe your ideal solution

The nv-morpheus/utilities repo gets the reusable workflows and each repo calls them

Additional context

The nice thing is there's no logic change between the workflows at all, so we can make the existing env variables into default input variables and not even need to supply them in the caller workflows.

Code of Conduct

jarmak-nv commented 1 year ago

Suggestion from @cwharris - open a new repo for reusable workflows to prevent utilities from getting too cluttered and having a focused location for these, and potential future reusable workflows.

I like this approach for his reasons, and because it does feel awk having utilities be the only repo that both houses and calls the workflows.