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.

Sign up for GitHub

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

Jump to bottom

Parameters in YAML DSL are not recognized #2441

Closed
nicolaferraro opened this issue Jun 24, 2021 · 0 comments · Fixed by #2442
Closed

Parameters in YAML DSL are not recognized #2441

nicolaferraro opened this issue Jun 24, 2021 · 0 comments · Fixed by #2442
Assignees
@nicolaferraro
Labels
kind/bug Something isn't working
Milestone
1.5.0

Comments

@nicolaferraro
Copy link
Member

nicolaferraro commented Jun 24, 2021 •
edited
Loading

And this makes traits such as cron and knative to not work as expected, because the operator skips them.

So, e.g. this works:

- from:
    uri: "cron:tab?schedule=*+*+*+*+?"
# ...

But this doesn't:

- from:
    uri: "cron:tab"
    parameters:
      schedule: "* * * * ?"
# ...

Even if they are equivalent at runtime level, they are not at operator side.
@nicolaferraro nicolaferraro added the kind/bug Something isn't working label Jun 24, 2021
@nicolaferraro nicolaferraro added this to the 1.5.0 milestone Jun 24, 2021
@nicolaferraro nicolaferraro self-assigned this Jun 24, 2021
nicolaferraro added a commit to nicolaferraro/camel-k that referenced this issue Jun 24, 2021
@nicolaferraro
Fix apache#2441: parse parameters in YAML DSL
323909c
@nicolaferraro nicolaferraro mentioned this issue Jun 24, 2021
Merged
nicolaferraro added a commit that referenced this issue Jun 24, 2021
@nicolaferraro
Fix #2441: parse parameters in YAML DSL
287dce0
nicolaferraro added a commit to nicolaferraro/camel-k that referenced this issue Jun 24, 2021
@nicolaferraro
Fix apache#2441: parse parameters in YAML DSL
8889764
nicolaferraro added a commit to jboss-fuse/camel-k that referenced this issue Jun 24, 2021
@nicolaferraro
Merge pull request #42 from nicolaferraro/fix-yaml-params
10d0362 
Backport Fix apache#2441: parse parameters in YAML DSL
@nicolaferraro nicolaferraro mentioned this issue Jul 2, 2021
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@nicolaferraro nicolaferraro

Labels
kind/bug Something isn't working
Projects
None yet
Milestone
1.5.0
Development

Successfully merging a pull request may close this issue.

Fix #2441: parse parameters in YAML DSL nicolaferraro/camel-k
1 participant
@nicolaferraro