mirror of
https://github.com/docker/docker-bench-security.git
synced 2024-11-01 00:21:45 +01:00
check if restart policy is 5 or less
Signed-off-by: Thomas Sjögren <konstruktoid@users.noreply.github.com>
This commit is contained in:
parent
b7a5284ce4
commit
ab2190819d
1 changed files with 7 additions and 7 deletions
|
@ -600,27 +600,27 @@ check_5_14() {
|
|||
for s in $(docker service ls --format '{{.Name}}'); do
|
||||
if echo $container_name | grep -q "$s"; then
|
||||
task_id=$(docker inspect "$c" --format '{{.Name}}' | awk -F '.' '{print $NF}')
|
||||
# a container name could arbitrary include a service one: it belongs to a service (created by Docker
|
||||
# a container name could arbitrary include a service one: it belongs to a service (created by Docker
|
||||
# as part of the service), if the container task ID matches one of the task IDs of the service.
|
||||
if docker service ps --no-trunc "$s" --format '{{.ID}}' | grep -q "$task_id"; then
|
||||
spolicy=$(docker inspect --format MaxAttempts='{{ .Spec.TaskTemplate.RestartPolicy.MaxAttempts }}' "$s")
|
||||
restart_policy=$(docker inspect --format '{{ .Spec.TaskTemplate.RestartPolicy.MaxAttempts }}' "$s")
|
||||
break
|
||||
fi
|
||||
fi
|
||||
done
|
||||
fi
|
||||
cpolicy=$(docker inspect --format MaximumRetryCount='{{ .HostConfig.RestartPolicy.MaximumRetryCount }}' "$c")
|
||||
restart_policy=$(docker inspect --format '{{ .HostConfig.RestartPolicy.MaximumRetryCount }}' "$c")
|
||||
|
||||
if [ "$cpolicy" != "MaximumRetryCount=5" ] && [ "$spolicy" != "MaxAttempts=5" ]; then
|
||||
if [ "$restart_policy" -gt "5" ]; then
|
||||
# If it's the first container, fail the test
|
||||
if [ $fail -eq 0 ]; then
|
||||
warn -s "$check"
|
||||
warn " * MaximumRetryCount is not set to 5: $c"
|
||||
warn " * MaximumRetryCount is not set to 5 or less: $c"
|
||||
maxretry_unset_containers="$maxretry_unset_containers $c"
|
||||
fail=1
|
||||
continue
|
||||
fi
|
||||
warn " * MaximumRetryCount is not set to 5: $c"
|
||||
warn " * MaximumRetryCount is not set to 5 or less: $c"
|
||||
maxretry_unset_containers="$maxretry_unset_containers $c"
|
||||
fi
|
||||
done
|
||||
|
@ -630,7 +630,7 @@ check_5_14() {
|
|||
logcheckresult "PASS"
|
||||
return
|
||||
fi
|
||||
logcheckresult "WARN" "Containers with MaximumRetryCount not set to 5" "$maxretry_unset_containers"
|
||||
logcheckresult "WARN" "Containers with MaximumRetryCount not set to 5 or less" "$maxretry_unset_containers"
|
||||
}
|
||||
|
||||
check_5_15() {
|
||||
|
|
Loading…
Reference in a new issue