testShell too slow

Bug #1597366 reported by Michał Sawicz
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
unity8 (Ubuntu)
Fix Released
High
Albert Astals Cid

Bug Description

Our CI turnaround has gotten worse - to over two hours for a single MP.

Part of this is testShell, which takes 25mins *after* all other tests finished already. That suite alone takes 50mins [1]:

00:22:21.633 PASS : qmltestrunner::Shell::initTestCase()
[...]
00:48:33.484 ********* Finished testing of ListViewWithPageHeaderTest *********
[...]
01:12:00.463 PASS : qmltestrunner::Shell::cleanupTestCase()

We need to split this suite up at least, apart from speeding it up.

[1] https://unity8-jenkins.ubuntu.com/job/test-0-autopkgtest/1151/label=amd64,release=vivid+overlay,testname=qmluitests.sh/consoleFull

ProblemType: Bug
DistroRelease: Ubuntu 16.04
Package: unity8 8.12+16.04.20160627.5-0ubuntu1 [origin: LP-PPA-ci-train-ppa-service-stable-phone-overlay]
ProcVersionSignature: Ubuntu 4.4.0-24.43-generic 4.4.10
Uname: Linux 4.4.0-24-generic x86_64
NonfreeKernelModules: zfs zunicode zcommon znvpair zavl
ApportVersion: 2.20.1-0ubuntu2.1
Architecture: amd64
CurrentDesktop: Unity
Date: Wed Jun 29 16:22:02 2016
InstallationDate: Installed on 2016-05-06 (53 days ago)
InstallationMedia: Ubuntu 16.04 LTS "Xenial Xerus" - Release amd64 (20160420.1)
SourcePackage: unity8
UpgradeStatus: No upgrade log present (probably fresh install)

Related branches

Revision history for this message
Michał Sawicz (saviq) wrote :
Revision history for this message
Michał Sawicz (saviq) wrote :

Totals: 93 passed, 0 failed, 0 skipped, 0 blacklisted
********* Finished testing of qmltestrunner *********
ninja -C builddir xvfbtestShell 3358,84s user 46,51s system 170% cpu 33:15,88 total

WTF?

Revision history for this message
Michał Sawicz (saviq) wrote :

Looking at it it just stays there waiting for something, restarting the shell between phone and tablet modes.

Changed in unity8 (Ubuntu):
assignee: nobody → Albert Astals Cid (aacid)
status: Triaged → In Progress
Revision history for this message
Launchpad Janitor (janitor) wrote :
Download full text (3.3 KiB)

This bug was fixed in the package unity8 - 8.14+16.10.20160922-0ubuntu1

---------------
unity8 (8.14+16.10.20160922-0ubuntu1) yakkety; urgency=medium

  [ Albert Astals Cid ]
  * LVWPH: update clipItem height when list height changes (LP:
    #1606935)
  * Put the touchdown shape inside a loader
  * No need for the touchdown in the card for the cardtool
  * Add bottom gradient to the Show Less floating button
  * Remove artShapeSize from non cardtool cards
  * LVWPH: Fix case in which header was shown incorrectly
  * LVWPH: Fix items cut on top in the dash (or at least some instances
    of it)
  * Reduce calls to CardCreatorCache.getCardComponent while the
    component is being created (LP: #1615675)
  * Dash::test_cardIconStyle change compare into tryCompareFunction
  * Make PreviewActionsTest::test_comboButton more stable
  * Adapt onShiftedContentXChanged to work when the content changes very
    abruptly
  * Make tryGenericScopeView show the scope correctly
  * Make sure the spinner does not get too close to the title text (LP:
    #1597392)
  * Show "Pull to refresh" in white when overlaid in low luminance
    colors (LP: #1596849)
  * Make test_Shell non ultra slow again (LP: #1597366)
  * Improve findChild calls inside tryCompareFunction.

  [ Andrea Cimitan ]
  * override some Qt Components definitions so we can have sane default
    values for flick speeds
  * Elide the label text inside recent searches panel (LP: #1611796)
  * Add a PreviewSingleton module to store some data for previews (LP:
    #1595235)
  * Use PreviewSingleton for PreviewRatingInput and PreviewCommentInput
    (LP: #1595235)
  * rework GenericScopeView PullToRefresh test
  * use mouseFlick instead touchFlick for manage_dash_move_current
  * add a couple of waiting tricks for a flaky dash test

  [ Daniel d'Andrada ]
  * Set progress bar indeterminate when processing signal received. (LP:
    #1249349)
  * Implement cursor confinement (LP: #1590099)

  [ Josh Arenson ]
  * Add a frontend to the sessions model and enable a session chooser in
    the greeter.

  [ Lukáš Tinkl ]
  * Implement edge maximizing (aka window snapping) (LP: #1602628)
  * On the PC platform (as opposed to running on $devices), use the
    "mute" action instead of silent mode
  * Respect Fitt's law wrt the window control buttons in panel (LP:
    #1611959)
  * Fix 2 failing color-related tests

  [ Marco Trevisan (Treviño) ]
  * Indicators, mocks: add fake indicators menuitem to populate mocks
    with different menu types
  * IndicatorsClient: use PageHeader and ListItemLayout's

  [ Michael Terry ]
  * Support launching apps inside a unity8 session from the greeter and
    support emergency dialing inside the greeter.
  * Fix tryShell to actually show appropriate backgrounds for each mock
    user.
  * Add an indicator to the greeter when a user is logged in (only shown
    if we have more than one user)
  * Hide the greeter OSK if indicators are open and don't show the edge
    tutorial during an emergency call.
  * Switch from lockscreen PIN pad to a passcode entry box that uses the
    OSK.
  * Use the default system wallpaper instead of our custom one.
  * Update look of infographi...

Read more...

Changed in unity8 (Ubuntu):
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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