Implement editing model servers

Signed-off-by: Mark Winter <mark.winter@navercorp.com>

Author: markwinter

.github/workflows/docker-publish-releasing.yml

@@ -0,0 +1,58 @@
+name: Models web app Docker Publisher (Releasing)
+
+on:
+  push:
+    # Publish `master` as Docker `latest` image.
+    branches:
+      - master
+
+    paths:
+      - releasing/VERSION
+
+  # Run tests for any PRs.
+  pull_request:
+
+env:
+  IMAGE_NAME: models-web-app
+
+jobs:
+  # Run tests.
+  # See also https://docs.docker.com/docker-hub/builds/automated-testing/
+  test:
+    runs-on: ubuntu-latest
+
+    steps:
+      - uses: actions/checkout@v2
+
+      - name: Run tests
+        run: |
+          docker build . --file Dockerfile
+  # Push image to GitHub Packages.
+  # See also https://docs.docker.com/docker-hub/builds/
+  push:
+    # Ensure test job passes before pushing image.
+    needs: test
+
+    runs-on: ubuntu-latest
+    if: github.event_name == 'push'
+
+    steps:
+      - uses: actions/checkout@v2
+
+      - name: Build image
+        run: |
+          docker build . --file Dockerfile --tag ${IMAGE_NAME}
+      - name: Log into registry
+        run: docker login -u ${{ secrets.DOCKER_USER }} -p ${{ secrets.DOCKER_PASSWORD }}
+
+      - name: Push image
+        run: |
+          IMAGE_ID=kserve/$IMAGE_NAME
+          # Change all uppercase to lowercase
+          IMAGE_ID=$(echo $IMAGE_ID | tr '[A-Z]' '[a-z]')
+          VERSION=$(cat ./releasing/VERSION)
+
+          echo IMAGE_ID=$IMAGE_ID
+          echo VERSION=$VERSION
+          docker tag $IMAGE_NAME $IMAGE_ID:$VERSION
+          docker push $IMAGE_ID:$VERSION

.github/workflows/flake-lint.yml

@@ -0,0 +1,22 @@
+name: Python checks
+
+on:
+  push:
+    branches: [ master ]
+  pull_request:
+    branches: [ master ]
+jobs:
+  flake8-lint:
+    runs-on: ubuntu-latest
+    name: Lint
+    steps:
+      - name: Check out source repository
+        uses: actions/checkout@v2
+      - name: Set up Python environment
+        uses: actions/setup-python@v1
+        with:
+          python-version: "3.7"
+      - name: flake8 Lint
+        uses: py-actions/flake8@v1
+        with:
+          path: backend

.gitignore

@@ -1,3 +1,24 @@
 **/__pycache__/
 **/.vscode/
 **/static/*
+**/fonts/
+
+# Swap
+[._]*.s[a-v][a-z]
+!*.svg  # comment out if you don't need vector files
+[._]*.sw[a-p]
+[._]s[a-rt-v][a-z]
+[._]ss[a-gi-z]
+[._]sw[a-p]
+
+# Session
+Session.vim
+Sessionx.vim
+
+# Temporary
+.netrwhist
+*~
+# Auto-generated tag files
+tags
+# Persistent undo
+[._]*.un~

Dockerfile

@@ -6,7 +6,7 @@ RUN apt-get update && apt-get install git -y
 WORKDIR /kf
 RUN git clone https://github.com/kubeflow/kubeflow.git && \
     cd kubeflow && \
-    git checkout e6fdf51
+    git checkout d1da825
 
 # --- Build the backend kubeflow-wheel ---
 FROM python:3.7-slim-buster AS backend-kubeflow-wheel
@@ -18,7 +18,7 @@ COPY --from=fetch-kubeflow-kubeflow $BACKEND_LIB .
 RUN python setup.py sdist bdist_wheel
 
 # --- Build the frontend kubeflow library ---
-FROM node:10 AS frontend-kubeflow-lib
+FROM node:12-buster-slim AS frontend-kubeflow-lib
 
 WORKDIR /src
 
