aboutsummaryrefslogtreecommitdiffhomepage
path: root/site/dev
diff options
context:
space:
mode:
authorGravatar hcm <hcm@skia.org>2015-08-07 10:09:03 -0700
committerGravatar Commit bot <commit-bot@chromium.org>2015-08-07 10:09:03 -0700
commit767276e4bd97ebea690884d156862d5e6e8aa7fd (patch)
tree4bef43fdc0f7f3e9f92a0b887acf3a84c4613ded /site/dev
parent922c8b13c512c3287509936795735c1b31bedba9 (diff)
Gold Workflows Documentation
BUG=skia: NOTRY=true DOCS_PREVIEW= https://skia.org/?cl=1276663003 Review URL: https://codereview.chromium.org/1276663003
Diffstat (limited to 'site/dev')
-rw-r--r--site/dev/testing/BlameView.pngbin0 -> 192389 bytes
-rw-r--r--site/dev/testing/ByTest.pngbin0 -> 204022 bytes
-rw-r--r--site/dev/testing/ClusterConfig.pngbin0 -> 218419 bytes
-rw-r--r--site/dev/testing/Digests.pngbin0 -> 417670 bytes
-rw-r--r--site/dev/testing/DotDiagram.pngbin0 -> 319870 bytes
-rw-r--r--site/dev/testing/Grid.pngbin0 -> 328877 bytes
-rw-r--r--site/dev/testing/Ignores.pngbin0 -> 259943 bytes
-rw-r--r--site/dev/testing/IssueHighlight.pngbin0 -> 193256 bytes
-rw-r--r--site/dev/testing/Search.pngbin0 -> 210544 bytes
-rw-r--r--site/dev/testing/skiagold.md150
10 files changed, 150 insertions, 0 deletions
diff --git a/site/dev/testing/BlameView.png b/site/dev/testing/BlameView.png
new file mode 100644
index 0000000000..8884dbcf25
--- /dev/null
+++ b/site/dev/testing/BlameView.png
Binary files differ
diff --git a/site/dev/testing/ByTest.png b/site/dev/testing/ByTest.png
new file mode 100644
index 0000000000..75857f1674
--- /dev/null
+++ b/site/dev/testing/ByTest.png
Binary files differ
diff --git a/site/dev/testing/ClusterConfig.png b/site/dev/testing/ClusterConfig.png
new file mode 100644
index 0000000000..6854d9527d
--- /dev/null
+++ b/site/dev/testing/ClusterConfig.png
Binary files differ
diff --git a/site/dev/testing/Digests.png b/site/dev/testing/Digests.png
new file mode 100644
index 0000000000..efd811fb49
--- /dev/null
+++ b/site/dev/testing/Digests.png
Binary files differ
diff --git a/site/dev/testing/DotDiagram.png b/site/dev/testing/DotDiagram.png
new file mode 100644
index 0000000000..7bf8b990b8
--- /dev/null
+++ b/site/dev/testing/DotDiagram.png
Binary files differ
diff --git a/site/dev/testing/Grid.png b/site/dev/testing/Grid.png
new file mode 100644
index 0000000000..192d340a0b
--- /dev/null
+++ b/site/dev/testing/Grid.png
Binary files differ
diff --git a/site/dev/testing/Ignores.png b/site/dev/testing/Ignores.png
new file mode 100644
index 0000000000..a4200ca36d
--- /dev/null
+++ b/site/dev/testing/Ignores.png
Binary files differ
diff --git a/site/dev/testing/IssueHighlight.png b/site/dev/testing/IssueHighlight.png
new file mode 100644
index 0000000000..c21fda9b99
--- /dev/null
+++ b/site/dev/testing/IssueHighlight.png
Binary files differ
diff --git a/site/dev/testing/Search.png b/site/dev/testing/Search.png
new file mode 100644
index 0000000000..4e8491e3c9
--- /dev/null
+++ b/site/dev/testing/Search.png
Binary files differ
diff --git a/site/dev/testing/skiagold.md b/site/dev/testing/skiagold.md
new file mode 100644
index 0000000000..9fc9bbafbe
--- /dev/null
+++ b/site/dev/testing/skiagold.md
@@ -0,0 +1,150 @@
+Skia Gold
+=========
+
+Recommended Workflows
+---------------------
+### How to best use Gold for commonly faced problems ###
+
+These instructions will refer to various views which are accessible via the left
+navigation on [gold.skia.org](https://gold.skia.org/).
+View access is public, triage access is granted to
+Skia contributors. You must be logged in to triage.
+
+Problem #1: As sheriff, I need to triage and “assign” many incoming new images.
+-------------------------------------------------------------------------------
+Solution today:
+
+* Access the By Blame view to see digests needing triage and associated
+ owners/CLs
+ + Only untriaged digests will be shown by default
+ + Blame is not sorted in any particular order
+ + Digests are clustered by runs and the most minimal set of blame
+
+<img src=BlameView.png style="margin-left:30px" align="left" width="800"/> <br clear="left">
+
+* Select digests for triage
+ + Digests will be listed in order with largest difference first
+ + Click to open the digest view with detailed information
+
+<img src=Digests.png style="margin-left:40px" align="left" width="780"/> <br clear="left">
+
+* Open bugs for identified owner(s)
+ + The digest detail view has a link to open a bug from the UI
+ + Via the Gold UI or when manually entering a bug, copy the full URL of
+ single digest into a bug report
+ + The URL reference to the digest in Issue Tracker will link the bug to
+ the digest in Gold
+
+<img src="IssueHighlight.png" style="margin-left:60px" align="left" width="720" border=1/> <br clear="left">
+
+<br>
+
+Future improvements:
+
+* Smarter, more granular blamelist
+
+<br>
+
+Problem #2: As a developer, I need to land a CL that may change many images.
+----------------------------------------------------------------------------
+To find your results:
+
+* Immediately following commit, access the By Blame view to find untriaged
+ digest groupings associated with your ID
+* Click on one of the clusters including your CL to triage
+* Return to the By Blame view to walk through all untriaged digests involving
+ your change
+* Note: It is not yet implemented in the UI but possible to filter the view
+ by CL. Delete hashes in the URL to only include the hash for your CL.
+
+<img src=BlameView.png style="margin-left:30px" align="left" width="800"/> <br clear="left">
+
+To rebaseline images:
+
+* Access the Ignores view and create a new, short-interval (hours) ignore for
+ the most affected configuration(s)
+
+<img src=Ignores.png style="margin-left:30px" align="left" width="800"/> <br clear="left">
+
+
+* Click on the Ignore to bring up a search view filtered by the affected
+ configuration(s)
+* Mark untriaged images as positive (or negative if appropriate)
+* Follow one of two options for handling former positives:
+ + Leave former positives as-is and let them fall off with time if there is
+ low risk of recurrence
+ + Mark former positives as negative if needed to verify the change moving
+ forward
+
+Future improvements:
+
+* Trybot support prior to commit, with view limited to your CL
+* Pre-triage prior to commit that will persist when the CL lands
+
+<br>
+
+Problem #3: As a developer or infrastructure engineer, I need to add a new or updated config.
+---------------------------------------------------------------------------------------------
+(ie: new bot, test mode, environment change)
+
+Solution today:
+
+* Follow the process for rebaselining images:
+ + Wait for the bot/test/config to be committed and show up in the Gold UI
+ + Access the Ignores view and create a short-interval ignore for the
+ configuration(s)
+ + Triage the ignores for that config to identify positive images
+ + Delete the ignore
+
+Future improvements:
+
+* Introduction of a new or updated test can make use of try jobs and pre-triage.
+* New configs may be able to use these features as well.
+
+<br>
+
+Problem #4: As a developer, I need to analyze the details of a particular image digest.
+---------------------------------------------------------------------------------------
+Solution:
+
+* Access the By Test view
+
+<img src=ByTest.png style="margin-left:30px" align="left" width="800"/> <br clear="left">
+
+* Click the magnifier to filter by configuration
+* Access the Cluster view to see the distribution of digest results
+ + Use control-click to select and do a direct compare between data points
+ + Click on configurations under “parameters” to highlight data points and
+ compare
+
+<img src=ClusterConfig.png style="margin-left:30px" align="left" width="800"/> <br clear="left">
+
+* Access the Grid view to see NxN diffs
+
+<img src=Grid.png style="margin-left:30px" align="left" width="800"/> <br clear="left">
+
+* Access the Dot diagram to see history of commits for the trace
+ + Each dot represents a commit
+ + Each line represents a configuration
+ + Dot colors distinguish between digests
+
+<img src=DotDiagram.png style="margin-left:30px" align="left" width="800"/> <br clear="left">
+
+<br>
+
+Future improvements:
+
+* Large diff display of image vs image
+
+<br>
+
+Problem #5: As a developer, I need to find results for a particular configuration.
+----------------------------------------------------------------------------------
+Solution:
+
+* Access the Search view
+* Select any parameters desired to search across tests
+
+<img src=Search.png style="margin-left:30px" align="left" width="800"/> <br clear="left">
+
+