-
Notifications
You must be signed in to change notification settings - Fork 41
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
MGDAPI-6218 CRO release-v1.1.3 #723
MGDAPI-6218 CRO release-v1.1.3 #723
Conversation
3c86ec7
to
e2ba47e
Compare
Changes look good but not sure why you have an index image already , as the index should only be built off master after this merges. /lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: austincunningham 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 |
Checked the pipeline looks like you built the index off your branch . Not ideal if we had to make changes to your PR after its built. We got away with in this time I guess. |
….1.3 MGDAPI-6218 CRO release-v1.1.3
Overview
Jira: https://issues.redhat.com/browse/MGDAPI-6218
-- change Makefile: VERSION=1.1.3, PREV_VERSION=1.1.2
-- run
make gen/csv
Verification
Following verification was done for PR:
make cluster/prepare
make run
CRO Index image was prepared: https://quay.io/repository/integreatly/cloud-resource-operator?tab=tags&tag=index-v1.1.3 , using Pipeline https://master-jenkins-csb-intly.apps.ocp-c1.prod.psi.redhat.com/job/Playground/job/vmogilev/job/cro-release-2/
RHOAM installed successfully with new CRO