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

Possible configuration problem in the pubsub rabbitmq component #2764

Closed
arturotrenard opened this issue Apr 10, 2023 · 10 comments
Closed

Possible configuration problem in the pubsub rabbitmq component #2764

arturotrenard opened this issue Apr 10, 2023 · 10 comments
Assignees
Labels
kind/bug Something isn't working P1 size/XS 2 days of work stale
Milestone

Comments

@arturotrenard
Copy link
Contributor

Expected Behavior

Hello guys I think there is a problem here if you want to change the configuration

Actual Behavior

image

Steps to Reproduce the Problem

Release Note

RELEASE NOTE:

@arturotrenard arturotrenard added the kind/bug Something isn't working label Apr 10, 2023
@ItalyPaleAle
Copy link
Contributor

@arturotrenard I am not sure I understand the issue. Could you help us understand please?

@arturotrenard
Copy link
Contributor Author

Yes, of course, let's say that the user indicates that the queue should not be durable. In the code it is already stated that it is.

@ItalyPaleAle ItalyPaleAle added this to the v1.11 milestone Apr 11, 2023
@ItalyPaleAle ItalyPaleAle added P1 size/XS 2 days of work labels Apr 11, 2023
@ItalyPaleAle
Copy link
Contributor

Thanks, we'll get this fixed

@arturotrenard
Copy link
Contributor Author

Then I will write to you on discord about the performance issue. Because I have implemented a connection pool management for rabbitMQ and in my local tests without putting much iron and pointing to a standard RabbitMQ and the results exceed 20k/s, something that I did not achieve with the component and that is why I comment for the group. I have not
The ticket is still open because I want to have everything documented. And I thought of a v2 of the component but I mounted all the publishing code and with the management of the local pool in my service and it works great for me. Right now, I code the component and deploy it to dapr and it fails. So I want to document the ticket well so that it can be reviewed properly.

@SpikeWong
Copy link
Contributor

/assign

@berndverst
Copy link
Member

Once Dapr 1.11 is out please try to reproduce this problem @arturotrenard. Hopefully the PR from @SpikeWong actually addressed the issue.

@berndverst
Copy link
Member

@arturotrenard can you try out Dapr 1.11 RC5? https://github.com/dapr/dapr/releases/tag/v1.11.0-rc.5 to see if you can reproduce this?

@github-actions
Copy link

This issue has been automatically marked as stale because it has not had activity in the last 30 days. It will be closed in the next 7 days unless it is tagged (pinned, good first issue, help wanted or triaged/resolved) or other activity occurs. Thank you for your contributions.

@github-actions github-actions bot added the stale label Jun 29, 2023
@github-actions
Copy link

github-actions bot commented Jul 6, 2023

This issue has been automatically closed because it has not had activity in the last 37 days. If this issue is still valid, please ping a maintainer and ask them to label it as pinned, good first issue, help wanted or triaged/resolved. Thank you for your contributions.

@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale Jul 6, 2023
@arturotrenard
Copy link
Contributor Author

@arturotrenard can you try out Dapr 1.11 RC5? https://github.com/dapr/dapr/releases/tag/v1.11.0-rc.5 to see if you can reproduce this?
Hi, sorry for the delay, but I don't know why I hadn't received the email. So I'll be doing tests and if I see something weird I'll write to you.

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 P1 size/XS 2 days of work stale
Projects
None yet
Development

No branches or pull requests

4 participants