fix(host): setup routes from start to end (#18675)

yunionio · Nov 14, 2023 · d0e8ec1 · d0e8ec1
1 parent 421d903
commit d0e8ec1
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 9 deletions.
diff --git a/pkg/hostman/host_services.go b/pkg/hostman/host_services.go
@@ -15,8 +15,6 @@
 package hostman
 
 import (
-	"time"
-
 	execlient "yunion.io/x/executor/client"
 	"yunion.io/x/log"
 
@@ -79,12 +77,7 @@ func (host *SHostService) RunService() {
 
 	hostInstance := hostinfo.Instance()
 	if err := hostInstance.Init(); err != nil {
-		log.Errorf("Host instance init error: %v", err)
-		for {
-			// to catch error
-			time.Sleep(10 * time.Second)
-		}
-		// log.Fatalf("Host instance init error: %v", err)
+		log.Fatalf("Host instance init error: %v", err)
 	}
 
 	app_common.InitAuth(&options.HostOptions.CommonOptions, func() {

diff --git a/pkg/hostman/hostinfo/hostbridge/hostbridge.go b/pkg/hostman/hostinfo/hostbridge/hostbridge.go
@@ -273,7 +273,7 @@ func (d *SBaseBridgeDriver) SetupRoutes(routespecs []iproute2.RouteSpec) error {
 	bridgeIP := d.inter.Addr
 	bridgeMask := d.inter.Mask
 	br := d.bridge.String()
-	for i := len(routespecs) - 1; i >= 0; i-- {
+	for i := 0; i < len(routespecs); i++ {
 		errs := []error{}
 		routespec := routespecs[i]
 		if routespec.Dst.Contains(net.ParseIP(bridgeIP)) && bridgeMask.String() == routespec.Dst.Mask.String() {