Skip to content

Commit 1fa9952

Browse files
committed
Update scale sub-resource to return the same error everywhere
Signed-off-by: Maciej Szulik <[email protected]>
1 parent dba9475 commit 1fa9952

File tree

3 files changed

+3
-3
lines changed

3 files changed

+3
-3
lines changed

pkg/registry/apps/deployment/storage/storage.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -306,7 +306,7 @@ func (r *ScaleREST) Destroy() {
306306
func (r *ScaleREST) Get(ctx context.Context, name string, options *metav1.GetOptions) (runtime.Object, error) {
307307
obj, err := r.store.Get(ctx, name, options)
308308
if err != nil {
309-
return nil, errors.NewNotFound(apps.Resource("deployments/scale"), name)
309+
return nil, err
310310
}
311311
deployment := obj.(*apps.Deployment)
312312
scale, err := scaleFromDeployment(deployment)

pkg/registry/apps/replicaset/storage/storage.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -202,7 +202,7 @@ func (r *ScaleREST) Destroy() {
202202
func (r *ScaleREST) Get(ctx context.Context, name string, options *metav1.GetOptions) (runtime.Object, error) {
203203
obj, err := r.store.Get(ctx, name, options)
204204
if err != nil {
205-
return nil, errors.NewNotFound(apps.Resource("replicasets/scale"), name)
205+
return nil, err
206206
}
207207
rs := obj.(*apps.ReplicaSet)
208208
scale, err := scaleFromReplicaSet(rs)

pkg/registry/core/replicationcontroller/storage/storage.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -190,7 +190,7 @@ func (r *ScaleREST) Destroy() {
190190
func (r *ScaleREST) Get(ctx context.Context, name string, options *metav1.GetOptions) (runtime.Object, error) {
191191
obj, err := r.store.Get(ctx, name, options)
192192
if err != nil {
193-
return nil, errors.NewNotFound(autoscaling.Resource("replicationcontrollers/scale"), name)
193+
return nil, err
194194
}
195195
rc := obj.(*api.ReplicationController)
196196
return scaleFromRC(rc), nil

0 commit comments

Comments
 (0)