[tunnelbroker] Remove legacy AMQP connect function
Needs ReviewPublic
Actions

Authored by bartek on Thu, Oct 3, 1:42 PM.

Tags

None

Referenced Files

	F2896959: D13609.diff
	Fri, Oct 4, 9:27 PM

	F2888830: D13609.id44889.diff
	Fri, Oct 4, 5:47 AM

	F2888796: D13609.id.diff
	Fri, Oct 4, 5:47 AM

	F2888759: D13609.diff
	Fri, Oct 4, 5:45 AM

	Unknown Object (File)
	Thu, Oct 3, 8:45 PM

	Unknown Object (File)
	Thu, Oct 3, 6:34 PM

	Unknown Object (File)
	Thu, Oct 3, 4:24 PM

	Unknown Object (File)
	Thu, Oct 3, 4:18 PM

Subscribers

ashoat

tomek

Details

Reviewers

kamil
varun
will

Summary

Removed the old connection function. It's no longer used anywhere

Depends on D13608

Test Plan

cargo build

Diff Detail

Repository

rCOMM Comm

Branch

barthap/amqp

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

bartek created this revision.Thu, Oct 3, 1:42 PM

bartek held this revision as a draft.

Herald added subscribers: tomek, ashoat. · View Herald TranscriptThu, Oct 3, 1:42 PM

Harbormaster completed remote builds in B32060: Diff 44889.Thu, Oct 3, 2:27 PM

bartek added a child revision: D13610: [tunnelbroker] Recreate closed channels in pool.Fri, Oct 4, 3:36 AM

bartek published this revision for review.Fri, Oct 4, 4:08 AM

bartek edited the summary of this revision. (Show Details)

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

bartek added a child revision: D13611: [tunnelbroker] Recover AMQP before publishing messages.Fri, Oct 4, 8:21 AM

Revision Contents
Changeset List

Path

Size

services/

tunnelbroker/

src/

amqp.rs

15 lines

Diff 44889

View Options

services/tunnelbroker/src/amqp.rs

[tunnelbroker] Remove legacy AMQP connect functionNeeds ReviewPublicActions