Fix more problems

Author: lucacome

cmd/sync/aws.go

@@ -198,13 +198,13 @@ func (client *AWSClient) getInstancesInService(insIDtoIP map[string]string) ([]s
 	const maxItems = 50
 	var result []string
 	keys := reflect.ValueOf(insIDtoIP).MapKeys()
-	instanceIds := make([]string, len(keys))
+	instanceIDs := make([]string, len(keys))
 
 	for i := 0; i < len(keys); i++ {
-		instanceIds[i] = keys[i].String()
+		instanceIDs[i] = keys[i].String()
 	}
 
-	batches := prepareBatches(maxItems, instanceIds)
+	batches := prepareBatches(maxItems, instanceIDs)
 	for _, batch := range batches {
 		params := &autoscaling.DescribeAutoScalingInstancesInput{
 			InstanceIds: batch,

cmd/sync/aws_test.go

@@ -167,11 +167,11 @@ func areEqualUpstreamsAWS(u1 awsUpstream, u2 Upstream) bool {
 func TestPrepareBatches(t *testing.T) {
 	const maxItems = 3
 	ids := []string{"i-394ujfs", "i-dfdinf", "i-fsfsf", "i-8hr83hfwif", "i-nsnsnan"}
-	instanceIds := make([]string, len(ids))
+	instanceIDs := make([]string, len(ids))
 
-	copy(instanceIds, ids)
+	copy(instanceIDs, ids)
 
-	batches := prepareBatches(maxItems, instanceIds)
+	batches := prepareBatches(maxItems, instanceIDs)
 
 	if len(batches) > len(ids)/maxItems+1 {
 		t.Error("prepareBatches() didn't split the slice correctly")