kubernetes
/
kops
mirror of https://github.com/kubernetes/kops.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

refactor: NetworkLoadBalancer Name should match Name tag 

It was actually the terraform name, and didn't match the tag.

This change should have no externally-visible effect.
This commit is contained in:
justinsb 2024-01-28 16:16:32 -05:00
parent a55614910d
commit 169c96ba11
5 changed files with 10 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
pkg/model/awsmodel/api_loadbalancer.go
View File

@ -167,16 +167,15 @@ func (b *APILoadBalancerBuilder) Build(c *fi.CloudupModelBuilderContext) error {
klog.V(1).Infof("WARNING: You are overwriting the Load Balancers, Security Group. When this is done you are responsible for ensure the correct rules!") klog.V(1).Infof("WARNING: You are overwriting the Load Balancers, Security Group. When this is done you are responsible for ensure the correct rules!")
} }
tags := b.CloudTags(loadBalancerName, false) tags := b.CloudTags("", false)
for k, v := range b.Cluster.Spec.CloudLabels { for k, v := range b.Cluster.Spec.CloudLabels {
tags[k] = v tags[k] = v
} }
// Override the returned name to be the expected ELB name // Override the returned name to be the expected ELB name
tags["Name"] = "api." + b.ClusterName() tags["Name"] = "api." + b.ClusterName()
name := b.NLBName("api")
nlb = &awstasks.NetworkLoadBalancer{ nlb = &awstasks.NetworkLoadBalancer{
Name: &name, Name: fi.PtrTo(b.NLBName("api")),
Lifecycle: b.Lifecycle, Lifecycle: b.Lifecycle,
LoadBalancerName: fi.PtrTo(loadBalancerName), LoadBalancerName: fi.PtrTo(loadBalancerName),

2
pkg/model/awsmodel/bastion.go
View File

@ -321,7 +321,7 @@ func (b *BastionModelBuilder) Build(c *fi.CloudupModelBuilderContext) error {
{ {
loadBalancerName := b.LBName32("bastion") loadBalancerName := b.LBName32("bastion")
tags := b.CloudTags(loadBalancerName, false) tags := b.CloudTags("", false)
for k, v := range b.Cluster.Spec.CloudLabels { for k, v := range b.Cluster.Spec.CloudLabels {
tags[k] = v tags[k] = v
} }

2
pkg/model/names.go
View File

@ -98,7 +98,7 @@ func (b *KopsModelContext) CLBName(prefix string) string {
} }
func (b *KopsModelContext) NLBName(prefix string) string { func (b *KopsModelContext) NLBName(prefix string) string {
return strings.ReplaceAll(prefix+"-"+b.ClusterName(), ".", "-") return prefix + "." + b.ClusterName()
} }
func (b *KopsModelContext) NLBTargetGroupName(prefix string) string { func (b *KopsModelContext) NLBTargetGroupName(prefix string) string {

1
upup/pkg/fi/cloudup/awstasks/dnsname.go
View File

@ -160,7 +160,6 @@ func findDNSTargetNLB(cloud awsup.AWSCloud, aliasTarget *route53.AliasTarget, dn
if nameTag == "" { if nameTag == "" {
return nil, fmt.Errorf("Found NLB %q linked to DNS name %q, but it did not have a Name tag", loadBalancerName, fi.ValueOf(targetDNSName)) return nil, fmt.Errorf("Found NLB %q linked to DNS name %q, but it did not have a Name tag", loadBalancerName, fi.ValueOf(targetDNSName))
} }
nameTag = strings.Replace(nameTag, ".", "-", -1)
return &NetworkLoadBalancer{Name: fi.PtrTo(nameTag)}, nil return &NetworkLoadBalancer{Name: fi.PtrTo(nameTag)}, nil
} }
return nil, nil return nil, nil

7
upup/pkg/fi/cloudup/awstasks/network_load_balancer.go
View File

@ -870,7 +870,7 @@ func (_ *NetworkLoadBalancer) RenderTerraform(t *terraform.TerraformTarget, a, e
listenerTF.Protocol = elbv2.ProtocolEnumTcp listenerTF.Protocol = elbv2.ProtocolEnumTcp
} }
err = t.RenderResource("aws_lb_listener", fmt.Sprintf("%v-%v", *e.Name, listener.Port), listenerTF) err = t.RenderResource("aws_lb_listener", fmt.Sprintf("%v-%v", e.TerraformName(), listener.Port), listenerTF)
if err != nil { if err != nil {
return err return err
} }
@ -879,6 +879,11 @@ func (_ *NetworkLoadBalancer) RenderTerraform(t *terraform.TerraformTarget, a, e
return nil return nil
} }
func (e *NetworkLoadBalancer) TerraformName() string {
tfName := strings.Replace(fi.ValueOf(e.Name), ".", "-", -1)
return tfName
}
func (e *NetworkLoadBalancer) TerraformLink(params ...string) *terraformWriter.Literal { func (e *NetworkLoadBalancer) TerraformLink(params ...string) *terraformWriter.Literal {
prop := "id" prop := "id"
if len(params) > 0 { if len(params) > 0 {