Skip to content

Commit

Permalink
tests: more k8s-exec-rejected debug output
Browse files Browse the repository at this point in the history
Print more information useful for debugging. Also, use a separate YAML
file for this test, instead of reusing someone else's file.

Fixes: #8270

Signed-off-by: Dan Mihai <dmihai@microsoft.com>
  • Loading branch information
danmihai1 committed Nov 29, 2023
1 parent 1f1e537 commit 96deea5
Show file tree
Hide file tree
Showing 2 changed files with 29 additions and 3 deletions.
10 changes: 7 additions & 3 deletions tests/integration/kubernetes/k8s-exec-rejected.bats
Original file line number Diff line number Diff line change
Expand Up @@ -10,8 +10,8 @@ load "${BATS_TEST_DIRNAME}/tests_common.sh"

setup() {
get_pod_config_dir
pod_name="busybox"
pod_yaml="${pod_config_dir}/busybox-pod.yaml"
pod_name="policy-exec-rejected"
pod_yaml="${pod_config_dir}/k8s-policy-exec-rejected.yaml"
allow_all_except_exec_policy=$(base64 -w 0 "${pod_config_dir}/allow-all-except-exec-process.rego")
}

Expand All @@ -25,10 +25,14 @@ setup() {
kubectl create -f "${pod_yaml}"

# Wait for pod to start
echo "timeout=${timeout}"
kubectl wait --for=condition=Ready --timeout=$timeout pod "$pod_name"

# Try executing a command in the Pod - an action rejected by the agent policy.
kubectl exec "$pod_name" -- date 2>&1 | grep "ExecProcessRequest is blocked by policy"
exec_output=$(kubectl exec "$pod_name" -- date 2>&1) || true
echo "$exec_output"

echo "$exec_output" | grep "ExecProcessRequest is blocked by policy"
}

teardown() {
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,22 @@
#
# Copyright (c) 2023 Microsoft
#
# SPDX-License-Identifier: Apache-2.0
#
apiVersion: v1
kind: Pod
metadata:
name: policy-exec-rejected
spec:
terminationGracePeriodSeconds: 0
shareProcessNamespace: true
runtimeClassName: kata
containers:
- name: first-test-container
image: quay.io/prometheus/busybox:latest
env:
- name: CONTAINER_NAME
value: "first-test-container"
command:
- sleep
- "120"

0 comments on commit 96deea5

Please sign in to comment.