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

Allow workflows/cronworkflows to be resubmitted #18

Open
iameskild opened this issue Jul 21, 2023 · 8 comments
Open

Allow workflows/cronworkflows to be resubmitted #18

iameskild opened this issue Jul 21, 2023 · 8 comments

Comments

@iameskild
Copy link
Member

Currently there is a known limitation which prevents us from resubmitting workflows/cronworkflows. As such, this is preventing the Argo-Jupyter-Scheduler from being able to Resume paused cron-workflows and prevents cron-workflows from being updated (ie to run on a new schedule).

@dharhas
Copy link
Member

dharhas commented Jul 24, 2023

@iameskild When you have a moment, can you add details or point to an upstream issue that defines why this is? i.e. is this a limitation of Argo or an implementation issue etc.

@iameskild
Copy link
Member Author

@dharhas as far as a I can tell, this is a limitation of the Nebari-Workflow-Controller ( https://github.com/nebari-dev/nebari-workflow-controller#known-limitations ). Resubmitting workflows fail in general when using NWC.

@dharhas
Copy link
Member

dharhas commented Jul 24, 2023

Yes. But there are no details on why.

@iameskild
Copy link
Member Author

I'm not sure why this is a limitation on the NWC side. @Adam-D-Lewis if you have more information, do you mind chiming in?

@Adam-D-Lewis
Copy link
Member

Adam-D-Lewis commented Aug 2, 2023

Originally, I didn't support resubmission because I wasn't sure how to get the username from the creator-preferred-username label, but I think Eskild added a way to get the username from the creator-preferred-username, but I need to check that his solution applies here.

@Adam-D-Lewis
Copy link
Member

Adam-D-Lewis commented Aug 3, 2023

So I checked and unfortunately, the creator-preferred-username set by argo just strips off unsupported characters like @ and ! in the username so getting the creator on a resubmitted workflow is not possible.

It looks like there is a fix that was just committed 2 weeks ago and might be in the latest version of Argo Workflows if we want to upgrade (It should be in the v3.5 release of argo worfklows). With the fix we could just look at the creator label instead of the creator-preferred-username.
argoproj/argo-workflows#11414

@ericdatakelly
Copy link

I'm working on the doc for jupyter-scheduler and followed the warning to this issue. I see that argo-workflows is up to v3.5.6. Will this be fixed soon or should I leave the warning unmodified? https://www.nebari.dev/docs/tutorials/jupyter-scheduler#submit-jupyter-notebooks-to-run-on-a-schedule

@Adam-D-Lewis
Copy link
Member

Adam-D-Lewis commented May 23, 2024

@ericdatakelly Updating Argo workflows is desired, but not scheduled at the moment. We would accept a PR doing so, but we would need to update Jupyter scheduler and argo-workflow-controller as well before merging.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants