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

waitFor exits when nested task is marked as a Warning #6031

Open
janorga opened this issue Nov 21, 2024 · 0 comments
Open

waitFor exits when nested task is marked as a Warning #6031

janorga opened this issue Nov 21, 2024 · 0 comments
Labels
area/backend Needs backend code changes bug Something isn't working

Comments

@janorga
Copy link

janorga commented Nov 21, 2024

Describe the issue

WaitFor loop exits when a nested task in the loop exits with a Warning status, no matter the "waitfor" condition:

You can test it with:

id: myflow
namespace: mynamespace
tasks:
  - id: loop
    type: io.kestra.plugin.core.flow.WaitFor
    condition: "{{ outputs.check_migration_task.vars['status'] == 'FINISHED' }}"
    failOnMaxReached: true
    checkFrequency: 
      interval: PT5S
      maxDuration: PT135M
    tasks:

     ## forcing a Warning
    - id: allow_failure
      type: io.kestra.plugin.scripts.shell.Commands
      taskRunner:
        type: io.kestra.plugin.core.runner.Process
      allowFailure: true
      commands:
        - exit 1

    - id: check_migration_task
      type: io.kestra.plugin.scripts.shell.Commands
      taskRunner:
        type: io.kestra.plugin.scripts.runner.docker.Docker
        entryPoint: []
      containerImage: "ubuntu:latest"
      commands:
        - "echo '::{\"outputs\":{\"status\": \"INPROGRESS\"}}::'"

Expected behavior

WaitFor should only auto-exit on Failure status on a nested task but not with other status for a nested task. So a Warning in a nested task should keep on with the loop and evaluating the "waitfor" condition

Environment

  • Kestra Version: 0.19.11
  • Kubernetes deployment
@janorga janorga added area/backend Needs backend code changes area/frontend Needs frontend code changes bug Something isn't working labels Nov 21, 2024
@github-project-automation github-project-automation bot moved this to Backlog in Issues Nov 21, 2024
@MilosPaunovic MilosPaunovic removed the area/frontend Needs frontend code changes label Nov 21, 2024
@loicmathieu loicmathieu removed their assignment Nov 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/backend Needs backend code changes bug Something isn't working
Projects
Status: Backlog
Development

No branches or pull requests

3 participants