Bug 1881222: Annotate CSVs with the properties used during dependency resolution.

[benluddy]

This ensures that property information is not lost after an operator
is installed, and that the properties of an operator are the same
before and after installation. Preserving properties also allows
installed operators to satisfy dependencies on properties that cannot
be inferred from a ClusterServiceVersion spec, and is a step toward
unifying installed and available operators from the perspective of
resolution.

Reviewer Checklist

• Implementation matches the proposed design, or proposal is updated to match implementation
• Sufficient unit test coverage
• Sufficient end-to-end test coverage
• Docs updated or added to /docs
• Commit messages sensible and descriptive

[openshift-ci-robot]

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

[openshift-ci-robot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: benluddy

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

• OWNERS [benluddy]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[openshift-ci-robot]

@benluddy: This pull request references Bugzilla bug 1881222, which is invalid:

• expected the bug to target the "4.6.0" release, but it targets "4.7.0" instead

Comment /bugzilla refresh to re-evaluate validity if changes to the Bugzilla bug are made, or edit the title of this pull request to link to a different bug.

In response to this:

Bug 1881222: Annotate CSVs with the properties used during dependency resolution.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[ecordell]

/lgtm

[ecordell]

/bugzilla refresh

[openshift-ci-robot]

@ecordell: This pull request references Bugzilla bug 1881222, which is valid. The bug has been moved to the POST state.

3 validation(s) were run on this bug

• bug is open, matching expected state (open)
• bug target release (4.6.0) matches configured target release for branch (4.6.0)
• bug is in the state ASSIGNED, which is one of the valid states (NEW, ASSIGNED, ON_DEV, POST, POST)

In response to this:

/bugzilla refresh

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[openshift-bot]

/retest

Please review the full test history for this PR and help us cut down flakes.

[benluddy]

@benluddy: The following test failed, say /retest to rerun all failed tests:
Test name Commit Details Rerun command
ci/prow/e2e-aws-olm bcf0ace link /test e2e-aws-olm

Full PR test history. Your PR dashboard.

/hold

This looks like a problem with the index image fixture. It was last pushed July 22, and it's exhibiting a bug in its ListBundles response that was fixed a month later in operator-framework/operator-registry#417.

[benluddy]

/hold cancel

[openshift-ci-robot]

@benluddy: All pull requests linked via external trackers have merged:

• operator-framework/operator-lifecycle-manager#1781

Bugzilla bug 1881222 has been moved to the MODIFIED state.

In response to this:

Bug 1881222: Annotate CSVs with the properties used during dependency resolution.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.