Populate GVK for listed objects

Author: liyinan926

pkg/cache/cache_test.go

@@ -198,6 +198,19 @@ func CacheTest(createCacheFunc func(config *rest.Config, opts cache.Options) (ca
 					}
 				})
 
+				It("should be able to list objects with GVK populated", func() {
+					By("listing pods")
+					listObj := &kcorev1.PodList{}
+					Expect(informerCache.List(context.Background(), listObj)).To(Succeed())
+
+					By("verifying that the returned pods have GVK populated")
+					Expect(listObj.Items).NotTo(BeEmpty())
+					Expect(listObj.Items).Should(HaveLen(3))
+					for _, p := range listObj.Items {
+						Expect(p.GroupVersionKind().Empty()).To(BeFalse())
+					}
+				})
+
 				It("should be able to list objects by namespace", func() {
 					By("listing pods in test-namespace-1")
 					listObj := &kcorev1.PodList{}

pkg/cache/internal/cache_reader.go

@@ -125,6 +125,7 @@ func (c *CacheReader) List(_ context.Context, out runtime.Object, opts ...client
 		if !isObj {
 			return fmt.Errorf("cache contained %T, which is not an Object", obj)
 		}
+		obj.GetObjectKind().SetGroupVersionKind(c.groupVersionKind)
 		runtimeObjs = append(runtimeObjs, obj)
 	}
 	filteredItems, err := objectutil.FilterWithLabels(runtimeObjs, labelSel)