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 ODH overlay #35

Merged

Conversation

israel-hdez
Copy link

What this PR does / why we need it:

Augments the default profile with some changes expected by an ODH installation:

Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):

Feature/Issue validation/testing:

Tested by a simple run of kustomize build ./config/overlays/odh and checking that the manifest builds correctly. No additional testing was done other than simple build.

Checklist:

  • [N/A] Have you added unit/e2e tests that prove your fix is effective or that this feature works?
  • Has code been commented, particularly in hard-to-understand areas?
  • [N/A] Have you made corresponding changes to the documentation?

Release note:

NONE

Augments the `default` profile with some changes expected by an ODH installation:
* Removes the `Namespace` CR, because the ODH operator does not expect such resource. The Namespace is expected to be created in advance to later create a KfDef on it, where resources are going to be installed.
* Adds cluster roles, to extend the cluster's default user-facing roles with KServe privileges.

Signed-off-by: Edgar Hernández <23639005+israel-hdez@users.noreply.github.com>
Copy link

@Jooho Jooho left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@israel-hdez israel-hdez merged commit 23191b9 into opendatahub-io:release-v0.10 Jul 25, 2023
16 checks passed
@israel-hdez israel-hdez deleted the 884-885-odh-overlay branch August 2, 2023 17:20
Jooho added a commit to Jooho/kserve that referenced this pull request Jan 11, 2024
Add missing NS flags and minor changes to the kserve demo
Jooho pushed a commit to Jooho/kserve that referenced this pull request Jan 11, 2024
[Cherry-Pick] increase kserve controller limit resources(cpu/memory)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants