Skip to content

Conversation

everettraven
Copy link
Contributor

Description of the change:

  • Add a note in the bundle tutorial about specifying the index image used for handling SQLite bundles in versions of operator-sdk < v1.22.0 as FBC support wasn't enabled until v1.22.0
  • Add a FAQ section for the error that may be encountered when using operator-sdk run bundle with operator-sdk versions < v1.22.0

Motivation for the change:

Checklist

If the pull request includes user-facing changes, extra documentation is required:

Signed-off-by: Bryce Palmer <bpalmer@redhat.com>
@openshift-ci openshift-ci bot requested review from fabianvf and jmrodri October 4, 2022 19:15
Copy link
Contributor

@michaelryanpeter michaelryanpeter left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A little less surgical with this review. If this is too heavy-handed, please feel free to take what is useful and disregard the rest.

Signed-off-by: Bryce Palmer <bpalmer@redhat.com>
Copy link
Contributor

@michaelryanpeter michaelryanpeter left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@openshift-ci
Copy link

openshift-ci bot commented Oct 18, 2022

@michaelryanpeter: changing LGTM is restricted to collaborators

In response to this:

/lgtm

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.

Copy link
Member

@laxmikantbpandhare laxmikantbpandhare left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Nov 23, 2022
@laxmikantbpandhare laxmikantbpandhare merged commit f2ac68f into operator-framework:master Nov 23, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm Indicates that a PR is ready to be merged.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Failed to run bundle: install plan is not available for the subscription

3 participants