diff options
| author | Yuval Lifshitz <ylifshit@redhat.com> | 2023-06-16 17:10:19 +0200 |
|---|---|---|
| committer | Yuval Lifshitz <ylifshit@redhat.com> | 2023-06-21 10:06:17 +0200 |
| commit | 38d819b3d77f4b330b542940314e22c21bb5591d (patch) | |
| tree | 063682ba0edb966da3c02d645a990f19887ace65 /doc/radosgw/notifications.rst | |
| parent | Merge pull request #51827 from liu-chunmei/get_assert (diff) | |
| download | ceph-38d819b3d77f4b330b542940314e22c21bb5591d.tar.xz ceph-38d819b3d77f4b330b542940314e22c21bb5591d.zip | |
rgw/amqp: remove possible race conditions with the amqp connections
* simplify memory management of the connection by not using a unique_ptr
* simplify the logic by handling all issues inside the amqp manager
* fix iterator invalidation issue with miltiple n/acks
* allow different connections with different exchanges
* modify the unit tests according to the new behavior
Fixes:
* https://tracker.ceph.com/issues/61639
* https://tracker.ceph.com/issues/46127
Signed-off-by: Yuval Lifshitz <ylifshit@redhat.com>
Diffstat (limited to 'doc/radosgw/notifications.rst')
| -rw-r--r-- | doc/radosgw/notifications.rst | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/doc/radosgw/notifications.rst b/doc/radosgw/notifications.rst index e450dd9488a..1cc538432f3 100644 --- a/doc/radosgw/notifications.rst +++ b/doc/radosgw/notifications.rst @@ -188,8 +188,7 @@ Request parameters: specified CA will be used to authenticate the broker. The default CA will not be used. - amqp-exchange: The exchanges must exist and must be able to route messages - based on topics. This parameter is mandatory. Different topics that point - to the same endpoint must use the same exchange. + based on topics. This parameter is mandatory. - amqp-ack-level: No end2end acking is required. Messages may persist in the broker before being delivered to their final destinations. Three ack methods exist: |