@@ -30,7 +30,7 @@ COPY --from=fetch-kubeflow-kubeflow $LIB/ ./
 RUN npm run build
 
 # --- Build the frontend ---
-FROM node:12 AS frontend
+FROM node:12-buster-slim AS frontend
 
 WORKDIR /src
 COPY ./frontend/package*.json ./

Makefile

@@ -1,4 +1,4 @@
-IMG ?= gcr.io/kfserving/models-web-app
+IMG ?= kserve/models-web-app
 
 # We want the git tag to be the last commit to this directory so we don't
 # bump the image on unrelated changes.

OWNERS

@@ -1,5 +1,6 @@
 approvers:
   - kimwnasptd
+  - yuzisun
 reviewers:
   - elikatsis
   - StefanoFioravanzo

README.md

@@ -6,11 +6,11 @@ The web app currently works with `v1beta1` versions of `InferenceService` object
 
 ## Connect to the app
 
-The web app is installed alongside the other KFServing components, either in the `kfserving-system` or in the `kubeflow` namespace. There is a `VirtualService` that exposes the app via an Istio Ingress Gateway. Depending on the installation environment the following Ingress Gateway will be used.
+The web app is installed alongside the other KServe components, either in the `kserve` or in the `kubeflow` namespace. There is a `VirtualService` that exposes the app via an Istio Ingress Gateway. Depending on the installation environment the following Ingress Gateway will be used.
 
 | Installation mode | IngressGateway |
 | - | - |
-| Standalone KFServing | knative-ingress-gateway.knative-serving |
+| Standalone KServe | knative-ingress-gateway.knative-serving |
 | Kubeflow | kubeflow-gateway.kubeflow |
 
 To access the app you will need to navigate with your browser to
@@ -27,11 +27,11 @@ You can apply the mentioned configurations by doing the following commands:
 ```bash
 # edit the configmap
 # CONFIG=config/overlays/kubeflow/kustomization.yaml
-CONFIG=config/web-app/kustomization.yaml
+CONFIG=config/base/kustomization.yaml
 vim ${CONFIG}
 
 # Add the following env vars to the configMapGenerator's literals
-# for kfserving-models-web-app-config
+# for kserve-models-web-app-config
 - APP_PREFIX=/
 - APP_DISABLE_AUTH="True"
 - APP_SECURE_COOKIES="False"
@@ -83,7 +83,7 @@ cd dist/kubeflow
 npm link
 
 # run the app frontend
-cd $KFSERVING_REPO/web-app/frontend
+cd $KSERVE_MODELS_WEB_APP_REPO/frontend
 npm i
 npm link kubeflow
 npm run build:watch
@@ -95,7 +95,7 @@ npm run build:watch
 ```bash
 # create a virtual env and install deps
 # https://packaging.python.org/guides/installing-using-pip-and-virtual-environments/
-cd $KFSERVING_REPO/web-app/backend
+cd $KSERVE_MODELS_WEB_APP_REPO/backend
 python3.7 -m pip install --user virtualenv
 python3.7 -m venv web-apps-dev
 source web-apps-dev/bin/activate

backend/.flake8

@@ -0,0 +1,4 @@
+[flake8]
+docstring_convention = google
+exclude = assets,__init__.py,__pycache__
+ignore = D100,D104,D107,W503

backend/apps/common/__init__.py

@@ -1,3 +1,4 @@
+"""Package with the base code between backend versions."""
 import kubeflow.kubeflow.crud_backend as base
 from kubeflow.kubeflow.crud_backend import config, logging
 
@@ -8,6 +9,7 @@
 
 def create_app(name=__name__, static_folder="static",
                cfg: config.Config = None):
+    """Create the WSGI app."""
     cfg = config.Config() if cfg is None else cfg
 
     app = base.create_app(name, static_folder, cfg)

backend/apps/common/routes/__init__.py

@@ -1,3 +1,4 @@
+"""Common routes across backends."""
 from flask import Blueprint
 
 bp = Blueprint("base_routes", __name__)