-
Notifications
You must be signed in to change notification settings - Fork 1.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Replace OperatorSource with CatalogSource for backing service in Service Binding #25383
Replace OperatorSource with CatalogSource for backing service in Service Binding #25383
Conversation
3948425
to
4772410
Compare
Hi @vikram-redhat Could you please take a look at this? Thx |
@Preeticp PTAL |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@pmacik thank you for this PR.
I have made a minor style suggestion as per docs conventions. Other than that, it's an LGTM from me.
@pmacik just wanted to confirm, AFAIU, this PR needs to be merged to the 4.5, 4.6 and later versions, right? |
Yes. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@pmacik if you could please fix these comments I will ping the OCP docs team and ask them for a review and merge. Thank you.
/lgtm |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@pmacik and @Preeticp I have outlined a couple of suggestions below. Other than that, this PR looks good to me!
-
Please squash your commits into one.
-
Line 64 in the updated module file (
modules/odc-connecting-components.adoc
) still refers to the backingOperatorSource
. Please update this introduction to the procedure so that it references theCatalogSource
resource instead.
2c3c136
to
082b457
Compare
New changes are detected. LGTM label has been removed. |
@pneedle-rh Good catch, thx. I've fixed, rebased and squashed the commits. |
082b457
to
4634ee6
Compare
…ice Binding Co-authored-by: Preeti Chandrashekar <preetipagad@gmail.com>
4634ee6
to
778709d
Compare
@pneedle-rh There were conflicts with the base branch, so I rebased and fixed those conflicts. |
/cherrypick enterprise-4.5 |
@pneedle-rh: new pull request created: #28192 In response to this:
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. |
/cherrypick enterprise-4.6 |
@pneedle-rh: #25383 failed to apply on top of branch "enterprise-4.6":
In response to this:
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. |
/cherrypick enterprise-4.7 |
@pneedle-rh: #25383 failed to apply on top of branch "enterprise-4.7":
In response to this:
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. |
@pmacik and @Preeticp there were merge conflicts when cherry picking this PR to enterprise-4.6 and enterprise-4.7. The conflicts were due to the object API references having already been updated to reflect the new guidelines. I have now manually cherry picked this PR for those releases. The enterprise-4.5 cherry pick is also now complete. |
This PR is meant for ocp v4.5+
The docs shows installation of a backing service operator that is done via
OperatorSource
. That is obsolete in v4.5 and removed in v4.6, so this PR updates the docs by replacing theOperatorSource
withCatalogSource
.