Address review feedback

Signed-off-by: rashmigottipati <[email protected]>

Author: rashmigottipati

changelog/fragments/sdk-fbc-run-bundle(-upgrade).yaml

@@ -1,19 +1,17 @@
 entries:
   - description: >
-      Add File-Based Catalog support for both `run bundle` and `run bundle-upgrade` subcommands. 
-      Infer the image type and handle FBC scenarios accordingly for both commands. For FBC images,
-      Operator Registry APIs are called for rendering the bundle and/or the index image and serving these images
-      through the registry pod using a gRPC port. For images that are SQLite based, 
-      `opm registry add` and `opm serve` commands will be called during the registry pod creation. 
-      Support for these old opm commands will be deprecated and completely removed in the future. 
+      Add support for File-Based Catalog to the subcommands [operator-sdk run bundle](https://sdk.operatorframework.io/docs/cli/operator-sdk_run_bundle/#m-docsclioperator-sdk_run_bundle) 
+      and [run bundle-upgrade](https://sdk.operatorframework.io/docs/cli/operator-sdk_run_bundle-upgrade/) so that 
+      new indexes created by these subcommands are using the new format. 
+      Users are able to pass in an index catalog with FBC format via the flag option `--index-image`.
 
     # kind is one of:
     # - addition
     # - change
     # - deprecation
     # - removal
     # - bugfix
-    kind: addition
+    kind: change
 
     # Is this a breaking change?
     breaking: false

internal/cmd/operator-sdk/run/bundle/cmd.go

@@ -36,7 +36,9 @@ The main purpose of this command is to streamline running the bundle without hav
 
 The ` + "`--index-image`" + ` flag specifies an index image in which to inject the given bundle. It can be specified to resolve dependencies for a bundle. 
 This is an optional flag which will default to ` + "`quay.io/operator-framework/opm:latest`." + `
-The index image provided should **NOT** already have the bundle.
+The index image provided should **NOT** already have the bundle. A limitation of the index image flag is that it does not check the upgrade graph
+as the annotations for channels are ignored but it is still a useful flag to have to validate the dependencies. 
+For example: It does not fail fast when the bundle version provided is <= ChannelHead.
 `,
 		Args:    cobra.ExactArgs(1),
 		PreRunE: func(*cobra.Command, []string) error { return cfg.Load() },

website/content/en/docs/cli/operator-sdk_run_bundle.md

@@ -14,7 +14,9 @@ The main purpose of this command is to streamline running the bundle without hav
 
 The `--index-image` flag specifies an index image in which to inject the given bundle. It can be specified to resolve dependencies for a bundle. 
 This is an optional flag which will default to `quay.io/operator-framework/opm:latest`.
-The index image provided should **NOT** already have the bundle.
+The index image provided should **NOT** already have the bundle. A limitation of the index image flag is that it does not check the upgrade graph
+as the annotations for channels are ignored but it is still a useful flag to have to validate the dependencies. 
+For example: It does not fail fast when the bundle version provided is <= ChannelHead.
 
 
 ```