Changes between Version 3 and Version 4 of QtWebKitGardening
- Timestamp:
- Jun 1, 2011 6:51:23 AM (13 years ago)
Legend:
- Unmodified
- Added
- Removed
- Modified
-
QtWebKitGardening
v3 v4 1 1 '''NOTE: This is a proposal in progress.''' 2 2 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: 3 The 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 5 There are four main tasks for the gardener: 4 6 5 7 * Reacting to bot redness. … … 9 11 10 12 Let's look at each one in detail... 13 14 == Reacting to bot redness == 11 15 12 16 == Maintaining the LayoutTests skip lists == … … 39 43 40 44 For 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 48 The 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 50 The 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 52 Skip 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().