Allow less than three NTP servers (#607)

robertvolkmann · web-flow · commit 21b062c4bd7e · 2025-02-12T14:26:57.000+01:00
diff --git a/cmd/metal-api/internal/metal/machine.go b/cmd/metal-api/internal/metal/machine.go
@@ -14,8 +14,9 @@ import (
 
 	"github.com/asaskevich/govalidator"
 	"github.com/dustin/go-humanize"
-	mn "github.com/metal-stack/metal-lib/pkg/net"
 	"github.com/samber/lo"
+
+	mn "github.com/metal-stack/metal-lib/pkg/net"
 )
 
 // A MState is an enum which indicates the state of a machine
@@ -758,8 +759,8 @@ func (n NTPServers) Validate() error {
 		return nil
 	}
 
-	if len(n) < 3 || len(n) > 5 {
-		return errors.New("please specify a minimum of 3 and a maximum of 5 ntp servers")
+	if len(n) > 5 {
+		return errors.New("please specify a maximum of five ntp servers")
 	}
 
 	for _, ntpserver := range n {

Original file line number	Diff line number	Diff line change
`@@ -14,8 +14,9 @@ import (`
`14`	`14`
`15`	`15`	`"github.com/asaskevich/govalidator"`
`16`	`16`	`"github.com/dustin/go-humanize"`
`17`		`- mn "github.com/metal-stack/metal-lib/pkg/net"`
`18`	`17`	`"github.com/samber/lo"`
	`18`	`+`
	`19`	`+ mn "github.com/metal-stack/metal-lib/pkg/net"`
`19`	`20`	`)`
`20`	`21`
`21`	`22`	`// A MState is an enum which indicates the state of a machine`
`@@ -758,8 +759,8 @@ func (n NTPServers) Validate() error {`
`758`	`759`	`return nil`
`759`	`760`	`}`
`760`	`761`
`761`		`- if len(n) < 3 \|\| len(n) > 5 {`
`762`		`- return errors.New("please specify a minimum of 3 and a maximum of 5 ntp servers")`
	`762`	`+ if len(n) > 5 {`
	`763`	`+ return errors.New("please specify a maximum of five ntp servers")`
`763`	`764`	`}`
`764`	`765`
`765`	`766`	`for _, ntpserver := range n {`