Skip to content
New issue

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

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

Already on GitHub? # to your account

message dequeue panic #70

Closed
Glitchfix opened this issue Mar 23, 2023 · 0 comments · Fixed by portworx/ubbagent#1 · May be fixed by #83
Closed

message dequeue panic #70

Glitchfix opened this issue Mar 23, 2023 · 0 comments · Fixed by portworx/ubbagent#1 · May be fixed by #83

Comments

@Glitchfix
Copy link

I encountered an error when I send a message one but I see multiple invocations of the same message in retry.go
the message gets dequeued after the first success, when it attempts to do so in the second attempt the dequeue fails as the message is not in the queue anymore as it was dequeued in the first attempt.
This happens in case of service control endpoint and it is not an expected behaviour

To make things worse ubbagent panics resulting in the system to crash
I believe when this happens it should only log the error and move on

So we have 2 problems at hand that we must discuss:

  1. why should the dequeue panic when we can just log the error
  2. why would it try to send it twice in the first place when it already had a success sending it in the service control
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
1 participant