Skip to content

K
k3s
Commit 0ad16a18 authored by Justin Santa Barbara's avatar Justin Santa Barbara
Browse files
Options

Refactor findRouteTable to be less verbose

Thanks for the suggestion @cjcullen
parent a4e15cdf
Hide whitespace changes
Inline Side-by-side
Showing with 2 additions and 4 deletions
pkg/cloudprovider/aws/aws_routes.go
View file @ 0ad16a18
...@@ -24,13 +24,11 @@ import ( ...@@ -24,13 +24,11 @@ import (
) )
func (s *AWSCloud) findRouteTable(clusterName string) (*ec2.RouteTable, error) { func (s *AWSCloud) findRouteTable(clusterName string) (*ec2.RouteTable, error) {
request := &ec2.DescribeRouteTablesInput{}
filters := []*ec2.Filter{}
// This should be unnecessary (we already filter on TagNameKubernetesCluster, // This should be unnecessary (we already filter on TagNameKubernetesCluster,
// and something is broken if cluster name doesn't match, but anyway... // and something is broken if cluster name doesn't match, but anyway...
// TODO: All clouds should be cluster-aware by default // TODO: All clouds should be cluster-aware by default
filters = append(filters, newEc2Filter("tag:"+TagNameKubernetesCluster, clusterName)) filters := []*ec2.Filter{newEc2Filter("tag:"+TagNameKubernetesCluster, clusterName)}
request.Filters = s.addFilters(filters) request := &ec2.DescribeRouteTablesInput{Filters: s.addFilters(filters)}
tables, err := s.ec2.DescribeRouteTables(request) tables, err := s.ec2.DescribeRouteTables(request)
if err != nil { if err != nil {
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment