remove unused function (#4821)

Signed-off-by: zcq98 <[email protected]>
kubeovn · Dec 18, 2024 · 0a8adbf · 0a8adbf
1 parent 96ad245
commit 0a8adbf
Show file tree

Hide file tree

Showing 3 changed files with 0 additions and 17 deletions.
diff --git a/pkg/ovs/ovn-nb-suite_test.go b/pkg/ovs/ovn-nb-suite_test.go
@@ -1231,10 +1231,6 @@ func (suite *OvnClientTestSuite) Test_OvsCleanDuplicatePort() {
 	suite.testOvsCleanDuplicatePort()
 }
 
-func (suite *OvnClientTestSuite) Test_OvsSetPortTag() {
-	suite.testOvsSetPortTag()
-}
-
 func (suite *OvnClientTestSuite) Test_ValidatePortVendor() {
 	suite.testValidatePortVendor()
 }

diff --git a/pkg/ovs/ovs-vsctl.go b/pkg/ovs/ovs-vsctl.go
@@ -288,10 +288,6 @@ func CleanDuplicatePort(ifaceID, portName string) {
 	}
 }
 
-func SetPortTag(port, tag string) error {
-	return ovsSet("port", port, fmt.Sprintf("tag=%s", tag))
-}
-
 // ValidatePortVendor returns true if the port's external_ids:vendor=kube-ovn
 func ValidatePortVendor(port string) (bool, error) {
 	output, err := ovsFind("Port", "name", "external_ids:vendor="+util.CniTypeName)

diff --git a/pkg/ovs/ovs-vsctl_test.go b/pkg/ovs/ovs-vsctl_test.go
@@ -177,15 +177,6 @@ func (suite *OvnClientTestSuite) testOvsCleanDuplicatePort() {
 	CleanDuplicatePort("iface-id", "port-name")
 }
 
-func (suite *OvnClientTestSuite) testOvsSetPortTag() {
-	t := suite.T()
-	t.Parallel()
-
-	err := SetPortTag("port-name", "tag")
-	// ovs-vsctl cmd is not available in the test environment
-	require.Error(t, err)
-}
-
 func (suite *OvnClientTestSuite) testValidatePortVendor() {
 	t := suite.T()
 	t.Parallel()