From b2687c1a84e4b3b1be6700251e6ef7233427712a Mon Sep 17 00:00:00 2001 From: Tim Hockin Date: Wed, 25 Mar 2015 14:45:07 -0700 Subject: [PATCH] rename ResetBeforeCreate to PrepareForCreate --- pkg/api/rest/create.go | 11 ++++++----- pkg/api/rest/types.go | 4 ++-- pkg/registry/controller/rest.go | 4 ++-- pkg/registry/endpoint/rest.go | 4 ++-- pkg/registry/generic/etcd/etcd_test.go | 2 +- pkg/registry/minion/rest.go | 4 ++-- pkg/registry/namespace/rest.go | 4 ++-- pkg/registry/namespace/rest_test.go | 2 +- pkg/registry/pod/rest.go | 4 ++-- pkg/registry/resourcequota/rest.go | 4 ++-- pkg/registry/resourcequota/rest_test.go | 2 +- 11 files changed, 23 insertions(+), 22 deletions(-) diff --git a/pkg/api/rest/create.go b/pkg/api/rest/create.go index 27f0d34c397..9089554a9d2 100644 --- a/pkg/api/rest/create.go +++ b/pkg/api/rest/create.go @@ -34,16 +34,17 @@ type RESTCreateStrategy interface { // NamespaceScoped returns true if the object must be within a namespace. NamespaceScoped() bool - // ResetBeforeCreate is invoked on create before validation to remove any fields - // that may not be persisted. - ResetBeforeCreate(obj runtime.Object) + // PrepareForCreate is invoked on create before validation to normalize + // the object. For example: remove fields that are not to be persisted, + // sort order-insensitive list fields, etc. + PrepareForCreate(obj runtime.Object) // Validate is invoked after default fields in the object have been filled in before // the object is persisted. Validate(obj runtime.Object) fielderrors.ValidationErrorList } // BeforeCreate ensures that common operations for all resources are performed on creation. It only returns -// errors that can be converted to api.Status. It invokes ResetBeforeCreate, then GenerateName, then Validate. +// errors that can be converted to api.Status. It invokes PrepareForCreate, then GenerateName, then Validate. // It returns nil if the object should be created. func BeforeCreate(strategy RESTCreateStrategy, ctx api.Context, obj runtime.Object) error { objectMeta, kind, kerr := objectMetaAndKind(strategy, obj) @@ -58,7 +59,7 @@ func BeforeCreate(strategy RESTCreateStrategy, ctx api.Context, obj runtime.Obje } else { objectMeta.Namespace = api.NamespaceNone } - strategy.ResetBeforeCreate(obj) + strategy.PrepareForCreate(obj) api.FillObjectMetaSystemFields(ctx, objectMeta) api.GenerateName(strategy, objectMeta) diff --git a/pkg/api/rest/types.go b/pkg/api/rest/types.go index fe839781d71..6ed38e998f8 100644 --- a/pkg/api/rest/types.go +++ b/pkg/api/rest/types.go @@ -60,8 +60,8 @@ func (svcStrategy) NamespaceScoped() bool { return true } -// ResetBeforeCreate clears fields that are not allowed to be set by end users on creation. -func (svcStrategy) ResetBeforeCreate(obj runtime.Object) { +// PrepareForCreate clears fields that are not allowed to be set by end users on creation. +func (svcStrategy) PrepareForCreate(obj runtime.Object) { service := obj.(*api.Service) service.Status = api.ServiceStatus{} } diff --git a/pkg/registry/controller/rest.go b/pkg/registry/controller/rest.go index 4ca1e22707d..2a7890748b9 100644 --- a/pkg/registry/controller/rest.go +++ b/pkg/registry/controller/rest.go @@ -42,8 +42,8 @@ func (rcStrategy) NamespaceScoped() bool { return true } -// ResetBeforeCreate clears the status of a replication controller before creation. -func (rcStrategy) ResetBeforeCreate(obj runtime.Object) { +// PrepareForCreate clears the status of a replication controller before creation. +func (rcStrategy) PrepareForCreate(obj runtime.Object) { controller := obj.(*api.ReplicationController) controller.Status = api.ReplicationControllerStatus{} } diff --git a/pkg/registry/endpoint/rest.go b/pkg/registry/endpoint/rest.go index b41b72709e1..9cad99e5d0e 100644 --- a/pkg/registry/endpoint/rest.go +++ b/pkg/registry/endpoint/rest.go @@ -43,8 +43,8 @@ func (endpointsStrategy) NamespaceScoped() bool { return true } -// ResetBeforeCreate clears fields that are not allowed to be set by end users on creation. -func (endpointsStrategy) ResetBeforeCreate(obj runtime.Object) { +// PrepareForCreate clears fields that are not allowed to be set by end users on creation. +func (endpointsStrategy) PrepareForCreate(obj runtime.Object) { _ = obj.(*api.Endpoints) } diff --git a/pkg/registry/generic/etcd/etcd_test.go b/pkg/registry/generic/etcd/etcd_test.go index 72580ec691a..d5bb4c984e7 100644 --- a/pkg/registry/generic/etcd/etcd_test.go +++ b/pkg/registry/generic/etcd/etcd_test.go @@ -43,7 +43,7 @@ type testRESTStrategy struct { func (t *testRESTStrategy) NamespaceScoped() bool { return t.namespaceScoped } func (t *testRESTStrategy) AllowCreateOnUpdate() bool { return t.allowCreateOnUpdate } -func (t *testRESTStrategy) ResetBeforeCreate(obj runtime.Object) {} +func (t *testRESTStrategy) PrepareForCreate(obj runtime.Object) {} func (t *testRESTStrategy) Validate(obj runtime.Object) fielderrors.ValidationErrorList { return nil } diff --git a/pkg/registry/minion/rest.go b/pkg/registry/minion/rest.go index 418ae7b00a7..ed7e4ef8758 100644 --- a/pkg/registry/minion/rest.go +++ b/pkg/registry/minion/rest.go @@ -53,8 +53,8 @@ func (nodeStrategy) AllowCreateOnUpdate() bool { return false } -// ResetBeforeCreate clears fields that are not allowed to be set by end users on creation. -func (nodeStrategy) ResetBeforeCreate(obj runtime.Object) { +// PrepareForCreate clears fields that are not allowed to be set by end users on creation. +func (nodeStrategy) PrepareForCreate(obj runtime.Object) { _ = obj.(*api.Node) // Nodes allow *all* fields, including status, to be set. } diff --git a/pkg/registry/namespace/rest.go b/pkg/registry/namespace/rest.go index 371524991ed..d9379ed3929 100644 --- a/pkg/registry/namespace/rest.go +++ b/pkg/registry/namespace/rest.go @@ -43,8 +43,8 @@ func (namespaceStrategy) NamespaceScoped() bool { return false } -// ResetBeforeCreate clears fields that are not allowed to be set by end users on creation. -func (namespaceStrategy) ResetBeforeCreate(obj runtime.Object) { +// PrepareForCreate clears fields that are not allowed to be set by end users on creation. +func (namespaceStrategy) PrepareForCreate(obj runtime.Object) { // on create, status is active namespace := obj.(*api.Namespace) namespace.Status = api.NamespaceStatus{ diff --git a/pkg/registry/namespace/rest_test.go b/pkg/registry/namespace/rest_test.go index 0b36fa87843..9ac4d138484 100644 --- a/pkg/registry/namespace/rest_test.go +++ b/pkg/registry/namespace/rest_test.go @@ -33,7 +33,7 @@ func TestNamespaceStrategy(t *testing.T) { ObjectMeta: api.ObjectMeta{Name: "foo"}, Status: api.NamespaceStatus{Phase: api.NamespaceTerminating}, } - Strategy.ResetBeforeCreate(namespace) + Strategy.PrepareForCreate(namespace) if namespace.Status.Phase != api.NamespaceActive { t.Errorf("Namespaces do not allow setting phase on create") } diff --git a/pkg/registry/pod/rest.go b/pkg/registry/pod/rest.go index e92da88cc85..bc7f6fce73f 100644 --- a/pkg/registry/pod/rest.go +++ b/pkg/registry/pod/rest.go @@ -50,8 +50,8 @@ func (podStrategy) NamespaceScoped() bool { return true } -// ResetBeforeCreate clears fields that are not allowed to be set by end users on creation. -func (podStrategy) ResetBeforeCreate(obj runtime.Object) { +// PrepareForCreate clears fields that are not allowed to be set by end users on creation. +func (podStrategy) PrepareForCreate(obj runtime.Object) { pod := obj.(*api.Pod) pod.Status = api.PodStatus{ Host: pod.Spec.Host, diff --git a/pkg/registry/resourcequota/rest.go b/pkg/registry/resourcequota/rest.go index d800afc56ec..dbc351d2462 100644 --- a/pkg/registry/resourcequota/rest.go +++ b/pkg/registry/resourcequota/rest.go @@ -43,8 +43,8 @@ func (resourcequotaStrategy) NamespaceScoped() bool { return true } -// ResetBeforeCreate clears fields that are not allowed to be set by end users on creation. -func (resourcequotaStrategy) ResetBeforeCreate(obj runtime.Object) { +// PrepareForCreate clears fields that are not allowed to be set by end users on creation. +func (resourcequotaStrategy) PrepareForCreate(obj runtime.Object) { resourcequota := obj.(*api.ResourceQuota) resourcequota.Status = api.ResourceQuotaStatus{} } diff --git a/pkg/registry/resourcequota/rest_test.go b/pkg/registry/resourcequota/rest_test.go index f54eb3a0031..cc36b91c0cf 100644 --- a/pkg/registry/resourcequota/rest_test.go +++ b/pkg/registry/resourcequota/rest_test.go @@ -51,7 +51,7 @@ func TestResourceQuotaStrategy(t *testing.T) { }, }, } - Strategy.ResetBeforeCreate(resourceQuota) + Strategy.PrepareForCreate(resourceQuota) if resourceQuota.Status.Used != nil { t.Errorf("ResourceQuota does not allow setting status on create") }