Conversation
eero-t
left a comment
There was a problem hiding this comment.
Otherwise looks fine, but I wonder whether this should go under helm-charts/ instead?
I don't think so. This is not a Helm chart but a solution which uses external Helm chart. Those are two different things, and would not be good to put them into same place. |
I concur with @poussa |
|
KubeAI requires resources.limits so putting it back. |
f89b68b to
dcacffa
Compare
eero-t
left a comment
There was a problem hiding this comment.
Approved.
PS. before force-pushing updates, please pull the pre-ci commit first. This avoids re-introducing trailing white-space + table alignment issues, that pre-ci needs to fix again.
|
@chensuyue What's this |
I need to add entry to the docs index.rst file first to make infra ci/cd happy. I'll do that next. |
Signed-off-by: Sakari Poussa <sakari.poussa@intel.com>
mkbhanda
left a comment
There was a problem hiding this comment.
Minor typo, can approve
for more information, see https://pre-commit.ci
Description
KubeAI is an open-source AI inferencing operator. This PR contains documentation, installation instructions and deployment files for running KubeAI with OPEA inference services.
For now, OPEA enables a subset of the KubeAI features. In the future more KubeAI service will be added.
Features
The following features are enabled at the moment.
Issues
OIM Operator RFC
Type of change
List the type of change like below. Please delete options that are not relevant.
Dependencies
n/a
Tests
Manually tested on CPU and Gaudi clusters.