Project

General

Profile

Actions

action #132689

closed

[qem-dashboard] Incorrect behavior for Group Flavor feature size:M

Added by jbaier_cz over 1 year ago. Updated about 1 year ago.

Status:
Resolved
Priority:
High
Assignee:
Start date:
2023-07-13
Due date:
2023-10-21
% Done:

0%

Estimated time:
Tags:

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.

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

Actions #1

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.

Actions #2

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

Actions #3

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.

Actions #4

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)

Actions #6

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

Actions #7

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.)

Actions #8

Updated by okurz about 1 year ago

  • Target version changed from Tools - Next to Ready
Actions #9

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
Actions #10

Updated by kraih about 1 year ago

  • Assignee set to kraih
Actions #11

Updated by kraih about 1 year ago

  • Status changed from Workable to In Progress

Updated by kraih about 1 year ago

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.

Actions #13

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

Actions #15

Updated by kraih about 1 year ago

  • Status changed from In Progress to Feedback
Actions #16

Updated by kraih about 1 year ago

Created a new ticket for the CI problem: https://progress.opensuse.org/issues/137651

Actions #17

Updated by mgrifalconi about 1 year ago

That's great! Thank you!

Actions #18

Updated by livdywan about 1 year ago

  • Status changed from Feedback to Resolved
Actions

Also available in: Atom PDF