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

feat(telemetry)_: include device type in metrics #5669

Merged
merged 2 commits into from
Aug 30, 2024

Conversation

adklempner
Copy link
Contributor

@adklempner adklempner commented Aug 7, 2024

Initializes telemetry client with device type and includes in all calls to telemetry server.

At the end of injectAccountsIntoWakuService, finds the installation based on installation ID set in messenger, and calls Waku service to set it in the telemetry client. Does nothing if the telemetry client is not set.

Important changes:

Closes #

@adklempner adklempner force-pushed the feat/telemetry-device branch from c64f5d3 to 348e9f2 Compare August 7, 2024 04:20
@adklempner adklempner changed the title Feat/telemetry device feat(telemetry)_: include device type in metrics Aug 7, 2024
@status-im-auto
Copy link
Member

✔️ status-go/prs/tests-rpc/PR-5669#1 🔹 ~2 min 51 sec 🔹 c64f5d3 🔹 📦 tests-rpc package

@adklempner adklempner marked this pull request as ready for review August 7, 2024 04:24
@status-im-auto
Copy link
Member

status-im-auto commented Aug 7, 2024

Jenkins Builds

Click to see older builds (35)
Commit #️⃣ Finished (UTC) Duration Platform Result
✔️ c64f5d3 #1 2024-08-07 04:22:28 ~2 min tests-rpc 📄log
✔️ c64f5d3 #1 2024-08-07 04:24:11 ~4 min linux 📦zip
✔️ c64f5d3 #1 2024-08-07 04:24:39 ~4 min ios 📦zip
✔️ c64f5d3 #1 2024-08-07 04:25:29 ~5 min android 📦aar
✖️ c64f5d3 #1 2024-08-07 04:26:11 ~6 min tests 📄log
✔️ 348e9f2 #2 2024-08-07 04:25:13 ~2 min tests-rpc 📄log
✔️ 348e9f2 #2 2024-08-07 04:26:42 ~2 min linux 📦zip
✔️ 348e9f2 #2 2024-08-07 04:27:56 ~2 min android 📦aar
✔️ 348e9f2 #2 2024-08-07 04:27:59 ~2 min ios 📦zip
✖️ 348e9f2 #2 2024-08-07 04:32:16 ~5 min tests 📄log
✔️ 8ec4005 #3 2024-08-07 05:15:05 ~1 min android 📦aar
✔️ 8ec4005 #3 2024-08-07 05:15:40 ~2 min linux 📦zip
✔️ 8ec4005 #3 2024-08-07 05:15:40 ~2 min tests-rpc 📄log
✔️ 8ec4005 #3 2024-08-07 05:16:23 ~2 min ios 📦zip
✖️ 8ec4005 #3 2024-08-07 05:19:15 ~5 min tests 📄log
✔️ c44f9fc #4 2024-08-07 06:13:35 ~2 min android 📦aar
✔️ c44f9fc #4 2024-08-07 06:13:43 ~2 min tests-rpc 📄log
✔️ c44f9fc #4 2024-08-07 06:13:46 ~2 min linux 📦zip
✔️ c44f9fc #4 2024-08-07 06:14:11 ~2 min ios 📦zip
✔️ c44f9fc #4 2024-08-07 06:55:18 ~43 min tests 📄log
✔️ 4c5f52e #5 2024-08-07 19:31:19 ~2 min tests-rpc 📄log
✔️ 4c5f52e #5 2024-08-07 19:31:20 ~2 min android 📦aar
✔️ 4c5f52e #5 2024-08-07 19:31:32 ~2 min linux 📦zip
✔️ 4c5f52e #5 2024-08-07 19:32:24 ~3 min ios 📦zip
✔️ 4c5f52e #5 2024-08-07 20:13:12 ~43 min tests 📄log
✔️ a940bfd #6 2024-08-07 22:35:12 ~2 min android 📦aar
✔️ a940bfd #6 2024-08-07 22:35:24 ~2 min tests-rpc 📄log
✔️ a940bfd #6 2024-08-07 22:35:27 ~2 min linux 📦zip
✔️ a940bfd #6 2024-08-07 22:36:19 ~3 min ios 📦zip
✔️ a940bfd #6 2024-08-07 23:16:27 ~43 min tests 📄log
✔️ 711d6d7 #7 2024-08-14 01:49:40 ~1 min tests-rpc 📄log
✔️ 711d6d7 #7 2024-08-14 01:52:10 ~3 min ios 📦zip
✔️ 711d6d7 #7 2024-08-14 01:52:16 ~3 min linux 📦zip
✔️ 711d6d7 #7 2024-08-14 01:52:51 ~4 min android 📦aar
✔️ 711d6d7 #7 2024-08-14 02:33:01 ~44 min tests 📄log
Commit #️⃣ Finished (UTC) Duration Platform Result
✔️ 74f9d5b #8 2024-08-14 01:52:28 ~2 min tests-rpc 📄log
✔️ 74f9d5b #8 2024-08-14 01:54:50 ~2 min linux 📦zip
✔️ 74f9d5b #8 2024-08-14 01:55:19 ~2 min android 📦aar
✔️ 74f9d5b #8 2024-08-14 01:55:53 ~3 min ios 📦zip
✔️ 74f9d5b #8 2024-08-14 03:17:20 ~44 min tests 📄log
✔️ a734ddf #9 2024-08-30 04:58:13 ~2 min tests-rpc 📄log
✔️ a734ddf #9 2024-08-30 04:59:30 ~3 min ios 📦zip
✔️ a734ddf #9 2024-08-30 04:59:38 ~3 min linux 📦zip
✔️ a734ddf #9 2024-08-30 05:00:17 ~4 min android 📦aar
✔️ a734ddf #9 2024-08-30 05:28:21 ~32 min tests 📄log

@adklempner adklempner force-pushed the feat/telemetry-device branch 2 times, most recently from 8ec4005 to c44f9fc Compare August 7, 2024 06:11
@igor-sirotin igor-sirotin self-requested a review August 7, 2024 07:35
Copy link
Collaborator

@igor-sirotin igor-sirotin left a comment

Choose a reason for hiding this comment

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

My main objection is accessing telemetry client through WakuV2Service, it's very confusing. The DeviceType is taken from Messenger, which also holds the TelemetryClient.

Otherwise it's good, thank you!

telemetry/client.go Outdated Show resolved Hide resolved
api/geth_backend.go Outdated Show resolved Hide resolved
api/geth_backend.go Outdated Show resolved Hide resolved
@adklempner adklempner force-pushed the feat/telemetry-device branch from c44f9fc to 4c5f52e Compare August 7, 2024 19:28
Copy link
Collaborator

@igor-sirotin igor-sirotin left a comment

Choose a reason for hiding this comment

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

Thanks for the improvements 👍

@adklempner adklempner force-pushed the feat/telemetry-device branch 2 times, most recently from 711d6d7 to 74f9d5b Compare August 14, 2024 01:49
@adklempner adklempner force-pushed the feat/telemetry-device branch from 74f9d5b to a734ddf Compare August 30, 2024 04:55
Copy link
Member

@vpavlin vpavlin left a comment

Choose a reason for hiding this comment

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

lgtm

@adklempner adklempner merged commit 27d02d5 into develop Aug 30, 2024
11 checks passed
@adklempner adklempner deleted the feat/telemetry-device branch August 30, 2024 15:59
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.

4 participants