Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: ignore error when input artifacts optional. Fixes #13564 #13567

Merged

Conversation

shuangkun
Copy link
Member

@shuangkun shuangkun commented Sep 6, 2024

Fixed the problem of workflow failure when using optional Artifacts and containing subpaths and loops.

Fixes: #13564

Motivation

Modifications

Verification

Co-authored-by: shuangkun <tsk2013uestc@163.com>
Co-authored-by: sherwinkoo29 <sherwinkoo@163.com>
Signed-off-by: shuangkun <tsk2013uestc@163.com>
@terrytangyuan terrytangyuan merged commit b375647 into argoproj:main Sep 6, 2024
28 checks passed
@agilgur5 agilgur5 added the area/artifacts S3/GCP/OSS/Git/HDFS etc label Sep 6, 2024
@agilgur5 agilgur5 added this to the v3.5.x patches milestone Sep 6, 2024
Joibel pushed a commit to pipekit/argo-workflows that referenced this pull request Sep 19, 2024
…argoproj#13567)

Signed-off-by: shuangkun <tsk2013uestc@163.com>
Co-authored-by: sherwinkoo29 <sherwinkoo@163.com>
Joibel pushed a commit that referenced this pull request Sep 20, 2024
Signed-off-by: shuangkun <tsk2013uestc@163.com>
Co-authored-by: sherwinkoo29 <sherwinkoo@163.com>
@agilgur5 agilgur5 changed the title fix: ignore error when input artifacts optional. Fixes:#13564 fix: ignore error when input artifacts optional. Fixes #13564 Sep 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/artifacts S3/GCP/OSS/Git/HDFS etc
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Workflow failed when using optional subpath in withParam
3 participants