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

Support set integration name for kamel run #40

Closed
lburgazzoli opened this Issue Sep 11, 2018 · 4 comments

Comments

Projects
None yet
3 participants
@lburgazzoli
Contributor

lburgazzoli commented Sep 11, 2018

the kamel run command determine the integration name using the integration file name but it would be nice to let the user set the name of the integration with an additional flag.

@lburgazzoli

This comment has been minimized.

Show comment
Hide comment
@lburgazzoli

lburgazzoli Sep 11, 2018

Contributor

@oscerd do you mind having a look ?

Contributor

lburgazzoli commented Sep 11, 2018

@oscerd do you mind having a look ?

@nicolaferraro

This comment has been minimized.

Show comment
Hide comment
@nicolaferraro

nicolaferraro Sep 11, 2018

Contributor

I'll add also that the current mapping is trivial, e.g. MyIntegration.java -> myintegration.

A better name would be my-integration.

We should also make sure to respect Kubernetes conventions.

Contributor

nicolaferraro commented Sep 11, 2018

I'll add also that the current mapping is trivial, e.g. MyIntegration.java -> myintegration.

A better name would be my-integration.

We should also make sure to respect Kubernetes conventions.

@lburgazzoli

This comment has been minimized.

Show comment
Hide comment
@lburgazzoli

lburgazzoli Sep 11, 2018

Contributor

Thre is also a problem when the file is located on a different directory than the current one i.e. runtime/examples/Sample.java becomes runtimeexamplessample

Contributor

lburgazzoli commented Sep 11, 2018

Thre is also a problem when the file is located on a different directory than the current one i.e. runtime/examples/Sample.java becomes runtimeexamplessample

@oscerd

This comment has been minimized.

Show comment
Hide comment
@oscerd

oscerd Sep 12, 2018

Contributor

Sure

Contributor

oscerd commented Sep 12, 2018

Sure

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment