From be33d2c9d1466755ae8b29edf7082f82cc088c61 Mon Sep 17 00:00:00 2001
From: Samuel Walker <sfxworks@gmail.com>
Date: Mon, 30 Mar 2020 19:45:47 -0400
Subject: [PATCH] Update the image version in the example file (#8)

* Update the image version in the example file

https://github.com/itzg/mc-router/issues/7

Deployed it with an older version. Works but ends up spiking in CPU usage. Easy to overlook.

Also, for good measure/best practice, added resource request/limit.

* Bad syntax on CPU limit value
---
 docs/k8s-example-auto.yaml | 9 ++++++++-
 1 file changed, 8 insertions(+), 1 deletion(-)

diff --git a/docs/k8s-example-auto.yaml b/docs/k8s-example-auto.yaml
index 00dacab..9e697e8 100644
--- a/docs/k8s-example-auto.yaml
+++ b/docs/k8s-example-auto.yaml
@@ -63,7 +63,7 @@ spec:
     spec:
       serviceAccountName: mc-router
       containers:
-      - image: itzg/mc-router:1.1.1
+      - image: itzg/mc-router:latest
         name: mc-router
         args: ["--api-binding", ":8080", "--in-kube-cluster"]
         ports:
@@ -71,6 +71,13 @@ spec:
           containerPort: 25565
         - name: web
           containerPort: 8080
+        resources:
+          requests:
+            memory: 50Mi
+            cpu: "100m"
+          limits:
+            memory: 100Mi
+            cpu: "250m"
 ---
 apiVersion: v1
 kind: Service