(courtesy of Martin Robinson, who introduced the notion of Garderner in WebKit world of fame) 1. Keep the tree green: the gardener on a given day will diagnose all bot redness and try to form a hypothesis for each failure. In my experience there are several types of failures: a. The GTK+ port is lacking some feature or has a bug: In this case the gardener will figure out exactly what the problem is, open a bug with as much information as possible including test diffs and links to relevant bugs. The gardener will CC any people involved in WebKitGTK+ who would might know how to fix the problem. b. A test is exposing a general WebKit bug that only GTK+ triggers (often the case with JavaScriptCore and inspector tests): In this case the gardener will leave a comment on the original bug with the test diff or stack trace (for something like an assertion failure) and offer to gather more information for the original patch author (even resorting to debugging directly on the bots if it cannot be reproduced locally). c. A test is missing expectations or needs new expectations: The gardener will generate new expectations after verifying that the tests seem to work when run manually or by looking at the expected results. d. A DRT feature is missing: The gardener will open a bug. The idea is that we will completely avoid skipped tests with no information about the failure. 2. If the tree isn't red, the gardener should be able to choose how to spend the day: a. Generating results for tests that do not have results (soon this will include pixel results as well). b. Diagnosing and opening bugs for failures that do not have open bugs (there's a huge list). This also includes unskipping tests that are passing. c. Implementing missing DRT features. There are a ton of these and we need to kill them. Using DRTSupportGtk we can do it without making API decisions. 3. At the end of the day the gardener should keep a very simple log for tomorrow's gardener, so he/she can follow up with any pending investigations. This will also help us keep track of newly skipped tests. See the [#Gardeninglog Gardening log] entry at the bottom of this wiki page. Check also chrome gardening instructions: http://dev.chromium.org/developers/how-tos/webkit-gardening == Gardening schedule == ||= Day =||= Gardener =|| || Mon 07 Apr || clopez || || Tue 08 Apr || clopez || || Wed 09 Apr || aperez || || Thu 10 Apr || dpino || || Fri 11 Apr || eocanha || || Sat 12 Apr || || || Sun 13 Apr || || || Mon 14 Apr || elima || || Tue 15 Apr || ltilve || || Wed 16 Apr || aperez || || Thu 17 Apr || elima || || Fri 18 Apr || dpino || || Sat 19 Apr || || || Sun 20 Apr || || || Mon 21 Apr || clopez || || Tue 22 Apr || ltilve || || Wed 23 Apr || clopez || || Thu 24 Apr || elima || || Fri 25 Apr || eocanha || || Sat 26 Apr || || || Sun 27 Apr || || || Mon 28 Apr || || || Tue 29 Apr || ltilve || || Wed 30 Apr || elima || || Thu 01 May || clopez || || Fri 02 May || || || Sat 03 May || || || Sun 04 May || || || Mon 05 May || || || Tue 06 May || ltilve || || Wed 07 May || || || Thu 08 May || || || Fri 09 May || ~~eocanha~~ || || Sat 10 May || || || Sun 11 May || || || Mon 12 May || || || Tue 13 May || ltilve || || Wed 14 May || clopez || || Thu 15 May || || || Fri 16 May || eocanha || || Sat 17 May || || || Sun 18 May || || || Mon 19 May || || || Tue 20 May || ltilve || || Wed 21 May || || || Thu 22 May || || || Fri 23 May || || || Sat 24 May || || || Sun 25 May || || || Mon 26 May || || || Tue 27 May || || || Wed 28 May || || || Thu 29 May || || || Fri 30 May || || || Sat 31 May || || || Sun 01 May || || == Gardening log == * Please put new log entries at the top. {{{#!td {{{#!td style="background: #eef" Done: * Rebaselined the tests affected by r168641, r168543, r168350 and r168674. * Opened the following bugs: * http://webkit.org/b/132909 * http://webkit.org/b/132910 * http://webkit.org/b/132914 * http://webkit.org/b/132919 * http://webkit.org/b/132921 * http://webkit.org/b/132923 * Updated the highdpi list for bug http://webkit.org/b/131347 * Commented on bug: http://webkit.org/b/132886 * Committed the following changes: * http://webkit.org/b/132925 Pending: * Check the remaining failures. -- !clopez@igalia.com - Thu May 15 2014 }}} {{{#!td style="background: #eef" Done: * Checked Layout test failures on r168676 * http://build.webkit.org/builders/GTK%20Linux%2064-bit%20Release%20%28Tests%29/builds/108 * Updated the following bugs: * http://webkit.org/b/132232 * Committed the following changes: * http://webkit.org/b/132858 * http://webkit.org/b/132873 -- !ltilve@igalia.com - Thu May 13 2014 }}} {{{#!td style="background: #eef" Done: * Checked Layout test failures on r168358 * http://build.webkit.org/builders/GTK%20Linux%2064-bit%20Release/builds/47158 * Updated the following bugs: * http://webkit.org/b/132328 * http://webkit.org/b/132255 * Opened the following bugs: * http://webkit.org/b/132329 * Committed the following changes: * http://webkit.org/b/132614 * http://webkit.org/b/132644 -- !ltilve@igalia.com - Thu May 7 2014 }}} {{{#!td style="background: #eef" Done: * r168118 Broke the GTK build. * Tracked here: http://webkit.org/b/132447 -- !clopez@igalia.com - Thu May 01 2014 }}} {{{#!td style="background: #eef" Done: * Rebaselined all the (128) tests affected by the new multi-column mode enabled on r168046. * Opened the following bugs: * http://webkit.org/b/132425 * Updated the following bugs: * http://webkit.org/b/132275 * http://webkit.org/b/77631 * http://webkit.org/b/128061 * Committed the following changes: * http://webkit.org/b/132426 Pending: * Check the remaining failures. -- !clopez@igalia.com - Thu May 01 2014 }}} {{{#!td style="background: #eef" Done: * Checked Layout test failures on r167923 * http://build.webkit.org/builders/GTK%20Linux%2064-bit%20Release/builds/46925 * Opened the following bugs: * http://webkit.org/b/132329 * http://webkit.org/b/132361 -- !ltilve@igalia.com - Thu Apr 29 2014 }}} {{{#!td style="background: #eef" Done: * Done: * Flagged tons of tests and created tons of bugs in three entries: * https://trac.webkit.org/changeset/167881 * https://trac.webkit.org/changeset/167882 * https://trac.webkit.org/changeset/167886 * https://trac.webkit.org/changeset/167888 -- !calvaris@igalia.com - Thu Apr 28 2014 }}} {{{#!td style="background: #eef" Done: * Opened the following bugs (flaky tests): * http://webkit.org/b/132181 * http://webkit.org/b/132182 * http://webkit.org/b/132183 * http://webkit.org/b/132184 * http://webkit.org/b/132185 * Updated the following test expectations for the previous bugs: * http://webkit.org/b/132187 Pending: * Check the remaining failures. -- !eocanha@igalia.com - Fri Apr 25 }}} {{{#!td style="background: #eef" Done: * Changed on the release bot (http://build.webkit.org/builders/GTK%20Linux%2064-bit%20Release) the following: * Removed the environment variable XVFB_SCREEN_DEPTH=8 * Removed deprecated environment variable WEBKIT_TESTFONTS * Did an upgrade of the system (Debian testing) * Opened the following bugs: * http://webkit.org/b/132126 * The removal of the environment variable XVFB_SCREEN_DEPTH=8 caused new failures on a dozen of tests. \\ I reported here all of them and updated the text expectations accordingly. * http://webkit.org/b/132128 * http://webkit.org/b/132134 * Updated the following bugs: * http://webkit.org/b/131223 * Committed the following changes: * http://webkit.org/b/132129 Notes: * If you notice that a test started failing on r167754 it could be because \\ of the removal of the XVFB_SCREEN_DEPTH=8 variable. Let me know that \\ or update http://webkit.org/b/132126 -- !clopez@igalia.com - Thu Apr 24 2014 }}} {{{#!td style="background: #eef" Done: * Checked Layout test failures on r167649 * http://build.webkit.org/builders/GTK%20Linux%2064-bit%20Release/builds/46703 * Opened the following bugs: * http://webkit.org/b/132119 * http://webkit.org/b/132135 * Committed the following changes: * http://webkit.org/b/132138 Pending: * Check the remaining failures. -- !elima@igalia.com - Thu Apr 24 2014 }}} {{{#!td style="background: #eef" Done: * Checked Layout test failures on r167649 * http://build.webkit.org/builders/GTK%20Linux%2064-bit%20Release/builds/46703 * Opened the following bugs: * http://webkit.org/b/132046 * http://webkit.org/b/132054 * http://webkit.org/b/132056 * Committed the following changes: * http://webkit.org/b/132058 * http://webkit.org/b/132051 Pending: * Check the remaining failures. -- !ltilve@igalia.com - Mon Apr 22 2014 }}} {{{#!td style="background: #eef" Done: * Checked Layout test failures on r167589 * http://build.webkit.org/builders/GTK%20Linux%2064-bit%20Release/builds/46673 * Opened the following bugs: * http://webkit.org/b/131934 * Updated the following bugs: * http://webkit.org/b/129330 * http://webkit.org/b/117689 * Committed the following changes: * http://webkit.org/b/131937 Pending: * Check the remaining failures. -- !clopez@igalia.com - Mon Apr 21 2014 }}} {{{#!td style="background: #eef" Done: * Checked API tests failures on r167424 * http://build.webkit.org/builders/GTK%20Linux%2064-bit%20Release/builds/46563 * Updated the following bugs: * http://webkit.org/b/131619 (Closed after checking that test is now passing) * http://webkit.org/b/118420 (Checked that test is now passing) * http://webkit.org/b/119840 (Checked that test is now passing) * Opened the following bugs: * http://webkit.org/b/131793 * http://webkit.org/b/131794 * Committed the following changes: * http://webkit.org/b/131795 -- !elima@igalia.com - Thu Apr 17 2014 }}} {{{#!td style="background: #eef" Done: * Opened the following bugs: * http://webkit.org/b/131737 * http://webkit.org/b/131743 Pending: * Check the remaining failures. -- !aperez@igalia.com - Wed Apr 16 2014 }}} {{{#!td style="background: #eef" Done: * Checked API tests failures on r167347 * http://build.webkit.org/builders/GTK%20Linux%2064-bit%20Release/builds/46527 * Opened the following bugs: * http://webkit.org/b/131731 * Updated the following bugs: * http://webkit.org/b/85037 * http://webkit.org/b/117689 * http://webkit.org/b/120305 * http://webkit.org/b/120306 * http://webkit.org/b/121970 * http://webkit.org/b/122571 * http://webkit.org/b/125068 * Committed the following changes: * http://webkit.org/b/131736 Pending: * Check the remaining failures. -- !clopez@igalia.com - Wed Apr 16 2014 }}} {{{#!td style="background: #eef" Done: * Checked failures on r167304 * http://build.webkit.org/builders/GTK%20Linux%2064-bit%20Release/builds/46496 * Committed the following changes: * http://webkit.org/b/131686 Pending: * Check the remaining failures. -- !ltilve@igalia.com - Tue Apr 15 2014 }}} {{{#!td style="background: #eef" Done: * Checked failures on r167246 * http://build.webkit.org/builders/GTK%20Linux%2064-bit%20Release/builds/46454 * Opened the following bugs: * http://webkit.org/b/131616 (this is a general bug to fix test-related scripts) * http://webkit.org/b/131619 * http://webkit.org/b/131620 * Committed the following changes: * http://webkit.org/b/131624 Pending: * Check the remaining failures. -- !elima@igalia.com - Fri Apr 14 2014 }}} {{{#!td style="background: #eef" Done: * Checked failures on r167125 * http://build.webkit.org/builders/GTK%20Linux%2064-bit%20Release/builds/46384 * Opened the following bugs: * http://webkit.org/b/131540 * http://webkit.org/b/131546 * http://webkit.org/b/131545 * Updated the following bugs: * http://webkit.org/b/95428 * http://webkit.org/b/130971 * Committed the following changes: * http://webkit.org/b/131549 Pending: * Check the remaining failures. -- !clopez@igalia.com - Fri Apr 11 2014 }}} {{{#!td style="background: #eef" Done: * Opened the following bugs: * http://webkit.org/b/131533 * http://webkit.org/b/131534 * http://webkit.org/b/131535 * Updated the following test expectations: * http://webkit.org/b/131537 Pending: * Check the remaining failures. -- !eocanha@igalia.com - Fri Apr 11 2014 }}} {{{#!td style="background: #eef" Done: * Checked both text-only and image-only failures on r167072 * http://build.webkit.org/builders/GTK%20Linux%2064-bit%20Release/builds/46344 * Opened the following bugs: * http://webkit.org/b/131496 * http://webkit.org/b/131498 * http://webkit.org/b/131501 * Updated the following test expectations: * http://webkit.org/b/131504 Pending: * Check the remaining failures. -- !clopez@igalia.com - Thu Apr 10 2014 }}} {{{#!td style="background: #eef" Done: * Checked the status of `garden-o-matic`: * Works fine for WebKitGTK+ port when using a Git clone. * Usage instructions in the [[Rebaseline]] page. !aperez@igalia.com - Wed 9 Apr 2014 }}} {{{#!td style="background: #eef" Done: * Started checking the reported Unexpected text-only failures on r166926 * http://build.webkit.org/builders/GTK%20Linux%2064-bit%20Release/builds/46239/ * Opened the following bugs: * http://webkit.org/b/131380 * http://webkit.org/b/131377 * Updated test expectations accordingly and rebaselined one test: * http://webkit.org/b/131382 Pending: * Check the remaining failures. -- !clopez@igalia.com - Tue Apr 8 2014 }}} }}}