From #5265 review (CodeRabbit Major). register_domain_subscribers marks a DomainGroup done the moment its group_first_time(…) block is entered, not after every subscribe_global inside returns Some. A transient failure (global bus not yet initialized) only logs a warning but the group is marked done, so no later call retries — that subscriber stays absent for the process lifetime. Pre-existing pattern at every group_first_time(DomainGroup::…) site, not dedup-specific. Fix: mark done only after all registrations succeed, or make individual registrations retryable. Deferred from #5265.
From #5265 review (CodeRabbit Major).
register_domain_subscribersmarks aDomainGroupdone the moment itsgroup_first_time(…)block is entered, not after everysubscribe_globalinside returnsSome. A transient failure (global bus not yet initialized) only logs a warning but the group is marked done, so no later call retries — that subscriber stays absent for the process lifetime. Pre-existing pattern at everygroup_first_time(DomainGroup::…)site, not dedup-specific. Fix: mark done only after all registrations succeed, or make individual registrations retryable. Deferred from #5265.