fix(cmx): Use pointer for collect-report bool (#559)

Signed-off-by: Kyle Squizzato <[email protected]>

Author: squizzi

cli/cmd/network_update.go

@@ -71,7 +71,7 @@ func (r *runners) updateNetwork(cmd *cobra.Command, args []string) error {
 	}
 
 	if cmd.Flags().Changed("collect-report") {
-		opts.CollectReport = r.args.updateNetworkCollectReport
+		opts.CollectReport = &r.args.updateNetworkCollectReport
 	}
 
 	// Update the network

pkg/kotsclient/network_update.go

@@ -13,7 +13,7 @@ import (
 
 type UpdateNetworkRequest struct {
 	Policy        string `json:"policy"`
-	CollectReport bool   `json:"collect_report"`
+	CollectReport *bool  `json:"collect_report,omitempty"`
 }
 
 type UpdateNetworkResponse struct {
@@ -23,7 +23,7 @@ type UpdateNetworkResponse struct {
 
 type UpdateNetworkOpts struct {
 	Policy        string `json:"policy"`
-	CollectReport bool   `json:"collect_report"`
+	CollectReport *bool  `json:"collect_report,omitempty"`
 }
 
 type UpdateNetworkErrorResponse struct {
@@ -34,7 +34,10 @@ type UpdateNetworkErrorResponse struct {
 
 // UpdateNetworkPolicy updates a network's policy setting
 func (c *VendorV3Client) UpdateNetwork(networkID string, opts UpdateNetworkOpts) (*types.Network, error) {
-	req := UpdateNetworkRequest(opts)
+	req := UpdateNetworkRequest{
+		Policy:        opts.Policy,
+		CollectReport: opts.CollectReport,
+	}
 	return c.doUpdateNetworkRequest(networkID, req)
 }