sentry-kubernetes / charts

Easily deploy Sentry on your Kubernetes Cluster
MIT License
1.09k stars 514 forks source link

warning: skipped value for kafka.config: Not a table. #1370

Closed yujiaao closed 1 month ago

yujiaao commented 3 months ago

According to https://artifacthub.io/packages/helm/sentry/sentry I try install on windows wsl with Ubuntu 22.04.4 LTS.

$ microk8s helm install sentry sentry/sentry
coalesce.go:175: warning: skipped value for kafka.config: Not a table.
Error: INSTALLATION FAILED: failed post-install: job failed: DeadlineExceeded

I have try helm uninstall sentry then reinstall, but still failed.

Please, people, help me!

lbcd commented 3 months ago

Same for me. I can move forward by executing:

kubectl exec -it sentry-web-xxxxxxxx -n sentry -- sentry upgrade

But I have an error after login and configuring first page.

lbcd commented 3 months ago

You can try: helm upgrade sentry sentry/sentry

or / and doing all hook manualy (https://github.com/sentry-kubernetes/charts/tree/develop/charts/sentry/templates/hooks) :

And that works for me :).

lbcd commented 3 months ago

I get others problems when I try to log some issues in sentry with my application.

So finally I remove everything an I launch:

helm template sentry sentry/sentry -f values.yaml > sentry-manifest.yaml
kubectl apply -f sentry-manifest.yaml

I let some time to containers to start. I get some error on one time jobs (hooks) that doesn't wait containers to be ready. So I extract jobs from sentry-manifest.yaml to put them in single files and I start them manually one by one.

kubectl apply -f hook/sentry-db-check.yaml
kubectl apply -f hook/sentry-db-init.yaml
kubectl apply -f hook/sentry-snuba-migrate.yaml
kubectl apply -f hook/user-create.yaml

At the end I had an ingress :

apiVersion: networking.k8s.io/v1
kind: Ingress
metadata:
  name: sentry
  annotations:
    nginx.ingress.kubernetes.io/use-regex: "true"
    nginx.ingress.kubernetes.io/proxy-protocol-headers: "HTTP_X_FORWARDED_PROTO=https"
spec:
  ingressClassName: nginx
  rules:
  - host: sentry.xxxxxxx.com
    http:
      paths:
      - path: /api/store
        pathType: ImplementationSpecific
        backend:
          service:
            name: sentry-relay
            port:
              number: 3000
      - path: /api/[1-9][0-9]*/(.*)
        pathType: ImplementationSpecific
        backend:
          service:
            name: sentry-relay
            port:
              number: 3000
      - path: /
        pathType: Prefix
        backend:
          service:
            name: sentry-web
            port:
              number: 9000
  tls:
  - hosts:
    - sentry.xxxxxxx.com
    secretName: xxxxxxx-com-wild-card-2023-2024

And all works!

ivoistaken commented 3 months ago

just increase activeDeadlineSeconds in values.yaml (e.g. 1000, 2000, whatever works for you)

hooks:
  activeDeadlineSeconds: 1000
patsevanton commented 2 months ago

Add --timeout 3600s to helm install for fix issue

lbcd commented 2 months ago

just increase activeDeadlineSeconds in values.yaml (e.g. 1000, 2000, whatever works for you)

hooks:
  activeDeadlineSeconds: 1000

This work for me. Thank you @ivoistaken @patsevanton I had tried this --timeout 3600s to helm command but it doesn't work for me. By default I letf it on the command line but I'm pretty sur that hooks -> activeDeadlineSeconds is what make me success to install sentry.

Thank you for your help guys.

patsevanton commented 2 months ago

Work with increase timeout and activeDeadlineSeconds. I created a few commits in the main branch. You can see https://github.com/sentry-kubernetes/charts/commit/78de49b0f94633cf098aff320a79d7a48443b9a5 and https://github.com/sentry-kubernetes/charts/pull/1406

patsevanton commented 2 months ago

Issue fixed in 25.2.0. Please close the issue.

patsevanton commented 1 month ago

@Mokto Please close issue

vld1234 commented 1 month ago

Issue fixed in 25.2.0. Please close the issue.

Have the same issue. It doesn't help.

patsevanton commented 1 month ago

@vld1234 What's the problem? What's the mistake?

vld1234 commented 1 month ago

@vld1234 What's the problem? What's the mistake?

coalesce.go:237: warning: skipped value for kafka.config: Not a table.

patsevanton commented 1 month ago

the discussion in this issue was about: "Error: INSTALLATION FAILED: failed post-install: job failed: DeadlineExceeded"

write a new issue for "coalesce.go:237: warning: missing value for kafka.config: Is not a table".