-
Notifications
You must be signed in to change notification settings - Fork 163
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
Restructure the manifest files #2906
Restructure the manifest files #2906
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #2906 +/- ##
==========================================
- Coverage 79.34% 78.53% -0.82%
==========================================
Files 1139 1139
Lines 24284 24171 -113
Branches 6186 6099 -87
==========================================
- Hits 19268 18982 -286
- Misses 5016 5189 +173 see 141 files with indirect coverage changes Continue to review full report in Codecov by Sentry.
|
ce3ef2a
to
7c392ba
Compare
7c392ba
to
2002af4
Compare
/hold Waiting on confirmation with the Operator team and managing a good time to release this where it doesn't toss a release into a mess. Might be for 2.12 RHOAI and 2.15 ODH -- TBD. |
16055a2
to
1882ce9
Compare
1882ce9
to
0b75ca4
Compare
This comment was marked as resolved.
This comment was marked as resolved.
447f179
to
072b2e2
Compare
843d4a8
to
91e66dd
Compare
94ae219
to
e71c8c0
Compare
Tested with Wen on her custom catalog source images -- going to be merging this on Tuesday. Monday I'm OOO and today is Code Freeze for 2.11 -- just to avoid the noise, we will wait until post that. |
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.
I've left a nit and a question, so far the rest looks great.
I've managed to get a deployment in ODH and I've managed to get a cluster to install the rhoai image, it's giving me some issues so I'll take a look tomorrow with @zdtsw
/lgtm Everything seems great in rhoai too |
[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 |
https://issues.redhat.com/browse/RHOAIENG-5506
Description
Restructure the manifest files for the 3 types of deployments.
The goal is to make pure entry points and self link all needed parts, that way the operator doesn't need to know the internals of our manifests folder layout.
The three entry points are:
/manifests/odh/kustomization.yaml
- ODH Operator/manifests/rhoai/onprem/kustomization.yaml
- RHOAI Operator for Self Managed deployments (onprem is an old name)/manifests/rhoai/addon/kustomization.yaml
- RHOAI OPerator for Managed deploymentsHow Has This Been Tested?
Added testing steps in the root README.md -- these steps will only apply going forward and don't help with the change itself as the operator owned a lot of the functionality.
Need to get the answers to the following questions:
params.env
need to live for RHOAI to make use of itkustomize build
commands -- see the root README for moreTopic with Operator Team (slack)
OUT OF SCOPE of this PR is handling the out of date kustomize format. I'll follow up with proper updates to that when I'm not moving 100 some odd files.
Test Impact
None, manifest files.
Request review criteria:
Self checklist (all need to be checked):
If you have UI changes:
After the PR is posted & before it merges:
main
I'll need help working through this -- cc @zdtsw @VaishnaviHire @lucferbux