This job view page is being replaced by Spyglass soon. Check out the new job view.
PRandrewsykim: support configuration of kube-proxy IPVS tcp,tcpfin,udp timeout
ResultFAILURE
Tests 4 failed / 33 succeeded
Started2019-11-22 00:18
Elapsed42m18s
Revision460a3615900d6f1d4c26f6e102656d2967fdb3df
Refs 85517

Test Failures


verify bazel 23s

make verify WHAT=bazel
diff -Naupr -x _output /home/prow/go/src/k8s.io/kubernetes/pkg/proxy/apis/config/v1alpha1/BUILD /tmp/verify-bazel.epGvGm/go/src/k8s.io/kubernetes/pkg/proxy/apis/config/v1alpha1/BUILD
--- /home/prow/go/src/k8s.io/kubernetes/pkg/proxy/apis/config/v1alpha1/BUILD	2019-11-22 00:18:08.207725582 +0000
+++ /tmp/verify-bazel.epGvGm/go/src/k8s.io/kubernetes/pkg/proxy/apis/config/v1alpha1/BUILD	2019-11-22 00:18:39.782755786 +0000
@@ -25,7 +25,6 @@ go_library(
         "//staging/src/k8s.io/apimachinery/pkg/conversion:go_default_library",
         "//staging/src/k8s.io/apimachinery/pkg/runtime:go_default_library",
         "//staging/src/k8s.io/apimachinery/pkg/runtime/schema:go_default_library",
-        "//staging/src/k8s.io/component-base/config/v1alpha1:go_default_library",
         "//staging/src/k8s.io/kube-proxy/config/v1alpha1:go_default_library",
         "//vendor/k8s.io/utils/pointer:go_default_library",
     ],

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

Filter through log files | View test history on testgrid


verify generated-files 0.09s

