-
Notifications
You must be signed in to change notification settings - Fork 107
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
feat(dspv2): update manifests path for new overlay struct #891
feat(dspv2): update manifests path for new overlay struct #891
Conversation
/lgtm |
Signed-off-by: Wen Zhou <wenzhou@redhat.com>
Signed-off-by: Wen Zhou <wenzhou@redhat.com>
re-testing to see if it works with "main" dspo after new commits. if it works, it is fine to get this PR out |
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.
Tested successfully
/lgtm
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: VaishnaviHire 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 |
…b-io#891) * feat(dspv2): update manifests path Signed-off-by: Wen Zhou <wenzhou@redhat.com> * update: remove apply on "base" since overlays/odh does include "base" Signed-off-by: Wen Zhou <wenzhou@redhat.com> --------- Signed-off-by: Wen Zhou <wenzhou@redhat.com>
Description
ref : https://issues.redhat.com/browse/RHOAIENG-4223
How Has This Been Tested?
NEW
also manual tested with new build quay.io/wenzhou/opendatahub-operator-catalog:v2.9.3309-2 for ODH and it works now.
OLD
live build with dspo v2.0.0 tag
quay.io/wenzhou/opendatahub-operator-catalog:v2.9.3309-1
I did some local fix for the broken manifests from dspv2 repo, so the change in this PR is working, but we need to sort out problem in dpsv2 first
Merge criteria: