Bug 2011774
| Summary: | Toggled repos in Discover jump to the bottom of the list | ||||||||
|---|---|---|---|---|---|---|---|---|---|
| Product: | [Fedora] Fedora | Reporter: | Kamil Páral (Red Hat) <kparal> | ||||||
| Component: | plasma-discover | Assignee: | Rex Dieter <rdieter> | ||||||
| Status: | CLOSED ERRATA | QA Contact: | Fedora Extras Quality Assurance <extras-qa> | ||||||
| Severity: | unspecified | Docs Contact: | |||||||
| Priority: | unspecified | ||||||||
| Version: | 35 | CC: | awilliam, geraldo.simiao.kutz, gmarr, kde-sig, rdieter, robatino | ||||||
| Target Milestone: | --- | Keywords: | CommonBugs | ||||||
| Target Release: | --- | ||||||||
| Hardware: | Unspecified | ||||||||
| OS: | Unspecified | ||||||||
| Whiteboard: | https://ask.fedoraproject.org/t/we-are-testing-a-new-common-issues-category-and-process/18916 RejectedBlocker AcceptedFreezeException https://fedoraproject.org/wiki/Common_F35_bugs#discover-repo-jump | ||||||||
| Fixed In Version: | plasma-discover-5.23.2-1.fc35 | Doc Type: | If docs needed, set a value | ||||||
| Doc Text: | Story Points: | --- | |||||||
| Clone Of: | Environment: | ||||||||
| Last Closed: | 2021-11-16 16:34:43 UTC | Type: | Bug | ||||||
| Regression: | --- | Mount Type: | --- | ||||||
| Documentation: | --- | CRM: | |||||||
| Verified Versions: | Category: | --- | |||||||
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |||||||
| Cloudforms Team: | --- | Target Upstream Version: | |||||||
| Embargoed: | |||||||||
| Bug Depends On: | |||||||||
| Bug Blocks: | 1891956 | ||||||||
| Attachments: |
|
||||||||
|
Description
Kamil Páral (Red Hat)
2021-10-07 10:55:43 UTC
Created attachment 1830328 [details]
repo jump and redraw issue (video)
Created attachment 1830329 [details]
rpm -qa
Please report upstream. I saw the undefinted thing too, but appeared to be temporary while config is re-read. Tested with plasma-discover-5.23.0-1.fc35 and the bug is still there. Please note that the reshuffled rpm repo order survives reboots, i.e. the order change is permanent. The "undefined" glitch is temporary, until the page is forced to be redrawn. Upstream bug report: https://bugs.kde.org/show_bug.cgi?id=443583 Since bug 2011305 is an accepted blocker now, and the proposed update for it [1] introduces/uncovers this bug, I'm going to mark this for a blocker discussion as well. It's not fully technically correct (this bug can't be a blocker before that update is pushed stable), but a) I want others to be aware of it, and b) it's very likely that this bug has been already present in Discover for some time, it just couldn't be tested due to bug 2011305. [1] https://bodhi.fedoraproject.org/updates/FEDORA-2021-a32a5c05d4 Discussed during the 2021-10-11 blocker review meeting: [0] The decision to classify this bug as a "RejectedBlocker (Final)" and an "AcceptedFreezeException (Final)" was made as we agreed that while this looks bad, the operation does work, so critical functionality is maintained. It's accepted as an FE issue if it can be fixed relatively safely, as it does look pretty bad. [0] https://meetbot.fedoraproject.org/fedora-blocker-review/2021-10-11/f35-blocker-review.2021-10-11-16.00.txt FEDORA-2021-4f68774f07 has been submitted as an update to Fedora 35. https://bodhi.fedoraproject.org/updates/FEDORA-2021-4f68774f07 https://bodhi.fedoraproject.org/updates/FEDORA-2021-4f68774f07 should fix the "other repos go to 'undefined'" part of this. It doesn't solve the order changing part, though. Aleix is working on a bigger fix that would avoid the whole "refresh the entire interface whenever you touch a PK repo" thing, but we don't think that's likely safe to rush in for Final. Tested it and just like @awilliam said: no more the "undefined" problem, thats fixed. https://invent.kde.org/plasma/discover/-/merge_requests/192 is the PR that would address the rest of this, probably. (In reply to Geraldo Simião from comment #10) > Tested it and just like @awilliam said: no more the "undefined" > problem, thats fixed. Confirmed. FEDORA-2021-4f68774f07 has been pushed to the Fedora 35 testing repository. Soon you'll be able to install the update with the following command: `sudo dnf upgrade --enablerepo=updates-testing --advisory=FEDORA-2021-4f68774f07` You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2021-4f68774f07 See also https://fedoraproject.org/wiki/QA:Updates_Testing for more information on how to test updates. FEDORA-2021-4f68774f07 has been pushed to the Fedora 35 stable repository. If problem still persists, please make note of it in this bug report. The remaining issue is that the toggled repos jump around in the list. https://koji.fedoraproject.org/koji/taskinfo?taskID=77634062 is a scratch build with the fix for this. Please test it quite hard as it changes the logic of how the PK source list gets refreshed, so see if you can break that...enable and disable a bunch of sources, add sources, remove them, leave and re-enter the page, stuff like that. Adam, the repo jumping is fixed. I only found one problem in my testing. If you quickly click twice on a repo checkbox, you can make it not reflect the actual state. So e.g. if updates-testing is disabled (unchecked), and you quickly click twice on its checkbox, it gets checked and unchecked again, i.e. it looks as disabled. But in fact, only the first action got performed in the code beneath, and updates-testing is currently enabled. Switching the tab to e.g. About and back to Settings redraws the checkbox into a proper (checked) state. I'm not able to verify whether this is a new bug or it existed before, because it couldn't be triggered before (the repos jumped around, and if you tried this with the bottom-most repo, the whole page scrolled up, preventing you from doing a double click). thanks. yeah, I've managed to trigger bugs in the other backends by doing the same thing, but it strikes me as a fairly unlikely 'real world' thing, it's more something you and me would do because we like breaking stuff. :P So if that's the only problem, I think the fix is good, but...on the whole, since it's Monday, it might be best to hold off pulling the fix into the final compose. I think I'll submit it but leave it to be a 0-day update. FEDORA-2021-35e9884fd8 has been submitted as an update to Fedora 35. https://bodhi.fedoraproject.org/updates/FEDORA-2021-35e9884fd8 FEDORA-2021-f7d19c8901 has been pushed to the Fedora 35 testing repository. Soon you'll be able to install the update with the following command: `sudo dnf upgrade --enablerepo=updates-testing --advisory=FEDORA-2021-f7d19c8901` You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2021-f7d19c8901 See also https://fedoraproject.org/wiki/QA:Updates_Testing for more information on how to test updates. seems to be fixed here after the update https://bodhi.fedoraproject.org/updates/FEDORA-2021-f7d19c8901 FEDORA-2021-f7d19c8901 has been pushed to the Fedora 35 stable repository. If problem still persists, please make note of it in this bug report. |