Skip to content
This repository was archived by the owner on Nov 16, 2020. It is now read-only.

Update Knative version Readme #708

Open
wants to merge 4 commits into
base: knative
Choose a base branch
from
Open

Update Knative version Readme #708

Changes from 3 commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
ab92f84
Update Knative version Readme
pzmrzy Oct 31, 2018
f10e4c6
Fix jq format, Add note to dockerhub username
pzmrzy Oct 31, 2018
7cb41d2
Add optional step for tiller
pzmrzy Oct 31, 2018
e63e448
Add cluster zone env var
pzmrzy Nov 1, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
41 changes: 25 additions & 16 deletions README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -41,10 +41,10 @@ The diagram below illustrates the different components which make up the Dispatc

2. Create GKE cluster:
```bash
K8S_VERSION=1.10.7-gke.6
export K8S_VERSION=1.10.7-gke.6
export CLUSTER_NAME=dispatch-knative
gcloud container clusters create -m n1-standard-4 --cluster-version ${K8S_VERSION} ${CLUSTER_NAME}
gcloud container clusters get-credentials ${CLUSTER_NAME}
gcloud container clusters create -m n1-standard-4 --cluster-version ${K8S_VERSION} ${CLUSTER_NAME} --zone us-west1-c
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe make the zone an env var too?

gcloud container clusters get-credentials ${CLUSTER_NAME} --zone us-west1-c
```

3. Install Knative:
Expand All @@ -69,13 +69,15 @@ Installing Dispatch depends on having a Kubernetes cluster with the Knative comp
export RELEASE_NAME="dispatch-server"
export MINIO_USERNAME="dispatch"
export MINIO_PASSWORD="dispatch"
export INGRESS_IP=$(kubectl get service -n istio-system knative-ingressgateway -o json | jq -r .status.loadBalancer.ingress[].ip)
export DOCKER_REPOSITORY="{dockerhub_username}"
export INGRESS_IP=$(kubectl get service -n istio-system knative-ingressgateway -o json | jq -r ".status.loadBalancer.ingress[0].ip")
```

> Note: You may need to modify `DISPATCH_SERVER_DOCKER_REPOSITORY` in `scripts/images.sh` to your dockerhub username
2. Build and publish a dispatch image:
```bash
PUSH_IMAGES=1 make images
```
> **NOTE**: You may need to `docker login` before push

3. The previous command will output a configuration file `values.yaml`:
```yaml
Expand All @@ -88,35 +90,38 @@ Installing Dispatch depends on having a Kubernetes cluster with the Knative comp
password: ********
```

4. Deploy via helm chart (if helm is not installed and initialized, do that first):
4. Helm Init and grant cluster admin privileges to tiller
```bash
helm init
Copy link
Contributor

Choose a reason for hiding this comment

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

I would probably add a new optional step above for helm init and also mentions that tiller requires cluster admin privileges (refer the legacy installation steps)

Copy link
Contributor

Choose a reason for hiding this comment

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

Nit: helm dep up instead of the verbose command

kubectl create clusterrolebinding tiller-cluster-admin --clusterrole=cluster-admin --serviceaccount=kube-system:default
```

5. Deploy via helm chart (if helm is not installed and initialized, do that first):
```bash
helm dep up ./charts/dispatch/
helm upgrade -i --debug ${RELEASE_NAME} ./charts/dispatch --namespace ${DISPATCH_NAMESPACE} -f values.yaml
```
> **NOTE**: Use following to create cluster role binding for tiller:
>```bash
>kubectl create clusterrolebinding tiller-cluster-admin --clusterrole=cluster-admin --serviceaccount=kube-system:default
>```

5. Reconfigure Knative serving (need to whitelist our internal repository):
6. Reconfigure Knative serving (need to whitelist our internal repository):
```bash
./scripts/configure-knative.sh
```

6. Build the CLI (substitute darwin for linux if needed):
7. Build the CLI (substitute darwin for linux if needed):
```bash
make cli-darwin
# Create symlink to binary
ln -s `pwd`/bin/dispatch-darwin /usr/local/bin/dispatch
```

7. Create the Dispatch config:
8. Create the Dispatch config:
```bash
cat << EOF > config.json
{
"current": "${RELEASE_NAME}",
"contexts": {
"${RELEASE_NAME}": {
"host": "$(kubectl -n ${DISPATCH_NAMESPACE} get service ${RELEASE_NAME}-nginx-ingress-controller -o json | jq -r .status.loadBalancer.ingress[].ip)",
"host": "$(kubectl -n ${DISPATCH_NAMESPACE} get service ${RELEASE_NAME}-nginx-ingress-controller -o json | jq -r ".status.loadBalancer.ingress[].ip")",
Copy link
Contributor

Choose a reason for hiding this comment

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

Should these be single quotes?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Both works, then single quotes seems better

"port": 443,
"scheme": "https",
"insecure": true
Expand All @@ -128,10 +133,10 @@ Installing Dispatch depends on having a Kubernetes cluster with the Knative comp
export DISPATCH_CONFIG=`pwd`/config.json
```

8. Test out your install:
9. Test out your install:
First, create an baseimage:
```bash
dispatch create baseimage python3-base dispatchframework/python3-base:0.0.13-knative
dispatch create base-image python3-base dispatchframework/python3-base:0.0.13-knative
Created baseimage: python3-base
```
Then, create an image:
Expand Down Expand Up @@ -184,4 +189,8 @@ Installing Dispatch depends on having a Kubernetes cluster with the Knative comp
curl -v http://${INGRESS_IP}/hello?name=Jon -H 'Host: default.dispatch-server.dispatch.local'
```

10. Delete dispatch server
```bash
helm delete dispatch-server
```
For a more complete quickstart see the [developer documentation](#documentation)