Checkbox GUI (Legacy)

whitelist ordering lost in checkbox-gui

Bug #1323201 reported by Zygmunt Krynicki
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Checkbox GUI (Legacy)
Fix Released
Critical
Sylvain Pineau
Checkbox GUI (Legacy) 0.18

Bug Description

After my recent fix for quirky whitelist filtering (bug 1302388) I noted that the GUI may ignore job ordering information and now I'm sure it is exactly the case.

I've prepared a simple whitelitst with two jobs

stub_package
stub_package2

In exactly that order

The ordering at runtime was stub_package2 followed by stub_package

So there you have it.

Related branches

lp:~sylvain-pineau/checkbox/fix-1323201
Zygmunt Krynicki (community): Approve
Sylvain Pineau (community): Needs Resubmitting
Diff: 28 lines (+13/-4)
1 file modified
checkbox-gui/gui-engine/gui-engine.cpp (+13/-4)
lp:ubuntu/utopic/checkbox
Revision history for this message
Jerry Kao (jerry.kao) wrote :

System Greenwood2 has suspend hand issue. When running Plainbox for full test, the order of test case is not consistent. After finish test case 5x of 209, disk/detect, system run suspend/suspend_advanced and hang. After reboot and continue for next test case, plainbox run one test case and run suspend/suspend_advanced (108 of 209) again. System hang again. Test can never be completed.

tags: added: ce-qa-providers
Revision history for this message
Zygmunt Krynicki (zyga) wrote :

Jerry, can you explain how that is relevant to this bug? Plainbox is not at fault here (I'm assuming you're referring to plainbox-the-command-line-tool here).

This bug is not related to suspend resume functionality so I'd like to ask you to open a new issue instead.

Sylvain Pineau (sylvain-pineau)
tags: added: 14-04-blocker
Sylvain Pineau (sylvain-pineau)
Changed in checkbox-gui:
assignee: Zygmunt Krynicki (zkrynicki) → Sylvain Pineau (sylvain-pineau)
status: Confirmed → In Progress
Revision history for this message
Sylvain Pineau (sylvain-pineau) wrote :

To restore the whitelist ordering, the idea is to use the test selection as the mask for the desired job list (The same sort of fix applied to checkbox-ng). And a QSet replacing the frozenset to improve the lookup speed.

Sylvain Pineau (sylvain-pineau)
Changed in checkbox-gui:
status: In Progress → Fix Committed
Sylvain Pineau (sylvain-pineau)
Changed in checkbox-gui:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.