feat_: call telemetry upon error pushing envelope

adklempner commented 4 days ago

Sends request to telemetry upon error publishing envelope.

If telemetry is enabled and the publish function set in func (w *Waku) broadcast() fails, then the envelope details and resulting error will be pushed to the telemetry client for sending to the service. Also:

adds configurable telemetry period to determine how much time must pass between each http request made to the telemetry service (for now it's not used when creating the client to make these changes work without having to change desktop). Defaults to 10 seconds.
adds flag to cli for making ~60% of messages to fail
includes refactor to avoid blocking routine from #5399

Important changes:

[x] Depends on https://github.com/status-im/telemetry/pull/22

Closes #5260

status-im-auto commented 4 days ago

Jenkins Builds

Click to see older builds (20)

| :grey_question: | Commit | :hash: | Finished (UTC) | Duration | Platform | Result | |-|-|-|-|-|-|-| | :heavy_multiplication_x: | 3043ed79 | [#1](https://ci.status.im/job/status-go/job/prs/job/tests/job/PR-5430/1/) | 2024-06-27 04:03:46 | ~1 min | `tests` | [:page_facing_up:`log`](https://ci.status.im/job/status-go/job/prs/job/tests/job/PR-5430/1//consoleText) | | :heavy_check_mark: | 3043ed79 | [#1](https://ci.status.im/job/status-go/job/prs/job/linux/job/PR-5430/1/) | 2024-06-27 04:06:06 | ~3 min | `linux` | [:package:`zip`](https://status-im-prs.ams3.cdn.digitaloceanspaces.com/status-go-desktop-240627-040217-3043ed-pr5430.zip) | | :heavy_check_mark: | 3043ed79 | [#1](https://ci.status.im/job/status-go/job/prs/job/ios/job/PR-5430/1/) | 2024-06-27 04:06:13 | ~3 min | `ios` | [:package:`zip`](https://status-im-prs.ams3.cdn.digitaloceanspaces.com/status-go-ios-240627-040217-3043ed-pr5430.zip) | | :heavy_check_mark: | 3043ed79 | [#1](https://ci.status.im/job/status-go/job/prs/job/android/job/PR-5430/1/) | 2024-06-27 04:07:37 | ~5 min | `android` | [:package:`aar`](https://status-im-prs.ams3.cdn.digitaloceanspaces.com/status-go-android-240627-040217-3043ed-pr5430.aar) | | | | | | | | | | :heavy_multiplication_x: | eb6169f0 | [#2](https://ci.status.im/job/status-go/job/prs/job/tests/job/PR-5430/2/) | 2024-06-27 04:20:24 | ~57 sec | `tests` | [:page_facing_up:`log`](https://ci.status.im/job/status-go/job/prs/job/tests/job/PR-5430/2//consoleText) | | :heavy_check_mark: | eb6169f0 | [#2](https://ci.status.im/job/status-go/job/prs/job/ios/job/PR-5430/2/) | 2024-06-27 04:22:04 | ~2 min | `ios` | [:package:`zip`](https://status-im-prs.ams3.cdn.digitaloceanspaces.com/status-go-ios-240627-041922-eb6169-pr5430.zip) | | :heavy_check_mark: | eb6169f0 | [#2](https://ci.status.im/job/status-go/job/prs/job/linux/job/PR-5430/2/) | 2024-06-27 04:23:06 | ~3 min | `linux` | [:package:`zip`](https://status-im-prs.ams3.cdn.digitaloceanspaces.com/status-go-desktop-240627-041922-eb6169-pr5430.zip) | | :heavy_check_mark: | eb6169f0 | [#2](https://ci.status.im/job/status-go/job/prs/job/android/job/PR-5430/2/) | 2024-06-27 04:24:41 | ~5 min | `android` | [:package:`aar`](https://status-im-prs.ams3.cdn.digitaloceanspaces.com/status-go-android-240627-041922-eb6169-pr5430.aar) | | | | | | | | | | :heavy_multiplication_x: | e0a5e836 | [#3](https://ci.status.im/job/status-go/job/prs/job/tests/job/PR-5430/3/) | 2024-06-27 04:21:34 | ~58 sec | `tests` | [:page_facing_up:`log`](https://ci.status.im/job/status-go/job/prs/job/tests/job/PR-5430/3//consoleText) | | :heavy_check_mark: | e0a5e836 | [#3](https://ci.status.im/job/status-go/job/prs/job/ios/job/PR-5430/3/) | 2024-06-27 04:24:53 | ~2 min | `ios` | [:package:`zip`](https://status-im-prs.ams3.cdn.digitaloceanspaces.com/status-go-ios-240627-042221-e0a5e8-pr5430.zip) | | :heavy_check_mark: | e0a5e836 | [#3](https://ci.status.im/job/status-go/job/prs/job/linux/job/PR-5430/3/) | 2024-06-27 04:25:33 | ~2 min | `linux` | [:package:`zip`](https://status-im-prs.ams3.cdn.digitaloceanspaces.com/status-go-desktop-240627-042322-e0a5e8-pr5430.zip) | | :heavy_check_mark: | e0a5e836 | [#3](https://ci.status.im/job/status-go/job/prs/job/android/job/PR-5430/3/) | 2024-06-27 04:27:11 | ~2 min | `android` | [:package:`aar`](https://status-im-prs.ams3.cdn.digitaloceanspaces.com/status-go-android-240627-042456-e0a5e8-pr5430.aar) | | | | | | | | | | :heavy_multiplication_x: | bb855c37 | [#4](https://ci.status.im/job/status-go/job/prs/job/tests/job/PR-5430/4/) | 2024-06-27 04:36:47 | ~1 min | `tests` | [:page_facing_up:`log`](https://ci.status.im/job/status-go/job/prs/job/tests/job/PR-5430/4//consoleText) | | :heavy_check_mark: | bb855c37 | [#4](https://ci.status.im/job/status-go/job/prs/job/android/job/PR-5430/4/) | 2024-06-27 04:37:50 | ~2 min | `android` | [:package:`aar`](https://status-im-prs.ams3.cdn.digitaloceanspaces.com/status-go-android-240627-043532-bb855c-pr5430.aar) | | :heavy_check_mark: | bb855c37 | [#4](https://ci.status.im/job/status-go/job/prs/job/linux/job/PR-5430/4/) | 2024-06-27 04:37:56 | ~2 min | `linux` | [:package:`zip`](https://status-im-prs.ams3.cdn.digitaloceanspaces.com/status-go-desktop-240627-043536-bb855c-pr5430.zip) | | :heavy_check_mark: | bb855c37 | [#4](https://ci.status.im/job/status-go/job/prs/job/ios/job/PR-5430/4/) | 2024-06-27 04:38:14 | ~2 min | `ios` | [:package:`zip`](https://status-im-prs.ams3.cdn.digitaloceanspaces.com/status-go-ios-240627-043532-bb855c-pr5430.zip) | | | | | | | | | | :heavy_multiplication_x: | b884ee0b | [#5](https://ci.status.im/job/status-go/job/prs/job/tests/job/PR-5430/5/) | 2024-06-27 04:39:04 | ~2 min | `tests` | [:page_facing_up:`log`](https://ci.status.im/job/status-go/job/prs/job/tests/job/PR-5430/5//consoleText) | | :heavy_check_mark: | b884ee0b | [#5](https://ci.status.im/job/status-go/job/prs/job/android/job/PR-5430/5/) | 2024-06-27 04:39:45 | ~1 min | `android` | [:package:`aar`](https://status-im-prs.ams3.cdn.digitaloceanspaces.com/status-go-android-240627-043806-b884ee-pr5430.aar) | | :heavy_check_mark: | b884ee0b | [#5](https://ci.status.im/job/status-go/job/prs/job/linux/job/PR-5430/5/) | 2024-06-27 04:40:19 | ~2 min | `linux` | [:package:`zip`](https://status-im-prs.ams3.cdn.digitaloceanspaces.com/status-go-desktop-240627-043817-b884ee-pr5430.zip) | | :heavy_check_mark: | b884ee0b | [#5](https://ci.status.im/job/status-go/job/prs/job/ios/job/PR-5430/5/) | 2024-06-27 04:41:09 | ~2 min | `ios` | [:package:`zip`](https://status-im-prs.ams3.cdn.digitaloceanspaces.com/status-go-ios-240627-043830-b884ee-pr5430.zip) |

:grey_question:	Commit	:hash:	Finished (UTC)	Duration	Platform	Result
:heavy_check_mark:	8bddacfa	#6	2024-06-27 04:58:36	~2 min	`android`	:package:`aar`
:heavy_check_mark:	8bddacfa	#6	2024-06-27 04:58:50	~2 min	`linux`	:package:`zip`
:heavy_check_mark:	8bddacfa	#6	2024-06-27 04:58:57	~2 min	`ios`	:package:`zip`
:heavy_check_mark:	8bddacfa	#6	2024-06-27 05:38:02	~41 min	`tests`	:page_facing_up:`log`

:heavy_check_mark:	e919e702	#7	2024-06-28 03:39:51	~1 min	`android`	:package:`aar`
:heavy_check_mark:	e919e702	#7	2024-06-28 03:40:14	~2 min	`linux`	:package:`zip`
:heavy_check_mark:	e919e702	#7	2024-06-28 03:40:58	~2 min	`ios`	:package:`zip`
:heavy_check_mark:	e919e702	#7	2024-06-28 04:19:45	~41 min	`tests`	:page_facing_up:`log`

anastasiyaig commented 3 days ago

Pls dont merge yet, we want to test it real quick with autotests

cc @churik please post the results here when done

churik commented 3 days ago

https://github.com/status-im/status-mobile/pull/20595#issuecomment-2196531204 All good from mobile side, thank you for waiting!

anastasiyaig commented 3 days ago

alright , lets merge, thanks!

Samyoul commented 3 days ago

This kind of activity seems like something we could automate in Github.

Steps:

🧙 🪄 A magic trigger fires.
Copy from target status-go PR to memory:
- Latest commit hash <HASH>
- Branch name <NAME>
Branch from latest status-mobile and status-desktop develop (or target branch(es))
Name new client branch go-only/<NAME>
Call ./scripts/update-status-go.sh <HASH> and the desktop equivalent
Commit change to status-go-version.json and the desktop equivalent
Push new branch go-only/<NAME> to status-mobile and status-desktop
Create new PR for branch go-only/<NAME>
status-mobile or status-desktop PR is added to a Project board and column notifying QA teams that their smoke test prs are ready for smoking.

I don't know what we could use to trigger this, may be moving the status-go PR into a Project pipeline?

@churik would this be helpful @siddarthkay is this feasible in your opinion? @anastasiyaig do you know if this process (or an adaption) would work for Desktop?

siddarthkay commented 3 days ago

ah yes @Samyoul : Please copy and paste this into a new issue and assign it to me I could get this done 👍🏻 @J-Son89 also had discussed this with me but this work got lost in the list of to-dos I have :)

status-im / status-go

feat_: call telemetry upon error pushing envelope #5430

Jenkins Builds