action #132689
closed[qem-dashboard] Incorrect behavior for Group Flavor feature size:M
0%
Description
[qem-dashboard] Incorrect behavior for Group Flavor feature
Observation¶
Blocked by Tests view show incorrect results when Group Flavor is toggled on (the default). In some cases, the flavors with errors are hidden. See the attached pictures of the same incident with Grouped Flavors off and on.
From Updated by mgrifalconi 1 day ago
Priority changed from Normal to High
Hello, now the issue is still present but it become worse as you are not able to 'ungroup flavors' anymore (maybe a regression caused by the new feature of having search parameters in the url?) Clicking the "group flavor" checkbox now does nothing.
One example is http://dashboard.qam.suse.de/incident/30825 (filter out 30825 in blocked page) where you see no red boxes on SLE Micro (link of green box is https://openqa.suse.de/tests/overview?build=20231003-1&distri=sle-micro&groupid=420&version=5.5&flavor=Default-Updates) but opening the link of the incident you can see the failures: https://openqa.suse.de/tests/overview?version=5.5&groupid=420&flavor=Default-RT-SelfInstall-Updates&distri=sle-micro&build=20231003-1
Proposing a higher prio here since it completely hides failures now with no workaround (other than clicking every single incident)Updated by mkittler about 20 hours ago
Clicking the "group flavor" checkbox now does nothing.
I was under the same impression after https://github.com/openSUSE/qem-dashboard/pull/359 had been deployed. However, it turned out that the current dataset simply didn't have anything that could be grouped. I have also cross-checked by importing an older production database dump locally. This dataset still had groupable entries and the grouping still worked after my change. So I don't think that "group flavor" doing nothing is related to my change - it is simply because there's nothing to group.
I'm not sure whether I can follow further paragraphs in your comment. You mention a few links but none of those is about the http://dashboard.qam.suse.de/blocked page the "Group flavor" feature is about.
(Note that "Group flavor" is only supposed to group by the flavor if the distribution and version are identical as well. After a quick look on http://dashboard.qam.suse.de/blocked I couldn't find such an instance.)
Acceptance criteria¶
- AC1: Both views (grouped or not) provides correct (i.e. the same) information
- AC2: Failures are visible in both views
- AC3: Counts add up between the grouped and ungrouped view
Suggestions¶
- Import the database dump to reproduce locally
- Inspect the logic behind "Group Flavor"
- For local testing data try the database dump from
/home/lurklur/bug-poo-132689.sql.xz
on qam2.suse.de, reproduce the failed red box not showing up based on grouping selection as visible in screenshots, fix the displaying so that a failed red box with corresponding entries show up in both cases grouped and not-grouped - Make a test case for the failing case
Files
Updated by kraih over 1 year ago
I've saved a database dump on qam2.suse.de
under /home/lurklur/bug-poo-132689.sql.xz
. So this should be fairly easy to replicate locally with a checkout of qem-dashboard.
Updated by okurz over 1 year ago
- Target version changed from Ready to future
Looks annoying but I guess the workaround is to not use the grouping for now
Updated by mkittler over 1 year ago
I guess when multiple boxes are combined then the color of the combined box should be the "most dangerous" color, e.g. red wins over green (and not vice versa as shown in the screenshot in the ticket description). That the grouping is happening at all is supposedly expected behavior.
Updated by mgrifalconi about 1 year ago
- Priority changed from Normal to High
Hello, now the issue is still present but it become worse as you are not able to 'ungroup flavors' anymore (maybe a regression caused by the new feature of having search parameters in the url?) Clicking the "group flavor" checkbox now does nothing.
One example is http://dashboard.qam.suse.de/incident/30825 (filter out 30825 in blocked page) where you see no red boxes on SLE Micro (link of green box is https://openqa.suse.de/tests/overview?build=20231003-1&distri=sle-micro&groupid=420&version=5.5&flavor=Default-Updates) but opening the link of the incident you can see the failures: https://openqa.suse.de/tests/overview?version=5.5&groupid=420&flavor=Default-RT-SelfInstall-Updates&distri=sle-micro&build=20231003-1
Proposing a higher prio here since it completely hides failures now with no workaround (other than clicking every single incident)
Updated by szarate about 1 year ago
Updated by okurz about 1 year ago
- Target version changed from future to Tools - Next
@mkittler would you be able to look into this if it's related to your recent PR https://github.com/openSUSE/qem-dashboard/pull/359
Updated by mkittler about 1 year ago
Clicking the "group flavor" checkbox now does nothing.
I was under the same impression after https://github.com/openSUSE/qem-dashboard/pull/359 had been deployed. However, it turned out that the current dataset simply didn't have anything that could be grouped. I have also cross-checked by importing an older production database dump locally. This dataset still had groupable entries and the grouping still worked after my change. So I don't think that "group flavor" doing nothing is related to my change - it is simply because there's nothing to group.
I'm not sure whether I can follow further paragraphs in your comment. You mention a few links but none of those is about the http://dashboard.qam.suse.de/blocked page the "Group flavor" feature is about.
(Note that "Group flavor" is only supposed to group by the flavor if the distribution and version are identical as well. After a quick look on http://dashboard.qam.suse.de/blocked I couldn't find such an instance.)
Updated by okurz about 1 year ago
- Target version changed from Tools - Next to Ready
Updated by livdywan about 1 year ago
- Subject changed from [qem-dashboard] Incorrect behavior for Group Flavor feature to [qem-dashboard] Incorrect behavior for Group Flavor feature size:M
- Description updated (diff)
- Status changed from New to Workable
Updated by kraih about 1 year ago
- File expected-groupFlavors-true.png expected-groupFlavors-true.png added
- File expected-groupFlavors-false.png expected-groupFlavors-false.png added
Looks like the problem is a bit more complex than i expected. I've bisected the commit that broke displaying failed tests in the grouped view and it seems this commit was responsible. Afterwards this commit by a different author completely broke grouping by flavors. I've attached two screenshots for what grouped and ungrouped results are supposed to look like.
Updated by openqa_review about 1 year ago
- Due date set to 2023-10-21
Setting due date based on mean cycle time of SUSE QE Tools
Updated by kraih about 1 year ago
Updated by kraih about 1 year ago
Created a new ticket for the CI problem: https://progress.opensuse.org/issues/137651
Updated by livdywan about 1 year ago
- Status changed from Feedback to Resolved
http://dashboard.qam.suse.de/blocked Looks awesome!