Merge pull request #1462 from ioito/hotfix/qx-aws-backend-ip

ioito · web-flow · commit 45526ea9cc40 · 2025-09-01T15:57:27.000+08:00
fix(region): sync aws lb backend ip
diff --git a/pkg/multicloud/aws/loadbalancerbackend.go b/pkg/multicloud/aws/loadbalancerbackend.go
@@ -21,6 +21,7 @@ import (
 	api "yunion.io/x/cloudmux/pkg/apis/compute"
 	"yunion.io/x/cloudmux/pkg/cloudprovider"
 	"yunion.io/x/cloudmux/pkg/multicloud"
+	"yunion.io/x/pkg/util/netutils"
 )
 
 type SElbBackends struct {
@@ -100,7 +101,11 @@ func (self *SElbBackend) Update(ctx context.Context, opts *cloudprovider.SLoadba
 }
 
 func (self *SElbBackend) GetIpAddress() string {
-	return ""
+	_, err := netutils.NewIPV4Addr(self.Target.Id)
+	if err != nil {
+		return ""
+	}
+	return self.Target.Id
 }
 
 func (self *SRegion) SyncElbBackend(backendId, serverId string, oldPort, newPort int) error {

Original file line number	Diff line number	Diff line change
`@@ -21,6 +21,7 @@ import (`
`21`	`21`	`api "yunion.io/x/cloudmux/pkg/apis/compute"`
`22`	`22`	`"yunion.io/x/cloudmux/pkg/cloudprovider"`
`23`	`23`	`"yunion.io/x/cloudmux/pkg/multicloud"`
	`24`	`+ "yunion.io/x/pkg/util/netutils"`
`24`	`25`	`)`
`25`	`26`
`26`	`27`	`type SElbBackends struct {`
`@@ -100,7 +101,11 @@ func (self SElbBackend) Update(ctx context.Context, opts cloudprovider.SLoadba`
`100`	`101`	`}`
`101`	`102`
`102`	`103`	`func (self *SElbBackend) GetIpAddress() string {`
`103`		`- return ""`
	`104`	`+ _, err := netutils.NewIPV4Addr(self.Target.Id)`
	`105`	`+ if err != nil {`
	`106`	`+ return ""`
	`107`	`+ }`
	`108`	`+ return self.Target.Id`
`104`	`109`	`}`
`105`	`110`
`106`	`111`	`func (self *SRegion) SyncElbBackend(backendId, serverId string, oldPort, newPort int) error {`