-
Notifications
You must be signed in to change notification settings - Fork 113
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
[openshift_setup] Add support for overriding dependent operators CatalogSource #2694
[openshift_setup] Add support for overriding dependent operators CatalogSource #2694
Conversation
Skipping CI for Draft Pull Request. |
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/ecec170bd3104576a89ea2a78c57819c ✔️ openstack-k8s-operators-content-provider SUCCESS in 1h 26m 33s |
d4b8f9e
to
c30c162
Compare
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/eb7e705be4114ebda9f0a21729642e38 ✔️ openstack-k8s-operators-content-provider SUCCESS in 1h 24m 56s |
c30c162
to
203c5ef
Compare
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/d66b44e84dfc459d973537c0f6d299b0 ✔️ openstack-k8s-operators-content-provider SUCCESS in 3h 09m 34s |
52409d6
to
c281822
Compare
roles/openshift_setup/tasks/patch_dependent_operators_source.yml
Outdated
Show resolved
Hide resolved
c281822
to
ec2eb3d
Compare
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.
Non blocking: It would be good to also add the new variables in the readme for the openshift_setup role
[APPROVALNOTIFIER] This PR is APPROVED Approval requirements bypassed by manually added approval. This pull-request has been approved by: 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:
Approvers can indicate their approval by writing |
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.
LGTM, minor request before approving.
ec2eb3d
to
ca4536e
Compare
…logSource This is needed to easily test pre-release versions of OCP. OpenStack depends on a few operators, these operators don't easily expose pre-release versions for installation. A hack like this patch allows a user to create a new CatalogSource for the current GA version of OCP and patch the dependent operators to install from it.
ca4536e
to
d074a72
Compare
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/0952a813edb8438f880cd53662a0f89d ✔️ openstack-k8s-operators-content-provider SUCCESS in 3h 28m 35s |
recheck |
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/f9023d780d474bda92687ad04524b219 ✔️ openstack-k8s-operators-content-provider SUCCESS in 3h 28m 19s |
recheck |
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.
LGTM
This is needed to easily test pre-release versions of OCP. OpenStack
depends on a few operators, these operators don't easily expose
pre-release versions for installation. A hack like this patch allows a
user to create a new CatalogSource for the current GA version of OCP and
patch the dependent operators to install from it.
JIRA: https://issues.redhat.com/browse/OSPRH-13706