Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

#8476 breaks in-tree tests #9118

Closed
c0d3z3r0 opened this issue Aug 2, 2019 · 1 comment
Closed

#8476 breaks in-tree tests #9118

c0d3z3r0 opened this issue Aug 2, 2019 · 1 comment
Labels
Component: Test Suite Indicates an issue with the test framework or a test case

Comments

@c0d3z3r0
Copy link
Contributor

c0d3z3r0 commented Aug 2, 2019

System information

Type Version/Name
Distribution Name debian
Distribution Version testing
Linux Kernel not relevant
Architecture amd64
ZFS Version master
SPL Version -

Describe the problem you're observing

Looks like the last change of #8476 - adding sudo - broke in-tree tests.

Describe how to reproduce the problem

$ ./scripts/zfs-tests.sh -xv -t tests/functional/cli_user/misc/dbufstat_001_pos

--- Cleanup ---
Removing pool(s):      
Removing dm(s):       
Removing loopback(s):  
Removing files(s):     

--- Configuration ---
Runfile:         /var/tmp/zfs-tests.28717.run
STF_TOOLS:       /usr/src/zfs/tests/test-runner
STF_SUITE:       /usr/src/zfs/tests/zfs-tests
STF_PATH:        /usr/src/zfs/bin
FILEDIR:         /var/tmp
FILES:           /var/tmp/file-vdev0 /var/tmp/file-vdev1 /var/tmp/file-vdev2
LOOPBACKS:       /dev/loop0 /dev/loop1 /dev/loop2 
DISKS:           loop0 loop1 loop2
NUM_DISKS:       3
FILESIZE:        4G
ITERATIONS:      1
TAGS:            functional
STACK_TRACER:    no
Keep pool(s):    rpool
Missing util(s): arp pax quotaon setenforce umask uuidgen wait 

/usr/src/zfs/tests/test-runner/bin/test-runner.py  -c /var/tmp/zfs-tests.28717.run -T functional -i /usr/src/zfs/tests/zfs-tests -I 1
Test: /usr/src/zfs/tests/zfs-tests/tests/functional/cli_user/misc/setup (run as root) [00:09] [PASS]
Test: /usr/src/zfs/tests/zfs-tests/tests/functional/cli_user/misc/dbufstat_001_pos (run as root) [00:00] [FAIL]
Test: /usr/src/zfs/tests/zfs-tests/tests/functional/cli_user/misc/cleanup (run as root) [00:01] [PASS]

Results Summary
PASS	   2
FAIL	   1

Running Time:	00:00:11
Percent passed:	66.7%
Log directory:	/var/tmp/test_results/20190802T185749

Tests with results other than PASS that are expected:

Tests with result of PASS that are unexpected:

Tests with results other than PASS that are unexpected:
    FAIL cli_user/misc/dbufstat_001_pos (expected PASS)

After reverting the sudo stuff in dbufstat_001_pos.ksh:

$ ./scripts/zfs-tests.sh -xv -t tests/functional/cli_user/misc/dbufstat_001_pos

--- Cleanup ---
Removing pool(s):      
Removing dm(s):       
Removing loopback(s):  
Removing files(s):     

--- Configuration ---
Runfile:         /var/tmp/zfs-tests.2930.run
STF_TOOLS:       /usr/src/zfs/tests/test-runner
STF_SUITE:       /usr/src/zfs/tests/zfs-tests
STF_PATH:        /usr/src/zfs/bin
FILEDIR:         /var/tmp
FILES:           /var/tmp/file-vdev0 /var/tmp/file-vdev1 /var/tmp/file-vdev2
LOOPBACKS:       /dev/loop0 /dev/loop1 /dev/loop2 
DISKS:           loop0 loop1 loop2
NUM_DISKS:       3
FILESIZE:        4G
ITERATIONS:      1
TAGS:            functional
STACK_TRACER:    no
Keep pool(s):    rpool
Missing util(s): arp pax quotaon setenforce umask uuidgen wait 

