Skip to content

Commit bf07298

Browse files
committed
Revert "Passing Resource Pool names to the VyOS/Networking Script to allow setting hostname based on Resource Pool"
This reverts commit 9c86f62.
1 parent dbc1fa2 commit bf07298

5 files changed

Lines changed: 9 additions & 12 deletions

File tree

‎internal/cloning/cloning_service.go‎

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -202,7 +202,6 @@ func (cs *CloningService) CloneTemplate(req CloneRequest) error {
202202
// Store router info for later operations
203203
clonedRouters = append(clonedRouters, RouterInfo{
204204
TargetName: target.Name,
205-
PoolName: target.PoolName,
206205
RouterType: routerType,
207206
PodNumber: target.PodNumber,
208207
Node: bestNode,
@@ -345,7 +344,7 @@ func (cs *CloningService) CloneTemplate(req CloneRequest) error {
345344
}
346345

347346
log.Printf("Configuring pod router for %s (Pod: %d, VMID: %d)", routerInfo.TargetName, routerInfo.PodNumber, routerInfo.VMID)
348-
err = cs.ProxmoxService.ConfigurePodRouter(routerInfo.PodNumber, routerInfo.Node, routerInfo.VMID, routerInfo.RouterType, routerInfo.PoolName)
347+
err = cs.ProxmoxService.ConfigurePodRouter(routerInfo.PodNumber, routerInfo.Node, routerInfo.VMID, routerInfo.RouterType)
349348
if err != nil {
350349
errors = append(errors, fmt.Sprintf("failed to configure pod router for %s: %v", routerInfo.TargetName, err))
351350
}

‎internal/cloning/types.go‎

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -119,7 +119,6 @@ type CloneRequest struct {
119119

120120
type RouterInfo struct {
121121
TargetName string
122-
PoolName string
123122
RouterType string
124123
PodNumber int
125124
Node string

‎internal/proxmox/networking.go‎

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,7 @@ func (s *ProxmoxService) GetRouterType(router VM) (string, error) {
4040
}
4141

4242
// ConfigurePodRouter configures the pod router with proper networking settings
43-
func (s *ProxmoxService) ConfigurePodRouter(podNumber int, node string, vmid int, routerType string, poolName string) error {
43+
func (s *ProxmoxService) ConfigurePodRouter(podNumber int, node string, vmid int, routerType string) error {
4444
config := RouterConfig{
4545
WANScriptPath: s.Config.WANScriptPath,
4646
VIPScriptPath: s.Config.VIPScriptPath,
@@ -115,10 +115,9 @@ func (s *ProxmoxService) ConfigurePodRouter(podNumber int, node string, vmid int
115115
case "vyos":
116116
reqBody := map[string]any{
117117
"command": []string{
118-
config.VYOSScriptPath,
119-
fmt.Sprintf("%d", podNumber),
120-
config.WANIPBase,
121-
poolName,
118+
"sh",
119+
"-c",
120+
fmt.Sprintf("sed -i -e 's/{{THIRD_OCTET}}/%d/g;s/{{NETWORK_PREFIX}}/%s/g' %s", podNumber, config.WANIPBase, config.VYOSScriptPath),
122121
},
123122
}
124123

@@ -130,7 +129,7 @@ func (s *ProxmoxService) ConfigurePodRouter(podNumber int, node string, vmid int
130129

131130
_, err := s.RequestHelper.MakeRequest(execReq)
132131
if err != nil {
133-
return fmt.Errorf("failed to run VyOS setup script: %v", err)
132+
return fmt.Errorf("failed to make IP change request: %v", err)
134133
}
135134

136135
default:

‎internal/proxmox/pools.go‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -446,7 +446,7 @@ func (s *ProxmoxService) CreateTemplatePool(creator string, name string, addRout
446446
log.Printf("Third octect is %d", octect)
447447

448448
log.Printf("Configuring router")
449-
err = s.ConfigurePodRouter(octect, bestNode, routerVMID, routerType, poolName)
449+
err = s.ConfigurePodRouter(octect, bestNode, routerVMID, routerType)
450450
if err != nil {
451451
return fmt.Errorf("failed to configure router for %s: %v", routerType, err)
452452
}

‎internal/proxmox/types.go‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ type ProxmoxConfig struct {
2626
RouterWaitTimeout time.Duration `envconfig:"ROUTER_WAIT_TIMEOUT" default:"120s"`
2727
WANScriptPath string `envconfig:"WAN_SCRIPT_PATH" default:"/home/update-wan-ip.sh"`
2828
VIPScriptPath string `envconfig:"VIP_SCRIPT_PATH" default:"/home/update-wan-vip.sh"`
29-
VYOSScriptPath string `envconfig:"VYOS_SCRIPT_PATH" default:"/config/scripts/setup.sh"`
29+
VYOSScriptPath string `envconfig:"VYOS_SCRIPT_PATH" default:"/config/scripts/vyos-postconfig-bootup.script"`
3030
WANIPBase string `envconfig:"WAN_IP_BASE" default:"172.16."`
3131
Nodes []string // Parsed from NodesStr
3232
APIToken string // Computed from TokenID and TokenSecret
@@ -76,7 +76,7 @@ type Service interface {
7676

7777
// Networking
7878
GetRouterType(router VM) (string, error)
79-
ConfigurePodRouter(podNumber int, node string, vmid int, routerType string, poolName string) error
79+
ConfigurePodRouter(podNumber int, node string, vmid int, routerType string) error
8080
SetPodVnet(poolName string, vnetName string, routerVMID int) error
8181
GetUsedVNets() ([]VNet, error)
8282
CreateTemplatePool(creator string, name string, addRouter bool, vms []VM) error

0 commit comments

Comments
 (0)