Rename machine to machineType

Author: dannawang0221

test/e2e/tests/setup_e2e_test.go

@@ -37,7 +37,7 @@ var (
 	project         = flag.String("project", "", "Project to run tests in")
 	serviceAccount  = flag.String("service-account", "", "Service account to bring up instance with")
 	zones           = flag.String("zones", "us-central1-c,us-central1-b", "Zones to run tests in. If there are multiple zones, separate each by comma")
-	machine         = flag.String("machine", "n1-standard-1", "Type of machine to provision instance on")
+	machineType     = flag.String("machine-type", "n1-standard-1", "Type of machine to provision instance on")
 	imageURL        = flag.String("image-url", "projects/debian-cloud/global/images/family/debian-11", "OS image url to get image from")
 	runInProw       = flag.Bool("run-in-prow", false, "If true, use a Boskos loaned project and special CI service accounts and ssh keys")
 	deleteInstances = flag.Bool("delete-instances", false, "Delete the instances after tests run")
@@ -93,7 +93,7 @@ var _ = BeforeSuite(func() {
 			nodeID := fmt.Sprintf("gce-pd-csi-e2e-%s", curZone)
 			klog.Infof("Setting up node %s\n", nodeID)
 
-			i, err := remote.SetupInstance(*project, curZone, nodeID, *machine, *serviceAccount, *imageURL, computeService)
+			i, err := remote.SetupInstance(*project, curZone, nodeID, *machineType, *serviceAccount, *imageURL, computeService)
 			if err != nil {
 				klog.Fatalf("Failed to setup instance %v: %v", nodeID, err)
 			}

test/remote/instance.go

@@ -45,10 +45,10 @@ const (
 )
 
 type InstanceInfo struct {
-	project string
-	zone    string
-	name    string
-	machine string
+	project     string
+	zone        string
+	name        string
+	machineType string
 
 	// External IP is filled in after instance creation
 	externalIP string
@@ -68,12 +68,12 @@ func (i *InstanceInfo) GetNodeID() string {
 	return common.CreateNodeID(i.project, i.zone, i.name)
 }
 
-func CreateInstanceInfo(project, instanceZone, name, machine string, cs *compute.Service) (*InstanceInfo, error) {
+func CreateInstanceInfo(project, instanceZone, name, machineType string, cs *compute.Service) (*InstanceInfo, error) {
 	return &InstanceInfo{
-		project: project,
-		zone:    instanceZone,
-		name:    name,
-		machine: machine,
+		project:     project,
+		zone:        instanceZone,
+		name:        name,
+		machineType: machineType,
 
 		computeService: cs,
 	}, nil
@@ -94,7 +94,7 @@ func (i *InstanceInfo) CreateOrGetInstance(imageURL, serviceAccount string) erro
 
 	inst := &compute.Instance{
 		Name:        i.name,
-		MachineType: machineType(i.zone, i.machine),
+		MachineType: machineType(i.zone, i.machineType),
 		NetworkInterfaces: []*compute.NetworkInterface{
 			{
 				AccessConfigs: []*compute.AccessConfig{
@@ -215,8 +215,8 @@ func getTimestamp() string {
 	return fmt.Sprintf(time.Now().Format(timestampFormat))
 }
 
-func machineType(zone, machine string) string {
-	return fmt.Sprintf("zones/%s/machineTypes/%s", zone, machine)
+func machineType(zone, machineType string) string {
+	return fmt.Sprintf("zones/%s/machineTypes/%s", zone, machineType)
 }
 
 // Create default SSH filewall rule if it does not exist

test/remote/setup-teardown.go

@@ -54,9 +54,9 @@ type processes struct {
 }
 
 // SetupInstance sets up the specified GCE Instance for E2E testing and returns a handle to the instance object for future use.
-func SetupInstance(instanceProject, instanceZone, instanceName, instanceMachine, instanceServiceAccount, instanceImageURL string, cs *compute.Service) (*InstanceInfo, error) {
+func SetupInstance(instanceProject, instanceZone, instanceName, instanceMachineType, instanceServiceAccount, instanceImageURL string, cs *compute.Service) (*InstanceInfo, error) {
 	// Create the instance in the requisite zone
-	instance, err := CreateInstanceInfo(instanceProject, instanceZone, instanceName, instanceMachine, cs)
+	instance, err := CreateInstanceInfo(instanceProject, instanceZone, instanceName, instanceMachineType, cs)
 	if err != nil {
 		return nil, err
 	}