Added version to expected labels (#93)

redhat-cop · Jul 27, 2020 · 76e721d · 76e721d
1 parent acb7840
commit 76e721d
Show file tree

Hide file tree

Showing 27 changed files with 143 additions and 1 deletion.
diff --git a/_test/all-namespaces/ocp/bestpractices/all.yml b/_test/all-namespaces/ocp/bestpractices/all.yml
@@ -20,6 +20,7 @@ objects:
     labels:
       app.kubernetes.io/name: Foo
       app.kubernetes.io/instance: Bar
+      app.kubernetes.io/version: v1
       app.kubernetes.io/component: FooBar
       app.kubernetes.io/part-of: Foo
       app.kubernetes.io/managed-by: Bar
@@ -29,6 +30,7 @@ objects:
       matchLabels:
         app.kubernetes.io/name: Foo
         app.kubernetes.io/instance: Bar
+        app.kubernetes.io/version: v1
         app.kubernetes.io/component: FooBar
         app.kubernetes.io/part-of: Foo
         app.kubernetes.io/managed-by: Bar
@@ -40,6 +42,7 @@ objects:
     labels:
       app.kubernetes.io/name: Foo
       app.kubernetes.io/instance: Bar
+      app.kubernetes.io/version: v1
       app.kubernetes.io/component: FooBar
       app.kubernetes.io/part-of: Foo
       app.kubernetes.io/managed-by: Bar
@@ -51,6 +54,7 @@ objects:
     selector:
       app.kubernetes.io/name: Foo
       app.kubernetes.io/instance: Bar
+      app.kubernetes.io/version: v1
       app.kubernetes.io/component: FooBar
       app.kubernetes.io/part-of: Foo
       app.kubernetes.io/managed-by: Bar
@@ -64,6 +68,7 @@ objects:
       matchLabels:
         app.kubernetes.io/name: Foo
         app.kubernetes.io/instance: Bar
+        app.kubernetes.io/version: v1
         app.kubernetes.io/component: FooBar
         app.kubernetes.io/part-of: Foo
         app.kubernetes.io/managed-by: Bar
@@ -73,6 +78,7 @@ objects:
     labels:
       app.kubernetes.io/name: Foo
       app.kubernetes.io/instance: Bar
+      app.kubernetes.io/version: v1
       app.kubernetes.io/component: FooBar
       app.kubernetes.io/part-of: Foo
       app.kubernetes.io/managed-by: Bar
@@ -83,6 +89,7 @@ objects:
       matchLabels:
         app.kubernetes.io/name: Foo
         app.kubernetes.io/instance: Bar
+        app.kubernetes.io/version: v1
         app.kubernetes.io/component: FooBar
         app.kubernetes.io/part-of: Foo
         app.kubernetes.io/managed-by: Bar
@@ -91,6 +98,7 @@ objects:
         labels:
           app.kubernetes.io/name: Foo
           app.kubernetes.io/instance: Bar
+          app.kubernetes.io/version: v1
           app.kubernetes.io/component: FooBar
           app.kubernetes.io/part-of: Foo
           app.kubernetes.io/managed-by: Bar
@@ -145,6 +153,7 @@ objects:
     labels:
       app.kubernetes.io/name: Foo
       app.kubernetes.io/instance: Bar
+      app.kubernetes.io/version: v1
       app.kubernetes.io/component: FooBar
       app.kubernetes.io/part-of: Foo
       app.kubernetes.io/managed-by: Bar
@@ -156,6 +165,7 @@ objects:
         labels:
           app.kubernetes.io/name: Foo
           app.kubernetes.io/instance: Bar
+          app.kubernetes.io/version: v1
           app.kubernetes.io/component: FooBar
           app.kubernetes.io/part-of: Foo
           app.kubernetes.io/managed-by: Bar

diff --git a/policy/ocp/bestpractices/common-k8s-labels-notset/src.rego b/policy/ocp/bestpractices/common-k8s-labels-notset/src.rego
@@ -17,7 +17,7 @@ violation[msg] {
 is_common_labels_set(metadata) {
   metadata.labels["app.kubernetes.io/name"]
   metadata.labels["app.kubernetes.io/instance"]
-  #metadata.labels["app.kubernetes.io/version"]
+  metadata.labels["app.kubernetes.io/version"]
   metadata.labels["app.kubernetes.io/component"]
   metadata.labels["app.kubernetes.io/part-of"]
   metadata.labels["app.kubernetes.io/managed-by"]

diff --git a/policy/ocp/bestpractices/common-k8s-labels-notset/test_data/integration/list.yml b/policy/ocp/bestpractices/common-k8s-labels-notset/test_data/integration/list.yml
@@ -12,6 +12,7 @@ items:
       matchLabels:
         app.kubernetes.io/name: Foo
         app.kubernetes.io/instance: Bar
+        app.kubernetes.io/version: v1
         app.kubernetes.io/component: FooBar
         app.kubernetes.io/part-of: Foo
         app.kubernetes.io/managed-by: Bar
@@ -20,6 +21,7 @@ items:
         labels:
           app.kubernetes.io/name: Foo
           app.kubernetes.io/instance: Bar
+          app.kubernetes.io/version: v1
           app.kubernetes.io/component: FooBar
           app.kubernetes.io/part-of: Foo
           app.kubernetes.io/managed-by: Bar
@@ -72,6 +74,7 @@ items:
         labels:
           app.kubernetes.io/name: Foo
           app.kubernetes.io/instance: Bar
+          app.kubernetes.io/version: v1
           app.kubernetes.io/component: FooBar
           app.kubernetes.io/part-of: Foo
           app.kubernetes.io/managed-by: Bar

diff --git a/policy/ocp/bestpractices/container-env-maxmemory-notset/test_data/integration/list.yml b/policy/ocp/bestpractices/container-env-maxmemory-notset/test_data/integration/list.yml
@@ -8,6 +8,7 @@ items:
     labels:
       app.kubernetes.io/name: Foo
       app.kubernetes.io/instance: Bar
+      app.kubernetes.io/version: v1
       app.kubernetes.io/component: FooBar
       app.kubernetes.io/part-of: Foo
       app.kubernetes.io/managed-by: Bar
@@ -18,6 +19,7 @@ items:
       matchLabels:
         app.kubernetes.io/name: Foo
         app.kubernetes.io/instance: Bar
+        app.kubernetes.io/version: v1
         app.kubernetes.io/component: FooBar
         app.kubernetes.io/part-of: Foo
         app.kubernetes.io/managed-by: Bar
@@ -26,6 +28,7 @@ items:
         labels:
           app.kubernetes.io/name: Foo
           app.kubernetes.io/instance: Bar
+          app.kubernetes.io/version: v1
           app.kubernetes.io/component: FooBar
           app.kubernetes.io/part-of: Foo
           app.kubernetes.io/managed-by: Bar
@@ -66,6 +69,7 @@ items:
     labels:
       app.kubernetes.io/name: Foo
       app.kubernetes.io/instance: Bar
+      app.kubernetes.io/version: v1
       app.kubernetes.io/component: FooBar
       app.kubernetes.io/part-of: Foo
       app.kubernetes.io/managed-by: Bar
@@ -77,6 +81,7 @@ items:
         labels:
           app.kubernetes.io/name: Foo
           app.kubernetes.io/instance: Bar
+          app.kubernetes.io/version: v1
           app.kubernetes.io/component: FooBar
           app.kubernetes.io/part-of: Foo
           app.kubernetes.io/managed-by: Bar

diff --git a/policy/ocp/bestpractices/container-image-latest/test_data/integration/list.yml b/policy/ocp/bestpractices/container-image-latest/test_data/integration/list.yml
@@ -8,6 +8,7 @@ items:
     labels:
       app.kubernetes.io/name: Foo
       app.kubernetes.io/instance: Bar
+      app.kubernetes.io/version: v1
       app.kubernetes.io/component: FooBar
       app.kubernetes.io/part-of: Foo
       app.kubernetes.io/managed-by: Bar
@@ -18,6 +19,7 @@ items:
       matchLabels:
         app.kubernetes.io/name: Foo
         app.kubernetes.io/instance: Bar
+        app.kubernetes.io/version: v1
         app.kubernetes.io/component: FooBar
         app.kubernetes.io/part-of: Foo
         app.kubernetes.io/managed-by: Bar
@@ -26,6 +28,7 @@ items:
         labels:
           app.kubernetes.io/name: Foo
           app.kubernetes.io/instance: Bar
+          app.kubernetes.io/version: v1
           app.kubernetes.io/component: FooBar
           app.kubernetes.io/part-of: Foo
           app.kubernetes.io/managed-by: Bar
@@ -71,6 +74,7 @@ items:
     labels:
       app.kubernetes.io/name: Foo
       app.kubernetes.io/instance: Bar
+      app.kubernetes.io/version: v1
       app.kubernetes.io/component: FooBar
       app.kubernetes.io/part-of: Foo
       app.kubernetes.io/managed-by: Bar
@@ -82,6 +86,7 @@ items:
         labels:
           app.kubernetes.io/name: Foo
           app.kubernetes.io/instance: Bar
+          app.kubernetes.io/version: v1
           app.kubernetes.io/component: FooBar
           app.kubernetes.io/part-of: Foo
           app.kubernetes.io/managed-by: Bar

diff --git a/policy/ocp/bestpractices/container-java-xmx-set/test_data/integration/list.yml b/policy/ocp/bestpractices/container-java-xmx-set/test_data/integration/list.yml
@@ -8,6 +8,7 @@ items:
     labels:
       app.kubernetes.io/name: Foo
       app.kubernetes.io/instance: Bar
+      app.kubernetes.io/version: v1
       app.kubernetes.io/component: FooBar
       app.kubernetes.io/part-of: Foo
       app.kubernetes.io/managed-by: Bar
@@ -18,6 +19,7 @@ items:
       matchLabels:
         app.kubernetes.io/name: Foo
         app.kubernetes.io/instance: Bar
+        app.kubernetes.io/version: v1
         app.kubernetes.io/component: FooBar
         app.kubernetes.io/part-of: Foo
         app.kubernetes.io/managed-by: Bar
@@ -26,6 +28,7 @@ items:
         labels:
           app.kubernetes.io/name: Foo
           app.kubernetes.io/instance: Bar
+          app.kubernetes.io/version: v1
           app.kubernetes.io/component: FooBar
           app.kubernetes.io/part-of: Foo
           app.kubernetes.io/managed-by: Bar
@@ -72,6 +75,7 @@ items:
     labels:
       app.kubernetes.io/name: Foo
       app.kubernetes.io/instance: Bar
+      app.kubernetes.io/version: v1
       app.kubernetes.io/component: FooBar
       app.kubernetes.io/part-of: Foo
       app.kubernetes.io/managed-by: Bar
@@ -82,6 +86,7 @@ items:
       matchLabels:
         app.kubernetes.io/name: Foo
         app.kubernetes.io/instance: Bar
+        app.kubernetes.io/version: v1
         app.kubernetes.io/component: FooBar
         app.kubernetes.io/part-of: Foo
         app.kubernetes.io/managed-by: Bar
@@ -90,6 +95,7 @@ items:
         labels:
           app.kubernetes.io/name: Foo
           app.kubernetes.io/instance: Bar
+          app.kubernetes.io/version: v1
           app.kubernetes.io/component: FooBar
           app.kubernetes.io/part-of: Foo
           app.kubernetes.io/managed-by: Bar
@@ -136,6 +142,7 @@ items:
     labels:
       app.kubernetes.io/name: Foo
       app.kubernetes.io/instance: Bar
+      app.kubernetes.io/version: v1
       app.kubernetes.io/component: FooBar
       app.kubernetes.io/part-of: Foo
       app.kubernetes.io/managed-by: Bar
@@ -146,6 +153,7 @@ items:
       matchLabels:
         app.kubernetes.io/name: Foo
         app.kubernetes.io/instance: Bar
+        app.kubernetes.io/version: v1
         app.kubernetes.io/component: FooBar
         app.kubernetes.io/part-of: Foo
         app.kubernetes.io/managed-by: Bar
@@ -154,6 +162,7 @@ items:
         labels:
           app.kubernetes.io/name: Foo
           app.kubernetes.io/instance: Bar
+          app.kubernetes.io/version: v1
           app.kubernetes.io/component: FooBar
           app.kubernetes.io/part-of: Foo
           app.kubernetes.io/managed-by: Bar
@@ -201,6 +210,7 @@ items:
     labels:
       app.kubernetes.io/name: Foo
       app.kubernetes.io/instance: Bar
+      app.kubernetes.io/version: v1
       app.kubernetes.io/component: FooBar
       app.kubernetes.io/part-of: Foo
       app.kubernetes.io/managed-by: Bar
@@ -212,6 +222,7 @@ items:
         labels:
           app.kubernetes.io/name: Foo
           app.kubernetes.io/instance: Bar
+          app.kubernetes.io/version: v1
           app.kubernetes.io/component: FooBar
           app.kubernetes.io/part-of: Foo
           app.kubernetes.io/managed-by: Bar
@@ -260,6 +271,7 @@ items:
     labels:
       app.kubernetes.io/name: Foo
       app.kubernetes.io/instance: Bar
+      app.kubernetes.io/version: v1
       app.kubernetes.io/component: FooBar
       app.kubernetes.io/part-of: Foo
       app.kubernetes.io/managed-by: Bar
@@ -271,6 +283,7 @@ items:
         labels:
           app.kubernetes.io/name: Foo
           app.kubernetes.io/instance: Bar
+          app.kubernetes.io/version: v1
           app.kubernetes.io/component: FooBar
           app.kubernetes.io/part-of: Foo
           app.kubernetes.io/managed-by: Bar
@@ -319,6 +332,7 @@ items:
     labels:
       app.kubernetes.io/name: Foo
       app.kubernetes.io/instance: Bar
+      app.kubernetes.io/version: v1
       app.kubernetes.io/component: FooBar
       app.kubernetes.io/part-of: Foo
       app.kubernetes.io/managed-by: Bar
@@ -330,6 +344,7 @@ items:
         labels:
           app.kubernetes.io/name: Foo
           app.kubernetes.io/instance: Bar
+          app.kubernetes.io/version: v1
           app.kubernetes.io/component: FooBar
           app.kubernetes.io/part-of: Foo
           app.kubernetes.io/managed-by: Bar

diff --git a/policy/ocp/bestpractices/container-labelkey-inconsistent/test_data/integration/list.yml b/policy/ocp/bestpractices/container-labelkey-inconsistent/test_data/integration/list.yml
@@ -8,6 +8,7 @@ items:
     labels:
       app.kubernetes.io/name: Foo
       app.kubernetes.io/instance: Bar
+      app.kubernetes.io/version: v1
       app.kubernetes.io/component: FooBar
       app.kubernetes.io/part-of: Foo
       app.kubernetes.io/managed-by: Bar
@@ -20,6 +21,7 @@ items:
       matchLabels:
         app.kubernetes.io/name: Foo
         app.kubernetes.io/instance: Bar
+        app.kubernetes.io/version: v1
         app.kubernetes.io/component: FooBar
         app.kubernetes.io/part-of: Foo
         app.kubernetes.io/managed-by: Bar
@@ -28,6 +30,7 @@ items:
         labels:
           app.kubernetes.io/name: Foo
           app.kubernetes.io/instance: Bar
+          app.kubernetes.io/version: v1
           app.kubernetes.io/component: FooBar
           app.kubernetes.io/part-of: Foo
           app.kubernetes.io/managed-by: Bar
@@ -75,6 +78,7 @@ items:
     labels:
       app.kubernetes.io/name: Foo
       app.kubernetes.io/instance: Bar
+      app.kubernetes.io/version: v1
       app.kubernetes.io/component: FooBar
       app.kubernetes.io/part-of: Foo
       app.kubernetes.io/managed-by: Bar
@@ -88,6 +92,7 @@ items:
         labels:
           app.kubernetes.io/name: Foo
           app.kubernetes.io/instance: Bar
+          app.kubernetes.io/version: v1
           app.kubernetes.io/component: FooBar
           app.kubernetes.io/part-of: Foo
           app.kubernetes.io/managed-by: Bar

diff --git a/.../ocp/bestpractices/container-liveness-readinessprobe-equal/test_data/integration/list.yml b/.../ocp/bestpractices/container-liveness-readinessprobe-equal/test_data/integration/list.yml
@@ -8,6 +8,7 @@ items:
     labels:
       app.kubernetes.io/name: Foo
       app.kubernetes.io/instance: Bar
+      app.kubernetes.io/version: v1
       app.kubernetes.io/component: FooBar
       app.kubernetes.io/part-of: Foo
       app.kubernetes.io/managed-by: Bar
@@ -18,6 +19,7 @@ items:
       matchLabels:
         app.kubernetes.io/name: Foo
         app.kubernetes.io/instance: Bar
+        app.kubernetes.io/version: v1
         app.kubernetes.io/component: FooBar
         app.kubernetes.io/part-of: Foo
         app.kubernetes.io/managed-by: Bar
@@ -26,6 +28,7 @@ items:
         labels:
           app.kubernetes.io/name: Foo
           app.kubernetes.io/instance: Bar
+          app.kubernetes.io/version: v1
           app.kubernetes.io/component: FooBar
           app.kubernetes.io/part-of: Foo
           app.kubernetes.io/managed-by: Bar
@@ -72,6 +75,7 @@ items:
     labels:
       app.kubernetes.io/name: Foo
       app.kubernetes.io/instance: Bar
+      app.kubernetes.io/version: v1
       app.kubernetes.io/component: FooBar
       app.kubernetes.io/part-of: Foo
       app.kubernetes.io/managed-by: Bar
@@ -83,6 +87,7 @@ items:
         labels:
           app.kubernetes.io/name: Foo
           app.kubernetes.io/instance: Bar
+          app.kubernetes.io/version: v1
           app.kubernetes.io/component: FooBar
           app.kubernetes.io/part-of: Foo
           app.kubernetes.io/managed-by: Bar