From d36376f86c8944f6955f537a1a12f4072f8d7403 Mon Sep 17 00:00:00 2001
From: Tonis Tiigi <tonistiigi@gmail.com>
Date: Sat, 19 Mar 2016 12:56:45 -0700
Subject: [PATCH] Convert libnetwork stats directly to api types

Signed-off-by: Tonis Tiigi <tonistiigi@gmail.com>
---
 daemon/daemon.go        | 63 +++++++++++++----------------------------
 daemon/stats_freebsd.go | 14 ---------
 2 files changed, 19 insertions(+), 58 deletions(-)
 delete mode 100644 daemon/stats_freebsd.go

diff --git a/daemon/daemon.go b/daemon/daemon.go
index a01a8b4d9b..c971b8d855 100644
--- a/daemon/daemon.go
+++ b/daemon/daemon.go
@@ -70,9 +70,7 @@ import (
 	"github.com/docker/go-connections/nat"
 	"github.com/docker/libnetwork"
 	nwconfig "github.com/docker/libnetwork/config"
-	lntypes "github.com/docker/libnetwork/types"
 	"github.com/docker/libtrust"
-	"github.com/opencontainers/runc/libcontainer"
 	"golang.org/x/net/context"
 )
 
@@ -1528,50 +1526,40 @@ func (daemon *Daemon) GetContainerStats(container *container.Container) (*types.
 		return nil, err
 	}
 
-	// Retrieve the nw statistics from libnetwork and inject them in the Stats
-	var nwStats []*libcontainer.NetworkInterface
-	if nwStats, err = daemon.getNetworkStats(container); err != nil {
+	if stats.Networks, err = daemon.getNetworkStats(container); err != nil {
 		return nil, err
 	}
 
-	stats.Networks = make(map[string]types.NetworkStats)
-	for _, iface := range nwStats {
-		// For API Version >= 1.21, the original data of network will
-		// be returned.
-		stats.Networks[iface.Name] = types.NetworkStats{
-			RxBytes:   iface.RxBytes,
-			RxPackets: iface.RxPackets,
-			RxErrors:  iface.RxErrors,
-			RxDropped: iface.RxDropped,
-			TxBytes:   iface.TxBytes,
-			TxPackets: iface.TxPackets,
-			TxErrors:  iface.TxErrors,
-			TxDropped: iface.TxDropped,
-		}
-	}
-
 	return stats, nil
 }
 
-func (daemon *Daemon) getNetworkStats(c *container.Container) ([]*libcontainer.NetworkInterface, error) {
-	var list []*libcontainer.NetworkInterface
-
+func (daemon *Daemon) getNetworkStats(c *container.Container) (map[string]types.NetworkStats, error) {
 	sb, err := daemon.netController.SandboxByID(c.NetworkSettings.SandboxID)
 	if err != nil {
-		return list, err
+		return nil, err
 	}
 
-	stats, err := sb.Statistics()
+	lnstats, err := sb.Statistics()
 	if err != nil {
-		return list, err
+		return nil, err
 	}
 
-	// Convert libnetwork nw stats into libcontainer nw stats
-	for ifName, ifStats := range stats {
-		list = append(list, convertLnNetworkStats(ifName, ifStats))
+	stats := make(map[string]types.NetworkStats)
+	// Convert libnetwork nw stats into engine-api stats
+	for ifName, ifStats := range lnstats {
+		stats[ifName] = types.NetworkStats{
+			RxBytes:   ifStats.RxBytes,
+			RxPackets: ifStats.RxPackets,
+			RxErrors:  ifStats.RxErrors,
+			RxDropped: ifStats.RxDropped,
+			TxBytes:   ifStats.TxBytes,
+			TxPackets: ifStats.TxPackets,
+			TxErrors:  ifStats.TxErrors,
+			TxDropped: ifStats.TxDropped,
+		}
 	}
 
-	return list, nil
+	return stats, nil
 }
 
 // newBaseContainer creates a new container with its initial
@@ -1675,19 +1663,6 @@ func (daemon *Daemon) reloadClusterDiscovery(config *Config) error {
 	return nil
 }
 
-func convertLnNetworkStats(name string, stats *lntypes.InterfaceStatistics) *libcontainer.NetworkInterface {
-	n := &libcontainer.NetworkInterface{Name: name}
-	n.RxBytes = stats.RxBytes
-	n.RxPackets = stats.RxPackets
-	n.RxErrors = stats.RxErrors
-	n.RxDropped = stats.RxDropped
-	n.TxBytes = stats.TxBytes
-	n.TxPackets = stats.TxPackets
-	n.TxErrors = stats.TxErrors
-	n.TxDropped = stats.TxDropped
-	return n
-}
-
 func validateID(id string) error {
 	if id == "" {
 		return fmt.Errorf("Invalid empty id")
diff --git a/daemon/stats_freebsd.go b/daemon/stats_freebsd.go
deleted file mode 100644
index 44c330aab6..0000000000
--- a/daemon/stats_freebsd.go
+++ /dev/null
@@ -1,14 +0,0 @@
-package daemon
-
-import (
-	"github.com/docker/engine-api/types"
-	"github.com/opencontainers/runc/libcontainer"
-)
-
-// convertStatsToAPITypes converts the libcontainer.Stats to the api specific
-// structs. This is done to preserve API compatibility and versioning.
-func convertStatsToAPITypes(ls *libcontainer.Stats) *types.StatsJSON {
-	// TODO FreeBSD. Refactor accordingly to fill in stats.
-	s := &types.StatsJSON{}
-	return s
-}