Paths

Table of Contentst

[services] Tunnelbroker - Add `updateSessionItemIsOnline` database method
ClosedPublic
Actions

Authored by • max on Sep 16 2022, 11:51 AM.

Details

Reviewers

• jon
tomek
• karol

Commits

rCOMMe4f0d2ebe260: [services] Tunnelbroker - Add `updateSessionItemIsOnline` database method

Summary

This diff introduces adding of updateSessionItemIsOnline database method.

To send notifications if the device is offline we need to distinguish the device state. To do this we will ping the device and change the device status (online/offline) in the IsOnline field (D5164) using the introduced method.

More context is available in the Linear task description.

Related Linear task: ENG-1766

Test Plan

Successfully built.
Passed a unit test in D5166.

Diff Detail

Repository

rCOMM Comm

Branch

add-update-online-method

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

• max created this revision.Sep 16 2022, 11:51 AM

• max held this revision as a draft.

Herald added a reviewer: • jon. · View Herald TranscriptSep 16 2022, 11:51 AM

Herald added subscribers: • abosh, • karol, atul and 2 others. · View Herald Transcript

• max added a parent revision: D5164: [services] Tunnelbroker - Update `SessionItem` database entity with the `IsOnline` field.Sep 16 2022, 11:51 AM

Harbormaster completed remote builds in B12227: Diff 16775.Sep 16 2022, 11:59 AM

• max added a child revision: D5166: [services] Tunnelbroker - Add test for the `updateSessionItemIsOnline` database method.Sep 16 2022, 12:08 PM

• max edited the summary of this revision. (Show Details)Sep 16 2022, 12:21 PM

• max edited the test plan for this revision. (Show Details)

• max added a reviewer: tomek.

• max published this revision for review.Sep 16 2022, 12:25 PM

• max mentioned this in D5166: [services] Tunnelbroker - Add test for the `updateSessionItemIsOnline` database method.

Herald added a reviewer: • karol. · View Herald TranscriptSep 16 2022, 12:26 PM

• max added a child revision: D5175: [services] Tunnelbroker - Update device online status in `Get` handler.Sep 19 2022, 5:58 AM

tomek accepted this revision.Sep 19 2022, 6:02 AM

tomek added inline comments.

services/tunnelbroker/src/Database/DatabaseManager.cpp
109–110 ↗	(On Diff #16775)	We're not updating checkpoint time

This revision is now accepted and ready to land.Sep 19 2022, 6:02 AM

• max marked an inline comment as done.Sep 19 2022, 6:03 AM

• max added inline comments.

services/tunnelbroker/src/Database/DatabaseManager.cpp
109–110 ↗	(On Diff #16775)	We're not updating checkpoint time Wow, sorry about that. I'll fix this. Thanks @tomek ;)

• max mentioned this in D5173: [services] Tunnelbroker - Update Protobuf file with the `ping`.Sep 19 2022, 6:32 AM

• max mentioned this in D5174: [services] Tunnelbroker - Add ping implementation to the `Get` handler.Sep 19 2022, 6:35 AM

Rebased on parent changes. Error message was fixed.

Harbormaster failed remote builds in B12311: Diff 16899!Sep 20 2022, 5:45 AM

Harbormaster completed remote builds in B12311: Diff 16899.Sep 20 2022, 5:57 AM

Closed by commit rCOMMe4f0d2ebe260: [services] Tunnelbroker - Add `updateSessionItemIsOnline` database method (authored by • max). · Explain WhySep 20 2022, 6:19 AM

This revision was automatically updated to reflect the committed changes.

• max added a commit: rCOMMe4f0d2ebe260: [services] Tunnelbroker - Add `updateSessionItemIsOnline` database method.

• max mentioned this in rCOMMf6cc2d93dede: [services] Tunnelbroker - Add test for the `updateSessionItemIsOnline` database….

• max removed a child revision: D5175: [services] Tunnelbroker - Update device online status in `Get` handler.Sep 20 2022, 6:21 AM

• max mentioned this in rCOMMbf9ad730ef50: [services] Tunnelbroker - Update Protobuf file with the `ping`.Sep 21 2022, 2:29 PM

• max mentioned this in rCOMM5d2497b42072: [services] Tunnelbroker - Add ping implementation to the `Get` handler.Sep 23 2022, 1:51 PM

Revision Contents
Changeset List

Path

Size

services/

tunnelbroker/

src/

Database/

DatabaseManager.h

3 lines

DatabaseManager.cpp

39 lines

Diff 16899

View Options

services/tunnelbroker/src/Database/DatabaseManager.h