-
Notifications
You must be signed in to change notification settings - Fork 981
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 install script for KServe and ModelMesh #2032
Add install script for KServe and ModelMesh #2032
Conversation
/restest |
/retest |
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.
Tried it out, and works great. Some minor comments in line.
hack/install_kserve_mm.sh
Outdated
if [[ ! -z $user_ns_array ]]; then | ||
for USER_NS in "${user_ns_array[@]}"; do | ||
kubectl create ns ${USER_NS} || true | ||
echo "Installing ModelMesh in namespace: ${USER_NS}..." |
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.
nit: Maybe "Enabling ModelMesh for namespace:" would be more accurate?
hack/install_kserve_mm.sh
Outdated
|
||
kubectl create ns ${CTLR_NS} || true | ||
cd "${C_DIR}/modelmesh-serving" | ||
./scripts/install.sh -n ${CTLR_NS} --quickstart --dev-mode-logging |
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.
Wondering if dev-mode-logging shouldn't be on by default since there is considerable DEBUG log spam with this enabled.
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.
Yeah, most people won't care about logging. Just removed it.
Add install script for KServe and ModelMesh so the user can quickly experiment features provided by two repos. The script will install all prerequisits knative, istio, cert manager, etcd, and minio, and create namespaces when needed. The repo branch variables can be changed to install different versions of KServe and ModelMesh. Signed-off-by: Chin Huang <chhuang@us.ibm.com>
ffb2e2c
to
86a3477
Compare
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: chinhuang007, yuzisun 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 |
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.
/lgtm
Add install script for KServe and ModelMesh so the user can quickly experiment features provided by two repos. The script will install all prerequisits knative, istio, cert manager, etcd, and minio, and create namespaces when needed. The repo branch variables can be changed to install different versions of KServe and ModelMesh. Signed-off-by: Chin Huang <chhuang@us.ibm.com> Signed-off-by: alexagriffith <agriffith96@gmail.com>
Add install script for KServe and ModelMesh so the user can
quickly experiment features provided by two repos.
The script will install all prerequisits knative, istio, cert
manager, etcd, and minio, and create namespaces when needed.
The repo branch variables can be changed to install different
versions of KServe and ModelMesh.
Signed-off-by: Chin Huang chhuang@us.ibm.com
What this PR does / why we need it:
Which issue(s) this PR fixes (optional, in
fixes #<issue number>(, fixes #<issue_number>, ...)
format, will close the issue(s) when PR gets merged):Fixes #
Special notes for your reviewer:
Release note: