Use Grab when getting nested MaxSpeed to avoid null pointer panic

Author: Renier Morales

softlayer/resource_softlayer_lb_vpx_ha.go

@@ -2,15 +2,16 @@ package softlayer
 
 import (
 	"fmt"
+	"log"
+	"strconv"
+	"strings"
+	"time"
+
 	"github.com/hashicorp/terraform/helper/schema"
 	"github.com/minsikl/netscaler-nitro-go/client"
 	dt "github.com/minsikl/netscaler-nitro-go/datatypes"
 	"github.com/minsikl/netscaler-nitro-go/op"
 	"github.com/softlayer/softlayer-go/session"
-	"log"
-	"strconv"
-	"strings"
-	"time"
 )
 
 func resourceSoftLayerLbVpxHa() *schema.Resource {
@@ -137,7 +138,7 @@ func configureHA(nClient1 *client.NitroClient, nClient2 *client.NitroClient, sta
 	// 7. VPX1 : Sync files
 	hafiles := dt.HafilesReq{
 		Hafiles: &dt.Hafiles{
-			[]string{"all"},
+			Mode: []string{"all"},
 		},
 	}
 	err = nClient1.Add(&hafiles, "action=sync")

softlayer/resource_softlayer_virtual_guest.go

@@ -540,8 +540,9 @@ func resourceSoftLayerVirtualGuestRead(d *schema.ResourceData, meta interface{})
 
 	d.Set(
 		"network_speed",
-		sl.Get(
-			result.PrimaryNetworkComponent.MaxSpeed,
+		sl.Grab(
+			result,
+			"PrimaryNetworkComponent.MaxSpeed",
 			d.Get("network_speed").(int),
 		),
 	)

vendor/github.com/softlayer/softlayer-go/sl/helpers.go

@@ -39,80 +39,80 @@
 		{
 			"checksumSHA1": "KeNIYJQUdE6o196SaYuLMmjEPZM=",
 			"path": "github.com/softlayer/softlayer-go/config",
-			"revision": "b916637c357676dd73de892c4e51db494e5c407c",
-			"revisionTime": "2016-10-25T15:10:44Z"
+			"revision": "7b0d5dcf966fa8d383989312c4127ea214e84c08",
+			"revisionTime": "2016-11-14T15:40:38Z"
 		},
 		{
 			"checksumSHA1": "YFRxGJKFs1KJVMCimD2v5Lck/HU=",
 			"path": "github.com/softlayer/softlayer-go/datatypes",
-			"revision": "b916637c357676dd73de892c4e51db494e5c407c",
-			"revisionTime": "2016-10-25T15:10:44Z"
+			"revision": "7b0d5dcf966fa8d383989312c4127ea214e84c08",
+			"revisionTime": "2016-11-14T15:40:38Z"
 		},
 		{
 			"checksumSHA1": "nmqOPqyvDmGdvAntqvg/e0VfdbY=",
 			"path": "github.com/softlayer/softlayer-go/examples",
-			"revision": "b916637c357676dd73de892c4e51db494e5c407c",
-			"revisionTime": "2016-10-25T15:10:44Z"
+			"revision": "7b0d5dcf966fa8d383989312c4127ea214e84c08",
+			"revisionTime": "2016-11-14T15:40:38Z"
 		},
 		{
 			"checksumSHA1": "tZ3W1bwC/B9lAGW7007CwaZl6C4=",
 			"path": "github.com/softlayer/softlayer-go/filter",
-			"revision": "b916637c357676dd73de892c4e51db494e5c407c",
-			"revisionTime": "2016-10-25T15:10:44Z"
+			"revision": "7b0d5dcf966fa8d383989312c4127ea214e84c08",
+			"revisionTime": "2016-11-14T15:40:38Z"
 		},
 		{
 			"checksumSHA1": "8w7mbrWkcP0yCAdX3BpPr1vndrY=",
 			"path": "github.com/softlayer/softlayer-go/helpers/hardware",
-			"revision": "b916637c357676dd73de892c4e51db494e5c407c",
-			"revisionTime": "2016-10-25T15:10:44Z"
+			"revision": "7b0d5dcf966fa8d383989312c4127ea214e84c08",
+			"revisionTime": "2016-11-14T15:40:38Z"
 		},
 		{
 			"checksumSHA1": "iIil9KYj6J7ZEK4dG9UirsFuq1Y=",
 			"path": "github.com/softlayer/softlayer-go/helpers/location",
-			"revision": "b916637c357676dd73de892c4e51db494e5c407c",
-			"revisionTime": "2016-10-25T15:10:44Z"
+			"revision": "7b0d5dcf966fa8d383989312c4127ea214e84c08",
+			"revisionTime": "2016-11-14T15:40:38Z"
 		},
 		{
 			"checksumSHA1": "G3kKx93f2BtEvrApz/xB3yy1BuQ=",
 			"path": "github.com/softlayer/softlayer-go/helpers/network",
-			"revision": "b916637c357676dd73de892c4e51db494e5c407c",
-			"revisionTime": "2016-10-25T15:10:44Z"
+			"revision": "7b0d5dcf966fa8d383989312c4127ea214e84c08",
+			"revisionTime": "2016-11-14T15:40:38Z"
 		},
 		{
 			"checksumSHA1": "bGG9wihg55FY7q/kHVE5KQV8XmE=",
 			"path": "github.com/softlayer/softlayer-go/helpers/order",
-			"revision": "b916637c357676dd73de892c4e51db494e5c407c",
-			"revisionTime": "2016-10-25T15:10:44Z"
+			"revision": "7b0d5dcf966fa8d383989312c4127ea214e84c08",
+			"revisionTime": "2016-11-14T15:40:38Z"
 		},
 		{
 			"checksumSHA1": "P/4mxQUWq3TBfriw7yZF3IuNdOs=",
 			"path": "github.com/softlayer/softlayer-go/helpers/product",
-			"revision": "b916637c357676dd73de892c4e51db494e5c407c",
-			"revisionTime": "2016-10-25T15:10:44Z"
+			"revision": "7b0d5dcf966fa8d383989312c4127ea214e84c08",
+			"revisionTime": "2016-11-14T15:40:38Z"
 		},
 		{
 			"checksumSHA1": "AE7CLBIxLRpDneGO0RiDAifDMoM=",
 			"path": "github.com/softlayer/softlayer-go/helpers/virtual",
-			"revision": "b916637c357676dd73de892c4e51db494e5c407c",
-			"revisionTime": "2016-10-25T15:10:44Z"
+			"revision": "7b0d5dcf966fa8d383989312c4127ea214e84c08",
+			"revisionTime": "2016-11-14T15:40:38Z"
 		},
 		{
 			"checksumSHA1": "65dzedbCq23p7rADD3Ehvv4oxGs=",
 			"path": "github.com/softlayer/softlayer-go/services",
-			"revision": "b916637c357676dd73de892c4e51db494e5c407c",
-			"revisionTime": "2016-10-25T15:10:44Z"
+			"revision": "7b0d5dcf966fa8d383989312c4127ea214e84c08",
+			"revisionTime": "2016-11-14T15:40:38Z"
 		},
 		{
 			"checksumSHA1": "sNorXGAP6WIpphnKbmfsgaRAp34=",
 			"path": "github.com/softlayer/softlayer-go/session",
-			"revision": "b916637c357676dd73de892c4e51db494e5c407c",
-			"revisionTime": "2016-10-25T15:10:44Z"
+			"revision": "7b0d5dcf966fa8d383989312c4127ea214e84c08",
+			"revisionTime": "2016-11-14T15:40:38Z"
 		},
 		{
-			"checksumSHA1": "IwQnOvXL7IpCVwOHxyI7HgGD9gE=",
+			"checksumSHA1": "ftAY+Gacd77mJM7mhcVkUbtAqQ8=",
 			"path": "github.com/softlayer/softlayer-go/sl",
-			"revision": "b916637c357676dd73de892c4e51db494e5c407c",
-			"revisionTime": "2016-10-25T15:10:44Z"
+			"revision": "7b0d5dcf966fa8d383989312c4127ea214e84c08",
+			"revisionTime": "2016-11-14T15:40:38Z"
 		},
 		{
 			"checksumSHA1": "Mr4ur60bgQJnQFfJY0dGtwWwMPE=",