Merge pull request #791 from njhale/fix-back-support

fix(packageserver): add struct tags to PackageManifestList

Author: openshift-merge-robot

pkg/package-server/apis/openapi/zz_generated.openapi.go

@@ -10,10 +10,10 @@ import (
 // PackageManifestList is a list of PackageManifest objects.
 // +k8s:deepcopy-gen:interfaces=k8s.io/apimachinery/pkg/runtime.Object
 type PackageManifestList struct {
-	metav1.TypeMeta
-	metav1.ListMeta
+	metav1.TypeMeta `json:",inline"`
+	metav1.ListMeta `json:"metadata,omitempty"`
 
-	Items []PackageManifest
+	Items []PackageManifest `json:"items"`
 }
 
 // PackageManifest holds information about a package, which is a reference to one (or more)

pkg/package-server/apis/operators/v1/packagemanifest_types.go

@@ -108,4 +108,10 @@ func TestPackageManifestLoading(t *testing.T) {
 	require.NotNil(t, pm)
 	require.Equal(t, packageName, pm.GetName())
 	require.Equal(t, expectedStatus, pm.Status)
+
+	// Get a PackageManifestList and ensure it has the correct items
+	pmList, err := pmc.OperatorsV1().PackageManifests(testNamespace).List(metav1.ListOptions{})
+	require.NoError(t, err, "could not access package manifests list meta")
+	require.NotNil(t, pmList.ListMeta, "package manifest list metadata empty")
+	require.NotNil(t, pmList.Items)
 }