]> git.proxmox.com Git - mirror_zfs.git/commitdiff
Update codecov.io behavior
authorBrian Behlendorf <behlendorf1@llnl.gov>
Wed, 18 Oct 2017 17:07:02 +0000 (10:07 -0700)
committerGitHub <noreply@github.com>
Wed, 18 Oct 2017 17:07:02 +0000 (10:07 -0700)
Update the codecov.yml included in the repository to behave as
originally intended.  This can be refined as needed.

* Always post coverage results to the GitHub PR after two builds
  have been uploaded.  This is the normal case since there will
  be a build uploaded for both kernel and user coverage results.

* Adjust red -> yellow -> green coloring in the web interface.
  Due to the number of unlikely error conditions which are hard
  to force consider 90% coverage an excellent level of coverage.

* Allow a 1% variance in coverage between test runs.  This is
  approximately 10x larger than the typical variance observed
  which leaves us a reasonable margin to prevent false positives.

* Always post a new smaller comment to PRs which does not include
  a file list.  Old coverage reports are removed.

Reviewed by: Prakash Surya <prakash.surya@delphix.com>
Signed-off-by: Brian Behlendorf <behlendorf1@llnl.gov>
Closes #6765

.github/codecov.yml

index e74c59a0fc9ba1172e84d40ca7692f471e9a0c72..ab0f11d7966ffea8c1f2196194226311137423be 100644 (file)
@@ -1,30 +1,21 @@
 codecov:
   notify:
-    require_ci_to_pass: no
+    require_ci_to_pass: false  # always post
+    after_n_builds: 2          # user and kernel
 
 coverage:
-  precision: 2
-  round: down
-  range: "50...100"
+  precision: 2                 # 2 digits of precision
+  range: "50...90"             # red -> yellow -> green
 
   status:
     project:
       default:
-        threshold: 1%
+        threshold: 1%          # allow 1% coverage variance
 
     patch:
       default:
-        threshold: 1%
-
-parsers:
-  gcov:
-    branch_detection:
-      conditional: yes
-      loop: yes
-      method: no
-      macro: no
+        threshold: 1%          # allow 1% coverage variance
 
 comment:
-  layout: "header, sunburst, diff"
-  behavior: default
-  require_changes: no
+  layout: "reach, diff, flags, footer"
+  behavior: "new"              # delete old, post new