[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

Branch

barthap/amqp

Lint

No Lint Coverage

Unit

No Test Coverage

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 44889

View Options

services/tunnelbroker/src/amqp.rs

[tunnelbroker] Remove legacy AMQP connect functionClosedPublicActions