Page MenuHomePhabricator
Differential D5165

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

Authored by • max on Sep 16 2022, 11:51 AM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Dec 10, 1:07 AM
Unknown Object (File)
Tue, Dec 3, 11:29 PM
Unknown Object (File)
Wed, Nov 27, 9:05 PM
Unknown Object (File)
Wed, Nov 27, 8:19 PM
Unknown Object (File)
Wed, Nov 27, 8:01 PM
Unknown Object (File)
Nov 23 2024, 11:39 PM
Unknown Object (File)
Oct 26 2024, 2:48 AM
Unknown Object (File)
Oct 26 2024, 2:48 AM
View All Files
Subscribers
• abosh
ashoat
atul
• karol
tomek

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
  1. Successfully built.
  2. 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
• max updated this revision to Diff 16899.Sep 20 2022, 5:44 AM
• max marked an inline comment as done.

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

PathSize
services/
tunnelbroker/
src/
Database/
3 lines
39 lines

Diff 16899

View Options

services/tunnelbroker/src/Database/DatabaseManager.h

Loading...
View Options

services/tunnelbroker/src/Database/DatabaseManager.cpp

Loading...