From 5ded29f74e125b1f56b356ec185003b954a69ae2 Mon Sep 17 00:00:00 2001
From: Jim Bugwadia <jim@nirmata.com>
Date: Tue, 5 Nov 2019 12:28:44 -0800
Subject: [PATCH] temp update for debugging

---
 pkg/testrunner/scenario.go                    |  4 ++++
 .../add_safe-to-evict_annotation.yaml         | 22 ++++++++++++++++---
 .../scenario_mutate_safe-to-evict.yaml        |  8 +++++--
 3 files changed, 29 insertions(+), 5 deletions(-)

diff --git a/pkg/testrunner/scenario.go b/pkg/testrunner/scenario.go
index 5faaa79051..625fbbcce6 100644
--- a/pkg/testrunner/scenario.go
+++ b/pkg/testrunner/scenario.go
@@ -3,6 +3,7 @@ package testrunner
 import (
 	"bytes"
 	"encoding/json"
+	"flag"
 	"io/ioutil"
 	"os"
 	ospath "path"
@@ -442,6 +443,9 @@ func loadPolicy(t *testing.T, path string) *kyverno.ClusterPolicy {
 }
 
 func testScenario(t *testing.T, path string) {
+	flag.Set("logtostderr", "true")
+	flag.Set("v", "8")
+
 	scenario, err := loadScenario(t, path)
 	if err != nil {
 		t.Error(err)
diff --git a/samples/best_practices/add_safe-to-evict_annotation.yaml b/samples/best_practices/add_safe-to-evict_annotation.yaml
index ac468ecbae..2e45dccd49 100644
--- a/samples/best_practices/add_safe-to-evict_annotation.yaml
+++ b/samples/best_practices/add_safe-to-evict_annotation.yaml
@@ -4,10 +4,12 @@ metadata:
   name: "annotate-emptyDir"
   annotations:
     policies.kyverno.io/category: AutoScaling
-    policies.kyverno.io/description: 
+    policies.kyverno.io/description: The Kubernetes cluster autoscaler does not evict pods that 
+      use hostPath or emptyDir volumes. To allow eviction the annotation 
+      cluster-autoscaler.kubernetes.io/safe-to-evict=true is added to the pod. 
 spec: 
   rules: 
-  - name: "add-safe-to-evict-annotation"
+  - name: "empty-dir-add-safe-to-evict"
     match: 
       resources: 
         kinds: 
@@ -19,4 +21,18 @@ spec:
             +(cluster-autoscaler.kubernetes.io/safe-to-evict): true
         spec:          
           volumes: 
-          - (emptyDir): {}
\ No newline at end of file
+          - (emptyDir): {}
+  - name: "host-path-add-safe-to-evict"
+    match: 
+      resources: 
+        kinds: 
+        - "Pod"
+    mutate: 
+      overlay:
+        metadata:
+          annotations:
+            +(cluster-autoscaler.kubernetes.io/safe-to-evict): true
+        spec:          
+          volumes: 
+          - (hostPath):
+            path: "*"
diff --git a/test/scenarios/samples/best_practices/scenario_mutate_safe-to-evict.yaml b/test/scenarios/samples/best_practices/scenario_mutate_safe-to-evict.yaml
index fade477510..e6b4ea9f93 100644
--- a/test/scenarios/samples/best_practices/scenario_mutate_safe-to-evict.yaml
+++ b/test/scenarios/samples/best_practices/scenario_mutate_safe-to-evict.yaml
@@ -13,7 +13,11 @@ expected:
         namespace: ''
         name: pod-with-emptydir
       rules:
-        - name: add-safe-to-evict-annotation
+        - name: empty-dir-add-safe-to-evict
           type: Mutation
           success: true
-          message: "successfully processed overlay"
\ No newline at end of file
+          message: "successfully processed overlay"
+        - name: host-path-add-safe-to-evict
+          type: Mutation
+          success: false
+          message: "successfully processed overlay"          
\ No newline at end of file