Paths

Table of Contentst

Differential D5304

[Docs] Update tunnelbroker configuration section
ClosedPublic
Actions

Authored by • jon on Oct 5 2022, 11:07 AM.

Tags

None

Referenced Files

	F3247731: D5304.id17367.diff
	Fri, Nov 15, 6:30 AM

	F3247282: D5304.diff
	Fri, Nov 15, 4:31 AM

	Unknown Object (File)
	Tue, Oct 22, 12:06 PM

	Unknown Object (File)
	Tue, Oct 22, 11:54 AM

	Unknown Object (File)
	Tue, Oct 22, 11:54 AM

	Unknown Object (File)
	Tue, Oct 22, 11:54 AM

	Unknown Object (File)
	Tue, Oct 22, 12:26 AM

	Unknown Object (File)
	Oct 12 2024, 11:25 AM

Subscribers

Details

Reviewers

• max
tomek
ashoat
varun

Commits

rCOMM6193b9635fbf: [Docs] Update tunnelbroker configuration section

Summary

Update tunnelbroker docs to reflect new state-of-the-world, in that
it requires a default keyserver now.

Add configuration files to .gitignore to avoid developers clobbering each
others dev environments.

Requirement added in https://phab.comm.dev/D2903

https://linear.app/comm/issue/ENG-1971

Depends on D5246

Test Plan

N/A

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

• jon created this revision.Oct 5 2022, 11:07 AM

Herald added subscribers: • abosh, atul. · View Herald TranscriptOct 5 2022, 11:07 AM

• jon edited the summary of this revision. (Show Details)Oct 5 2022, 11:08 AM

• jon added a parent revision: D5246: [Nix] Add localstack-down.

Correct installation directions

https://phab.comm.dev/D2903 also made $HOME/tunnelbroker/tunnelbroker.ini the configuration path.

Harbormaster completed remote builds in B12637: Diff 17367.Oct 5 2022, 11:18 AM

Harbormaster completed remote builds in B12638: Diff 17368.Oct 5 2022, 11:25 AM

• jon requested review of this revision.Oct 5 2022, 11:25 AM

This makes sense to me, but @max should take a look. (Might take a sec since he's on a trip)

Git-ignoring the tunnelbroker.ini looks reasonable to me, as well as updating the docs reflecting what the configuration file must contain. Thanks, @jon !

This revision is now accepted and ready to land.Oct 6 2022, 6:14 AM

Closed by commit rCOMM6193b9635fbf: [Docs] Update tunnelbroker configuration section (authored by • jon). · Explain WhyOct 10 2022, 3:06 PM

This revision was automatically updated to reflect the committed changes.

• jon added a commit: rCOMM6193b9635fbf: [Docs] Update tunnelbroker configuration section.

Revision Contents
Changeset List

Path

Size

1 line

docs/

dev_services.md

6 lines

services/

tunnelbroker/

tunnelbroker.ini

Diff 17473

.gitignore

Loading...

docs/dev_services.md

Loading...

services/tunnelbroker/tunnelbroker.ini

Loading...