Skip to content

Commit

Permalink
CR improvement
Browse files Browse the repository at this point in the history
  • Loading branch information
programmer04 committed May 9, 2023
1 parent 5188fe6 commit 68f9362
Showing 1 changed file with 10 additions and 13 deletions.
23 changes: 10 additions & 13 deletions pkg/utils/kubernetes/networking/services.go
Original file line number Diff line number Diff line change
Expand Up @@ -93,37 +93,34 @@ func WaitForConnectionOnServicePort(ctx context.Context, c kubernetes.Interface,
// Recreate EndpointSlices for the lb service with proper metadata.
tempEndpointSlices := endpointSlices.DeepCopy().Items
for i := range tempEndpointSlices {
epsName := fmt.Sprintf("%s-%d", lbServiceName, i)
tempEndpointSlices[i].ObjectMeta = metav1.ObjectMeta{
Namespace: namespace,
Name: fmt.Sprintf("%s-%d", lbServiceName, i),
Name: epsName,
Labels: map[string]string{
discoveryv1.LabelServiceName: lbServiceName, // Maps EndpointSlice to Service.
correspondingSvcNameLabel: name,
},
}
_, err = c.DiscoveryV1().EndpointSlices(namespace).Create(ctx, &tempEndpointSlices[i], metav1.CreateOptions{})
if err != nil {
if _, err = c.DiscoveryV1().EndpointSlices(namespace).Create(ctx, &tempEndpointSlices[i], metav1.CreateOptions{}); err != nil {
return err
}
}

defer func() {
for _, eps := range tempEndpointSlices {
err := c.DiscoveryV1().EndpointSlices(namespace).Delete(ctx, eps.Name, metav1.DeleteOptions{})
// For each successfully created temporary EndpointSlice ensure deletion on return from the function.
defer func(namespace, epsName string) {
err := c.DiscoveryV1().EndpointSlices(namespace).Delete(ctx, epsName, metav1.DeleteOptions{})
if err != nil && !errors.IsNotFound(err) {
fmt.Printf("failed to delete endpoints %s/%s after testing, error %v\n",
namespace, eps.Name, err,
fmt.Printf("failed to delete EndpointSlice %s/%s after testing, error %v\n",
namespace, epsName, err,
)
}
}
}()
}(namespace, epsName)
}
}

_, err = c.CoreV1().Services(namespace).Create(ctx, tempLoadBalancer, metav1.CreateOptions{})
if err != nil {
return err
}

defer func() {
err := c.CoreV1().Services(namespace).Delete(ctx, lbServiceName, metav1.DeleteOptions{})
if err != nil && !errors.IsNotFound(err) {
Expand Down

0 comments on commit 68f9362

Please sign in to comment.