Remove gopkg.in/yaml.v2 (#583)

* rm "gopkg.in/yaml.v2"

* fix UnmarshalYAML for Networks & Ulimits

go.mod

@@ -54,7 +54,6 @@ require (
 	google.golang.org/genproto v0.0.0-20211027162914-98a5263abeca // indirect
 	google.golang.org/grpc v1.41.0
 	google.golang.org/protobuf v1.27.1
-	gopkg.in/yaml.v2 v2.4.0
 	gopkg.in/yaml.v3 v3.0.0-20210107192922-496545a6307b
 	xorm.io/builder v0.3.9
 	xorm.io/xorm v1.2.5

pipeline/frontend/yaml/types/network.go

@@ -72,16 +72,12 @@ func handleNetwork(name string, value interface{}) (*Network, error) {
 		}, nil
 	}
 	switch v := value.(type) {
-	case map[interface{}]interface{}:
+	case map[string]interface{}:
 		network := &Network{
 			Name: name,
 		}
 		for mapKey, mapValue := range v {
-			name, ok := mapKey.(string)
+			switch mapKey {
-			if !ok {
-				return &Network{}, fmt.Errorf("Cannot unmarshal '%v' to type %T into a string value", name, name)
-			}
-			switch name {
 			case "aliases":
 				aliases, ok := mapValue.([]interface{})
 				if !ok {

pipeline/frontend/yaml/types/network_test.go

@@ -3,7 +3,6 @@ package types
 import (
 	"testing"
 
-	todo_yaml "gopkg.in/yaml.v2"
 	"gopkg.in/yaml.v3"
 
 	"github.com/stretchr/testify/assert"
@@ -16,8 +15,7 @@ func TestMarshalNetworks(t *testing.T) {
 	}{
 		{
 			networks: Networks{},
-			expected: `{}
+			expected: "{}\n",
-`,
 		},
 		{
 			networks: Networks{
@@ -158,8 +156,8 @@ func TestUnmarshalNetworks(t *testing.T) {
 	}
 	for _, network := range networks {
 		actual := &Networks{}
-		err := todo_yaml.Unmarshal([]byte(network.yaml), actual)
+		err := yaml.Unmarshal([]byte(network.yaml), actual)
-		assert.Nil(t, err)
+		assert.NoError(t, err)
-		assert.Equal(t, network.expected, actual, "should be equal")
+		assert.EqualValues(t, network.expected, actual)
 	}
 }

pipeline/frontend/yaml/types/ulimit.go

@@ -37,7 +37,7 @@ func (u *Ulimits) UnmarshalYAML(unmarshal func(interface{}) error) error {
 			case int:
 				soft = int64(mv)
 				hard = int64(mv)
-			case map[interface{}]interface{}:
+			case map[string]interface{}:
 				if len(mv) != 2 {
 					return fmt.Errorf("Failed to unmarshal Ulimit: %#v", mapValue)
 				}

pipeline/frontend/yaml/types/ulimit_test.go

@@ -3,7 +3,6 @@ package types
 import (
 	"testing"
 
-	todo_yaml "gopkg.in/yaml.v2"
 	"gopkg.in/yaml.v3"
 
 	"github.com/stretchr/testify/assert"
@@ -119,7 +118,7 @@ nofile:
 
 	for _, ulimit := range ulimits {
 		actual := &Ulimits{}
-		err := todo_yaml.Unmarshal([]byte(ulimit.yaml), actual)
+		err := yaml.Unmarshal([]byte(ulimit.yaml), actual)
 
 		assert.Nil(t, err)
 		assert.Equal(t, ulimit.expected, actual, "should be equal")

vendor/modules.txt

@@ -812,7 +812,6 @@ google.golang.org/protobuf/types/known/timestamppb
 # gopkg.in/ini.v1 v1.63.2
 gopkg.in/ini.v1
 # gopkg.in/yaml.v2 v2.4.0
-## explicit
 gopkg.in/yaml.v2
 # gopkg.in/yaml.v3 v3.0.0-20210107192922-496545a6307b
 ## explicit