fix: use container as cause in KSV104 check

[nikpivkin]

We should only use objects as causes in kubernetes checks , since only they contain the locations used to display the findings.

Config:

---
apiVersion: v1
kind: Pod
metadata:
  name: hello-sysctls
spec:
  containers:
  - name: hello
    image: busybox
    command:
    - sh
    - "-c"
    - echo 'Hello' && sleep 1h
    securityContext:
      seccompProfile:
        type: Unconfined

Before:

AVD-KSV-0104 (MEDIUM): container "hello" of pod "hello-sysctls" in "default" namespace should specify a seccomp profile
════════════════════════════════════════════════════════════════════════════════════════════════════════════════════════════════════════════════════════
A program inside the container can bypass Seccomp protection policies.

See https://avd.aquasec.com/misconfig/ksv104
────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────
 test.yaml:1
────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────
   1 [ ---
────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────

After:

AVD-KSV-0104 (MEDIUM): container "hello" of pod "hello-sysctls" in "default" namespace should specify a seccomp profile
════════════════════════════════════════════════════════════════════════════════════════════════════════════════════════════════════════════════════════
A program inside the container can bypass Seccomp protection policies.

See https://avd.aquasec.com/misconfig/ksv104
────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────
 test.yaml:8-16
────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────
   8 ┌   - name: hello
   9 │     image: busybox
  10 │     command:
  11 │     - sh
  12 │     - "-c"
  13 │     - echo 'Hello' && sleep 1h
  14 │     securityContext:
  15 │       seccompProfile:
  16 └         type: Unconfined
────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────

[simar7]

In our tests, do we not assert the cause? Should we do so or would that make the tests too brittle?

[simar7]

left a comment but we can merge this in for now.

[nikpivkin]

We do not check the cause in Rego tests because the test data is simplified. We can add such a check to the example tests.