make verify WHAT=generated-files
Unexpected dirty working directory:

  diff --git a/pkg/proxy/apis/config/v1alpha1/zz_generated.conversion.go b/pkg/proxy/apis/config/v1alpha1/zz_generated.conversion.go
  index 6384fab914..ca2fcdea11 100644
  --- a/pkg/proxy/apis/config/v1alpha1/zz_generated.conversion.go
  +++ b/pkg/proxy/apis/config/v1alpha1/zz_generated.conversion.go
  @@ -26,6 +26,7 @@ import (
   	v1 "k8s.io/apimachinery/pkg/apis/meta/v1"
   	conversion "k8s.io/apimachinery/pkg/conversion"
   	runtime "k8s.io/apimachinery/pkg/runtime"
  +	configv1alpha1 "k8s.io/component-base/config/v1alpha1"
   	v1alpha1 "k8s.io/kube-proxy/config/v1alpha1"
   	config "k8s.io/kubernetes/pkg/proxy/apis/config"
   )
  @@ -98,8 +99,7 @@ func autoConvert_v1alpha1_KubeProxyConfiguration_To_config_KubeProxyConfiguratio
   	out.EnableProfiling = in.EnableProfiling
   	out.ClusterCIDR = in.ClusterCIDR
   	out.HostnameOverride = in.HostnameOverride
  -	// TODO: Inefficient conversion - can we improve it?
  -	if err := s.Convert(&in.ClientConnection, &out.ClientConnection, 0); err != nil {
  +	if err := configv1alpha1.Convert_v1alpha1_ClientConnectionConfiguration_To_config_ClientConnectionConfiguration(&in.ClientConnection, &out.ClientConnection, s); err != nil {
   		return err
   	}
   	if err := Convert_v1alpha1_KubeProxyIPTablesConfiguration_To_config_KubeProxyIPTablesConfiguration(&in.IPTables, &out.IPTables, s); err != nil {
  @@ -136,8 +136,7 @@ func autoConvert_config_KubeProxyConfiguration_To_v1alpha1_KubeProxyConfiguratio
   	out.EnableProfiling = in.EnableProfiling
   	out.ClusterCIDR = in.ClusterCIDR
   	out.HostnameOverride = in.HostnameOverride
  -	// TODO: Inefficient conversion - can we improve it?
  -	if err := s.Convert(&in.ClientConnection, &out.ClientConnection, 0); err != nil {
  +	if err := configv1alpha1.Convert_config_ClientConnectionConfiguration_To_v1alpha1_ClientConnectionConfiguration(&in.ClientConnection, &out.ClientConnection, s); err != nil {
   		return err
   	}
   	if err := Convert_config_KubeProxyIPTablesConfiguration_To_v1alpha1_KubeProxyIPTablesConfiguration(&in.IPTables, &out.IPTables, s); err != nil {
!!! [1122 00:31:03] Call tree:
!!! [1122 00:31:03]  1: hack/make-rules/../../hack/verify-generated-files.sh:25 kube::util::ensure_clean_working_dir(...)
				
				Click to see stdout/stderrfrom junit_verify.xml

Filter through log files | View test history on testgrid


verify generated-files-remake 0.20s

make verify WHAT=generated-files-remake
Unexpected dirty working directory:

  diff --git a/pkg/proxy/apis/config/v1alpha1/zz_generated.conversion.go b/pkg/proxy/apis/config/v1alpha1/zz_generated.conversion.go
  index 6384fab914..ca2fcdea11 100644
  --- a/pkg/proxy/apis/config/v1alpha1/zz_generated.conversion.go
  +++ b/pkg/proxy/apis/config/v1alpha1/zz_generated.conversion.go
  @@ -26,6 +26,7 @@ import (
   	v1 "k8s.io/apimachinery/pkg/apis/meta/v1"
   	conversion "k8s.io/apimachinery/pkg/conversion"
   	runtime "k8s.io/apimachinery/pkg/runtime"
  +	configv1alpha1 "k8s.io/component-base/config/v1alpha1"
   	v1alpha1 "k8s.io/kube-proxy/config/v1alpha1"
   	config "k8s.io/kubernetes/pkg/proxy/apis/config"
   )
  @@ -98,8 +99,7 @@ func autoConvert_v1alpha1_KubeProxyConfiguration_To_config_KubeProxyConfiguratio
   	out.EnableProfiling = in.EnableProfiling
   	out.ClusterCIDR = in.ClusterCIDR
   	out.HostnameOverride = in.HostnameOverride
  -	// TODO: Inefficient conversion - can we improve it?
  -	if err := s.Convert(&in.ClientConnection, &out.ClientConnection, 0); err != nil {
  +	if err := configv1alpha1.Convert_v1alpha1_ClientConnectionConfiguration_To_config_ClientConnectionConfiguration(&in.ClientConnection, &out.ClientConnection, s); err != nil {
   		return err
   	}
   	if err := Convert_v1alpha1_KubeProxyIPTablesConfiguration_To_config_KubeProxyIPTablesConfiguration(&in.IPTables, &out.IPTables, s); err != nil {
  @@ -136,8 +136,7 @@ func autoConvert_config_KubeProxyConfiguration_To_v1alpha1_KubeProxyConfiguratio
   	out.EnableProfiling = in.EnableProfiling
   	out.ClusterCIDR = in.ClusterCIDR
   	out.HostnameOverride = in.HostnameOverride
  -	// TODO: Inefficient conversion - can we improve it?
  -	if err := s.Convert(&in.ClientConnection, &out.ClientConnection, 0); err != nil {
  +	if err := configv1alpha1.Convert_config_ClientConnectionConfiguration_To_v1alpha1_ClientConnectionConfiguration(&in.ClientConnection, &out.ClientConnection, s); err != nil {
   		return err
   	}
   	if err := Convert_config_KubeProxyIPTablesConfiguration_To_v1alpha1_KubeProxyIPTablesConfiguration(&in.IPTables, &out.IPTables, s); err != nil {
!!! [1122 00:31:03] Call tree:
!!! [1122 00:31:03]  1: hack/make-rules/../../hack/verify-generated-files-remake.sh:24 kube::util::ensure_clean_working_dir(...)
				
				Click to see stdout/stderrfrom junit_verify.xml

Filter through log files | View test history on testgrid


verify golint 2m7s

make verify WHAT=golint
Errors from golint:
pkg/util/ipvs/testing/fake.go:208:1: exported method FakeIPVS.ConfigureTimeouts should have comment or be unexported

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 33 Passed Tests