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

Duplicated trait set as modeline and kamel run parameters causes error #2466

Closed
claudio4j opened this issue Jun 29, 2021 · 0 comments · Fixed by #2467
Closed

Duplicated trait set as modeline and kamel run parameters causes error #2466

claudio4j opened this issue Jun 29, 2021 · 0 comments · Fixed by #2467

Comments

@claudio4j
Copy link
Contributor

executing this command:

kamel run --name metrics --trait prometheus.enabled=true  Metrics.java -w --dependency mvn:com.github.openshift-integration:camel-k-example-metrics:1.0.0-SNAPSHOT

I got this message:

Modeline options have been loaded from source files
Full command: kamel run --name metrics --trait prometheus.enabled=true Metrics.java -w --dependency mvn:com.github.openshift-integration:camel-k-example-metrics:1.0.0-SNAPSHOT --trait=prometheus.enabled=true 
Error: 1 error(s) decoding:

* 'enabled' expected type 'bool', got unconvertible type '[]string'

Metrics.java is taken from: https://github.com/apache/camel-k/blob/main/e2e/yaks/openshift/monitoring/Metrics.java .
It seems the "enabled" property is both inferred from the integration and got from command line, causing a sort of double specification error ("type '[]string'").
claudio4j added a commit to claudio4j/camel-k that referenced this issue Jun 29, 2021
@claudio4j
fix(cli) Duplicated trait set as modeline and kamel run parameters ca…
5a7a99a 
…uses error

Fix apache#2466
@claudio4j claudio4j mentioned this issue Jun 29, 2021
Merged
claudio4j added a commit to claudio4j/camel-k that referenced this issue Jul 1, 2021
@claudio4j
fix(cli) Duplicated trait set as modeline and kamel run parameters ca…
969cf51 
…uses error

Fix apache#2466
claudio4j added a commit to claudio4j/camel-k that referenced this issue Jul 1, 2021
@claudio4j
fix(cli) Duplicated trait set as modeline and kamel run parameters ca…
72895eb 
…uses error

Fix apache#2466
claudio4j added a commit to claudio4j/camel-k that referenced this issue Jul 1, 2021
@claudio4j
fix(cli) Duplicated trait set as modeline and kamel run parameters ca…
c8f733e 
…uses error

Fix apache#2466
claudio4j added a commit to claudio4j/camel-k that referenced this issue Jul 1, 2021
@claudio4j
fix(cli) Duplicated trait set as modeline and kamel run parameters ca…
6ef892b 
…uses error

Fix apache#2466
claudio4j added a commit to claudio4j/camel-k that referenced this issue Jul 1, 2021
@claudio4j
fix(cli) Duplicated trait set as modeline and kamel run parameters ca…
ea71f89 
…uses error

Fix apache#2466
claudio4j added a commit to claudio4j/camel-k that referenced this issue Jul 1, 2021
@claudio4j
fix(cli) Duplicated trait set as modeline and kamel run parameters ca…
de68e7e 
…uses error

Fix apache#2466
claudio4j added a commit to claudio4j/camel-k that referenced this issue Jul 5, 2021
@claudio4j
fix(cli) Duplicated trait set as modeline and kamel run parameters ca…
fadd66a 
…uses error

Fix apache#2466
nicolaferraro pushed a commit that referenced this issue Jul 9, 2021
@claudio4j @nicolaferraro
fix(cli) Duplicated trait set as modeline and kamel run parameters ca…
17e0913 
…uses error

Fix #2466
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix(cli) Duplicated trait set as modeline and kamel run parameters causes error claudio4j/camel-k
1 participant
@claudio4j