Changes between Version 3 and Version 4 of QtWebKitGardening


Ignore:
Timestamp:
Jun 1, 2011 6:51:23 AM (8 years ago)
Author:
andreas.kling@nokia.com
Comment:

--

Legend:

Unmodified
Added
Removed
Modified
  • QtWebKitGardening

    v3 v4  
    11'''NOTE: This is a proposal in progress.'''
    22
    3 The purpose of QtWebKit gardening is to keep the Qt build and test bots green. There are a number of tasks for the gardener:
     3The purpose of QtWebKit gardening is to keep the Qt build and test bots green. It's a responsibility shared by all QtWebKit developers, though to be efficient you do need WebKit commit privileges.
     4
     5There are four main tasks for the gardener:
    46
    57 * Reacting to bot redness.
     
    911
    1012Let's look at each one in detail...
     13
     14== Reacting to bot redness ==
    1115
    1216== Maintaining the LayoutTests skip lists ==
     
    3943
    4044For the text results, this is often as simple as running "webkit-patch rebaseline" and grabbing the results from the Qt bot. The screenshot (PNG) results are a little trickier, as they are currently not dumped by the bot, and need to be generated manually. These tests should be generated in an environment equivalent to that of the bot, i.e [http://webkit.sed.hu/blog/20101028/qtwebkit-builder-and-tester-virtual-machine the Q-BAT VM from Szeged].
     45
     46== Maintaining the Qt API tests ==
     47
     48The Qt API tests live in Source/WebKit/qt/tests/ and are part of the bot rotation. Although a failing API test currently does not make the bot flip red, it's important that we maintain the tests regardless.
     49
     50The procedure is very similar to the LayoutTests, though we should have slightly higher tolerance for non-Qt WebKit developers inadvertently breaking our API tests. (This because our API takes some "artistic liberties" with regard to layering violations in more than a few places.)
     51
     52Skip the failing test (or the failing sub-portion of the test if you are able to) by adding a [http://doc.qt.nokia.com/4.7/qtest.html#QSKIP] QSKIP() statement. Open a bug about the failure, "Qt, QtTriaged, Regression", CC'ing the appropriate people, and adding a comment with a reference to the bug# along with the QSKIP().