Bug 1889714 - opm index export doesn't properly recreate graph using semver mode
Summary: opm index export doesn't properly recreate graph using semver mode
Keywords:
Status: POST
Alias: None
Product: OpenShift Container Platform
Classification: Red Hat
Component: OLM
Version: 4.6
Hardware: Unspecified
OS: Unspecified
high
high
Target Milestone: ---
: 4.7.0
Assignee: Ankita Thomas
QA Contact: Jian Zhang
URL:
Whiteboard:
: 1889716 (view as bug list)
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2020-10-20 12:53 UTC by Kevin Rizza
Modified: 2020-11-03 11:01 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed:
Target Upstream Version:


Attachments (Terms of Use)


Links
System ID Priority Status Summary Last Updated
Github operator-framework operator-registry pull 504 None open Bug 1889714: ensure replaces and skips field on exported bundles 2020-11-27 03:13:50 UTC

Description Kevin Rizza 2020-10-20 12:53:07 UTC
Description of problem:

Currently, when using `opm index export` to backfill the index image back to the appregistry nested directory format, opm generates the set of bundles included in a package and pulls the csvs for each bundle into the directory. Since those bundles become the source of truth for all graph information, any mode that takes that source of truth away from the replaces chain in the set of csvs is lost. Instead, opm should respect the changes made to the graph defined in the index image and modify the exported manifests to generate a replaces graph that properly encapsulates the information generated by index add commands using semver and semver-skippatch modes.


Version-Release number of selected component (if applicable):

4.6

How reproducible:

Always

Steps to Reproduce:
1. Add a set of bundles that do not have `replaces` set in their csvs using `opm index add` to an index image
2. `opm index export` that index image

Actual results:

The csvs do not have replaces set, the graphs those bundles represent will not link all of the bundles down from latest version to oldest version.

Expected results:

Exported csvs have replaces set properly so that the eventual app-registry blob can be unpacked by a cluster at runtime with the correct graph as defined in the index image.

Comment 1 Kevin Rizza 2020-10-20 18:27:30 UTC
*** Bug 1889716 has been marked as a duplicate of this bug. ***

Comment 2 Luiz Carvalho 2020-10-21 19:42:29 UTC
Any estimation of when this will be ready?

Comment 3 Nick Hale 2020-10-23 17:22:34 UTC
This is assigned to be worked on in the upcoming sprint.


Note You need to log in before you can comment on or make changes to this bug.