github/kubernetes
1
0
Fork 0
mirror of https://github.com/k3s-io/kubernetes.git synced 2025-07-23 03:41:45 +00:00
Code Issues Projects Releases Wiki Activity

fix integration endpointslicemirroring

Browse Source 
it can happen that there are multiple slices, and some of them
outdated, so we relax the test to check that at least one
has the corresponding fields mirrored.
This commit is contained in:
Antonio Ojea 2021-03-09 22:16:05 +01:00
parent eeccc9f9c0
commit 4c8be7793c
1 changed files with 7 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
test/integration/endpointslice/endpointslicemirroring_test.go
View File

@ -377,13 +377,13 @@ func TestEndpointSliceMirroringUpdates(t *testing.T) {
if !apiequality.Semantic.DeepEqual(epAddresses, sliceAddresses) {
t.Logf("Expected EndpointSlice to have the same IP addresses, expected %v got %v", epAddresses, sliceAddresses)
return false, nil
continue
}
// check labels were mirrored
if !isSubset(customEndpoints.Labels, endpointSlice.Labels) {
t.Logf("Expected EndpointSlice to mirror labels, expected %v to be in received %v", customEndpoints.Labels, endpointSlice.Labels)
return false, nil
continue
}
// check annotations but endpoints.kubernetes.io/last-change-trigger-time were mirrored
@ -396,10 +396,13 @@ func TestEndpointSliceMirroringUpdates(t *testing.T) {
}
if !apiequality.Semantic.DeepEqual(annotations, endpointSlice.Annotations) {
t.Logf("Expected EndpointSlice to mirror annotations, expected %v received %v", customEndpoints.Annotations, endpointSlice.Annotations)
return false, nil
continue
}
// This is a temporary workaround for https://github.com/kubernetes/kubernetes/issues/100033.
// TODO(robscott): When that issue is fixed, update this test to expect all EndpointSlices to meet this criteria.
return true, nil
}
return true, nil
return false, nil
})
if err != nil {
t.Fatalf("Timed out waiting for conditions: %v", err)