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

Subscribing Webhook fails despite Webhook pods are running #2282

Closed
steffh opened this issue Aug 12, 2021 · 0 comments · Fixed by #2284
Closed

Subscribing Webhook fails despite Webhook pods are running #2282

steffh opened this issue Aug 12, 2021 · 0 comments · Fixed by #2284
Assignees

Comments

@steffh
Copy link
Collaborator

steffh commented Aug 12, 2021

Describe the bug

As an Airy core user running version 0.28.0,
having the webhook component defined and the configuration applied,
an authenticated request to /webhooks.subscribe returns 409 - { "message": "The webhook component needs to be enabled. Learn more: https://airy.co/docs/core/api/webhook" }
despite the relevant webhook pods are running

To Reproduce

  1. Apply configuration for airy.yaml:
kubernetes:
  appImageTag: 0.28.0
  containerRegistry: ghcr.io/airyhq
  namespace: default
  ngrokEnabled: "false"
  host: SOMEHOST
  https: ""
  letsencryptemail: ""
security:
  allowedOrigins: '*'
  systemToken: "SOMETOKEN"
  jwtSecret: SOMESECRET
components:
  integration:
    webhook:
      name: somename
      maxBackoff: 10
  1. Checking if the pods are running:
[~/airy-aws] $kubectl get pods                               
NAME                                    READY   STATUS      RESTARTS   AGE
airy-controller-68584474fc-jn9wt        1/1     Running     0          2m33s
api-admin-77fc5569d6-wwz25              1/1     Running     0          6s
api-communication-5cbcc6cfb8-84295      1/1     Running     0          2m34s
api-websocket-bbc64484-m5bjb            1/1     Running     0          3m52s
beanstalkd-0                            2/2     Running     0          66d
frontend-chat-plugin-79db79cbdc-7pg8l   1/1     Running     0          9m47s
frontend-ui-d4494c76b-n2qgb             1/1     Running     0          9m45s
helm-runner-cck9z                       0/1     Completed   0          66d
kafka-0                                 1/1     Running     0          66d
provisioning-topics-k9zdf               0/1     Completed   0          66d
schema-registry-6c48f6f9f6-l7j5j        1/1     Running     0          66d
sources-chatplugin-cd78655dd-5pml4      1/1     Running     0          3m51s
wait-for-api-communication-p9nmg        0/1     Completed   0          66d
wait-for-frontend-chat-plugin-28rx9     0/1     Completed   0          66d
wait-for-frontend-ui-r8fzz              0/1     Completed   0          66d
webhook-consumer-7c9d9d8b98-xvwcc       1/1     Running     0          6m25s
webhook-publisher-575696bcb8-62m48      1/1     Running     0          6m25s
zookeeper-0                             1/1     Running     0          66d
  1. Send an authenticated request to /webhooks.subscribe:

Screenshot 2021-08-12 at 18 15 30

Expected behavior

The request should return HTTP 200 and the webhook pod should start sending events.

Environment:

  • Cloud: AWS
@steffh steffh added the bug label Aug 12, 2021
@chrismatix chrismatix self-assigned this Aug 13, 2021
@chrismatix chrismatix added this to To do in Engineering WIP via automation Aug 13, 2021
@chrismatix chrismatix moved this from To do to In progress in Engineering WIP Aug 13, 2021
Engineering WIP automation moved this from In progress to Done Aug 13, 2021
chrismatix added a commit that referenced this issue Aug 13, 2021
* [#2181] Hotfix: Chatplugin installation crashing without config

* [#2187] Hotfix chatplugin bundle loading (#2188)

* [#2192] Hotfix inbox crashing when selecting conversations in filtered view (#2193)

* Inbox UI carousels facebook break the UI (#2222)

* fix scroll

* unnecessary space

* bump version

* [#2282] Hotfix for webhook registration failing despite all services healthy

* update changelog and version

Co-authored-by: ljupcovangelski <ljupco@airy.co>
Co-authored-by: Aitor Algorta <aitor@airy.co>
Co-authored-by: Pascal Holy <pascal@airy.co>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

Successfully merging a pull request may close this issue.

2 participants