Minor comment edits

client/network.go

@@ -49,6 +49,9 @@ func (net Network) Compare(obj interface{}) bool {
 type CreateNetworkRequest struct {
 	// The first set of members are shared between bonded and non bonded networks
 	// These should be kept in sync with the CreateBondedNetworkRequest struct
+	// Refactoring these fields to an embedded struct means that the caller must
+	// know the embedded structs existance. Since the list is relatively small
+	// this was deemed a more appropriate tradeoff to make the API nicer.
 	Automatic       bool   `mapstructure:"automatic"`
 	DefaultIsLocked bool   `mapstructure:"defaultIsLocked"`
 	Pool            string `mapstructure:"pool"`

xoa/resource_xenorchestra_bonded_network_test.go

@@ -28,7 +28,7 @@ func TestAccXOBondedNetwork_createAndPlanWithOmittedPIF(t *testing.T) {
 					resource.TestCheckResourceAttrSet(resourceName, "mtu")),
 			},
 			{
-				// Asserting that pif_ids was set in the previous set failed. This
+				// Asserting that pif_ids was set in the previous step failed. This
 				// second step is to verify that even though it's not persisted in the
 				// state that it still correctly detects changes to the resource when
 				// the pif_ids argument is modified.