This job view page is being replaced by Spyglass soon. Check out the new job view.
PRcmluciano: Adding PathType to Ingress
ResultFAILURE
Tests 2 failed / 35 succeeded
Started2020-02-28 18:35
Elapsed54m56s
Revisione6592069f0624574fb7155a0f417a3c5c5963a3e
Refs 88587

Test Failures


verify gofmt 18s

make verify WHAT=gofmt
diff -u ./pkg/apis/networking/validation/validation.go.orig ./pkg/apis/networking/validation/validation.go
--- ./pkg/apis/networking/validation/validation.go.orig	2020-02-28 19:06:37.287034501 +0000
+++ ./pkg/apis/networking/validation/validation.go	2020-02-28 19:06:37.287034501 +0000
@@ -188,7 +188,6 @@
 // name.
 var ValidateIngressName = apimachineryvalidation.NameIsDNSSubdomain
 
-
 // Validation options for new beta to GA transitions for HTTP PathType
 type IngressValidationOptions struct {
 	requireRegexPath bool

Run ./hack/update-gofmt.sh
				
				Click to see stdout/stderrfrom junit_verify.xml

Filter through log files | View test history on testgrid


verify golint 2m24s

make verify WHAT=golint
Errors from golint:
pkg/apis/networking/validation/validation.go:192:1: comment on exported type IngressValidationOptions should be of the form "IngressValidationOptions ..." (with optional leading article)

Please review the above warnings. You can test via "golint" and commit the result.
If the above warnings do not make sense, you can exempt this package from golint
checking by adding it to hack/.golint_failures (if your reviewer is okay with it).
				
				Click to see stdout/stderrfrom junit_verify.xml

Filter through log files | View test history on testgrid


Show 35 Passed Tests