Skip to content
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

Add support for build manifests from different location in Dockerfile #43

Closed
wants to merge 3 commits into from

Conversation

zdtsw
Copy link

@zdtsw zdtsw commented Aug 24, 2023

ref: opendatahub-io#357

rebase after API change, test:
Screenshot from 2023-08-30 11-28-21

Screenshot from 2023-08-30 11-29-37

Screenshot from 2023-08-30 11-37-08

Test:
local image: quay.io/wenzhou/opendatahub-operator-catalog:v2.8.424-2
when enable all component

Screenshot from 2023-08-28 18-53-12
has no problem to start pods.

@zdtsw zdtsw changed the title Add support for build manifests from different location Add support for build manifests from different location in Dockerfile Aug 24, 2023
@VaishnaviHire
Copy link

/hold

Should not cherry-pick if its merge in upstream.

@zdtsw
Copy link
Author

zdtsw commented Aug 25, 2023

/hold

Should not cherry-pick if its merge in upstream.

this changes include 2 parts:

  1. dockerfile change, this is needed for upstream to get the dashboard, kserver etc away from odh-manifests
  2. another part of the dockerfile change is only for the monitoring part ( ODH odh-manifests does not have this monitoring change, RHODS odh-manifests has this)

we can have 1st part in the upstream and sync down to RHODS,
but for part2, we still need add it in RHODS operator, unless we want to add monitoring resource into upstream

- fetch manifests from component repo
- componnets cannot fetch from component repo, then from odh-manifests
- use red-hat-data-services for git org in script
- since odh-modelmesh-controller is moved out from model-mesh need copy
- kserve does not have 'base' folder in component repo, change path
  ref: https://github.com/opendatahub-io/odh-manifests/blob/master/kserve/hack/kustomization.yaml

Signed-off-by: Wen Zhou <wenzhou@redhat.com>
@zdtsw
Copy link
Author

zdtsw commented Sep 14, 2023

we will still need the change in not only sync from ODH @VaishnaviHire
main reason is the organization is different, in ODH is it opendatahu-io all the repo comes from, here is using red-hat-data-services

after that, changes should be fine to sync.

but have this PR hold, still upstream is merged and incubation for a bit time

@zdtsw
Copy link
Author

zdtsw commented Sep 27, 2023

close this one, will use "monitroing-intergration" to get all out

@zdtsw zdtsw closed this Sep 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants