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

Helm chart 0.13.1 failed to install #4497

Closed
anhpngt opened this issue Jun 21, 2023 · 5 comments · Fixed by #4684
Closed

Helm chart 0.13.1 failed to install #4497

anhpngt opened this issue Jun 21, 2023 · 5 comments · Fixed by #4684
Assignees
Labels
area/continuous integration Related to CI and automated testing kind/bug Something isn't working

Comments

@anhpngt
Copy link
Contributor

anhpngt commented Jun 21, 2023

I was unable to install camel-k Helm chart v0.13.1 and encountered the following error

Helm release camel-k not installed. Installing...
Error: INSTALLATION FAILED: failed to install CRD crds/crd-binding.yaml: no objects visited
deploying "camel-k": install: exit status 1

It seems like this file has no objects, which is not accepted by Helm.

@squakez
Copy link
Contributor

squakez commented Jun 21, 2023

It is strange that it fails against that version as that one should be pointing to release 1.12.1 which does not contain such leftover value.

@squakez squakez added kind/bug Something isn't working area/installation Installation and Topology labels Jun 21, 2023
@alagusundarams
Copy link

Even with 1.12.1 the same issue occurs and after removing the empty crd-binding.yaml it worked.

@squakez
Copy link
Contributor

squakez commented Jun 22, 2023

Thanks. Then we may have 2 problems. The first one is that we need to remove that crd-binding.yaml from main. The second one is that the installation should use the CRDs provided for the specific version, not the ones in main. I am not very familiar with the helm installation procedure, but we must make sure that every version is bundled or use the correct CRD definitions.

squakez added a commit to squakez/camel-k that referenced this issue Jun 23, 2023
squakez added a commit to squakez/camel-k that referenced this issue Jun 23, 2023
@squakez
Copy link
Contributor

squakez commented Jun 23, 2023

I've applied a patch that should work as soon as the repos are synched. However I'm leaving this issue open because we should find an automated way to add the correct chart when releasing from release branches. Moreover, we should have an E2E test to verify that the chart is working as expected before releasing.

squakez added a commit that referenced this issue Jun 23, 2023
@squakez
Copy link
Contributor

squakez commented Jun 23, 2023

Fixed:

$ helm install my-camel-k camel-k/camel-k --version 0.13.1 --set platform.build.registry.address=my-registry.io
NAME: my-camel-k
LAST DEPLOYED: Fri Jun 23 12:20:55 2023
NAMESPACE: default
STATUS: deployed
REVISION: 1
TEST SUITE: None
NOTES:
Connect the world with Camel K on Kubernetes!

@squakez squakez added area/continuous integration Related to CI and automated testing and removed area/installation Installation and Topology labels Jun 26, 2023
realMartinez pushed a commit to realMartinez/camel-k that referenced this issue Jun 28, 2023
@squakez squakez self-assigned this Aug 21, 2023
squakez added a commit to squakez/camel-k that referenced this issue Aug 22, 2023
squakez added a commit to squakez/camel-k that referenced this issue Aug 22, 2023
squakez added a commit to squakez/camel-k that referenced this issue Aug 23, 2023
squakez added a commit to squakez/camel-k that referenced this issue Aug 23, 2023
squakez added a commit that referenced this issue Aug 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/continuous integration Related to CI and automated testing kind/bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants