ovs: fix ct_state parsing when only one flag is specified

Author: Nicolas Bouliane

ovs/matchparser.go

@@ -257,9 +257,9 @@ func parseCTState(value string) (Match, error) {
 		value = strings.ReplaceAll(value, "-", " -")
 		value = strings.Trim(value, " ")
 	} else {
-		// There are only two valid format for ct_state:
-		// `ct_state=+est+trk` and `ct_state=est|trk`
-		return nil, errors.New("ct_state format is invalid")
+		// Assume only one state is specified: "ct_state=trk"
+		// "trk" => "+trk"
+		value = "+" + value
 	}
 
 	states := strings.Fields(value)

ovs/matchparser_test.go

@@ -73,6 +73,11 @@ func Test_parseMatch(t *testing.T) {
 				UnsetState(CTStateNew),
 			),
 		},
+		{
+			s:           "ct_state=trk",
+			m:           ConnectionTrackingState("+trk"),
+			skipMarshal: true,
+		},
 		{
 			s:           "ct_state=est|trk",
 			m:           ConnectionTrackingState("+est", "+trk"),