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

update agent telemetry url to be cloud function instead of posthog #15085

Merged
merged 6 commits into from May 29, 2023

Conversation

andrewm4894
Copy link
Contributor

@andrewm4894 andrewm4894 commented May 23, 2023

Summary

Replace posthog with a gcp cloud function for agent events that do not need to be in posthog.

  • update url for agent events from daemon\anonymous-statistics.sh.in
  • update url for agent install events from packaging\installer\kickstart.sh
  • update docs where relevant
Test Plan

Validate that events are flowing in as expected (into BigQuery via the cloud function) from agents running this feature branch.

Additional Information
For users: How does this change affect me?

internal slack thread with some more info and background

@github-actions github-actions bot added area/docs area/packaging Packaging and operating systems support labels May 23, 2023
@andrewm4894 andrewm4894 marked this pull request as ready for review May 23, 2023 12:43
@andrewm4894 andrewm4894 changed the title update telemetry url to be cloud function instead of posthog update agent telemetry url to be cloud function instead of posthog May 23, 2023
@andrewm4894 andrewm4894 self-assigned this May 23, 2023
Copy link
Contributor

@thiagoftsm thiagoftsm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

After a chat with @andrewm4894 we could confirm PR is working as expected, LGTM!

@andrewm4894
Copy link
Contributor Author

@netdata/agent-sre are some of the checks here stuck perhaps?

@andrewm4894
Copy link
Contributor Author

I am going to wait until Monday to merge this so can watch it close once its merged.

@andrewm4894 andrewm4894 merged commit 430c0ec into netdata:master May 29, 2023
125 checks passed
@andrewm4894 andrewm4894 deleted the update-agent-telemetry-url branch May 29, 2023 07:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/daemon area/docs area/packaging Packaging and operating systems support
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants