From f1a7e78d4b076fe661495dc28c6544b66f7fd67f Mon Sep 17 00:00:00 2001
From: Benjamin Elder <bentheelder@google.com>
Date: Sat, 17 Aug 2019 00:22:48 -0700
Subject: [PATCH 1/2] remove unnecessary ignore

---
 test/typecheck/main.go | 20 --------------------
 1 file changed, 20 deletions(-)

diff --git a/test/typecheck/main.go b/test/typecheck/main.go
index b6c55506af8..851b54926ac 100644
--- a/test/typecheck/main.go
+++ b/test/typecheck/main.go
@@ -103,26 +103,6 @@ func newAnalyzer(platform string) *analyzer {
 }
 
 func (a *analyzer) handleError(err error) {
-	if e, ok := err.(types.Error); ok {
-		// useful for some ignores:
-		// path := e.Fset.Position(e.Pos).String()
-		ignore := false
-		// TODO(rmmh): read ignores from a file, so this code can
-		// be Kubernetes-agnostic. Unused ignores should be treated as
-		// errors, to ensure coverage isn't overly broad.
-		if strings.Contains(e.Msg, "GetOpenAPIDefinitions") {
-			// TODO(rmmh): figure out why this happens.
-			// cmd/kube-apiserver/app/server.go:392:70
-			// test/integration/framework/master_utils.go:131:84
-			ignore = true
-		}
-		if ignore {
-			if *verbose {
-				fmt.Println("ignoring error:", err)
-			}
-			return
-		}
-	}
 	a.errors = append(a.errors, err.Error())
 	if *serial {
 		fmt.Fprintf(os.Stderr, "%sERROR(%s) %s\n", logPrefix, a.platform, err)

From 6b038f3d1eab375796bf18338fbf3158229d403c Mon Sep 17 00:00:00 2001
From: Benjamin Elder <bentheelder@google.com>
Date: Sat, 17 Aug 2019 00:33:24 -0700
Subject: [PATCH 2/2] swap BenTheElder for rmmh in typecheck / go-srcimporter
 reviewers

---
 test/typecheck/OWNERS             | 2 +-
 third_party/go-srcimporter/OWNERS | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/test/typecheck/OWNERS b/test/typecheck/OWNERS
index eeab9960bf8..ec2e1481307 100644
--- a/test/typecheck/OWNERS
+++ b/test/typecheck/OWNERS
@@ -1,8 +1,8 @@
 # See the OWNERS docs at https://go.k8s.io/owners
 
 reviewers:
+  - BenTheElder
   - cblecker
-  - rmmh
 approvers:
   - cblecker
   - rmmh
diff --git a/third_party/go-srcimporter/OWNERS b/third_party/go-srcimporter/OWNERS
index eeab9960bf8..ec2e1481307 100644
--- a/third_party/go-srcimporter/OWNERS
+++ b/third_party/go-srcimporter/OWNERS
@@ -1,8 +1,8 @@
 # See the OWNERS docs at https://go.k8s.io/owners
 
 reviewers:
+  - BenTheElder
   - cblecker
-  - rmmh
 approvers:
   - cblecker
   - rmmh