rgw/notification: Continue committing notification events for other topics subscribed to bucket even if there is failure to load any single topic .

kchheda3 · kchheda3 · commit 9f4ea76d4d32 · 2024-05-21T14:46:08.000-04:00
Fixes https://tracker.ceph.com/issues/66097 Signed-off-by: kchheda3 <kchheda3@bloomberg.net>
diff --git a/src/rgw/driver/rados/rgw_notify.cc b/src/rgw/driver/rados/rgw_notify.cc
@@ -1099,7 +1099,7 @@ int publish_reserve(const DoutPrefixProvider* dpp,
           // either the topic is deleted but the corresponding notification
           // still exist or in v2 mode the notification could have synced first
           // but topic is not synced yet.
-          return 0;
+          continue;
         }
         ldpp_dout(res.dpp, 1)
             << "WARN: Using the stored topic from bucket notification struct."

Original file line number	Diff line number	Diff line change
`@@ -1099,7 +1099,7 @@ int publish_reserve(const DoutPrefixProvider* dpp,`
`1099`	`1099`	`// either the topic is deleted but the corresponding notification`
`1100`	`1100`	`// still exist or in v2 mode the notification could have synced first`
`1101`	`1101`	`// but topic is not synced yet.`
`1102`		`- return 0;`
	`1102`	`+ continue;`
`1103`	`1103`	`}`
`1104`	`1104`	`ldpp_dout(res.dpp, 1)`
`1105`	`1105`	`<< "WARN: Using the stored topic from bucket notification struct."`