From 17d6a929548ae0693affef54f36f03d27f00e603 Mon Sep 17 00:00:00 2001
From: Sebastiaan van Stijn <github@gone.nl>
Date: Thu, 9 Oct 2025 13:46:33 +0200
Subject: [PATCH] opts: deprecate ValidateMACAddress

It was a wrapper around net.ParseMAC from stdlib, so users should
use that directly.

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
---
 cli/command/container/opts.go |  5 +++--
 opts/opts.go                  |  2 ++
 opts/opts_test.go             | 14 --------------
 3 files changed, 5 insertions(+), 16 deletions(-)

diff --git a/cli/command/container/opts.go b/cli/command/container/opts.go
index a3aa58d2f30b..cff4e499496a 100644
--- a/cli/command/container/opts.go
+++ b/cli/command/container/opts.go
@@ -5,6 +5,7 @@ import (
 	"encoding/json"
 	"errors"
 	"fmt"
+	"net"
 	"os"
 	"path"
 	"path/filepath"
@@ -353,7 +354,7 @@ func parse(flags *pflag.FlagSet, copts *containerOptions, serverOS string) (*con
 
 	// Validate the input mac address
 	if copts.macAddress != "" {
-		if _, err := opts.ValidateMACAddress(copts.macAddress); err != nil {
+		if _, err := net.ParseMAC(strings.TrimSpace(copts.macAddress)); err != nil {
 			return nil, fmt.Errorf("%s is not a valid mac address", copts.macAddress)
 		}
 	}
@@ -874,7 +875,7 @@ func parseNetworkAttachmentOpt(ep opts.NetworkAttachmentOpts) (*network.Endpoint
 		}
 	}
 	if ep.MacAddress != "" {
-		if _, err := opts.ValidateMACAddress(ep.MacAddress); err != nil {
+		if _, err := net.ParseMAC(strings.TrimSpace(ep.MacAddress)); err != nil {
 			return nil, fmt.Errorf("%s is not a valid mac address", ep.MacAddress)
 		}
 		epConfig.MacAddress = ep.MacAddress
diff --git a/opts/opts.go b/opts/opts.go
index 6ea218130454..61e99ffb7791 100644
--- a/opts/opts.go
+++ b/opts/opts.go
@@ -191,6 +191,8 @@ func ValidateIPAddress(val string) (string, error) {
 }
 
 // ValidateMACAddress validates a MAC address.
+//
+// Deprecated: use [net.ParseMAC]. This function will be removed in the next release.
 func ValidateMACAddress(val string) (string, error) {
 	_, err := net.ParseMAC(strings.TrimSpace(val))
 	if err != nil {
diff --git a/opts/opts_test.go b/opts/opts_test.go
index 39e7fdea43c1..958495075d76 100644
--- a/opts/opts_test.go
+++ b/opts/opts_test.go
@@ -360,20 +360,6 @@ func sampleValidator(val string) (string, error) {
 	return "", fmt.Errorf("invalid key %s", k)
 }
 
-func TestValidateMACAddress(t *testing.T) {
-	if _, err := ValidateMACAddress(`92:d0:c6:0a:29:33`); err != nil {
-		t.Fatalf("ValidateMACAddress(`92:d0:c6:0a:29:33`) got %s", err)
-	}
-
-	if _, err := ValidateMACAddress(`92:d0:c6:0a:33`); err == nil {
-		t.Fatalf("ValidateMACAddress(`92:d0:c6:0a:33`) succeeded; expected failure on invalid MAC")
-	}
-
-	if _, err := ValidateMACAddress(`random invalid string`); err == nil {
-		t.Fatalf("ValidateMACAddress(`random invalid string`) succeeded; expected failure on invalid MAC")
-	}
-}
-
 func TestValidateLink(t *testing.T) {
 	valid := []string{
 		"name",