Simplify types of Instance.SecurityGroup and Instance.Networks

These both have a type of pointer to slice, which is:
* Redundant in Go
* Is marshalled identically in the CRD

To verify the latter, note that there are no changes to the CRD despite
the type change.

Also note that the values are unused.

The reason for fixing these types is that Semantic.DeepEqual can't
compare them correctly so they break the fuzzer tests.

Author: mdbooth

api/v1alpha5/types.go

@@ -162,8 +162,8 @@ type Instance struct {
 	Name           string            `json:"name,omitempty"`
 	Trunk          bool              `json:"trunk,omitempty"`
 	FailureDomain  string            `json:"failureDomain,omitempty"`
-	SecurityGroups *[]string         `json:"securigyGroups,omitempty"`
-	Networks       *[]Network        `json:"networks,omitempty"`
+	SecurityGroups []string          `json:"securigyGroups,omitempty"`
+	Networks       []Network         `json:"networks,omitempty"`
 	Subnet         string            `json:"subnet,omitempty"`
 	Tags           []string          `json:"tags,omitempty"`
 	Image          string            `json:"image,omitempty"`

api/v1alpha5/zz_generated.deepcopy.go

@@ -172,8 +172,8 @@ type Instance struct {
 	Name           string            `json:"name,omitempty"`
 	Trunk          bool              `json:"trunk,omitempty"`
 	FailureDomain  string            `json:"failureDomain,omitempty"`
-	SecurityGroups *[]string         `json:"securigyGroups,omitempty"`
-	Networks       *[]Network        `json:"networks,omitempty"`
+	SecurityGroups []string          `json:"securigyGroups,omitempty"`
+	Networks       []Network         `json:"networks,omitempty"`
 	Subnet         string            `json:"subnet,omitempty"`
 	Tags           []string          `json:"tags,omitempty"`
 	Image          string            `json:"image,omitempty"`