sig_cloud/diskimage-builder
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Merge "Ignore missing path in unmount_dir"

Browse Source
This commit is contained in:
Jenkins 2017-04-07 15:59:47 +00:00 committed by Gerrit Code Review
commit 347833856e
3 changed files with 11 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
diskimage_builder/lib/common-functions
View File

@ -346,7 +346,7 @@ function unmount_dir {
if [ ! -d $dir ]; then if [ ! -d $dir ]; then
echo "*** $dir is not a directory" echo "*** $dir is not a directory"
return 1 return 0
fi fi
# get rid of any symlink elements in the incoming path, because # get rid of any symlink elements in the incoming path, because

4
diskimage_builder/lib/img-functions
View File

@ -121,9 +121,7 @@ function finalise_base () {
fi fi
fi fi
# Cleanup /tmp in the guest, so there is less cruft left there # Cleanup /tmp in the guest, so there is less cruft left there
if [ -d "$TMP_MOUNT_PATH/tmp" ]; then unmount_dir $TMP_MOUNT_PATH/tmp
unmount_dir $TMP_MOUNT_PATH/tmp
fi
find $TMP_MOUNT_PATH/tmp -maxdepth 1 -mindepth 1 | xargs sudo rm -rf --one-file-system find $TMP_MOUNT_PATH/tmp -maxdepth 1 -mindepth 1 | xargs sudo rm -rf --one-file-system
# Truncate /var/log files in preparation for first boot # Truncate /var/log files in preparation for first boot
sudo find ${TMP_MOUNT_PATH}/var/log -type f -exec cp /dev/null '{}' \; sudo find ${TMP_MOUNT_PATH}/var/log -type f -exec cp /dev/null '{}' \;

10
tests/run_dib_library_tests.sh
View File

@ -6,7 +6,7 @@
# need a human in the loop. Thus it's mostly useful for developers # need a human in the loop. Thus it's mostly useful for developers
# during testing, but not so great for CI # during testing, but not so great for CI
source ../lib/common-functions source ../diskimage_builder/lib/common-functions
# #
# Directory mounting and unmounting # Directory mounting and unmounting
@ -44,6 +44,14 @@ else
echo "*** PASS all directories unmounted" echo "*** PASS all directories unmounted"
fi fi
# unmount missing dir
if unmount_dir /this/path/does/not/exist/but/this/should/not/fail; then
echo "*** PASS unmount_dir ignored a missing path"
else
echo "*** FAILED unmount_dir should ignore missing paths"
return 1
fi
# cleanup # cleanup
rm -rf $TMP_DIR rm -rf $TMP_DIR