Skip to content

fix: argument list to long error for large DAGs. Fixes #12453#12737

Open
AnaySh wants to merge 1 commit intokubeflow:masterfrom
AnaySh:fix-argument-list-too-long
Open

fix: argument list to long error for large DAGs. Fixes #12453#12737
AnaySh wants to merge 1 commit intokubeflow:masterfrom
AnaySh:fix-argument-list-too-long

Conversation

@AnaySh
Copy link
Contributor

@AnaySh AnaySh commented Jan 30, 2026

Description of your changes:

Fixes #12453 along with the PR to argo-workflows, argoproj/argo-workflows#15265.

Checklist:

@github-actions
Copy link

🎉 Welcome to the Kubeflow Pipelines repo! 🎉

Thanks for opening your first PR! We're excited to have you onboard 🚀

Next steps:

Feel free to ask questions in the comments.

@google-oss-prow
Copy link

Hi @AnaySh. Thanks for your PR.

I'm waiting for a kubeflow member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@droctothorpe
Copy link
Collaborator

/ok-to-test

@AnaySh AnaySh changed the title fix: argument list to long error for large dags fix: argument list to long error for large dags. Fixes #12453 Feb 2, 2026
@AnaySh AnaySh changed the title fix: argument list to long error for large dags. Fixes #12453 fix: argument list to long error for large DAGs. Fixes #12453 Feb 2, 2026
@AnaySh AnaySh force-pushed the fix-argument-list-too-long branch 2 times, most recently from 71c6e6e to 17b5cb7 Compare February 6, 2026 15:30
@droctothorpe
Copy link
Collaborator

/approve
/lgtm

Great work, @AnaySh!

@google-oss-prow
Copy link

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: droctothorpe

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Signed-off-by: Anay Sharma <anay20sharma11@gmail.com>
@droctothorpe droctothorpe force-pushed the fix-argument-list-too-long branch from 17b5cb7 to 193d532 Compare February 6, 2026 15:53
@google-oss-prow google-oss-prow bot removed the lgtm label Feb 6, 2026
@google-oss-prow
Copy link

New changes are detected. LGTM label has been removed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[bug] Large DAGs fail to run, hit argoexec argument limit

3 participants