Add readiness/liveness probes to k8s CaaS resources (#2187)

Author: RobertLucian

build/images.sh

@@ -27,7 +27,6 @@ api_images=(
 )
 
 dev_images=(
-  "downloader"
   "manager"
   "proxy"
   "async-gateway"

cmd/proxy/main.go

@@ -19,7 +19,6 @@ package main
 import (
 	"context"
 	"flag"
-	"io/ioutil"
 	"net/http"
 	"os"
 	"os/signal"
@@ -49,7 +48,6 @@ func main() {
 		userContainerPort int
 		maxConcurrency    int
 		maxQueueLength    int
-		probeDefPath      string
 		clusterConfigPath string
 	)
 
@@ -59,7 +57,6 @@ func main() {
 	flag.IntVar(&maxConcurrency, "max-concurrency", 0, "max concurrency allowed for user container")
 	flag.IntVar(&maxQueueLength, "max-queue-length", 0, "max request queue length for user container")
 	flag.StringVar(&clusterConfigPath, "cluster-config", "", "cluster config path")
-	flag.StringVar(&probeDefPath, "probe", "", "path to the desired probe json definition")
 	flag.Parse()
 
 	log := logging.GetLogger()
@@ -119,23 +116,7 @@ func main() {
 	)
 
 	promStats := proxy.NewPrometheusStatsReporter()
-
-	var readinessProbe *probe.Probe
-	if probeDefPath != "" {
-		jsonProbe, err := ioutil.ReadFile(probeDefPath)
-		if err != nil {
-			log.Fatal(err)
-		}
-
-		probeDef, err := probe.DecodeJSON(string(jsonProbe))
-		if err != nil {
-			log.Fatal(err)
-		}
-
-		readinessProbe = probe.NewProbe(probeDef, log)
-	} else {
-		readinessProbe = probe.NewDefaultProbe(target, log)
-	}
+	readinessProbe := probe.NewDefaultProbe(target, log)
 
 	go func() {
 		reportTicker := time.NewTicker(_reportInterval)
@@ -165,7 +146,7 @@ func main() {
 
 	servers := map[string]*http.Server{
 		"proxy": {
-			Addr:    ":" + strconv.Itoa(userContainerPort),
+			Addr:    ":" + strconv.Itoa(port),
 			Handler: proxy.Handler(breaker, httpProxy),
 		},
 		"admin": {

docs/clusters/management/create.md

@@ -99,7 +99,6 @@ The docker images used by the cluster can also be overridden. They can be config
 image_operator: quay.io/cortexlabs/operator:master
 image_controller_manager: quay.io/cortexlabs/controller-manager:master
 image_manager: quay.io/cortexlabs/manager:master
-image_downloader: quay.io/cortexlabs/downloader:master
 image_proxy: quay.io/cortexlabs/proxy:master
 image_async_gateway: quay.io/cortexlabs/async-gateway:master
 image_cluster_autoscaler: quay.io/cortexlabs/cluster-autoscaler:master

images/downloader/Dockerfile

@@ -34,8 +34,6 @@ function main() {
 function cluster_up() {
   create_eks
 
-  start_pre_download_images
-
   echo -n "￮ updating cluster configuration "
   setup_configmap
   echo "✓"
@@ -76,8 +74,6 @@ function cluster_up() {
 
   validate_cortex
 
-  await_pre_download_images
-
   echo -e "\ncortex is ready!"
   if [ "$CORTEX_OPERATOR_LOAD_BALANCER_SCHEME" == "internal" ]; then
     echo -e "\nnote: you will need to configure VPC Peering to connect to your cluster: https://docs.cortex.dev/v/${CORTEX_VERSION_MINOR}/"
@@ -324,65 +320,6 @@ function setup_istio() {
   output_if_error istio-${ISTIO_VERSION}/bin/istioctl install -f /workspace/istio.yaml
 }
 
-function start_pre_download_images() {
-  registry="quay.io/cortexlabs"
-  if [ -n "$CORTEX_DEV_DEFAULT_IMAGE_REGISTRY" ]; then
-    registry="$CORTEX_DEV_DEFAULT_IMAGE_REGISTRY"
-  fi
-  export CORTEX_IMAGE_PYTHON_HANDLER_CPU="${registry}/python-handler-cpu:${CORTEX_VERSION}"
-  export CORTEX_IMAGE_PYTHON_HANDLER_GPU="${registry}/python-handler-gpu:${CORTEX_VERSION}-cuda10.2-cudnn8"
-  export CORTEX_IMAGE_PYTHON_HANDLER_INF="${registry}/python-handler-inf:${CORTEX_VERSION}"
-  export CORTEX_IMAGE_TENSORFLOW_SERVING_CPU="${registry}/tensorflow-serving-cpu:${CORTEX_VERSION}"
-  export CORTEX_IMAGE_TENSORFLOW_SERVING_GPU="${registry}/tensorflow-serving-gpu:${CORTEX_VERSION}"
-  export CORTEX_IMAGE_TENSORFLOW_SERVING_INF="${registry}/tensorflow-serving-inf:${CORTEX_VERSION}"
-  export CORTEX_IMAGE_TENSORFLOW_HANDLER="${registry}/tensorflow-handler:${CORTEX_VERSION}"
-
-  envsubst < manifests/image-downloader-cpu.yaml | kubectl apply -f - &>/dev/null
-
-  has_gpu="false"
-  has_inf="false"
-
-  cluster_config_len=$(cat /in/cluster_${CORTEX_CLUSTER_NAME}_${CORTEX_REGION}.yaml | yq -r .node_groups | yq -r length)
-  for idx in $(seq 0 $(($cluster_config_len-1))); do
-    ng_instance_type=$(cat /in/cluster_${CORTEX_CLUSTER_NAME}_${CORTEX_REGION}.yaml | yq -r .node_groups[$idx].instance_type)
-    if [[ "$ng_instance_type" == p* || "$ng_instance_type" == g* ]]; then
-      has_gpu="true"
-    fi
-    if [[ "$ng_instance_type" == inf* ]]; then
-      has_inf="true"
-    fi
-  done
-
-  if [ "$has_gpu" == "true" ]; then
-    envsubst < manifests/image-downloader-gpu.yaml | kubectl apply -f - &>/dev/null
-  fi
-
-  if [ "$has_inf" == "true" ]; then
-    envsubst < manifests/image-downloader-inf.yaml | kubectl apply -f - &>/dev/null
-  fi
-}
-
-function await_pre_download_images() {
-  echo -n "￮ downloading docker images "
-  printed_dot="false"
-  for ds_name in image-downloader-cpu image-downloader-gpu image-downloader-inf; do
-    if ! kubectl get daemonset $ds_name > /dev/null 2>&1; then
-      continue
-    fi
-    i=0
-    until [ "$(kubectl get daemonset $ds_name -n=default -o 'jsonpath={.status.numberReady}')" == "$(kubectl get daemonset $ds_name -n=default -o 'jsonpath={.status.desiredNumberScheduled}')" ]; do
-      if [ $i -eq 120 ]; then break; fi  # give up after 6 minutes
-      echo -n "."
-      printed_dot="true"
-      ((i=i+1))
-      sleep 3
-    done
-    kubectl -n=default delete --ignore-not-found=true daemonset $ds_name &>/dev/null
-  done
-
-  if [ "$printed_dot" == "true" ]; then echo " ✓"; else echo "✓"; fi
-}
-
 function validate_cortex() {
   set +e
 

manager/install.sh

@@ -184,7 +184,7 @@ spec:
     - path: /metrics
       scheme: http
       interval: 10s
-      port: metrics
+      port: admin
       relabelings:
         - action: keep
           sourceLabels: [ __meta_kubernetes_pod_container_name ]
@@ -221,7 +221,7 @@ metadata:
 spec:
   jobLabel: "statsd-exporter"
   podMetricsEndpoints:
-    - port: metrics
+    - port: admin
       scheme: http
       path: /metrics
       interval: 20s

manager/manifests/image-downloader-cpu.yaml

@@ -33,8 +33,8 @@ var (
 	ProxyListeningPortStr   = "8888"
 	ProxyListeningPortInt32 = int32(8888)
 
-	MetricsPortStr   = "15000"
-	MetricsPortInt32 = int32(15000)
+	AdminPortStr   = "15000"
+	AdminPortInt32 = int32(15000)
 
 	AuthHeader = "X-Cortex-Authorization"