mirror of
https://github.com/kata-containers/kata-containers.git
synced 2025-06-24 06:27:39 +00:00
metrics: Add double quotes in all variables
This PR adds double quotes in all variables to have uniformity across all the gha-run.sh script. Signed-off-by: Gabriela Cervantes <gabriela.cervantes.tellez@intel.com>
This commit is contained in:
parent
cc8f0a24e4
commit
917576e6fb
@ -9,7 +9,7 @@ set -o errexit
|
|||||||
set -o nounset
|
set -o nounset
|
||||||
set -o pipefail
|
set -o pipefail
|
||||||
|
|
||||||
kata_tarball_dir=${2:-kata-artifacts}
|
kata_tarball_dir="${2:-kata-artifacts}"
|
||||||
metrics_dir="$(dirname "$(readlink -f "$0")")"
|
metrics_dir="$(dirname "$(readlink -f "$0")")"
|
||||||
source "${metrics_dir}/../common.bash"
|
source "${metrics_dir}/../common.bash"
|
||||||
|
|
||||||
@ -58,7 +58,7 @@ function install_kata() {
|
|||||||
# Removing previous kata installation
|
# Removing previous kata installation
|
||||||
sudo rm -rf "${katadir}"
|
sudo rm -rf "${katadir}"
|
||||||
|
|
||||||
pushd ${kata_tarball_dir}
|
pushd "${kata_tarball_dir}"
|
||||||
sudo tar -xvf "${kata_tarball}" -C "${destdir}"
|
sudo tar -xvf "${kata_tarball}" -C "${destdir}"
|
||||||
popd
|
popd
|
||||||
|
|
||||||
@ -77,9 +77,9 @@ function check_containerd_config_for_kata() {
|
|||||||
declare -r line2="runtime_type = \"io.containerd.kata.v2\""
|
declare -r line2="runtime_type = \"io.containerd.kata.v2\""
|
||||||
declare -r num_lines_containerd=2
|
declare -r num_lines_containerd=2
|
||||||
declare -r containerd_path="/etc/containerd/config.toml"
|
declare -r containerd_path="/etc/containerd/config.toml"
|
||||||
local count_matches=$(grep -ic "$line1\|$line2" ${containerd_path})
|
local count_matches=$(grep -ic "$line1\|$line2" "${containerd_path}")
|
||||||
|
|
||||||
if [ $count_matches = $num_lines_containerd ]; then
|
if [ "${count_matches}" = "${num_lines_containerd}" ]; then
|
||||||
info "containerd ok"
|
info "containerd ok"
|
||||||
else
|
else
|
||||||
info "overwriting containerd configuration w/ a valid one"
|
info "overwriting containerd configuration w/ a valid one"
|
||||||
|
Loading…
Reference in New Issue
Block a user