Bug 480947 - Discover is randomly restarting the search for updates after displaying the results for few seconds
Summary: Discover is randomly restarting the search for updates after displaying the r...
Status: RESOLVED FIXED
Alias: None
Product: Discover
Classification: Applications
Component: rpm-ostree Backend (other bugs)
Version First Reported In: 6.0.3
Platform: Fedora RPMs Linux
: NOR normal
Target Milestone: ---
Assignee: Plasma Bugs List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2024-02-06 12:59 UTC by Fernando M. Muniz
Modified: 2025-05-28 13:29 UTC (History)
5 users (show)

See Also:
Latest Commit:
Version Fixed/Implemented In:
Sentry Crash Report:


Attachments
An smaller scale of the issue. (2.48 MB, video/x-matroska)
2024-02-06 21:49 UTC, Fernando M. Muniz
Details
Notice the extra search flashing for few frames. (2.94 MB, video/x-matroska)
2024-02-06 21:57 UTC, Fernando M. Muniz
Details
The results are flashed at: 1:23, 2:32, 3:42, 4:52, 6:02, 7:12, and 8:21. (3.76 MB, video/mp4)
2024-10-28 17:37 UTC, Fernando M. Muniz
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Fernando M. Muniz 2024-02-06 12:59:11 UTC
Discover's update section is randomly restarting the search for updates after displaying/flashing the results for few seconds.

I don't have the technical knowledge to replicate this issue. I just noticed that it has a higher chance of happening when it displays many items to update.

There must be something that doesn't prevent the section from automatically searching for updates after displaying the results.

If this issue is found, please check if the same issue happens when searching for apps.
Comment 1 Nate Graham 2024-02-06 20:03:37 UTC
Can you attach a screen recording that shows the issue happening? Thanks a lot!
Comment 2 Fernando M. Muniz 2024-02-06 21:49:30 UTC
Created attachment 165628 [details]
An smaller scale of the issue.

Managed to record this issue on a very smaller scale. Normally the waiting time to display the updates again is much longer.
Comment 3 Fernando M. Muniz 2024-02-06 21:57:25 UTC
Created attachment 165629 [details]
Notice the extra search flashing for few frames.

You can notice that after the updates being displayed, it does a second search for them.

This is the issue. One that can take a very long time depending on the number of items.
Comment 4 Fernando M. Muniz 2024-02-06 22:34:35 UTC
There's also another issue in which instead of searching one more time after displaying the results, it keeps searching multiple times (very likely forever).

But I won't be able to record that specific glitch anytime soon, and in any case the video would be too big to be here.
And fixing the issue I've recorded would likely fix the looped version anyway.
Comment 5 Nate Graham 2024-02-07 17:44:42 UTC
Thanks, that's helpful. The fact that you are using Kinoite is very relevant as it has a whole special backend for it.
Comment 6 Fernando M. Muniz 2024-02-09 09:51:07 UTC
I don't know if this is the right place to ask this, but is there a reason to why searching for updates hides the lastest results?
Why can't it show the latest results while searching for updates at the same time?
Comment 7 Fernando M. Muniz 2024-02-14 14:31:06 UTC
Managed to record the long loop issue.

The results are flashed at:
1:23, 2:32, 3:42, 4:52, 6:02, 7:12, and 8:21.

https://www.youtube.com/watch?v=yi5UFATsGnE
Comment 8 Bug Janitor Service 2024-04-21 14:15:21 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/discover/-/merge_requests/818
Comment 9 Fernando M. Muniz 2024-10-28 17:37:32 UTC
Created attachment 175311 [details]
The results are flashed at: 1:23, 2:32, 3:42, 4:52, 6:02, 7:12, and 8:21.

I'm not currently using Fedora, so I don't know if this got fixed since it became "ASSIGNED".
Comment 10 Bug Janitor Service 2025-05-24 22:37:26 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/discover/-/merge_requests/1096
Comment 11 Timothée Ravier 2025-05-28 09:21:13 UTC
Git commit e0907397d9b8f0cc4223c358d654a094eff5163e by Timothée Ravier.
Committed on 28/05/2025 at 09:19.
Pushed by ravier into branch 'master'.

rpm-ostree: Fix rpm-ostree driver registration

Properly initilize the boolean responsible for trigerring the logic to
register Discover as an update driver to the rpm-ostree daemon.

This will keep the rpm-ostree daemon running (but mostly idle) as long
as the Discover main window is open, and will avoid re-loading the list
of deployments and update every ~60 seconds, when rpm-ostree would have
exited due to being idle, releasing the name on the bus.

Fixes: https://bugs.kde.org/show_bug.cgi?id=480947

M  +7    -1    libdiscover/backends/RpmOstreeBackend/RpmOstreeBackend.cpp
M  +3    -0    libdiscover/backends/RpmOstreeBackend/RpmOstreeBackend.h

https://invent.kde.org/plasma/discover/-/commit/e0907397d9b8f0cc4223c358d654a094eff5163e
Comment 12 Timothée Ravier 2025-05-28 13:29:31 UTC
Git commit 8b8f9fd32cf8bb40bd800b8e6777476277295a5b by Timothée Ravier.
Committed on 28/05/2025 at 09:24.
Pushed by ravier into branch 'Plasma/6.4'.

rpm-ostree: Fix rpm-ostree driver registration

Properly initilize the boolean responsible for trigerring the logic to
register Discover as an update driver to the rpm-ostree daemon.

This will keep the rpm-ostree daemon running (but mostly idle) as long
as the Discover main window is open, and will avoid re-loading the list
of deployments and update every ~60 seconds, when rpm-ostree would have
exited due to being idle, releasing the name on the bus.

Fixes: https://bugs.kde.org/show_bug.cgi?id=480947
(cherry picked from commit e0907397d9b8f0cc4223c358d654a094eff5163e)

M  +7    -1    libdiscover/backends/RpmOstreeBackend/RpmOstreeBackend.cpp
M  +3    -0    libdiscover/backends/RpmOstreeBackend/RpmOstreeBackend.h

https://invent.kde.org/plasma/discover/-/commit/8b8f9fd32cf8bb40bd800b8e6777476277295a5b