Cron: don't create minutely workflow on spec error

tooptoop4 commented 2 weeks ago

Pre-requisites

[X] I have double-checked my configuration
[X] I have tested with the :latest image tag (i.e. quay.io/argoproj/workflow-controller:latest) and can confirm the issue still exists on :latest. If not, I have explained why, in detail, in my description below.
[X] I have searched existing issues and could not find a match for this bug
[ ] I'd like to contribute the fix myself (see contributing guide)

What happened? What did you expect to happen?

if we push new cron workflow with some errors, argo could replace it with default one dummy workflow which will run every minute

https://github.com/argoproj/argo-workflows/blob/ba75efb228fa3fbab91e2b1c7689f6db3f3c3b62/ui/src/app/shared/examples.ts#L79-L87

We need option to change this default workflow to some invalid one, to prevent unnecessary dummy jobs run.

this is noted in a comment of the 3.6 blog post but i have also noticed this

Version(s)

3.5.11

Paste a minimal workflow that reproduces the issue. We must be able to run the workflow; don't enter a workflows that uses private images.

n/a

Logs from the workflow controller

kubectl logs -n argo deploy/workflow-controller | grep ${workflow}

Logs from in your workflow's wait container

kubectl logs -n argo -c wait -l workflows.argoproj.io/workflow=${workflow},workflow.argoproj.io/phase!=Succeeded

Joibel commented 2 weeks ago

Could you provide further information on the exact steps to reproduce this? I tried:

Create a bad CronWorkflow using kubectl it does not replace it with the one you linked.
Create a bad CronWorkflow using argo cli it just reports it as bad, does not submit it and does not replace it
Create a bad CronWorkflow using the web userinterface it reports it as bad in the user interface and does not submit it and does not replace it

github-actions[bot] commented 1 day ago

This issue has been automatically marked as stale because it has not had recent activity and needs more information. It will be closed if no further activity occurs.

tooptoop4 commented 1 day ago

🦀

agilgur5 commented 1 day ago

run every minute

This is also intentional for an example, so that you can easily see it run due to its quick schedule

argoproj / argo-workflows