Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Namespaced consumer groups doesn't work with Bulk pubsub on RabbitMQ #3663

Open
alicejgibbons opened this issue Feb 10, 2025 · 1 comment
Open
Labels
kind/bug Something isn't working

Comments

@alicejgibbons
Copy link

Expected Behavior

Namespaced consumer groups should work as documented here: https://docs.dapr.io/developing-applications/building-blocks/pubsub/howto-namespace/#with-namespace-consumer-groups

Actual Behavior

When adding the namespace configuration to rabbitmq component this fails to add the correct namespaced consumer group to the broker but instead does not take the namespace into account on topic or consumer group creation.

  - name: consumerID
    value: "{namespace}"

This works on regular pubsub, eg namespace= default and topic=orders
Image

On bulk pubsub, the publisher is just using orders topic.

Image

This was also tested and works fine on Redis so I believe is just a bug on RabbitMQ.

Image

Steps to Reproduce the Problem

Run the repro sample here: https://github.com/alicejgibbons/dapr-bulk-pubsub-rabbit

Release Note

RELEASE NOTE: FIX Bug in Dapr bulk pubsub with rabbitmq

@alicejgibbons alicejgibbons added the kind/bug Something isn't working label Feb 10, 2025
@eliajbo
Copy link

eliajbo commented Feb 11, 2025

Hello,

Here is how I could go around this issue temporarely when using the .NET SDK.

The issue is located at the use of daprClient.BulkPublishEventAsync specifically.
I could go around the problem by prefixing the topicName value with the required namespace and all the rest then works. Eg: namespaceTopic.

In Rmq management, you can see the exchange as namespaceTopic after the fix too instead of Topic.
In queues and streams section, you will see
namespace.pubsubname-subscribingAppId-namespaceTopic

So the fix is not needed for client.PublishEventAsync that already does this well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Something isn't working
Projects
Status: Backlog
Development

No branches or pull requests

2 participants