GitHub PushEvent at 2014-07-06 18:40:07
Repository: zfs-test
Actor: FransUrbo
Payload: {"shas":[["60443f260691220dbdc96697d352eea57115b9f9","ae7481973e6c9f63ea91e7cd1b77d8821b8e072d@bayour.com","* Minor spelling errors and fixes.\n* Replace hardcoded path to lockfs with environment variable.\n* Double free fixed in ctime_001_pos.\n* Rewrite get_rootfs() and get_rootpool() - Can't exit with log_fail()\n  if it can't find rootfs/rootpool. This because both functions is used\n  to put the output into a variable\n\n     rootpool=$(get_rootpool)\n\n  This causes the test to continue and 'rootpool' to contain the value\n\n     Can not get rootpool\n\n  The rest of the code will try to use this as valid data. Instead\n  return NULL from the functions and let the test script to do the\n  log_fail() if the return value was NULL.\n* When filling the pool (slog_014_pos test), only write 20MB (not 50MB)\n  - ZFS always holds back at least 32MB of free space.\n* Replace calls to zfs to get property, use get_prop().\n* On Linux, a space between the '#' and '!' (the 'hash-bang') will\n  make stuff fail in a very weird way.\n* Remove an empty line at the end of a function.\n* Create all test files in one go (fork them) and then wait() for them\n  to finish. Similar to other tests...\n* Before mounting a filesystem, make sure that we remove (recursivly) the\n  mountpoint.","Turbo Fredriksson",true],["f6c3441918b92d548ab07fac404fed360a22bb1a","ae7481973e6c9f63ea91e7cd1b77d8821b8e072d@bayour.com","Make test scripts executable.","Turbo Fredriksson",true],["64b14e3dc61b19622dfe4bb8e00f1cc793ee2671","ae7481973e6c9f63ea91e7cd1b77d8821b8e072d@bayour.com","* Checkout the Makefile.am from ZoL.\n  + Remove all ZoL related crud.\n  + Add a test, test_verify, cleanup and target to compile stuff we need.\n  + Create three 25GB files which we can use to create pools on, instead of\n    using raw disks/devices. Some tests need _a lot_ of space...\n  + Link Makefile.am to Makefile - we'll fix that later with autoconf rules.\n* Add basic makefiles for linux to compile the binaries required by test..\n  + Make the binaries we need compile and run on Linux.\n* Ignore the binary commands, *.o and *~ files.","Turbo Fredriksson",true],["8e2aac6e6214a4ed9ca08bbbebaf74f88f30b097","ae7481973e6c9f63ea91e7cd1b77d8821b8e072d@bayour.com","Test case run file for Linux which is a copy of openindiana.run.\n+ Since ZFS On Linux doesn't have the 'aclmode' property (yet), skip that when testing.\n+ On Linux, the cli_user/zpool_{iostat,list} tests needs to run as root.\n+ Disable the ACL test case. Doesn't work in Linux because we don't have ACLs.\n+ Disable functional/grow_{pool,replicas} - This directory cannot be run on raw files.\n+ Disable functional/zvol/zvol_misc - no dumpadm command\n+ Disable functional/{interop,mdb} - Doesn't make sense in Linux (no metadb/metainit command).\n+ Disable functional/privilege - no pfexec command or 'RBAC profile' (?)\n+ Disable functional/rootpool - we're not running from ZFS on root.\n+ Disable functional/xattr - no 'runat' command.\n+ Disable functional/zvol/zvol_ENOSPC - no 'dumpadm' command.\n+ Disable functional/mount/umount_001 - no umountall command on Linux.\n+ Disable functional/slog/slog_013_pos - Linux doesn't have a 'lofiadm' command\n+ Disable functional/delegate - ZoL does not have delegation\n+ Disable utils_test_007_pos in functional/utils_test - no 'fstyp' command.\n+ Disable zvol_swap_003_pos, zvol_swap_004_pos, zvol_swap_006_pos - no kstat or swapadd.\n+ Disable zvol_swap_005_pos in functional/zvol/zvol_swap - swapon/swapoff is to simple (don't allow size etc).\n+ Disable functional/threadsappend - no psrinfo command.\n+ Note that there's something wrong with the 'largest_file' command - fails but succeeds (!?)\n+ Note that file_write needs more work - hangs ZoL.\n+ Note that functional/rsend test 008 and 010 get's killed\n  Don't know if it's because of bug in ZFS or the test rig/setup","Turbo Fredriksson",true],["26740c3cfd22a8bacd2659ed47302572b5461a51","ae7481973e6c9f63ea91e7cd1b77d8821b8e072d@bayour.com","* Add a linux commands config file, rename the old and create a wrapper\n  that loads either of these depending on system.\n  + Export the env variable LINUX=1 if it's a Linux system.\n* Move TEST_BASE_DIR to /var/tmp to not clutter the root directory.\n* Special considerations for linux/bsd (tar options, /dev/zvol location etc).\n* Set MNTTAB=/etc/mtab on Linux and /etc/mnttab on BSD\n* Use getent instead of groupmod (which don't exist in Linux) in libtest.shlib:add_group().\n* For missing commands in Linux, run 'scripts/missing_cmd'\n  + include the command name missing\n  + exit with errorcode to make sure test fails.\n  + write to STDERR.\n* lsof could be used in place of lockfs, but without the -f option.","Turbo Fredriksson",true],["d9c20e2f0d4845ea2147f3ec89d237cad0c4b81b","ae7481973e6c9f63ea91e7cd1b77d8821b8e072d@bayour.com","* A script using truncate or dd that will emulate 'mkfile' for the tests.\n  + Support creating sparse files (add -s as valid option to script)\n  + Support giving value directly in bytes (divide by 1M before calling dd)\n  + Support giving value in Giga bytes (multiply with 1k.\n  + Disable creating sparse files in some tests - some of them need 'real' files to work.\n  + If the dir to the file don't exist, create it.\n* Script to generate logs for the FAIL only on stdout.\n* Replace hardcoded /dev/zvol/{dsk,rdsk} with ZVOL_[R]DEVDIR.\n* Some commands behave differently (tar, cpio, cp, ls, pax etc) on Linux.\n* NEWFS (defined to be ext2fs on Linux) creates a ext2 filesystem by default.\n  So use $NEWFS_DEFAULT_FS instead of hardcoded 'ufs'\n* Use /dev/urandom (which is faster) instead of /dev/random.\n* Use 'stat -c %[XYZ]' instead of 'ls ...' to get [amc]time.\n  + Between the stats in check_atime_updated(), wait two seconds to make\n    sure we see a change.\n    !! MIGHT ONLY BE LINUX RELEVANT !!\n* In Linux, we use a file in /var/tmp as our 'disk'. This means that\n  creating a filesystem on it can't be done on '$DEV_RDSKDIR///var/tmp/file'.\n  So if the disk path begins with a /, then use as-is.\n  + Ditto for mounts.","Turbo Fredriksson",true],["288970723482d475efa2ee1afcc11663e03bff37","ae7481973e6c9f63ea91e7cd1b77d8821b8e072d@bayour.com","In ismounted(), support finding ext2 mounts\nIn ismounted(), try to find zvols mounted.","Turbo Fredriksson",true],["b394212fb7830e15992599d830ae653fabb3b45f","ae7481973e6c9f63ea91e7cd1b77d8821b8e072d@bayour.com","* Replace hardcoded path to /etc/mnttab to MNTTAB from commands.cfg.\n* Fix is_physical_device() to recognize devs on Linux.\n* In add_group(), groupadd return 9 if group already exists - re-set it to 4\n  to avoid to much clutter in the case.\n  + On Linux, start adding groups at gid=1000, not 100.\n* Use getent instead of groupmod to find if a group exists or not.\n* Use LINUX instead of 'uname -a | grep' - faster.\n* Add support for Linux\n  + by using 'parted' in set_partition() and get_endslice().\n  + by using 'exportfs' in setup_nfs_server().\n  + by renaming find_disks() to find_disks_bsd() and write find_disks_linux(),\n    using a 'wrapper function' to call one of the two.\n* Since we're using exportfs to get shares, the share point is $1, not $2 as with the cmd 'share'.\n* Translate 'psrinfo -p' to Linux 'grep ^processor /proc/cpuinfo | wc -l' in is_mp().\n  + Substract one on the number of processors found in /proc/cpuinfo. 1 proc => 0 (not MP).","Turbo Fredriksson",true],["ce30481d611e4d2409c3b4c6346c007a53d9981c","ae7481973e6c9f63ea91e7cd1b77d8821b8e072d@bayour.com","Don't use hardcoded paths and command, use the variables defined in commands.cfg.","Turbo Fredriksson",true],["987eca231e99f2ad99883606c4694f82dd31e9e4","ae7481973e6c9f63ea91e7cd1b77d8821b8e072d@bayour.com","* Remove exact path to kill, true and sudo.\n* Add .{ksh,sh} to the command to allow to run without installing.\n  + Ditto for verifying file name.","Turbo Fredriksson",true],["bb70f5da0d59364cc0c754dee892b5b86fde1ec8","ae7481973e6c9f63ea91e7cd1b77d8821b8e072d@bayour.com","* Include default.cfg at the top instead of the bottom (to get command variable defines etc).\n* Remove Don't use hardcoded paths and command, use the variables defined in commands.cfg.\n* Some stuff isn't relevant on Linux.\n* Don't force set STF_{SUITE,TOOLS}. Use value from env if availible.\n* Rename find_disks() to find_disks_zfstest() to not clash with one from lib.\n* Since we moved the basic test dir (from / to /var/tmp), make sure they exist.","Turbo Fredriksson",true],["9ca50ddf87d05f9a64e598a1a66914eefaaed7ab","ae7481973e6c9f63ea91e7cd1b77d8821b8e072d@bayour.com","Unpack the 'upgrade pool' images in /var/tmp/test_results instead of the\npreviously created pool. If that is using loop files, a kernel crash\nwill happen on Linux.","Turbo Fredriksson",true],["467f0bdc929afecfccaee0eb20073d04480072e0","ae7481973e6c9f63ea91e7cd1b77d8821b8e072d@bayour.com","On Linux, we must use 'kpartx' after partitioning the raw disk file\nto create a loop device with partitions (partitioning a raw disk file\ndoes not create the parition files/devices).\n+ In Linux, the 'disk part' is called a 'partition' and is labeled\n  with a '<device>p<nr>' instead of '<device>s<nr>' (slice).\n+ Overwrite some variables defined in the config file.\n+  Cleanup (reorder/'prettify') the definition of the META_* variables.\n+ In Linux we create a ext2 filesystem by default, not ufs.","Turbo Fredriksson",true],["59b83634280fed3f4b8f0b30e096a73462f3755d","ae7481973e6c9f63ea91e7cd1b77d8821b8e072d@bayour.com","* Some tests works on Linux but fail on BSD. Use log_{must,mustnot} appropriately.\n* Replace a == with a != to verify a failure.","Turbo Fredriksson",true],["0483fa2f721c6e1bedd1109a17cb4665f7711322","ae7481973e6c9f63ea91e7cd1b77d8821b8e072d@bayour.com","Since Linux don't have 'coreadm', we use the procedure used on Linux instead.\n+ This means using the /proc filesystem and ulimit.","Turbo Fredriksson",true],["7dde0008a7d945ee592a6626532e368b96ee7b33","ae7481973e6c9f63ea91e7cd1b77d8821b8e072d@bayour.com","* In setup_snap_env(), support 'newfs' on Linux (really mke2fs) which behave a little different\n* In cleanup_env(), look for any mounted zvols - !!!NEEDS BSD VERIFICATION!!!","Turbo Fredriksson",true],["9792ad5af7fe008482c71d2319170e0b3f6b993d","ae7481973e6c9f63ea91e7cd1b77d8821b8e072d@bayour.com","The Linux equivalence of 'mount -F zfs' is 'mount -t zfs'.\n+ The 'dev' ('devices') option never show up in a mount list\n  Test this by trying to create a 'zero' device.","Turbo Fredriksson",true],["682f5c80f2a02b1468009625324ec59a05c4f377","ae7481973e6c9f63ea91e7cd1b77d8821b8e072d@bayour.com","unshare doesn't exist on Linux, instead exportfs is used. But it have\nquite different usage. And can't run as normal user...","Turbo Fredriksson",true],["4c8903692d34887396ff0f9a4f002f4e0a660abb","ae7481973e6c9f63ea91e7cd1b77d8821b8e072d@bayour.com","Remove leading slash to disk images in TESTDIR (ended up with double\nwhich made the test fail).","Turbo Fredriksson",true],["58f556565e33f0abf2ef57fbe940dda8ba0b320b","ae7481973e6c9f63ea91e7cd1b77d8821b8e072d@bayour.com","The expected return value from 'mmap_exec' on Linux is 22, not 13.","Turbo Fredriksson",true]],"size":35,"ref":"refs/heads/issue-1534","head":"9dae2e23f25aaf49f98d7a1e63effafcf3479f11"}