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

kamel --output option should not need deploy resources to cluster #886

Closed
chirino opened this issue Jul 29, 2019 · 1 comment · Fixed by #1876
Closed

kamel --output option should not need deploy resources to cluster #886

chirino opened this issue Jul 29, 2019 · 1 comment · Fixed by #1876
Labels
area/core Core features of the integration platform area/ux Improves user experience good first issue Does not require full understanding of the codebase
Milestone
1.4.0

Comments

@chirino
Copy link
Contributor

chirino commented Jul 29, 2019

running:

oc delete crd -l app=camel-k
kamel install --cluster-setup -o yaml

results in:

Error: cannot check CRD installation after 25 seconds

From my point of view, --output should not actually be installed or assume the CRDs are installed in the cluster. Someone might do this to get the k8s config into a file they send to an admin that wants to manually install the resources.
@nicolaferraro nicolaferraro mentioned this issue Jul 29, 2019
Closed
@nicolaferraro nicolaferraro added area/core Core features of the integration platform area/ux Improves user experience labels Jul 29, 2019
@nicolaferraro nicolaferraro added this to the 1.0.0 milestone Jul 29, 2019
@nicolaferraro nicolaferraro added the good first issue Does not require full understanding of the codebase label Mar 16, 2020
@nicolaferraro nicolaferraro modified the milestones: 1.0.0, post 1.0.0 May 26, 2020
@nicolaferraro nicolaferraro modified the milestones: 1.3.0, 1.4.0 Dec 22, 2020
@squakez
Copy link
Contributor

squakez commented Jan 5, 2021

I can have a look at this if nobody else is working on it.

squakez added a commit to squakez/camel-k that referenced this issue Jan 5, 2021
@squakez
Skip CRD installation if output flag enabled
cf000e0 
Closes apache#886
@squakez squakez mentioned this issue Jan 5, 2021
Merged
nicolaferraro pushed a commit that referenced this issue Jan 11, 2021
@squakez @nicolaferraro
Skip CRD installation if output flag enabled
0e3fa94 
Closes #886
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area/core Core features of the integration platform area/ux Improves user experience good first issue Does not require full understanding of the codebase
Projects
None yet
Milestone
1.4.0
Development

Successfully merging a pull request may close this issue.

Skip CRD installation if output flag enabled squakez/camel-k
3 participants
@chirino @nicolaferraro @squakez