Change:
<ins> /* Copyright 2021 The Kubernetes Authors. Licensed under the Apache License, Version 2.0 (the "License"); you may not use this file except in compliance with the License. You may obtain a copy of the License at http://www.apache.org/licenses/LICENSE-2.0 Unless required by applicable law or agreed to in writing, software distributed under the License is distributed on an "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the License for the specific language governing permissions and limitations under the License. */ package stats import ( "reflect" "testing" "time" "github.com/Microsoft/hcsshim" v1 "k8s.io/apimachinery/pkg/apis/meta/v1" statsapi "k8s.io/kubelet/pkg/apis/stats/v1alpha1" testingclock "k8s.io/utils/clock/testing" ) type fakeNetworkStatsProvider struct { containers []containerStats } type containerStats struct { container hcsshim.ContainerProperties hcsStats  []hcsshim.NetworkStats } func (s fakeNetworkStatsProvider) GetHNSEndpointStats(endpointName string) (*hcsshim.HNSEndpointStats, error) { eps := hcsshim.HNSEndpointStats{} for _, c := range s.containers { for _, stat := range c.hcsStats { if endpointName == stat.InstanceId { eps = hcsshim.HNSEndpointStats{ EndpointID:      stat.EndpointId, BytesSent:       stat.BytesSent, BytesReceived:   stat.BytesReceived, PacketsReceived: stat.PacketsReceived, PacketsSent:     stat.PacketsSent, } } } } return &eps, nil } func (s fakeNetworkStatsProvider) HNSListEndpointRequest() ([]hcsshim.HNSEndpoint, error) { uniqueEndpoints := map[string]*hcsshim.HNSEndpoint{} for _, c := range s.containers { for _, stat := range c.hcsStats { e, found := uniqueEndpoints[stat.EndpointId] if found { // add the container e.SharedContainers = append(e.SharedContainers, c.container.ID) continue } uniqueEndpoints[stat.EndpointId] = &hcsshim.HNSEndpoint{ Name:             stat.EndpointId, Id:               stat.EndpointId, SharedContainers: []string{c.container.ID}, } } } eps := []hcsshim.HNSEndpoint{} for _, ep := range uniqueEndpoints { eps = append(eps, *ep) } return eps, nil } func Test_criStatsProvider_listContainerNetworkStats(t *testing.T) { fakeClock := testingclock.NewFakeClock(time.Time{}) tests := []struct { name    string fields  fakeNetworkStatsProvider want    map[string]*statsapi.NetworkStats wantErr bool }{ { name: "basic example", fields: fakeNetworkStatsProvider{ containers: []containerStats{ { container: hcsshim.ContainerProperties{ ID: "c1", }, hcsStats: []hcsshim.NetworkStats{ { BytesReceived: 1, BytesSent:     10, EndpointId:    "test", InstanceId:    "test", }, }, }, { container: hcsshim.ContainerProperties{ ID: "c2", }, hcsStats: []hcsshim.NetworkStats{ { BytesReceived: 2, BytesSent:     20, EndpointId:    "test2", InstanceId:    "test2", }, }, }, }, }, want: map[string]*statsapi.NetworkStats{ "c1": { Time: v1.NewTime(fakeClock.Now()), InterfaceStats: statsapi.InterfaceStats{ Name:    "test", RxBytes: toP(1), TxBytes: toP(10), }, Interfaces: []statsapi.InterfaceStats{ { Name:    "test", RxBytes: toP(1), TxBytes: toP(10), }, }, }, "c2": { Time: v1.Time{}, InterfaceStats: statsapi.InterfaceStats{ Name:    "test2", RxBytes: toP(2), TxBytes: toP(20), }, Interfaces: []statsapi.InterfaceStats{ { Name:    "test2", RxBytes: toP(2), TxBytes: toP(20), }, }, }, }, wantErr: false, }, { name: "multiple containers same endpoint", fields: fakeNetworkStatsProvider{ containers: []containerStats{ { container: hcsshim.ContainerProperties{ ID: "c1", }, hcsStats: []hcsshim.NetworkStats{ { BytesReceived: 1, BytesSent:     10, EndpointId:    "test", InstanceId:    "test", }, }, }, { container: hcsshim.ContainerProperties{ ID: "c2", }, hcsStats: []hcsshim.NetworkStats{ { BytesReceived: 2, BytesSent:     20, EndpointId:    "test2", InstanceId:    "test2", }, }, }, { container: hcsshim.ContainerProperties{ ID: "c3", }, hcsStats: []hcsshim.NetworkStats{ { BytesReceived: 3, BytesSent:     30, EndpointId:    "test2", InstanceId:    "test3", }, }, }, }, }, want: map[string]*statsapi.NetworkStats{ "c1": { Time: v1.NewTime(fakeClock.Now()), InterfaceStats: statsapi.InterfaceStats{ Name:    "test", RxBytes: toP(1), TxBytes: toP(10), }, Interfaces: []statsapi.InterfaceStats{ { Name:    "test", RxBytes: toP(1), TxBytes: toP(10), }, }, }, "c2": { Time: v1.Time{}, InterfaceStats: statsapi.InterfaceStats{ Name:    "test2", RxBytes: toP(2), TxBytes: toP(20), }, Interfaces: []statsapi.InterfaceStats{ { Name:    "test2", RxBytes: toP(2), TxBytes: toP(20), }, }, }, "c3": { Time: v1.Time{}, InterfaceStats: statsapi.InterfaceStats{ Name:    "test2", RxBytes: toP(2), TxBytes: toP(20), }, Interfaces: []statsapi.InterfaceStats{ { Name:    "test2", RxBytes: toP(2), TxBytes: toP(20), }, }, }, }, wantErr: false, }, { name: "multiple stats instances of same interface only picks up first", fields: fakeNetworkStatsProvider{ containers: []containerStats{ { container: hcsshim.ContainerProperties{ ID: "c1", }, hcsStats: []hcsshim.NetworkStats{ { BytesReceived: 1, BytesSent:     10, EndpointId:    "test", InstanceId:    "test", }, { BytesReceived: 3, BytesSent:     30, EndpointId:    "test", InstanceId:    "test3", }, }, }, { container: hcsshim.ContainerProperties{ ID: "c2", }, hcsStats: []hcsshim.NetworkStats{ { BytesReceived: 2, BytesSent:     20, EndpointId:    "test2", InstanceId:    "test2", }, }, }, }, }, want: map[string]*statsapi.NetworkStats{ "c1": { Time: v1.NewTime(fakeClock.Now()), InterfaceStats: statsapi.InterfaceStats{ Name:    "test", RxBytes: toP(1), TxBytes: toP(10), }, Interfaces: []statsapi.InterfaceStats{ { Name:    "test", RxBytes: toP(1), TxBytes: toP(10), }, }, }, "c2": { Time: v1.Time{}, InterfaceStats: statsapi.InterfaceStats{ Name:    "test2", RxBytes: toP(2), TxBytes: toP(20), }, Interfaces: []statsapi.InterfaceStats{ { Name:    "test2", RxBytes: toP(2), TxBytes: toP(20), }, }, }, }, wantErr: false, }, { name: "multiple endpoints per container", fields: fakeNetworkStatsProvider{ containers: []containerStats{ { container: hcsshim.ContainerProperties{ ID: "c1", }, hcsStats: []hcsshim.NetworkStats{ { BytesReceived: 1, BytesSent:     10, EndpointId:    "test", InstanceId:    "test", }, { BytesReceived: 3, BytesSent:     30, EndpointId:    "test3", InstanceId:    "test3", }, }, }, { container: hcsshim.ContainerProperties{ ID: "c2", }, hcsStats: []hcsshim.NetworkStats{ { BytesReceived: 2, BytesSent:     20, EndpointId:    "test2", InstanceId:    "test2", }, }, }, }, }, want: map[string]*statsapi.NetworkStats{ "c1": { Time: v1.NewTime(fakeClock.Now()), InterfaceStats: statsapi.InterfaceStats{ Name:    "test", RxBytes: toP(1), TxBytes: toP(10), }, Interfaces: []statsapi.InterfaceStats{ { Name:    "test", RxBytes: toP(1), TxBytes: toP(10), }, { Name:    "test3", RxBytes: toP(3), TxBytes: toP(30), }, }, }, "c2": { Time: v1.Time{}, InterfaceStats: statsapi.InterfaceStats{ Name:    "test2", RxBytes: toP(2), TxBytes: toP(20), }, Interfaces: []statsapi.InterfaceStats{ { Name:    "test2", RxBytes: toP(2), TxBytes: toP(20), }, }, }, }, wantErr: false, }, } for _, tt := range tests { t.Run(tt.name, func(t *testing.T) { p := &criStatsProvider{ windowsNetworkStatsProvider: fakeNetworkStatsProvider{ containers: tt.fields.containers, }, clock: fakeClock, } got, err := p.listContainerNetworkStats() if (err != nil) != tt.wantErr { t.Errorf("listContainerNetworkStats() error = %v, wantErr %v", err, tt.wantErr) return } if !reflect.DeepEqual(got, tt.want) { t.Errorf("listContainerNetworkStats() got = %v, want %v", got, tt.want) } }) } } func toP(i uint64) *uint64 { return &i } </ins>

Discussion:
<!-- Please use this template while reporting a bug and provide as much info as possible. Not doing so may result in your bug not being addressed in a timely manner. Thanks! If the matter is security related, please disclose it privately via --When curling the endpoint for stats we don't see network stats. Network stats to be included when calling the summary. This works with dockershim for windows. This happens because Windows containerd uses hcsv2 to create containers and we query for the networks stats using hcsv1: This will be address eventually with move to CRI only metrics KEP: but is still several releases away. Kubernetes version (use ): Cloud provider or hardware configuration: OS (e.g: ): Kernel (e.g. ): Install tools: Network plugin and version (if this is a network-related bug): Others: /sig windows /assign
This issue is currently awaiting triage. If a SIG or subproject determines this is a relevant issue, they will accept it by applying the label and provide further guidance. The label can be by org members by writing in a comment. <detailsInstructions 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. </details
The Kubernetes project currently lacks enough contributors to adequately respond to all issues and PRs. This bot triages issues and PRs according to the following rules: After 90d of inactivity, is applied After 30d of inactivity since was applied, is applied After 30d of inactivity since was applied, the issue is closed You can: Mark this issue or PR as fresh with Mark this issue or PR as rotten with Close this issue or PR with Offer to help out with [Issue Triage][1] Please send feedback to sig-contributor-experience at . /lifecycle stale [1]: