Paths

Table of Contentst

[services] Tunnelbroker - Add device ping interval to Constants
ClosedPublic
Actions

Authored by • max on Sep 15 2022, 2:42 PM.

Details

Reviewers

varun
tomek
• karol
ashoat
• jon

Commits

rCOMM6a8801a49786: [services] Tunnelbroker - Add device ping interval to Constants

Summary

This diff introduces a ping interval to check if the device is online.
The default "beacon" value for pinging is 3000 ms in our keyserver, which seems reasonable.
More context can be found in the Linear task.

Related Linear task: ENG-1766

Test Plan

No test, just a constant value.

Diff Detail

Repository

rCOMM Comm

Branch

add-constants-ping-interval

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

• max created this revision.Sep 15 2022, 2:42 PM

• max held this revision as a draft.

Herald added a reviewer: • jon. · View Herald TranscriptSep 15 2022, 2:42 PM

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

• max published this revision for review.Sep 15 2022, 2:50 PM

• max edited the summary of this revision. (Show Details)

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

• max added reviewers: varun, tomek.

Herald added a reviewer: • karol. · View Herald TranscriptSep 15 2022, 2:51 PM

Harbormaster failed remote builds in B12173: Diff 16701!Sep 15 2022, 2:51 PM

Harbormaster completed remote builds in B12173: Diff 16701.Sep 15 2022, 3:00 PM

Change the interval to 3s accrording to our keyserver interval.

• max edited the summary of this revision. (Show Details)Sep 15 2022, 3:14 PM

Harbormaster completed remote builds in B12174: Diff 16702.Sep 15 2022, 3:23 PM

No context on correct path forward

ashoat accepted this revision.Sep 16 2022, 2:34 PM

tomek accepted this revision.Sep 19 2022, 12:31 PM

This revision is now accepted and ready to land.Sep 19 2022, 12:31 PM

Closed by commit rCOMM6a8801a49786: [services] Tunnelbroker - Add device ping interval to Constants. · Explain WhySep 19 2022, 12:37 PM

This revision was automatically updated to reflect the committed changes.

• max added a commit: rCOMM6a8801a49786: [services] Tunnelbroker - Add device ping interval to Constants.

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

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

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

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

Revision Contents
Changeset List

Path

Size

services/

tunnelbroker/

src/

Constants.h

1 line

Diff 16702

View Options

services/tunnelbroker/src/Constants.h

[services] Tunnelbroker - Add device ping interval to ConstantsClosedPublicActions