[tunnelbroker] Enable Auto-ACK for AMQP consumers
ClosedPublic
Actions

Authored by bartek on Oct 7 2024, 3:59 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sat, Nov 9, 3:55 AM

	Unknown Object (File)
	Sat, Nov 9, 2:22 AM

	Unknown Object (File)
	Mon, Nov 4, 9:31 PM

	Unknown Object (File)
	Fri, Nov 1, 11:16 AM

	Unknown Object (File)
	Tue, Oct 29, 9:32 AM

	Unknown Object (File)
	Wed, Oct 23, 8:43 PM

	Unknown Object (File)
	Oct 18 2024, 6:24 PM

	Unknown Object (File)
	Oct 18 2024, 6:24 PM

View All 21 Files

Subscribers

ashoat

tomek

Details

Reviewers

kamil

Commits

rCOMM75173926638f: [tunnelbroker] Enable Auto-ACK for AMQP consumers

Summary

Addresses ENG-9510.
Details in the comment.

Depends on D13621

Test Plan

Ran Commtest several times and observed RabbitMQ dashboard.
Test results at the bottom of this comment.

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

bartek created this revision.Oct 7 2024, 3:59 AM

bartek held this revision as a draft.

Herald added subscribers: tomek, ashoat. · View Herald TranscriptOct 7 2024, 3:59 AM

Harbormaster completed remote builds in B32085: Diff 44921.Oct 7 2024, 4:20 AM

bartek published this revision for review.Oct 7 2024, 4:31 AM

kamil accepted this revision.Oct 7 2024, 4:38 AM

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

Closed by commit rCOMM75173926638f: [tunnelbroker] Enable Auto-ACK for AMQP consumers (authored by bartek). · Explain WhyOct 7 2024, 7:29 AM

This revision was automatically updated to reflect the committed changes.

bartek added a commit: rCOMM75173926638f: [tunnelbroker] Enable Auto-ACK for AMQP consumers.

Revision Contents
Changeset List

Path

Size

services/

tunnelbroker/

src/

websockets/

session.rs

5 lines

Diff 44942

View Options

services/tunnelbroker/src/websockets/session.rs

[tunnelbroker] Enable Auto-ACK for AMQP consumersClosedPublicActions