Merge pull request microsoft#167 from microsoft/pragme/add-todo-probe

Add probes to stats-api
Ashengross · Apr 29, 2021 · 6419387 · 6419387
2 parents c1d8de0 + 7d21390
commit 6419387
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 0 deletions.
diff --git a/samples/todo-app/deployment.yaml b/samples/todo-app/deployment.yaml
@@ -102,6 +102,18 @@ spec:
       containers:
       - name: stats-api
         image: azdspublic/todo-app-stats-api
+        livenessProbe:
+          httpGet:
+            path: /hello
+            port: 80
+          initialDelaySeconds: 5
+          periodSeconds: 3
+        readinessProbe:
+          httpGet:
+            path: /hello
+            port: 80
+          initialDelaySeconds: 15
+          periodSeconds: 5
 ---
 apiVersion: v1
 kind: Service

diff --git a/samples/todo-app/stats-api/server.js b/samples/todo-app/stats-api/server.js
@@ -13,6 +13,11 @@ var cache = redis.createClient({
     password: process.env.REDIS_PASSWORD || undefined
 });
 
+app.get('/hello', function (req, res) {
+    console.log('hello!');
+    res.send('hello!');
+})
+
 app.get('/stats', function (req, res) {
     console.log('request for stats received with kubernetes-route-as header: %s', req.get('kubernetes-route-as'));
     cache.get('todosCreated', function (err, created) {