From 0ad16a187d7f5ffecda5e8b049a1da0e267a8a70 Mon Sep 17 00:00:00 2001 From: Justin Santa Barbara Date: Thu, 18 Jun 2015 17:08:32 -0700 Subject: [PATCH] Refactor findRouteTable to be less verbose Thanks for the suggestion @cjcullen --- pkg/cloudprovider/aws/aws_routes.go | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/pkg/cloudprovider/aws/aws_routes.go b/pkg/cloudprovider/aws/aws_routes.go index ba9b4d9da17..a87f61da613 100644 --- a/pkg/cloudprovider/aws/aws_routes.go +++ b/pkg/cloudprovider/aws/aws_routes.go @@ -24,13 +24,11 @@ import ( ) func (s *AWSCloud) findRouteTable(clusterName string) (*ec2.RouteTable, error) { - request := &ec2.DescribeRouteTablesInput{} - filters := []*ec2.Filter{} // This should be unnecessary (we already filter on TagNameKubernetesCluster, // and something is broken if cluster name doesn't match, but anyway... // TODO: All clouds should be cluster-aware by default - filters = append(filters, newEc2Filter("tag:"+TagNameKubernetesCluster, clusterName)) - request.Filters = s.addFilters(filters) + filters := []*ec2.Filter{newEc2Filter("tag:"+TagNameKubernetesCluster, clusterName)} + request := &ec2.DescribeRouteTablesInput{Filters: s.addFilters(filters)} tables, err := s.ec2.DescribeRouteTables(request) if err != nil {