[tunnelbroker] Remove legacy AMQP connect function
ClosedPublic
Actions

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

Details

Reviewers

kamil
varun
will

Commits

rCOMMb5a906ddabbb: [tunnelbroker] Remove legacy AMQP connect function

Summary

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

Depends on D13608

Test Plan

cargo build

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

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

bartek held this revision as a draft.

Herald added subscribers: tomek, ashoat. · View Herald TranscriptOct 3 2024, 1:42 PM

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

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

bartek published this revision for review.Oct 4 2024, 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.Oct 4 2024, 8:21 AM

kamil accepted this revision.Oct 7 2024, 2:24 AM

This revision is now accepted and ready to land.Oct 7 2024, 2:24 AM

Closed by commit rCOMMb5a906ddabbb: [tunnelbroker] Remove legacy AMQP connect function (authored by bartek). · Explain WhyOct 7 2024, 7:28 AM

This revision was automatically updated to reflect the committed changes.

bartek added a commit: rCOMMb5a906ddabbb: [tunnelbroker] Remove legacy AMQP connect function.

Revision Contents
Changeset List

Path

Size

services/

tunnelbroker/

src/

amqp.rs

15 lines

Diff 44938

View Options

services/tunnelbroker/src/amqp.rs

[tunnelbroker] Remove legacy AMQP connect functionClosedPublicActions