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

add telemetry traffic metrics #20589

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

Conversation

AntonEbel
Copy link
Contributor

@AntonEbel AntonEbel commented Sep 30, 2024

/nocl
/jenkins-pr-deps Graylog2/graylog-plugin-enterprise#8765
fixes point 1 and 3 of Graylog2/graylog-plugin-enterprise#8312

@kodjo-anipah
Copy link
Contributor

kodjo-anipah commented Oct 2, 2024

@bernd @thll just wanted to double check with you guys if it's ok for us to store the actual written msg size traffic to DWH in the existing TrafficDTO on serverside. I know that Dennis added a TelemetryClient, which in theory would also give us a way to directly push information to posthog, but this seems more natural

@bernd
Copy link
Member

bernd commented Oct 2, 2024

@bernd @thll just wanted to double check with you guys if it's ok for us to store the actual written msg size traffic to DWH in the existing TrafficDTO on serverside. I know that Dennis added a TelemetryClient, which in theory would also give us a way to directly push information to posthog, but this seems more natural

@kodjo-anipah We can't add metrics for commercial features to the open server code. We either make this pluggable or report directly from the Enterprise code.

@kodjo-anipah
Copy link
Contributor

@bernd @thll just wanted to double check with you guys if it's ok for us to store the actual written msg size traffic to DWH in the existing TrafficDTO on serverside. I know that Dennis added a TelemetryClient, which in theory would also give us a way to directly push information to posthog, but this seems more natural

@kodjo-anipah We can't add metrics for commercial features to the open server code. We either make this pluggable or report directly from the Enterprise code.

Thought so sorry, maybe should have voiced that thought directly @AntonEbel . @tellistone fyi we can't do it like we discussed so we have to put in a bit more effort

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

Successfully merging this pull request may close these issues.

3 participants