[tunnelbroker] Add error listener for AMQP
AcceptedPublic
Actions

Authored by bartek on Thu, Oct 3, 8:40 AM.

Details

Reviewers

kamil
varun
will

Summary

By default, lapin error logs are out of our control, on a separate thread, in library internal code.
This handler lets us print them with our errorType.

NOTE: This might cause increased alarm count for TB. In the future we want to filter them here.

Depends on D13592

Test Plan

Disconnected rabbitmq while tunnelbroker running. Error log appeared about IOError

Diff Detail

Repository

rCOMM Comm

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

bartek created this revision.Thu, Oct 3, 8:40 AM

bartek held this revision as a draft.

Herald added subscribers: tomek, ashoat. · View Herald TranscriptThu, Oct 3, 8:40 AM

bartek added a child revision: D13594: [tunnelbroker] Add exponential backoff to AMQP connection.Thu, Oct 3, 8:42 AM

Harbormaster completed remote builds in B32044: Diff 44871.Thu, Oct 3, 8:56 AM

bartek published this revision for review.Thu, Oct 3, 8:59 AM

bartek added inline comments.

services/tunnelbroker/src/amqp.rs
28–31	This might cause increased alarm count for TB. In the future we want to filter them here. Tunnelbroker logs have these anyways, but they're inside `lapin` code and have no `errorType` assigned so they don't trigger alarms
services/tunnelbroker/src/websockets/session.rs
687	Noticed this is about WS, not AMQP - `self.tx` is a Websocket transmitter

will accepted this revision.Thu, Oct 3, 9:10 AM

This revision is now accepted and ready to land.Thu, Oct 3, 9:10 AM

Revision Contents
Changeset List

Path

Size

services/

tunnelbroker/

src/

amqp.rs

7 lines

websockets/

session.rs

2 lines

Diff 44871

View Options

services/tunnelbroker/src/amqp.rs