Change:
Test pull-kubernetes-federation-e2e-gce for PR is failing. pull-kubernetes-federation-e2e-gce https://k8s- pull-kubernetes-federation-e2e-gce test this
/sig federation /assign
cc
Details on the types of failures and frequency: In the last two days in particular, the following tests have been timing out: [] Federated Services [Feature:Federation] with clusters Federated Service should create and update matching services in underlying clusters [] Federated Services [Feature:Federation] with clusters DNS should be able to discover a federated service [] Federated Services [Feature:Federation] with clusters Federated Service should recreate service shard in underlying clusters when service shard is deleted cc:
Thanks. And yes we are aware of it. is working on a fix - PR .
Hi I'm trying to find out how to follow up on this. Will there be some sort of notification for the PRs affected this?
I just re-queue that test on your PR. note that the test isn't blocking, so your PR can be reviewed/merged without this test passing.
good to know. Thanks!
you can't re-open an issue unless you authored it or you are assigned to it. <detailsIn response to : /reopen Instructions for interacting with me using PR comments are available . If you have questions or suggestions related to my behavior, please file an issue against the repository. I understand the commands that are listed . </details
/reopen
Reopening to have an issue to track this. This was thought to be resolved, and was made blocking in There's still something up, and it was reverted in I believe sig-federation is going to look at this more next week in order to figure out a more permanent solution to this.
Closing this in favor of issue which has linked runs which more accurately describe the problem.

Discussion:
} } } <ins> // If ExternalTrafficPolicy is not set in federated service, use the ExternalTrafficPolicy // defaulted to in federated cluster. if desiredService.Spec.ExternalTrafficPolicy == v1.ServiceExternalTrafficPolicyType("") { desiredService.Spec.ExternalTrafficPolicy = clusterService.Spec.ExternalTrafficPolicy } </ins> // Update existing service, if needed. if !Equivalent(desiredService, clusterService) {

// equivalent returns true if the two services are equivalent.  Fields which are expected to differ between // federated services and the underlying cluster services (e.g. ClusterIP, NodePort) are ignored. <del> func equivalent(federationService, clusterService v1.Service) bool { clusterService.Spec.ClusterIP = federationService.Spec.ClusterIP for i := range clusterService.Spec.Ports { clusterService.Spec.Ports[i].NodePort = federationService.Spec.Ports[i].NodePort </del> <ins> func equivalent(clusterService, federationService v1.Service) bool { federationService.Spec.ClusterIP = clusterService.Spec.ClusterIP for i := range federationService.Spec.Ports { federationService.Spec.Ports[i].NodePort = clusterService.Spec.Ports[i].NodePort } if federationService.Spec.ExternalTrafficPolicy == v1.ServiceExternalTrafficPolicyType("") { federationService.Spec.ExternalTrafficPolicy = clusterService.Spec.ExternalTrafficPolicy </ins> } if federationService.Name != clusterService.Name || federationService.Namespace != clusterService.Namespace {