github/kata-containers
1
0
Fork 1
mirror of https://github.com/kata-containers/kata-containers.git synced 2025-04-27 11:31:05 +00:00
Code Issues Projects Releases Wiki Activity

tests: k8s/tests_common: avoid masking return values

Browse Source 
Avoid masking command return values by declaring and only then assigning.

ShellCheck:

warning: Declare and assign separately to avoid masking return values. [SC2155]

Signed-off-by: Dan Mihai <dmihai@microsoft.com>
This commit is contained in:
Dan Mihai 2025-03-19 00:09:32 +00:00
parent b895e3b3e5
commit 59a70a2b28
1 changed files with 4 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
tests/integration/kubernetes/tests_common.sh
View File

@ -220,7 +220,8 @@ add_exec_to_policy_settings() {
shift
# Create a JSON array of strings containing all the args of the command to be allowed.
local exec_args=$(printf "%s\n" "$@" | jq -R | jq -sc)
local exec_args
exec_args=$(printf "%s\n" "$@" | jq -R | jq -sc)
# Change genpolicy settings to allow kubectl to exec the command specified by the caller.
local jq_command=".request_defaults.ExecProcessRequest.allowed_commands |= . + [${exec_args}]"
@ -289,7 +290,8 @@ add_allow_all_policy_to_yaml() {
# Previous version of yq was not ready to handle multiple objects in a single yaml.
# By default was changing only the first object.
# With yq>4 we need to make it explicit during the read and write.
local resource_kind=$(yq .kind "${yaml_file}" | head -1)
local resource_kind
resource_kind=$(yq .kind "${yaml_file}" | head -1)
case "${resource_kind}" in