kfp-tekton icon indicating copy to clipboard operation
kfp-tekton copied to clipboard

optimize step-copy-results-artifacts script to minimize etcd space

Open Tomcli opened this issue 2 years ago • 3 comments

/kind feature

Description: step-copy-results-artifacts script has too many variable that get stringify during compile time. We need to rewrite it in a way that it takes a variable as the inputs for source and destination path instead of raw string

Additional information: [Miscellaneous information that will assist in solving the issue.]

Tomcli avatar Jul 19 '22 17:07 Tomcli

related #997

Tomcli avatar Jul 19 '22 17:07 Tomcli

cc @ScrapCodes @yhwang not sure is one of you have time to take a look and reduce the script size in the generated yaml? There was few changes made into the big data passing and now the generated step-copy-results-artifacts script is getting way too big.

Tomcli avatar Jul 19 '22 22:07 Tomcli

Mainly the past few PRs are from different contributors are duplicating the src and dst python variables into raw strings. So we want to rewrite it in a way that to use Tekton variables to reduce the final yaml size.

Tomcli avatar Jul 19 '22 22:07 Tomcli

/assign

Tomcli avatar Aug 16 '22 21:08 Tomcli