/usr/src/zfs/tests/test-runner/bin/test-runner.py  -c /var/tmp/zfs-tests.2930.run -T functional -i /usr/src/zfs/tests/zfs-tests -I 1
Test: /usr/src/zfs/tests/zfs-tests/tests/functional/cli_user/misc/setup (run as root) [00:10] [PASS]
Test: /usr/src/zfs/tests/zfs-tests/tests/functional/cli_user/misc/dbufstat_001_pos (run as root) [00:01] [PASS]
Test: /usr/src/zfs/tests/zfs-tests/tests/functional/cli_user/misc/cleanup (run as root) [00:01] [PASS]

Results Summary
PASS	   3

Running Time:	00:00:12
Percent passed:	100.0%
Log directory:	/var/tmp/test_results/20190802T190803

Tests with results other than PASS that are expected:

Tests with result of PASS that are unexpected:

Tests with results other than PASS that are unexpected:

Include any warning/errors/backtraces from the system logs

Stderr when failing:

ASSERTION: dbufstat generates output and doesn't return an error code
sudo: dbufstat: command not found
ERROR: eval sudo dbufstat  > /dev/null exited 1
@behlendorf
Copy link
Contributor

@c0d3z3r0 PR #9196 has been opened with the proposed fix. Reviewers welcome.

@behlendorf behlendorf added the Component: Test Suite Indicates an issue with the test framework or a test case label Aug 21, 2019
ghost pushed a commit to zfsonfreebsd/ZoF that referenced this issue Aug 26, 2019
Commit a887d65 updated the dbufstats such that escalated privileges
are required.  Since all tests under cli_user are run with normal
privileges move this test case to a location where it will be run
required privileges.

Reviewed-by: John Kennedy <john.kennedy@delphix.com>
Reviewed-by: Ryan Moeller <ryan@ixsystems.com>
Reviewed-by: Michael Niewöhner <foss@mniewoehner.de>
Signed-off-by: Brian Behlendorf <behlendorf1@llnl.gov>
Closes openzfs#9118 
Closes openzfs#9196
tonyhutter pushed a commit to tonyhutter/zfs that referenced this issue Dec 24, 2019
Commit a887d65 updated the dbufstats such that escalated privileges
are required.  Since all tests under cli_user are run with normal
privileges move this test case to a location where it will be run
required privileges.

Reviewed-by: John Kennedy <john.kennedy@delphix.com>
Reviewed-by: Ryan Moeller <ryan@ixsystems.com>
Reviewed-by: Michael Niewöhner <foss@mniewoehner.de>
Signed-off-by: Brian Behlendorf <behlendorf1@llnl.gov>
Closes openzfs#9118
Closes openzfs#9196
tonyhutter pushed a commit to tonyhutter/zfs that referenced this issue Dec 27, 2019
Commit a887d65 updated the dbufstats such that escalated privileges
are required.  Since all tests under cli_user are run with normal
privileges move this test case to a location where it will be run
required privileges.

Reviewed-by: John Kennedy <john.kennedy@delphix.com>
Reviewed-by: Ryan Moeller <ryan@ixsystems.com>
Reviewed-by: Michael Niewöhner <foss@mniewoehner.de>
Signed-off-by: Brian Behlendorf <behlendorf1@llnl.gov>
Closes openzfs#9118
Closes openzfs#9196
tonyhutter pushed a commit that referenced this issue Jan 23, 2020
Commit a887d65 updated the dbufstats such that escalated privileges
are required.  Since all tests under cli_user are run with normal
privileges move this test case to a location where it will be run
required privileges.

Reviewed-by: John Kennedy <john.kennedy@delphix.com>
Reviewed-by: Ryan Moeller <ryan@ixsystems.com>
Reviewed-by: Michael Niewöhner <foss@mniewoehner.de>
Signed-off-by: Brian Behlendorf <behlendorf1@llnl.gov>
Closes #9118
Closes #9196
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component: Test Suite Indicates an issue with the test framework or a test case
Projects
None yet
Development

No branches or pull requests

2 participants