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

Sublfow task validator render bug #7271

Closed
ReLoutre opened this issue Feb 10, 2025 · 1 comment · Fixed by #7294
Closed

Sublfow task validator render bug #7271

ReLoutre opened this issue Feb 10, 2025 · 1 comment · Fixed by #7294
Assignees
Labels
area/backend Needs backend code changes area/frontend Needs frontend code changes bug Something isn't working kind/backport-needed Issue that needs to be backported to a previous version

Comments

@ReLoutre
Copy link

Describe the issue

Image

Flow validator do not render the dynamic output and so is unable validate successfully the yaml.
Yet on my instance it does execute successfully anyway and the error doesn't prevent me to save using CTRL-S.

Here's a really simple reproducer :

id: subflow_dynamic_property_bug_reproducer
namespace: iut.cri.bug_reproducers

tasks:
  - id: call_subflow_with_pebble_namespace
    type: io.kestra.plugin.core.flow.Subflow
    flowId: some_subflow
    namespace: "{{ flow.namespace }}"

With its called dummy sublfow :

id: some_subflow
namespace: iut.cri.bug_reproducers

tasks:
  - id: log
    type: io.kestra.plugin.core.log.Log
    message: hello world

Executing this reproducer does indeed render and execute successfully the sub flow :

Image

Environment

  • Kestra Version: 0.21.1
@ReLoutre ReLoutre added area/backend Needs backend code changes area/frontend Needs frontend code changes bug Something isn't working labels Feb 10, 2025
@github-project-automation github-project-automation bot moved this to Backlog in Issues Feb 10, 2025
@anna-geller
Copy link
Member

acc to Yann: "we now check if flow does exists, but as its pebble we can not validate it, I'll find a trick"

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 area/frontend Needs frontend code changes bug Something isn't working kind/backport-needed Issue that needs to be backported to a previous version
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

3 participants