Skip to content

Commit

Permalink
chore: replace ROCKs with rocks (#63)
Browse files Browse the repository at this point in the history
* chore: replace ROCKs with rocks

This commit replaces ROCKs with rocks to be in sync with Canonical's standard terminology.

Part of canonical/bundle-kubeflow#916
  • Loading branch information
DnPlas authored Jun 12, 2024
1 parent da46e26 commit 195b695
Show file tree
Hide file tree
Showing 10 changed files with 12 additions and 12 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/on_pull_request.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ on:
jobs:

on-pull-request:
name: Get ROCKs modified and build-scan-test them
name: Get rocks modified and build-scan-test them
uses: canonical/charmed-kubeflow-workflows/.github/workflows/get-rocks-modified-and-build-scan-test-publish.yaml@main
permissions:
pull-requests: read
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/on_push.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ on:
jobs:

on-push:
name: Get ROCKs modified and build-scan-test-publish them
name: Get rocks modified and build-scan-test-publish them
uses: canonical/charmed-kubeflow-workflows/.github/workflows/get-rocks-modified-and-build-scan-test-publish.yaml@main
permissions:
pull-requests: read
Expand Down
6 changes: 3 additions & 3 deletions CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -10,9 +10,9 @@ The KServe server images are a collection of different inference server runtimes

The Dockerfile for each of these images takes advantage of how each server is defined as a poetry package, using `poetry install` in the Dockerfile directly.

## Implementation details of the ROCKs in this repo
## Implementation details of the rocks in this repo

The ROCKs for the KServe servers require some atypical workarounds, mostly due to the upstream project using poetry to install its dependencies. These are documented here in detail, and briefly noted in the rockcraft.yaml files in this repository.
The rocks for the KServe servers require some atypical workarounds, mostly due to the upstream project using poetry to install its dependencies. These are documented here in detail, and briefly noted in the rockcraft.yaml files in this repository.

### Installing Python/pip via overlay-packages

Expand Down Expand Up @@ -70,4 +70,4 @@ The upstream install procedure results in `python` being executable, but our roc

## Integration testing the server images

For every inference server provided, upstream maintains an example usage in their [Model Serving Runtimes docs](https://kserve.github.io/website/master/modelserving/v1beta1/serving_runtime/). Each example includes a model for the given server, sample input, and an example `curl` call. These can be used for integration testing of the models without deploying KServe. See also the `README.md` files in the subdirs of this repo for some of these examples applied to our ROCKs.
For every inference server provided, upstream maintains an example usage in their [Model Serving Runtimes docs](https://kserve.github.io/website/master/modelserving/v1beta1/serving_runtime/). Each example includes a model for the given server, sample input, and an example `curl` call. These can be used for integration testing of the models without deploying KServe. See also the `README.md` files in the subdirs of this repo for some of these examples applied to our rocks.
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
@@ -1,2 +1,2 @@
# kserve-rocks
ROCKs for Kserve
rocks for Kserve
2 changes: 1 addition & 1 deletion lgbserver/tox.ini
Original file line number Diff line number Diff line change
Expand Up @@ -51,4 +51,4 @@ allowlist_externals =
echo
commands =
# TODO: Implement integration tests here
echo "WARNING: This is a placeholder test - no test is implemented here."
echo "WARNING: This is a placeholder test - no test is implemented here."
2 changes: 1 addition & 1 deletion paddleserver/tox.ini
Original file line number Diff line number Diff line change
Expand Up @@ -51,4 +51,4 @@ allowlist_externals =
echo
commands =
# TODO: Implement integration tests here
echo "WARNING: This is a placeholder test - no test is implemented here."
echo "WARNING: This is a placeholder test - no test is implemented here."
2 changes: 1 addition & 1 deletion pmmlserver/tox.ini
Original file line number Diff line number Diff line change
Expand Up @@ -51,4 +51,4 @@ allowlist_externals =
echo
commands =
# TODO: Implement integration tests here
echo "WARNING: This is a placeholder test - no test is implemented here."
echo "WARNING: This is a placeholder test - no test is implemented here."
2 changes: 1 addition & 1 deletion sklearnserver/tox.ini
Original file line number Diff line number Diff line change
Expand Up @@ -51,4 +51,4 @@ allowlist_externals =
echo
commands =
# TODO: Implement integration tests here
echo "WARNING: This is a placeholder test - no test is implemented here."
echo "WARNING: This is a placeholder test - no test is implemented here."
2 changes: 1 addition & 1 deletion storage-initializer/tox.ini
Original file line number Diff line number Diff line change
Expand Up @@ -51,4 +51,4 @@ allowlist_externals =
echo
commands =
# TODO: Implement integration tests here
echo "WARNING: This is a placeholder test - no test is implemented here."
echo "WARNING: This is a placeholder test - no test is implemented here."
2 changes: 1 addition & 1 deletion xgbserver/tox.ini
Original file line number Diff line number Diff line change
Expand Up @@ -51,4 +51,4 @@ allowlist_externals =
echo
commands =
# TODO: Implement integration tests here
echo "WARNING: This is a placeholder test - no test is implemented here."
echo "WARNING: This is a placeholder test - no test is implemented here."

0 comments on commit 195b695

Please sign in to comment.