Update images/virtualization-artifact/pkg/controller/moduleconfig/util.go

yaroslavborbat · diafour · web-flow · commit c965fb0c0ca7 · 2024-12-17T21:36:18.000+03:00
Co-authored-by: Ivan Mikheykin &lt;ivan.mikheykin@flant.com&gt;
Signed-off-by: Yaroslav Borbat &lt;86148689+yaroslavborbat@users.noreply.github.com&gt;
diff --git a/images/virtualization-artifact/pkg/controller/moduleconfig/util.go b/images/virtualization-artifact/pkg/controller/moduleconfig/util.go
@@ -51,7 +51,7 @@ func checkNodeAddressesOverlap(nodes []corev1.Node, excludedPrefixes []netip.Pre
 
 				for _, prefix := range excludedPrefixes {
 					if prefix.Contains(ip) {
-						return fmt.Errorf("node address %s may overlap with subnet %s", ip, prefix)
+						return fmt.Errorf("subnet %s is invalid, it should not contains addresses assigned to nodes: got node %s with IP %s", prefix, node.GetName(), ip)
 					}
 				}
 			}

Original file line number	Diff line number	Diff line change
`@@ -51,7 +51,7 @@ func checkNodeAddressesOverlap(nodes []corev1.Node, excludedPrefixes []netip.Pre`
`51`	`51`
`52`	`52`	`for _, prefix := range excludedPrefixes {`
`53`	`53`	`if prefix.Contains(ip) {`
`54`		`- return fmt.Errorf("node address %s may overlap with subnet %s", ip, prefix)`
	`54`	`+ return fmt.Errorf("subnet %s is invalid, it should not contains addresses assigned to nodes: got node %s with IP %s", prefix, node.GetName(), ip)`
`55`	`55`	`}`
`56`	`56`	`}`
`57`	`57`	`}`