Skip to content

Commit

Permalink
Fix the copy result issue when it's already tekton result path (kubef…
Browse files Browse the repository at this point in the history
  • Loading branch information
pugangxa authored Mar 11, 2021
1 parent 19c779b commit 7c1e4ec
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions sdk/python/kfp_tekton/compiler/_op_to_template.py
Original file line number Diff line number Diff line change
Expand Up @@ -246,6 +246,8 @@ def _process_parameters(processed_op: BaseOp,
need_copy_step = False
args.append(a)
s['args'] = args
if path == '/tekton/results/' + sanitize_k8s_name(name):
need_copy_step = False
# If file output path cannot be found/replaced, use emptyDir to copy it to the tekton/results path
if need_copy_step:
copy_results_step['script'] = copy_results_step['script'] + 'cp ' + path + ' $(results.%s.path);' \
Expand Down

0 comments on commit 7c1e4ec

Please sign in to comment.