diff --git a/internal/olm/operator/bundle/install.go b/internal/olm/operator/bundle/install.go index eecfcbdd92f..8058ae42d23 100644 --- a/internal/olm/operator/bundle/install.go +++ b/internal/olm/operator/bundle/install.go @@ -97,7 +97,7 @@ func (i *Install) setup(ctx context.Context) error { // check if index image adopts File-Based Catalog or SQLite index image format isFBCImage, err := fbcutil.IsFBC(ctx, i.IndexImageCatalogCreator.IndexImage) if err != nil { - return fmt.Errorf("error in upgrading the bundle %q that was installed traditionally", i.IndexImageCatalogCreator.BundleImage) + return fmt.Errorf("error determining whether index %q is FBC or SQLite based: %v", i.IndexImageCatalogCreator.IndexImage, err) } i.IndexImageCatalogCreator.HasFBCLabel = isFBCImage diff --git a/internal/olm/operator/registry/index_image.go b/internal/olm/operator/registry/index_image.go index edb66e16b6e..9ed02fff1b4 100644 --- a/internal/olm/operator/registry/index_image.go +++ b/internal/olm/operator/registry/index_image.go @@ -148,17 +148,17 @@ func handleTraditionalUpgrade(ctx context.Context, indexImage string, bundleImag // render the index image originalDeclCfg, err := fbcutil.RenderRefs(ctx, []string{indexImage}) if err != nil { - return "", fmt.Errorf("error in rendering index %q", indexImage) + return "", fmt.Errorf("error rendering index %q", indexImage) } // render the bundle image bundleDeclConfig, err := fbcutil.RenderRefs(ctx, []string{bundleImage}) if err != nil { - return "", fmt.Errorf("error in rendering index %q", bundleImage) + return "", fmt.Errorf("error rendering bundle image %q", bundleImage) } if len(bundleDeclConfig.Bundles) != 1 { - return "", errors.New("bundle image must have exactly one bundle") + return "", errors.New("rendered bundle must have exactly one bundle") } // search for the specific channel in which the upgrade needs to take place, and upgrade from the channel head