Change 100's Result DNS as pointer to remove empty DNS field

This commit changes DNS as pointer to remove empty DNS field in
Result. This will simplify Result format. This also changes
SPEC to mention all keys are optional, other than 'cniVerison'.

Signed-off-by: Tomofumi Hayashi <[email protected]>

Author: s1061123

SPEC.md

@@ -523,6 +523,7 @@ For certain operations, plugins must output result information. The output shoul
 ### ADD Success
 
 Plugins must output a JSON object with the following keys upon a successful `ADD` operation:
+All keys are optional other than `cniVersion`.
 
 - `cniVersion`: The same version supplied on input - the string "1.1.0"
 - `interfaces`: An array of all interfaces created by the attachment, including any host-level interfaces:

libcni/api_test.go

@@ -1003,7 +1003,7 @@ var _ = Describe("Invoking plugins", func() {
 						},
 					},
 					// DNS injected by last plugin
-					DNS: types.DNS{
+					DNS: &types.DNS{
 						Nameservers: []string{"1.2.3.4"},
 					},
 				}))

libcni/conf_test.go

@@ -496,7 +496,7 @@ var _ = Describe("Loading configuration from disk", func() {
 				Expect(err).NotTo(HaveOccurred())
 
 				Expect(resultConfig).To(Equal(&libcni.NetworkConfig{
-					Network: &types.NetConf{Name: "some-plugin", Type: "bridge", DNS: types.DNS{Nameservers: servers, Domain: "local"}},
+					Network: &types.NetConf{Name: "some-plugin", Type: "bridge", DNS: &types.DNS{Nameservers: servers, Domain: "local"}},
 					Bytes:   expectedPluginConfig,
 				}))
 			})

pkg/types/100/types.go

@@ -92,7 +92,7 @@ type Result struct {
 	Interfaces []*Interface   `json:"interfaces,omitempty"`
 	IPs        []*IPConfig    `json:"ips,omitempty"`
 	Routes     []*types.Route `json:"routes,omitempty"`
-	DNS        types.DNS      `json:"dns,omitempty"`
+	DNS        *types.DNS     `json:"dns,omitempty"`
 }
 
 // convertFrom100 does nothing except set the version; the types are the same
@@ -145,7 +145,7 @@ func convertFrom04x(from types.Result, toVersion string) (types.Result, error) {
 	fromResult := from.(*types040.Result)
 	toResult := &Result{
 		CNIVersion: toVersion,
-		DNS:        *fromResult.DNS.Copy(),
+		DNS:        fromResult.DNS.Copy(),
 		Routes:     []*types.Route{},
 	}
 	for _, fromIntf := range fromResult.Interfaces {

pkg/types/100/types_test.go

@@ -72,7 +72,7 @@ func testResult() *current.Result {
 			{Dst: *routev4, GW: routegwv4},
 			{Dst: *routev6, GW: routegwv6},
 		},
-		DNS: types.DNS{
+		DNS: &types.DNS{
 			Nameservers: []string{"1.2.3.4", "1::cafe"},
 			Domain:      "acompany.com",
 			Search:      []string{"somedomain.com", "otherdomain.net"},

pkg/types/types.go

@@ -64,7 +64,7 @@ type NetConf struct {
 	Type         string          `json:"type,omitempty"`
 	Capabilities map[string]bool `json:"capabilities,omitempty"`
 	IPAM         IPAM            `json:"ipam,omitempty"`
-	DNS          DNS             `json:"dns,omitempty"`
+	DNS          *DNS            `json:"dns,omitempty"`
 
 	RawPrevResult map[string]interface{} `json:"prevResult,omitempty"`
 	PrevResult    Result                 `json:"-"`

pkg/types/types_test.go

@@ -176,7 +176,7 @@ var _ = Describe("Types", func() {
 						Gateway:   net.ParseIP("abcd:1234:ffff::1"),
 					},
 				},
-				DNS: types.DNS{
+				DNS: &types.DNS{
 					Nameservers: []string{"1.2.3.4", "1::cafe"},
 					Domain:      "acompany.com",
 					Search:      []string{"somedomain.com", "otherdomain.net"},