Skip to content

Commit c7cd03c

Browse files
authored
Merge pull request kubernetes#76900 from SataQiu/fix-shellcheck-test-20190422
Fix shellcheck failures of test/e2e_node/environment/setup_host.sh
2 parents a73980e + c892bec commit c7cd03c

File tree

2 files changed

+8
-13
lines changed

2 files changed

+8
-13
lines changed

hack/.shellcheck_failures

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -62,6 +62,5 @@
6262
./test/cmd/template-output.sh
6363
./test/cmd/version.sh
6464
./test/e2e_node/conformance/run_test.sh
65-
./test/e2e_node/environment/setup_host.sh
6665
./test/e2e_node/gubernator.sh
6766
./test/images/image-util.sh

test/e2e_node/environment/setup_host.sh

Lines changed: 8 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -42,38 +42,35 @@ EOF
4242
fi
4343

4444
# For coreos, disable updates
45-
if $(sudo systemctl status update-engine &>/dev/null); then
45+
if sudo systemctl status update-engine &>/dev/null; then
4646
sudo systemctl mask update-engine locksmithd
4747
fi
4848

4949
# Fixup sudoers require tty
50-
sudo grep -q "# Defaults requiretty" /etc/sudoers
51-
if [ $? -ne 0 ] ; then
50+
if ! sudo grep -q "# Defaults requiretty" /etc/sudoers; then
5251
sudo sed -i 's/Defaults requiretty/# Defaults requiretty/' /etc/sudoers
5352
fi
5453

5554
# Install nsenter for ubuntu images
56-
cat /etc/*-release | grep "ID=ubuntu"
57-
if [ $? -eq 0 ]; then
55+
if cat /etc/*-release | grep "ID=ubuntu"; then
5856
if ! which nsenter > /dev/null; then
5957
echo "Do not find nsenter. Install it."
6058
NSENTER_BUILD_DIR=$(mktemp -d /tmp/nsenter-build-XXXXXX)
61-
cd $NSENTER_BUILD_DIR
59+
cd "$NSENTER_BUILD_DIR" || exit 1
6260
curl https://www.kernel.org/pub/linux/utils/util-linux/v2.31/util-linux-2.31.tar.gz | tar -zxf-
6361
sudo apt-get update
6462
sudo apt-get --yes install make
6563
sudo apt-get --yes install gcc
66-
cd util-linux-2.31
64+
cd util-linux-2.31 || exit 1
6765
./configure --without-ncurses
6866
make nsenter
6967
sudo cp nsenter /usr/local/bin
70-
rm -rf $NSENTER_BUILD_DIR
68+
rm -rf "$NSENTER_BUILD_DIR"
7169
fi
7270
fi
7371

7472
# Install docker
75-
hash docker 2>/dev/null
76-
if [ $? -ne 0 ]; then
73+
if ! hash docker 2>/dev/null; then
7774
# RHEL platforms should always install from RHEL repository
7875
# This will install the latest supported stable docker platform on RHEL
7976
if [ $OS_RHEL -eq 0 ]; then
@@ -93,8 +90,7 @@ id jenkins || sudo useradd jenkins -m
9390
sudo usermod -a -G docker jenkins
9491

9592
# install lxc
96-
cat /etc/*-release | grep "ID=debian"
97-
if [ $? -ne 0 ]; then
93+
if ! cat /etc/*-release | grep "ID=debian"; then
9894
hash apt-get 2>/dev/null
9995
if [ $? -ne 1 ]; then
10096
sudo apt-get install lxc -y

0 commit comments

Comments
 (0)