Skip to content

Commit 10c2cb3

Browse files
committed
Fix shellcheck warnings
1 parent 09a9476 commit 10c2cb3

File tree

1 file changed

+12
-12
lines changed

1 file changed

+12
-12
lines changed

check_kubernetes.sh

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
#!/bin/bash
2-
# shellcheck disable=SC2181,SC2207
2+
# shellcheck disable=SC2181,SC2207,SC2199,SC2076
33

44
##########################
55
# Perform checks against Kubernetes API or with tab help of kubectl utility
@@ -283,44 +283,44 @@ mode_pvc() {
283283
volumes_namespace=$(echo "$volumes_list" | jq -r ".pvcRef.namespace" | uniq)
284284
for pvc_volumes in "${volumes_list[@]}"; do
285285
[ -z "$pvc_volumes" ] && continue
286-
for volume_name in $(echo $pvc_volumes | jq -r ".name"); do
287-
volume_bytes_available=$(echo "$pvc_volumes" | jq -r ". | select(.name==\"$volume_name\") | .availableBytes")
286+
for volume_name in $(echo "$pvc_volumes" | jq -r ".name"); do
287+
#volume_bytes_available=$(echo "$pvc_volumes" | jq -r ". | select(.name==\"$volume_name\") | .availableBytes")
288288
volume_bytes_capacity=$(echo "$pvc_volumes" | jq -r ". | select(.name==\"$volume_name\") | .capacityBytes")
289289
volume_bytes_used=$(echo "$pvc_volumes" | jq -r ". | select(.name==\"$volume_name\") | .usedBytes")
290-
volume_inodes_free=$(echo "$pvc_volumes" | jq -r ". | select(.name==\"$volume_name\") | .inodesFree")
290+
#volume_inodes_free=$(echo "$pvc_volumes" | jq -r ". | select(.name==\"$volume_name\") | .inodesFree")
291291
volume_inodes_used=$(echo "$pvc_volumes" | jq -r ". | select(.name==\"$volume_name\") | .inodesUsed")
292292
volume_inodes_capacity=$(echo "$pvc_volumes" | jq -r ". | select(.name==\"$volume_name\") | .inodes")
293293
volume_bytes_utilization=$(echo "100 * $volume_bytes_used / $volume_bytes_capacity" | bc)
294294
volume_inodes_utilization=$(echo "100 * $volume_inodes_used / $volume_inodes_capacity" | bc)
295-
if [ "$volume_bytes_utilization" -gt "$WARN" -a "$volume_bytes_utilization" -lt "$CRIT" ]; then
295+
if [ "$volume_bytes_utilization" -gt "$WARN" ] && [ "$volume_bytes_utilization" -lt "$CRIT" ]; then
296296
echo "WARNING. High storage utilization on pvc $volume_name (namespace:$volumes_namespace): \
297297
$volume_bytes_utilization% ($volume_bytes_used/$volume_bytes_capacity Bytes)"
298-
WARN_ERROR=$(($WARN_ERROR+1))
298+
((WARN_ERROR++))
299299
fi
300300
if [ "$volume_bytes_utilization" -gt "$CRIT" ]; then
301301
echo "CRITICAL. Very high storage utilization on pvc $volume_name: \
302302
$volume_bytes_utilization% ($volume_bytes_used/$volume_bytes_capacity Bytes)"
303-
CRIT_ERROR=$(($CRIT_ERROR+1))
303+
((CRIT_ERROR++))
304304
fi
305-
if [ "$volume_inodes_utilization" -gt "$WARN" -a "$volume_inodes_utilization" -lt "$CRIT" ]; then
305+
if [ "$volume_inodes_utilization" -gt "$WARN" ] && [ "$volume_inodes_utilization" -lt "$CRIT" ]; then
306306
echo "WARNING. High inodes utilization on pvc $volume_name: \
307307
$volume_inodes_utilization% ($volume_inodes_used/$volume_inodes_capacity)"
308-
WARN_ERROR=$(($WARN_ERROR+1))
308+
((WARN_ERROR++))
309309
fi
310310
if [ "$volume_inodes_utilization" -gt "$CRIT" ]; then
311311
echo "CRITICAL. Very high inodes utilization on pvc $volume_name: \
312312
$volume_inodes_utilization% ($volume_inodes_used/$volume_inodes_capacity)"
313-
CRIT_ERROR=$(($CRIT_ERROR+1))
313+
((CRIT_ERROR++))
314314
fi
315315
done
316316
done
317317
done
318318
done
319319
done
320320

321-
if [ "$WARN_ERROR" -eq "0" -a "$CRIT_ERROR" -eq "0" ]; then
321+
if [ "$WARN_ERROR" -eq "0" ] && [ "$CRIT_ERROR" -eq "0" ]; then
322322
echo "OK. No problem on pvc storage"
323-
elif [ "$WARN_ERROR" -ne "0" -a "$CRIT_ERROR" -eq "0" ]; then
323+
elif [ "$WARN_ERROR" -ne "0" ] && [ "$CRIT_ERROR" -eq "0" ]; then
324324
exit 1
325325
elif [ "$CRIT_ERROR" -ne "0" ]; then
326326
exit 2

0 commit comments

Comments
 (0)