-
Notifications
You must be signed in to change notification settings - Fork 171
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
Alert RPCs through Alert Manager stop working if the first was not dismissed. #1779
Comments
Hi @mjuarez-ford, we tried to reproduce the issue on Manticore but we were not able to see the issue. We couldn't test the exact scenario on Sync because we don't have an app that has the permissions to send Alerts in the Background on Sync. would be able to see if the issue happens when you test on Manticore? also does the issue happen every time when you go through the same scenario or only rarely? |
Hi @bilal-alsharifi, I'll try it with Manticore, Yes happens all the time. |
@mjuarez-ford Sounds great. Can you send me info on Slack? |
sure @bilal-alsharifi, I just shared the info there. |
Hi, @bilal-alsharifi were you able to reproduce it? |
Hello @mjuarez-ford, Bilal is no longer working on this project. We have internally reassigned this task and will get back to you when we can. In the meantime, if you can provide (privately) a Sync app id and app name that have these permissions, that would be a great help in tracking this issue down. You can also provide the TDK logs, but they will be more difficult for our engineers to parse and less likely to be useful. Thank you! |
Hello @joeljfischer, is there any update on this issue? |
Hi @mjuarez-ford, this task has not yet been started, but it should be soon. We have been working through crashing bugs first, but the plan is still for this issue to be investigated and solved before the April release. |
Thanks so much for the feedback @joeljfischer. |
Hi @joeljfischer is there any news regarding this issue? |
Due to unforeseen circumstances, this bug will not be fixed in the upcoming release. I responded with more detail on Slack. |
Bug Report
Alerts stop working if the previous notification was not dismissed by the User.
Reproduction Steps
HMI_BACKGROUND
Expected Behavior
The user should keep receiving notifications.
Observed Behavior
On the SYNC logs, I verify that on the HMI the Alert RPC is not received. And on the
PresentAlertOperation
thestart()
is not executed.When everything is executing correctly on the
Queue
class the propertieshead
andtail
are alwaysnull
. But after it breaks it always adds them to the queue.If we follow the steps on the
Reproduction Steps
on thestep **5**
it could be that the TaskMaster in charge of the alert queue is no longer running or is waiting for a response.OS & Version Information
SDL Hello World
with the same codeTest Case, Sample Code, and / or Example App
The text was updated successfully, but these errors were encountered: