OCPBUGS-7910: Sort channels in lexicographical order in Packagemanife…

…st (#2925) See operator-framework/operator-registry#1069 for more details Signed-off-by: Anik <anikbhattacharya93@gmail.com> Upstream-repository: operator-lifecycle-manager Upstream-commit: e4b15326c297f610dbf569d771abd62a703b0eb2
anik120 · Mar 22, 2023 · 4d388fa · 4d388fa
1 parent 3a82ea6
commit 4d388fa
Show file tree

Hide file tree

Showing 3 changed files with 100 additions and 0 deletions.
diff --git a/staging/operator-lifecycle-manager/pkg/package-server/provider/registry.go b/staging/operator-lifecycle-manager/pkg/package-server/provider/registry.go
@@ -512,6 +512,9 @@ func (p *RegistryProvider) List(namespace string, selector labels.Selector) (*op
 
 func newPackageManifest(ctx context.Context, logger *logrus.Entry, pkg *api.Package, client *registryClient, entriesByChannel map[string][]operators.ChannelEntry) (*operators.PackageManifest, error) {
 	pkgChannels := pkg.GetChannels()
+	sort.Slice(pkgChannels, func(i, j int) bool {
+		return pkgChannels[i].Name < pkgChannels[j].Name
+	})
 	catsrc := client.catsrc
 	manifest := &operators.PackageManifest{
 		ObjectMeta: metav1.ObjectMeta{