Merge pull request #2343 from Poor12/fix-logs

k8s-ci-robot · web-flow · commit 73949d01fbbe · 2023-05-24T02:42:49.000-07:00
✨Fix logs in unstructured client
diff --git a/pkg/client/unstructured_client.go b/pkg/client/unstructured_client.go
@@ -224,11 +224,11 @@ func (uc *unstructuredClient) List(ctx context.Context, obj ObjectList, opts ...
 
 func (uc *unstructuredClient) GetSubResource(ctx context.Context, obj, subResourceObj Object, subResource string, opts ...SubResourceGetOption) error {
 	if _, ok := obj.(runtime.Unstructured); !ok {
-		return fmt.Errorf("unstructured client did not understand object: %T", subResource)
+		return fmt.Errorf("unstructured client did not understand object: %T", obj)
 	}
 
 	if _, ok := subResourceObj.(runtime.Unstructured); !ok {
-		return fmt.Errorf("unstructured client did not understand object: %T", obj)
+		return fmt.Errorf("unstructured client did not understand object: %T", subResourceObj)
 	}
 
 	if subResourceObj.GetName() == "" {
@@ -255,11 +255,11 @@ func (uc *unstructuredClient) GetSubResource(ctx context.Context, obj, subResour
 
 func (uc *unstructuredClient) CreateSubResource(ctx context.Context, obj, subResourceObj Object, subResource string, opts ...SubResourceCreateOption) error {
 	if _, ok := obj.(runtime.Unstructured); !ok {
-		return fmt.Errorf("unstructured client did not understand object: %T", subResourceObj)
+		return fmt.Errorf("unstructured client did not understand object: %T", obj)
 	}
 
 	if _, ok := subResourceObj.(runtime.Unstructured); !ok {
-		return fmt.Errorf("unstructured client did not understand object: %T", obj)
+		return fmt.Errorf("unstructured client did not understand object: %T", subResourceObj)
 	}
 
 	if subResourceObj.GetName() == "" {

Original file line number	Diff line number	Diff line change
`@@ -224,11 +224,11 @@ func (uc *unstructuredClient) List(ctx context.Context, obj ObjectList, opts ...`
`224`	`224`
`225`	`225`	`func (uc *unstructuredClient) GetSubResource(ctx context.Context, obj, subResourceObj Object, subResource string, opts ...SubResourceGetOption) error {`
`226`	`226`	`if _, ok := obj.(runtime.Unstructured); !ok {`
`227`		`- return fmt.Errorf("unstructured client did not understand object: %T", subResource)`
	`227`	`+ return fmt.Errorf("unstructured client did not understand object: %T", obj)`
`228`	`228`	`}`
`229`	`229`
`230`	`230`	`if _, ok := subResourceObj.(runtime.Unstructured); !ok {`
`231`		`- return fmt.Errorf("unstructured client did not understand object: %T", obj)`
	`231`	`+ return fmt.Errorf("unstructured client did not understand object: %T", subResourceObj)`
`232`	`232`	`}`
`233`	`233`
`234`	`234`	`if subResourceObj.GetName() == "" {`
`@@ -255,11 +255,11 @@ func (uc *unstructuredClient) GetSubResource(ctx context.Context, obj, subResour`
`255`	`255`
`256`	`256`	`func (uc *unstructuredClient) CreateSubResource(ctx context.Context, obj, subResourceObj Object, subResource string, opts ...SubResourceCreateOption) error {`
`257`	`257`	`if _, ok := obj.(runtime.Unstructured); !ok {`
`258`		`- return fmt.Errorf("unstructured client did not understand object: %T", subResourceObj)`
	`258`	`+ return fmt.Errorf("unstructured client did not understand object: %T", obj)`
`259`	`259`	`}`
`260`	`260`
`261`	`261`	`if _, ok := subResourceObj.(runtime.Unstructured); !ok {`
`262`		`- return fmt.Errorf("unstructured client did not understand object: %T", obj)`
	`262`	`+ return fmt.Errorf("unstructured client did not understand object: %T", subResourceObj)`
`263`	`263`	`}`
`264`	`264`
`265`	`265`	`if subResourceObj.GetName() == "" {`