feat: inbound metrics #678
Conversation
| @@ -11,3 +11,7 @@ camunda.operate.client.url=http://localhost:8081 | |||
| camunda.operate.client.username=demo | |||
| camunda.operate.client.password=demo | |||
| camunda.connector.webhook.enabled=true | |||
|
|
|||
| management.server.port=9080 | |||
There was a problem hiding this comment.
FYI: we will also need to update Helm charts: https://github.com/camunda/camunda-platform-helm/blob/main/charts/camunda-platform/templates/connectors/deployment.yaml#L131
Created a task here so that we don't forget.
I'll fix it closer to release or when I have time. Alternatively, you can fix that too, and I can help you with testing.
There was a problem hiding this comment.
Thanks, good point. I will try to fix this if I have the time until I'm off, otherwise I will let you know.
There was a problem hiding this comment.
@chillleader Do we have to introduce this change?
There was a problem hiding this comment.
No, not really. If it makes us change the Helm charts, we can leave it as is.
There was a problem hiding this comment.
@chillleader ok, just maybe don't commit on much time. For me it will take ~2-3 hours.
Also, camunda-platform will also need this change reflected. https://github.com/camunda/camunda-platform/blob/main/docker-compose.yaml#L151
There was a problem hiding this comment.
I removed this port change altogether so that we don't have to change the Helm charts.
Description
In this PR I added new metrics for inbound connectors and slightly refined the existing metrics.
activatedmetric related to connector trigger events.Grafana dashboard will follow in another PR.
Related issues
closes https://github.com/camunda/team-connectors/issues/290