Merge pull request #122472 from pacoxu/flowcontrolv1
flowcontrol: set the serialization version to v1
This commit is contained in:
		| @@ -8,7 +8,7 @@ | |||||||
|       "name": "flowschemas", |       "name": "flowschemas", | ||||||
|       "namespaced": false, |       "namespaced": false, | ||||||
|       "singularName": "flowschema", |       "singularName": "flowschema", | ||||||
|       "storageVersionHash": "9NnFrw7ZEmA=", |       "storageVersionHash": "GJVAJZSZBIw=", | ||||||
|       "verbs": [ |       "verbs": [ | ||||||
|         "create", |         "create", | ||||||
|         "delete", |         "delete", | ||||||
| @@ -36,7 +36,7 @@ | |||||||
|       "name": "prioritylevelconfigurations", |       "name": "prioritylevelconfigurations", | ||||||
|       "namespaced": false, |       "namespaced": false, | ||||||
|       "singularName": "prioritylevelconfiguration", |       "singularName": "prioritylevelconfiguration", | ||||||
|       "storageVersionHash": "+CwSrEWTDhc=", |       "storageVersionHash": "Kir5PVfvNeI=", | ||||||
|       "verbs": [ |       "verbs": [ | ||||||
|         "create", |         "create", | ||||||
|         "delete", |         "delete", | ||||||
|   | |||||||
| @@ -8,7 +8,7 @@ | |||||||
|       "name": "flowschemas", |       "name": "flowschemas", | ||||||
|       "namespaced": false, |       "namespaced": false, | ||||||
|       "singularName": "flowschema", |       "singularName": "flowschema", | ||||||
|       "storageVersionHash": "9NnFrw7ZEmA=", |       "storageVersionHash": "GJVAJZSZBIw=", | ||||||
|       "verbs": [ |       "verbs": [ | ||||||
|         "create", |         "create", | ||||||
|         "delete", |         "delete", | ||||||
| @@ -36,7 +36,7 @@ | |||||||
|       "name": "prioritylevelconfigurations", |       "name": "prioritylevelconfigurations", | ||||||
|       "namespaced": false, |       "namespaced": false, | ||||||
|       "singularName": "prioritylevelconfiguration", |       "singularName": "prioritylevelconfiguration", | ||||||
|       "storageVersionHash": "+CwSrEWTDhc=", |       "storageVersionHash": "Kir5PVfvNeI=", | ||||||
|       "verbs": [ |       "verbs": [ | ||||||
|         "create", |         "create", | ||||||
|         "delete", |         "delete", | ||||||
|   | |||||||
| @@ -41,9 +41,6 @@ func Install(scheme *runtime.Scheme) { | |||||||
| 	utilruntime.Must(flowcontrolv1beta3.AddToScheme(scheme)) | 	utilruntime.Must(flowcontrolv1beta3.AddToScheme(scheme)) | ||||||
| 	utilruntime.Must(flowcontrolv1.AddToScheme(scheme)) | 	utilruntime.Must(flowcontrolv1.AddToScheme(scheme)) | ||||||
|  |  | ||||||
| 	// TODO(#121119): This controls serialization order, for 1.29, we continue | 	utilruntime.Must(scheme.SetVersionPriority(flowcontrolv1.SchemeGroupVersion, flowcontrolv1beta3.SchemeGroupVersion, | ||||||
| 	//  to use v1beta3 as the serialization version because vN-1 understands that |  | ||||||
| 	//  level. In 1.30, we should set the serialization version to v1. |  | ||||||
| 	utilruntime.Must(scheme.SetVersionPriority(flowcontrolv1beta3.SchemeGroupVersion, flowcontrolv1.SchemeGroupVersion, |  | ||||||
| 		flowcontrolv1beta2.SchemeGroupVersion, flowcontrolv1beta1.SchemeGroupVersion)) | 		flowcontrolv1beta2.SchemeGroupVersion, flowcontrolv1beta1.SchemeGroupVersion)) | ||||||
| } | } | ||||||
|   | |||||||
| @@ -82,8 +82,8 @@ var GVRToStorageVersionHash = map[string]string{ | |||||||
| 	"admissionregistration.k8s.io/v1/mutatingwebhookconfigurations":    "Sqi0GUgDaX0=", | 	"admissionregistration.k8s.io/v1/mutatingwebhookconfigurations":    "Sqi0GUgDaX0=", | ||||||
| 	"admissionregistration.k8s.io/v1/validatingwebhookconfigurations":  "B0wHjQmsGNk=", | 	"admissionregistration.k8s.io/v1/validatingwebhookconfigurations":  "B0wHjQmsGNk=", | ||||||
| 	"events.k8s.io/v1/events":                                          "r2yiGXH7wu8=", | 	"events.k8s.io/v1/events":                                          "r2yiGXH7wu8=", | ||||||
| 	"flowcontrol.apiserver.k8s.io/v1beta3/flowschemas":                 "9NnFrw7ZEmA=", | 	"flowcontrol.apiserver.k8s.io/v1beta3/flowschemas":                 "GJVAJZSZBIw=", | ||||||
| 	"flowcontrol.apiserver.k8s.io/v1beta3/prioritylevelconfigurations": "+CwSrEWTDhc=", | 	"flowcontrol.apiserver.k8s.io/v1beta3/prioritylevelconfigurations": "Kir5PVfvNeI=", | ||||||
| 	"flowcontrol.apiserver.k8s.io/v1/flowschemas":                      "9NnFrw7ZEmA=", | 	"flowcontrol.apiserver.k8s.io/v1/flowschemas":                      "GJVAJZSZBIw=", | ||||||
| 	"flowcontrol.apiserver.k8s.io/v1/prioritylevelconfigurations":      "+CwSrEWTDhc=", | 	"flowcontrol.apiserver.k8s.io/v1/prioritylevelconfigurations":      "Kir5PVfvNeI=", | ||||||
| } | } | ||||||
|   | |||||||
| @@ -228,7 +228,7 @@ func GetEtcdStorageDataForNamespace(namespace string) map[schema.GroupVersionRes | |||||||
| 		gvr("flowcontrol.apiserver.k8s.io", "v1beta1", "flowschemas"): { | 		gvr("flowcontrol.apiserver.k8s.io", "v1beta1", "flowschemas"): { | ||||||
| 			Stub:             `{"metadata": {"name": "va2"}, "spec": {"priorityLevelConfiguration": {"name": "name1"}}}`, | 			Stub:             `{"metadata": {"name": "va2"}, "spec": {"priorityLevelConfiguration": {"name": "name1"}}}`, | ||||||
| 			ExpectedEtcdPath: "/registry/flowschemas/va2", | 			ExpectedEtcdPath: "/registry/flowschemas/va2", | ||||||
| 			ExpectedGVK:      gvkP("flowcontrol.apiserver.k8s.io", "v1beta3", "FlowSchema"), | 			ExpectedGVK:      gvkP("flowcontrol.apiserver.k8s.io", "v1", "FlowSchema"), | ||||||
| 		}, | 		}, | ||||||
| 		// -- | 		// -- | ||||||
|  |  | ||||||
| @@ -236,7 +236,7 @@ func GetEtcdStorageDataForNamespace(namespace string) map[schema.GroupVersionRes | |||||||
| 		gvr("flowcontrol.apiserver.k8s.io", "v1beta1", "prioritylevelconfigurations"): { | 		gvr("flowcontrol.apiserver.k8s.io", "v1beta1", "prioritylevelconfigurations"): { | ||||||
| 			Stub:             `{"metadata": {"name": "conf2"}, "spec": {"type": "Limited", "limited": {"assuredConcurrencyShares":3, "limitResponse": {"type": "Reject"}}}}`, | 			Stub:             `{"metadata": {"name": "conf2"}, "spec": {"type": "Limited", "limited": {"assuredConcurrencyShares":3, "limitResponse": {"type": "Reject"}}}}`, | ||||||
| 			ExpectedEtcdPath: "/registry/prioritylevelconfigurations/conf2", | 			ExpectedEtcdPath: "/registry/prioritylevelconfigurations/conf2", | ||||||
| 			ExpectedGVK:      gvkP("flowcontrol.apiserver.k8s.io", "v1beta3", "PriorityLevelConfiguration"), | 			ExpectedGVK:      gvkP("flowcontrol.apiserver.k8s.io", "v1", "PriorityLevelConfiguration"), | ||||||
| 		}, | 		}, | ||||||
| 		// -- | 		// -- | ||||||
|  |  | ||||||
| @@ -244,7 +244,7 @@ func GetEtcdStorageDataForNamespace(namespace string) map[schema.GroupVersionRes | |||||||
| 		gvr("flowcontrol.apiserver.k8s.io", "v1beta2", "flowschemas"): { | 		gvr("flowcontrol.apiserver.k8s.io", "v1beta2", "flowschemas"): { | ||||||
| 			Stub:             `{"metadata": {"name": "fs-1"}, "spec": {"priorityLevelConfiguration": {"name": "name1"}}}`, | 			Stub:             `{"metadata": {"name": "fs-1"}, "spec": {"priorityLevelConfiguration": {"name": "name1"}}}`, | ||||||
| 			ExpectedEtcdPath: "/registry/flowschemas/fs-1", | 			ExpectedEtcdPath: "/registry/flowschemas/fs-1", | ||||||
| 			ExpectedGVK:      gvkP("flowcontrol.apiserver.k8s.io", "v1beta3", "FlowSchema"), | 			ExpectedGVK:      gvkP("flowcontrol.apiserver.k8s.io", "v1", "FlowSchema"), | ||||||
| 		}, | 		}, | ||||||
| 		// -- | 		// -- | ||||||
|  |  | ||||||
| @@ -252,7 +252,7 @@ func GetEtcdStorageDataForNamespace(namespace string) map[schema.GroupVersionRes | |||||||
| 		gvr("flowcontrol.apiserver.k8s.io", "v1beta2", "prioritylevelconfigurations"): { | 		gvr("flowcontrol.apiserver.k8s.io", "v1beta2", "prioritylevelconfigurations"): { | ||||||
| 			Stub:             `{"metadata": {"name": "conf3"}, "spec": {"type": "Limited", "limited": {"assuredConcurrencyShares":3, "limitResponse": {"type": "Reject"}}}}`, | 			Stub:             `{"metadata": {"name": "conf3"}, "spec": {"type": "Limited", "limited": {"assuredConcurrencyShares":3, "limitResponse": {"type": "Reject"}}}}`, | ||||||
| 			ExpectedEtcdPath: "/registry/prioritylevelconfigurations/conf3", | 			ExpectedEtcdPath: "/registry/prioritylevelconfigurations/conf3", | ||||||
| 			ExpectedGVK:      gvkP("flowcontrol.apiserver.k8s.io", "v1beta3", "PriorityLevelConfiguration"), | 			ExpectedGVK:      gvkP("flowcontrol.apiserver.k8s.io", "v1", "PriorityLevelConfiguration"), | ||||||
| 		}, | 		}, | ||||||
| 		// -- | 		// -- | ||||||
|  |  | ||||||
| @@ -260,6 +260,7 @@ func GetEtcdStorageDataForNamespace(namespace string) map[schema.GroupVersionRes | |||||||
| 		gvr("flowcontrol.apiserver.k8s.io", "v1beta3", "flowschemas"): { | 		gvr("flowcontrol.apiserver.k8s.io", "v1beta3", "flowschemas"): { | ||||||
| 			Stub:             `{"metadata": {"name": "fs-2"}, "spec": {"priorityLevelConfiguration": {"name": "name1"}}}`, | 			Stub:             `{"metadata": {"name": "fs-2"}, "spec": {"priorityLevelConfiguration": {"name": "name1"}}}`, | ||||||
| 			ExpectedEtcdPath: "/registry/flowschemas/fs-2", | 			ExpectedEtcdPath: "/registry/flowschemas/fs-2", | ||||||
|  | 			ExpectedGVK:      gvkP("flowcontrol.apiserver.k8s.io", "v1", "FlowSchema"), | ||||||
| 		}, | 		}, | ||||||
| 		// -- | 		// -- | ||||||
|  |  | ||||||
| @@ -267,6 +268,7 @@ func GetEtcdStorageDataForNamespace(namespace string) map[schema.GroupVersionRes | |||||||
| 		gvr("flowcontrol.apiserver.k8s.io", "v1beta3", "prioritylevelconfigurations"): { | 		gvr("flowcontrol.apiserver.k8s.io", "v1beta3", "prioritylevelconfigurations"): { | ||||||
| 			Stub:             `{"metadata": {"name": "conf4"}, "spec": {"type": "Limited", "limited": {"nominalConcurrencyShares":3, "limitResponse": {"type": "Reject"}}}}`, | 			Stub:             `{"metadata": {"name": "conf4"}, "spec": {"type": "Limited", "limited": {"nominalConcurrencyShares":3, "limitResponse": {"type": "Reject"}}}}`, | ||||||
| 			ExpectedEtcdPath: "/registry/prioritylevelconfigurations/conf4", | 			ExpectedEtcdPath: "/registry/prioritylevelconfigurations/conf4", | ||||||
|  | 			ExpectedGVK:      gvkP("flowcontrol.apiserver.k8s.io", "v1", "PriorityLevelConfiguration"), | ||||||
| 		}, | 		}, | ||||||
| 		// -- | 		// -- | ||||||
|  |  | ||||||
| @@ -274,7 +276,6 @@ func GetEtcdStorageDataForNamespace(namespace string) map[schema.GroupVersionRes | |||||||
| 		gvr("flowcontrol.apiserver.k8s.io", "v1", "flowschemas"): { | 		gvr("flowcontrol.apiserver.k8s.io", "v1", "flowschemas"): { | ||||||
| 			Stub:             `{"metadata": {"name": "fs-3"}, "spec": {"priorityLevelConfiguration": {"name": "name1"}}}`, | 			Stub:             `{"metadata": {"name": "fs-3"}, "spec": {"priorityLevelConfiguration": {"name": "name1"}}}`, | ||||||
| 			ExpectedEtcdPath: "/registry/flowschemas/fs-3", | 			ExpectedEtcdPath: "/registry/flowschemas/fs-3", | ||||||
| 			ExpectedGVK:      gvkP("flowcontrol.apiserver.k8s.io", "v1beta3", "FlowSchema"), |  | ||||||
| 		}, | 		}, | ||||||
| 		// -- | 		// -- | ||||||
|  |  | ||||||
| @@ -282,7 +283,6 @@ func GetEtcdStorageDataForNamespace(namespace string) map[schema.GroupVersionRes | |||||||
| 		gvr("flowcontrol.apiserver.k8s.io", "v1", "prioritylevelconfigurations"): { | 		gvr("flowcontrol.apiserver.k8s.io", "v1", "prioritylevelconfigurations"): { | ||||||
| 			Stub:             `{"metadata": {"name": "conf5"}, "spec": {"type": "Limited", "limited": {"nominalConcurrencyShares":3, "limitResponse": {"type": "Reject"}}}}`, | 			Stub:             `{"metadata": {"name": "conf5"}, "spec": {"type": "Limited", "limited": {"nominalConcurrencyShares":3, "limitResponse": {"type": "Reject"}}}}`, | ||||||
| 			ExpectedEtcdPath: "/registry/prioritylevelconfigurations/conf5", | 			ExpectedEtcdPath: "/registry/prioritylevelconfigurations/conf5", | ||||||
| 			ExpectedGVK:      gvkP("flowcontrol.apiserver.k8s.io", "v1beta3", "PriorityLevelConfiguration"), |  | ||||||
| 		}, | 		}, | ||||||
| 		// -- | 		// -- | ||||||
|  |  | ||||||
|   | |||||||
		Reference in New Issue
	
	Block a user
	 Kubernetes Prow Robot
					Kubernetes Prow Robot