Merge pull request #5220 from nikhiljindal/camelCaseFieldSelectors

Changing v1beta3 field selectors to be camelCased

pkg/api/v1beta1/conversion.go

@@ -1355,12 +1355,12 @@ func init() {
 			case "name":
 				return "name", value, nil
 			case "DesiredState.Host":
-				return "Status.Host", value, nil
+				return "status.host", value, nil
 			case "DesiredState.Status":
 				podStatus := PodStatus(value)
 				var internalValue newer.PodPhase
 				newer.Scheme.Convert(&podStatus, &internalValue)
-				return "Status.Phase", string(internalValue), nil
+				return "status.phase", string(internalValue), nil
 			default:
 				return "", "", fmt.Errorf("field label not supported: %s", label)
 			}

pkg/api/v1beta2/conversion.go

@@ -1271,12 +1271,12 @@ func init() {
 			case "name":
 				return "name", value, nil
 			case "DesiredState.Host":
-				return "Status.Host", value, nil
+				return "status.host", value, nil
 			case "DesiredState.Status":
 				podStatus := PodStatus(value)
 				var internalValue newer.PodPhase
 				newer.Scheme.Convert(&podStatus, &internalValue)
-				return "Status.Phase", string(internalValue), nil
+				return "status.phase", string(internalValue), nil
 			default:
 				return "", "", fmt.Errorf("field label not supported: %s", label)
 			}

pkg/api/v1beta3/conversion.go

@@ -29,9 +29,9 @@ func init() {
 			switch label {
 			case "name":
 				fallthrough
-			case "Status.Phase":
+			case "status.phase":
 				fallthrough
-			case "Status.Host":
+			case "status.host":
 				return label, value, nil
 			default:
 				return "", "", fmt.Errorf("field label not supported: %s", label)

pkg/client/cache/listwatch_test.go

@@ -53,7 +53,7 @@ func getHostFieldLabel() string {
 	case "v1beta1", "v1beta2":
 		return "DesiredState.Host"
 	default:
-		return "Status.Host"
+		return "status.host"
 	}
 }
 

pkg/kubelet/config/apiserver.go

@@ -57,6 +57,6 @@ func getHostFieldLabel(apiVersion string) string {
 	case "v1beta1", "v1beta2":
 		return "DesiredState.Host"
 	default:
-		return "Status.Host"
+		return "status.host"
 	}
 }

pkg/registry/pod/etcd/etcd_test.go

@@ -337,16 +337,16 @@ func TestListPodListSelection(t *testing.T) {
 			label:       "label=qux",
 			expectedIDs: util.NewStringSet("qux"),
 		}, {
-			field:       "Status.Phase=Failed",
+			field:       "status.phase=Failed",
 			expectedIDs: util.NewStringSet("baz"),
 		}, {
-			field:       "Status.Host=barhost",
+			field:       "status.host=barhost",
 			expectedIDs: util.NewStringSet("bar"),
 		}, {
-			field:       "Status.Host=",
+			field:       "status.host=",
 			expectedIDs: util.NewStringSet("foo", "baz", "qux", "zot"),
 		}, {
-			field:       "Status.Host!=",
+			field:       "status.host!=",
 			expectedIDs: util.NewStringSet("bar"),
 		},
 	}

pkg/registry/pod/rest.go

@@ -132,8 +132,8 @@ func MatchPod(label, field labels.Selector) generic.Matcher {
 func PodToSelectableFields(pod *api.Pod) labels.Set {
 	return labels.Set{
 		"name":         pod.Name,
-		"Status.Phase": string(pod.Status.Phase),
+		"status.phase": string(pod.Status.Phase),
-		"Status.Host":  pod.Status.Host,
+		"status.host":  pod.Status.Host,
 	}
 }
 

plugin/pkg/scheduler/factory/factory.go

@@ -259,7 +259,7 @@ func getHostFieldLabel(apiVersion string) string {
 	case "v1beta1", "v1beta2":
 		return "DesiredState.Host"
 	default:
-		return "Status.Host"
+		return "status.host"
 	}
 }