Merge pull request #1145 from ovh/dev/aamstutz/fix-private-network-deletion

amstuta · web-flow · commit f5c7b7dcad9e · 2025-11-06T17:10:33.000+01:00
fix(ovh_cloud_project_network_private): Use the right route to delete regional network
diff --git a/ovh/resource_cloud_project_network_private.go b/ovh/resource_cloud_project_network_private.go
@@ -295,11 +295,12 @@ func resourceCloudProjectNetworkPrivateUpdate(d *schema.ResourceData, meta inter
 		}
 	}
 
+	regionOpenstackIDs := d.Get("regions_openstack_ids").(map[string]any)
 	for _, reg := range regionsToRemove {
-		endpoint = fmt.Sprintf("/cloud/project/%s/network/private/%s/region/%s",
+		endpoint = fmt.Sprintf("/cloud/project/%s/region/%s/network/%s",
 			url.PathEscape(serviceName),
-			url.PathEscape(d.Id()),
 			url.PathEscape(reg),
+			url.PathEscape(regionOpenstackIDs[reg].(string)),
 		)
 
 		if err := config.OVHClient.Delete(endpoint, params); err != nil {

Original file line number	Diff line number	Diff line change
`@@ -295,11 +295,12 @@ func resourceCloudProjectNetworkPrivateUpdate(d *schema.ResourceData, meta inter`
`295`	`295`	`}`
`296`	`296`	`}`
`297`	`297`
	`298`	`+ regionOpenstackIDs := d.Get("regions_openstack_ids").(map[string]any)`
`298`	`299`	`for _, reg := range regionsToRemove {`
`299`		`- endpoint = fmt.Sprintf("/cloud/project/%s/network/private/%s/region/%s",`
	`300`	`+ endpoint = fmt.Sprintf("/cloud/project/%s/region/%s/network/%s",`
`300`	`301`	`url.PathEscape(serviceName),`
`301`		`- url.PathEscape(d.Id()),`
`302`	`302`	`url.PathEscape(reg),`
	`303`	`+ url.PathEscape(regionOpenstackIDs[reg].(string)),`
`303`	`304`	`)`
`304`	`305`
`305`	`306`	`if err := config.OVHClient.Delete(endpoint, params); err != nil {`