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

Fission helm conflicts with Rancher #2920

Closed
sergeykorablin opened this issue Mar 11, 2024 · 1 comment · Fixed by #2948
Closed

Fission helm conflicts with Rancher #2920

sergeykorablin opened this issue Mar 11, 2024 · 1 comment · Fixed by #2948

Comments

@sergeykorablin
Copy link

Fission/Kubernetes version

$ fission version

$ kubectl version
Client Version: v1.28.2
Kustomize Version: v5.0.4-0.20230601165947-6ce0bf390ce3
Server Version: v1.27.11+k3s1

Kubernetes platform (e.g. Google Kubernetes Engine)

Describe the bug

$ helm install --version v1.20.1 --namespace $FISSION_NAMESPACE fission fission-charts/fission-all
            
Error: INSTALLATION FAILED: rendered manifests contain a resource that already exists. 
Unable to continue with install: MutatingWebhookConfiguration "mutating-webhook-configuration" in namespace "" exists and cannot be imported into the current release: invalid ownership metadata;
label validation error: missing key "app.kubernetes.io/managed-by": must be set to "Helm";
annotation validation error: missing key "meta.helm.sh/release-name": must be set to "fission";
annotation validation error: missing key "meta.helm.sh/release-namespace": must be set to "fission"

https://github.com/rancher/rancher/blob/268d751c68b264e6f50a23d360d4699ca131768d/pkg/crds/provisioningv2/capi-webhooks.yaml#L6
uses the same name for MutatingWebhookConfiguration

To Reproduce

  1. Install Rancher
helm install rancher rancher-latest/rancher \
  --namespace cattle-system \
  --set hostname=rancher.homelab.local \
  --set replicas=1 \
  --set bootstrapPassword=admin
  1. Try to install Fission
helm install --version v1.20.1 --namespace fission fission fission-charts/fission-all

Expected result

Actual result

Screenshots/Dump file

$ fission support dump

Additional context

@robert-cronin
Copy link
Contributor

robert-cronin commented May 27, 2024

This should just be a simple matter of choosing a unique name for the mutating webhook, perhaps I can give it a try? I think it is similar to #2825

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

Successfully merging a pull request may close this issue.

3 participants