Discussion:
[developer] [REVIEW] small zfstest fixes
Yuri Pankov
2013-10-31 21:03:31 UTC
Permalink
webrev: https://www.xvoid.org/illumos/webrev/il-zfstest-fixes/

usr/src/pkg/manifests/system-test-zfstest.mf: depend on
system/xopen/xcu4 to make /usr/xpg4/bin/sh avaiable

usr/src/test/zfs-tests/cmd/chg_usr_exec/chg_usr_exec.c: make it actually
say which binary it can't execl() in case of error

usr/src/test/zfs-tests/tests/functional/acl/acl.cfg: use 'zfstest' user
account instead of commonly used 'admin'

usr/src/test/zfs-tests/tests/functional/cli_root/zpool_get/zpool_get.cfg: add
new features fixing the functional/cli_root/zpool_get/zpool_get_003_pos
test.


-------------------------------------------
illumos-developer
Archives: https://www.listbox.com/member/archive/182179/=now
RSS Feed: https://www.listbox.com/member/archive/rss/182179/21175072-86d49504
Modify Your Subscription: https://www.listbox.com/member/?member_id=21175072&id_secret=21175072-abdf7b7e
Powered by Listbox: http://www.listbox.com
John Kennedy
2013-10-31 23:17:25 UTC
Permalink
Hey Yuri,
Post by Yuri Pankov
webrev: https://www.xvoid.org/illumos/webrev/il-zfstest-fixes/
usr/src/pkg/manifests/system-test-zfstest.mf: depend on system/xopen/xcu4 to
make /usr/xpg4/bin/sh avaiable
usr/src/test/zfs-tests/cmd/chg_usr_exec/chg_usr_exec.c: make it actually say
which binary it can't execl() in case of error
usr/src/test/zfs-tests/tests/functional/acl/acl.cfg: use 'zfstest' user
account instead of commonly used 'admin'
add new features fixing the functional/cli_root/zpool_get/zpool_get_003_pos
test.
LGTM. Thanks for doing these.
--
John Wren Kennedy

275 Middlefield Road, Suite 50
Menlo Park, CA 94025
http://www.delphix.com/


-------------------------------------------
illumos-developer
Archives: https://www.listbox.com/member/archive/182179/=now
RSS Feed: https://www.listbox.com/member/archive/rss/182179/21175072-86d49504
Modify Your Subscription: https://www.listbox.com/member/?member_id=21175072&id_secret=21175072-abdf7b7e
Powered by Listbox: http://www.listbox.com
Marcel Telka
2013-11-01 11:06:41 UTC
Permalink
Post by Yuri Pankov
webrev: https://www.xvoid.org/illumos/webrev/il-zfstest-fixes/
LGTM
--
+-------------------------------------------+
| Marcel Telka e-mail: ***@telka.sk |
| homepage: http://telka.sk/ |
| jabber: ***@jabber.sk |
+-------------------------------------------+


-------------------------------------------
illumos-developer
Archives: https://www.listbox.com/member/archive/182179/=now
RSS Feed: https://www.listbox.com/member/archive/rss/182179/21175072-86d49504
Modify Your Subscription: https://www.listbox.com/member/?member_id=21175072&id_secret=21175072-abdf7b7e
Powered by Listbox: http://www.listbox.com
Lionel Cons
2013-11-01 17:19:28 UTC
Permalink
Post by Yuri Pankov
webrev: https://www.xvoid.org/illumos/webrev/il-zfstest-fixes/
usr/src/pkg/manifests/system-test-zfstest.mf: depend on system/xopen/xcu4 to
make /usr/xpg4/bin/sh avaiable
usr/src/test/zfs-tests/cmd/chg_usr_exec/chg_usr_exec.c: make it actually say
which binary it can't execl() in case of error
usr/src/test/zfs-tests/tests/functional/acl/acl.cfg: use 'zfstest' user
account instead of commonly used 'admin'
add new features fixing the functional/cli_root/zpool_get/zpool_get_003_pos
test.
LGTM

Lionel


-------------------------------------------
illumos-developer
Archives: https://www.listbox.com/member/archive/182179/=now
RSS Feed: https://www.listbox.com/member/archive/rss/182179/21175072-86d49504
Modify Your Subscription: https://www.listbox.com/member/?member_id=21175072&id_secret=21175072-abdf7b7e
Powered by Listbox: http://www.listbox.com

Loading...