github/kubernetes
4
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

kubectl/describe: use endpointslices for describing endpoints

Browse Source 
Signed-off-by: Daman Arora <aroradaman@gmail.com>
This commit is contained in:
Daman Arora
2024-05-01 15:09:02 +05:30
parent df417aa9b8
commit 3236891023
2 changed files with 124 additions and 42 deletions
Download Patch File Download Diff File Expand all files Collapse all files

52
staging/src/k8s.io/kubectl/pkg/describe/describe.go
View File

@@ -2603,7 +2603,9 @@ func (i *IngressDescriber) Describe(namespace, name string, describerSettings De
} }
func (i *IngressDescriber) describeBackendV1beta1(ns string, backend *networkingv1beta1.IngressBackend) string { func (i *IngressDescriber) describeBackendV1beta1(ns string, backend *networkingv1beta1.IngressBackend) string {
endpoints, err := i.client.CoreV1().Endpoints(ns).Get(context.TODO(), backend.ServiceName, metav1.GetOptions{}) endpointSliceList, err := i.client.DiscoveryV1().EndpointSlices(ns).List(context.TODO(), metav1.ListOptions{
LabelSelector: fmt.Sprintf("%s=%s", discoveryv1.LabelServiceName, backend.ServiceName),
})
if err != nil { if err != nil {
return fmt.Sprintf("<error: %v>", err) return fmt.Sprintf("<error: %v>", err)
} }
@@ -2625,14 +2627,16 @@ func (i *IngressDescriber) describeBackendV1beta1(ns string, backend *networking
} }
} }
} }
return formatEndpoints(endpoints, sets.NewString(spName)) return formatEndpointSlices(endpointSliceList.Items, sets.New(spName))
} }
func (i *IngressDescriber) describeBackendV1(ns string, backend *networkingv1.IngressBackend) string { func (i *IngressDescriber) describeBackendV1(ns string, backend *networkingv1.IngressBackend) string {
if backend.Service != nil { if backend.Service != nil {
sb := serviceBackendStringer(backend.Service) sb := serviceBackendStringer(backend.Service)
endpoints, err := i.client.CoreV1().Endpoints(ns).Get(context.TODO(), backend.Service.Name, metav1.GetOptions{}) endpointSliceList, err := i.client.DiscoveryV1().EndpointSlices(ns).List(context.TODO(), metav1.ListOptions{
LabelSelector: fmt.Sprintf("%s=%s", discoveryv1.LabelServiceName, backend.Service.Name),
})
if err != nil { if err != nil {
return fmt.Sprintf("%v (<error: %v>)", sb, err) return fmt.Sprintf("%v (<error: %v>)", sb, err)
} }
@@ -2649,7 +2653,7 @@ func (i *IngressDescriber) describeBackendV1(ns string, backend *networkingv1.In
spName = sp.Name spName = sp.Name
} }
} }
ep := formatEndpoints(endpoints, sets.NewString(spName)) ep := formatEndpointSlices(endpointSliceList.Items, sets.New(spName))
return fmt.Sprintf("%s (%s)", sb, ep) return fmt.Sprintf("%s (%s)", sb, ep)
} }
if backend.Resource != nil { if backend.Resource != nil {
@@ -2961,12 +2965,14 @@ func (d *ServiceDescriber) Describe(namespace, name string, describerSettings De
return "", err return "", err
} }
endpoints, _ := d.CoreV1().Endpoints(namespace).Get(context.TODO(), name, metav1.GetOptions{}) endpointSliceList, _ := d.DiscoveryV1().EndpointSlices(namespace).List(context.TODO(), metav1.ListOptions{
LabelSelector: fmt.Sprintf("%s=%s", discoveryv1.LabelServiceName, name),
})
var events *corev1.EventList var events *corev1.EventList
if describerSettings.ShowEvents { if describerSettings.ShowEvents {
events, _ = searchEvents(d.CoreV1(), service, describerSettings.ChunkSize) events, _ = searchEvents(d.CoreV1(), service, describerSettings.ChunkSize)
} }
return describeService(service, endpoints, events) return describeService(service, endpointSliceList.Items, events)
} }
func buildIngressString(ingress []corev1.LoadBalancerIngress) string { func buildIngressString(ingress []corev1.LoadBalancerIngress) string {
@@ -2985,10 +2991,7 @@ func buildIngressString(ingress []corev1.LoadBalancerIngress) string {
return buffer.String() return buffer.String()
} }
func describeService(service *corev1.Service, endpoints *corev1.Endpoints, events *corev1.EventList) (string, error) { func describeService(service *corev1.Service, endpointSlices []discoveryv1.EndpointSlice, events *corev1.EventList) (string, error) {
if endpoints == nil {
endpoints = &corev1.Endpoints{}
}
return tabbedString(func(out io.Writer) error { return tabbedString(func(out io.Writer) error {
w := NewPrefixWriter(out) w := NewPrefixWriter(out)
w.Write(LEVEL_0, "Name:\t%s\n", service.Name) w.Write(LEVEL_0, "Name:\t%s\n", service.Name)
@@ -3049,7 +3052,7 @@ func describeService(service *corev1.Service, endpoints *corev1.Endpoints, event
if sp.NodePort != 0 { if sp.NodePort != 0 {
w.Write(LEVEL_0, "NodePort:\t%s\t%d/%s\n", name, sp.NodePort, sp.Protocol) w.Write(LEVEL_0, "NodePort:\t%s\t%d/%s\n", name, sp.NodePort, sp.Protocol)
} }
w.Write(LEVEL_0, "Endpoints:\t%s\n", formatEndpoints(endpoints, sets.NewString(sp.Name))) w.Write(LEVEL_0, "Endpoints:\t%s\n", formatEndpointSlices(endpointSlices, sets.New(sp.Name)))
} }
w.Write(LEVEL_0, "Session Affinity:\t%s\n", service.Spec.SessionAffinity) w.Write(LEVEL_0, "Session Affinity:\t%s\n", service.Spec.SessionAffinity)
if service.Spec.ExternalTrafficPolicy != "" { if service.Spec.ExternalTrafficPolicy != "" {
@@ -5419,39 +5422,38 @@ func translateTimestampSince(timestamp metav1.Time) string {
} }
// Pass ports=nil for all ports. // Pass ports=nil for all ports.
func formatEndpoints(endpoints *corev1.Endpoints, ports sets.String) string { func formatEndpointSlices(endpointSlices []discoveryv1.EndpointSlice, ports sets.Set[string]) string {
if len(endpoints.Subsets) == 0 { if len(endpointSlices) == 0 {
return "<none>" return "<none>"
} }
list := []string{} var list []string
max := 3 max := 3
more := false more := false
count := 0 count := 0
for i := range endpoints.Subsets { for i := range endpointSlices {
ss := &endpoints.Subsets[i] if len(endpointSlices[i].Ports) == 0 {
if len(ss.Ports) == 0 {
// It's possible to have headless services with no ports. // It's possible to have headless services with no ports.
for i := range ss.Addresses { for j := range endpointSlices[i].Endpoints {
if len(list) == max { if len(list) == max {
more = true more = true
} }
if !more { if !more {
list = append(list, ss.Addresses[i].IP) list = append(list, endpointSlices[i].Endpoints[j].Addresses[0])
} }
count++ count++
} }
} else { } else {
// "Normal" services with ports defined. // "Normal" services with ports defined.
for i := range ss.Ports { for j := range endpointSlices[i].Ports {
port := &ss.Ports[i] port := endpointSlices[i].Ports[j]
if ports == nil || ports.Has(port.Name) { if ports == nil || ports.Has(*port.Name) {
for i := range ss.Addresses { for k := range endpointSlices[i].Endpoints {
if len(list) == max { if len(list) == max {
more = true more = true
} }
addr := &ss.Addresses[i] addr := endpointSlices[i].Endpoints[k].Addresses[0]
if !more { if !more {
hostPort := net.JoinHostPort(addr.IP, strconv.Itoa(int(port.Port))) hostPort := net.JoinHostPort(addr, strconv.Itoa(int(*port.Port)))
list = append(list, hostPort) list = append(list, hostPort)
} }
count++ count++

106
staging/src/k8s.io/kubectl/pkg/describe/describe_test.go
View File

@@ -657,6 +657,7 @@ func TestDescribeService(t *testing.T) {
testCases := []struct { testCases := []struct {
name string name string
service *corev1.Service service *corev1.Service
endpointSlices []*discoveryv1.EndpointSlice
expected string expected string
}{ }{
{ {
@@ -684,6 +685,25 @@ func TestDescribeService(t *testing.T) {
HealthCheckNodePort: 32222, HealthCheckNodePort: 32222,
}, },
}, },
endpointSlices: []*discoveryv1.EndpointSlice{{
ObjectMeta: metav1.ObjectMeta{
Name: "bar-abcde",
Namespace: "foo",
Labels: map[string]string{
"kubernetes.io/service-name": "bar",
},
},
Endpoints: []discoveryv1.Endpoint{
{Addresses: []string{"10.244.0.1"}},
{Addresses: []string{"10.244.0.2"}},
{Addresses: []string{"10.244.0.3"}},
},
Ports: []discoveryv1.EndpointPort{{
Name: ptr.To("port-tcp"),
Port: ptr.To[int32](9527),
Protocol: ptr.To(corev1.ProtocolTCP),
}},
}},
expected: dedent.Dedent(` expected: dedent.Dedent(`
Name: bar Name: bar
Namespace: foo Namespace: foo
@@ -698,7 +718,7 @@ func TestDescribeService(t *testing.T) {
Port: port-tcp 8080/TCP Port: port-tcp 8080/TCP
TargetPort: 9527/TCP TargetPort: 9527/TCP
NodePort: port-tcp 31111/TCP NodePort: port-tcp 31111/TCP
Endpoints: <none> Endpoints: 10.244.0.1:9527,10.244.0.2:9527,10.244.0.3:9527
Session Affinity: None Session Affinity: None
External Traffic Policy: Local External Traffic Policy: Local
HealthCheck NodePort: 32222 HealthCheck NodePort: 32222
@@ -730,6 +750,45 @@ func TestDescribeService(t *testing.T) {
HealthCheckNodePort: 32222, HealthCheckNodePort: 32222,
}, },
}, },
endpointSlices: []*discoveryv1.EndpointSlice{
{
ObjectMeta: metav1.ObjectMeta{
Name: "bar-12345",
Namespace: "foo",
Labels: map[string]string{
"kubernetes.io/service-name": "bar",
},
},
Endpoints: []discoveryv1.Endpoint{
{Addresses: []string{"10.244.0.1"}},
{Addresses: []string{"10.244.0.2"}},
},
Ports: []discoveryv1.EndpointPort{{
Name: ptr.To("port-tcp"),
Port: ptr.To[int32](9527),
Protocol: ptr.To(corev1.ProtocolUDP),
}},
},
{
ObjectMeta: metav1.ObjectMeta{
Name: "bar-54321",
Namespace: "foo",
Labels: map[string]string{
"kubernetes.io/service-name": "bar",
},
},
Endpoints: []discoveryv1.Endpoint{
{Addresses: []string{"10.244.0.3"}},
{Addresses: []string{"10.244.0.4"}},
{Addresses: []string{"10.244.0.5"}},
},
Ports: []discoveryv1.EndpointPort{{
Name: ptr.To("port-tcp"),
Port: ptr.To[int32](9527),
Protocol: ptr.To(corev1.ProtocolUDP),
}},
},
},
expected: dedent.Dedent(` expected: dedent.Dedent(`
Name: bar Name: bar
Namespace: foo Namespace: foo
@@ -744,7 +803,7 @@ func TestDescribeService(t *testing.T) {
Port: port-tcp 8080/TCP Port: port-tcp 8080/TCP
TargetPort: targetPort/TCP TargetPort: targetPort/TCP
NodePort: port-tcp 31111/TCP NodePort: port-tcp 31111/TCP
Endpoints: <none> Endpoints: 10.244.0.1:9527,10.244.0.2:9527,10.244.0.3:9527 + 2 more...
Session Affinity: None Session Affinity: None
External Traffic Policy: Local External Traffic Policy: Local
HealthCheck NodePort: 32222 HealthCheck NodePort: 32222
@@ -776,6 +835,23 @@ func TestDescribeService(t *testing.T) {
HealthCheckNodePort: 32222, HealthCheckNodePort: 32222,
}, },
}, },
endpointSlices: []*discoveryv1.EndpointSlice{{
ObjectMeta: metav1.ObjectMeta{
Name: "bar-123ab",
Namespace: "foo",
Labels: map[string]string{
"kubernetes.io/service-name": "bar",
},
},
Endpoints: []discoveryv1.Endpoint{
{Addresses: []string{"10.244.0.1"}},
},
Ports: []discoveryv1.EndpointPort{{
Name: ptr.To("port-tcp"),
Port: ptr.To[int32](9527),
Protocol: ptr.To(corev1.ProtocolTCP),
}},
}},
expected: dedent.Dedent(` expected: dedent.Dedent(`
Name: bar Name: bar
Namespace: foo Namespace: foo
@@ -790,7 +866,7 @@ func TestDescribeService(t *testing.T) {
Port: port-tcp 8080/TCP Port: port-tcp 8080/TCP
TargetPort: targetPort/TCP TargetPort: targetPort/TCP
NodePort: port-tcp 31111/TCP NodePort: port-tcp 31111/TCP
Endpoints: <none> Endpoints: 10.244.0.1:9527
Session Affinity: None Session Affinity: None
External Traffic Policy: Local External Traffic Policy: Local
HealthCheck NodePort: 32222 HealthCheck NodePort: 32222
@@ -849,7 +925,11 @@ func TestDescribeService(t *testing.T) {
} }
for _, tc := range testCases { for _, tc := range testCases {
t.Run(tc.name, func(t *testing.T) { t.Run(tc.name, func(t *testing.T) {
fakeClient := fake.NewSimpleClientset(tc.service) objects := []runtime.Object{tc.service}
for i := range tc.endpointSlices {
objects = append(objects, tc.endpointSlices[i])
}
fakeClient := fake.NewSimpleClientset(objects...)
c := &describeClient{T: t, Namespace: "foo", Interface: fakeClient} c := &describeClient{T: t, Namespace: "foo", Interface: fakeClient}
d := ServiceDescriber{c} d := ServiceDescriber{c}
out, err := d.Describe("foo", "bar", DescriberSettings{ShowEvents: true}) out, err := d.Describe("foo", "bar", DescriberSettings{ShowEvents: true})
@@ -3087,7 +3167,7 @@ Rules:
Host Path Backends Host Path Backends
---- ---- -------- ---- ---- --------
foo.bar.com foo.bar.com
/foo default-backend:80 (<error: endpoints "default-backend" not found>) /foo default-backend:80 (<error: services "default-backend" not found>)
Annotations: <none> Annotations: <none>
Events: <none>` + "\n", Events: <none>` + "\n",
}, },
@@ -3103,7 +3183,7 @@ Rules:
Host Path Backends Host Path Backends
---- ---- -------- ---- ---- --------
foo.bar.com foo.bar.com
/foo default-backend:80 (<error: endpoints "default-backend" not found>) /foo default-backend:80 (<error: services "default-backend" not found>)
Annotations: <none> Annotations: <none>
Events: <none>` + "\n", Events: <none>` + "\n",
}, },
@@ -3216,12 +3296,12 @@ Labels: <none>
Namespace: foo Namespace: foo
Address: Address:
Ingress Class: test Ingress Class: test
Default backend: default-backend:80 (<error: endpoints "default-backend" not found>) Default backend: default-backend:80 (<error: services "default-backend" not found>)
Rules: Rules:
Host Path Backends Host Path Backends
---- ---- -------- ---- ---- --------
foo.bar.com foo.bar.com
/foo default-backend:80 (<error: endpoints "default-backend" not found>) /foo default-backend:80 (<error: services "default-backend" not found>)
Annotations: <none> Annotations: <none>
Events: <none>` + "\n", Events: <none>` + "\n",
}, },
@@ -3301,7 +3381,7 @@ Rules:
Host Path Backends Host Path Backends
---- ---- -------- ---- ---- --------
foo.bar.com foo.bar.com
/foo default-backend:80 (<error: endpoints "default-backend" not found>) /foo default-backend:80 (<error: services "default-backend" not found>)
Annotations: <none> Annotations: <none>
Events: <none>` + "\n", Events: <none>` + "\n",
}, },
@@ -3321,11 +3401,11 @@ Labels: <none>
Namespace: foo Namespace: foo
Address: Address:
Ingress Class: test Ingress Class: test
Default backend: default-backend:80 (<error: endpoints "default-backend" not found>) Default backend: default-backend:80 (<error: services "default-backend" not found>)
Rules: Rules:
Host Path Backends Host Path Backends
---- ---- -------- ---- ---- --------
* * default-backend:80 (<error: endpoints "default-backend" not found>) * * default-backend:80 (<error: services "default-backend" not found>)
Annotations: <none> Annotations: <none>
Events: <none> Events: <none>
`, `,
@@ -3369,11 +3449,11 @@ Labels: <none>
Namespace: foo Namespace: foo
Address: Address:
Ingress Class: <none> Ingress Class: <none>
Default backend: default-backend:80 (<error: endpoints "default-backend" not found>) Default backend: default-backend:80 (<error: services "default-backend" not found>)
Rules: Rules:
Host Path Backends Host Path Backends
---- ---- -------- ---- ---- --------
* * default-backend:80 (<error: endpoints "default-backend" not found>) * * default-backend:80 (<error: services "default-backend" not found>)
Annotations: <none> Annotations: <none>
Events: <none> Events: <none>
`, `,