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.

Sign up for GitHub

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

Jump to bottom

ZTS: Log test name to /dev/kmsg on Linux #13227

Merged
merged 1 commit into from
Mar 23, 2022
Merged

ZTS: Log test name to /dev/kmsg on Linux #13227

merged 1 commit into from
Mar 23, 2022

Conversation

tonyhutter
Copy link
Contributor

Motivation and Context

Easily match a kernel warning to a particular test

Description

Log each test we run to /dev/kmsg (on Linux), so if there's a kernel warning we'll be able to match it up to a particular test

How Has This Been Tested?

Ran locally

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Performance enhancement (non-breaking change which improves efficiency)
  • Code cleanup (non-breaking change which makes code smaller or more readable)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Library ABI change (libzfs, libzfs_core, libnvpair, libuutil and libzfsbootenv)
  • Documentation (a change to man pages or other documentation)

Checklist:

@tonyhutter tonyhutter added the Component: Test Suite Indicates an issue with the test framework or a test case label Mar 16, 2022
@tonyhutter tonyhutter force-pushed the zts-kmsg branch 3 times, most recently from ff7c025 to 20d7332 Compare March 21, 2022 22:18
@tonyhutter
ZTS: Log test name to /dev/kmsg on Linux
6072507 
Add a -K option to the test suite to log each test name to /dev/kmsg
(on Linux), so if there's a kernel warning we'll be able to match
it up to a particular test.

Signed-off-by: Tony Hutter <hutter2@llnl.gov>
@tonyhutter
Copy link
Contributor Author

This should be good to go

@jwk404 jwk404 added the Status: Accepted Ready to integrate (reviewed, tested) label Mar 23, 2022
@jwk404 jwk404 merged commit b73505c into openzfs:master Mar 23, 2022
@gmelikov gmelikov mentioned this pull request Mar 24, 2022
Merged
13 tasks
nicman23 pushed a commit to nicman23/zfs that referenced this pull request Aug 22, 2022
@tonyhutter @nicman23
ZTS: Log test name to /dev/kmsg on Linux
83e2283 
Add a -K option to the test suite to log each test name to /dev/kmsg
(on Linux), so if there's a kernel warning we'll be able to match
it up to a particular test.

Reviewed-by: John Kennedy <john.kennedy@delphix.com>
Signed-off-by: Tony Hutter <hutter2@llnl.gov>
Closes  openzfs#13227
nicman23 pushed a commit to nicman23/zfs that referenced this pull request Aug 22, 2022
@tonyhutter @nicman23
ZTS: Log test name to /dev/kmsg on Linux
0323e2c 
Add a -K option to the test suite to log each test name to /dev/kmsg
(on Linux), so if there's a kernel warning we'll be able to match
it up to a particular test.

Reviewed-by: John Kennedy <john.kennedy@delphix.com>
Signed-off-by: Tony Hutter <hutter2@llnl.gov>
Closes  openzfs#13227
lundman pushed a commit to openzfsonwindows/openzfs that referenced this pull request Sep 2, 2022
@tonyhutter @lundman
ZTS: Log test name to /dev/kmsg on Linux
8c771d8 
Add a -K option to the test suite to log each test name to /dev/kmsg
(on Linux), so if there's a kernel warning we'll be able to match
it up to a particular test.

Reviewed-by: John Kennedy <john.kennedy@delphix.com>
Signed-off-by: Tony Hutter <hutter2@llnl.gov>
Closes  openzfs#13227
beren12 pushed a commit to beren12/zfs that referenced this pull request Sep 19, 2022
@tonyhutter @beren12
ZTS: Log test name to /dev/kmsg on Linux
a2dcab9 
Add a -K option to the test suite to log each test name to /dev/kmsg
(on Linux), so if there's a kernel warning we'll be able to match
it up to a particular test.

Reviewed-by: John Kennedy <john.kennedy@delphix.com>
Signed-off-by: Tony Hutter <hutter2@llnl.gov>
Closes  openzfs#13227
andrewc12 pushed a commit to andrewc12/openzfs that referenced this pull request Sep 23, 2022
@tonyhutter @andrewc12
ZTS: Log test name to /dev/kmsg on Linux
aa5dc63 
Add a -K option to the test suite to log each test name to /dev/kmsg
(on Linux), so if there's a kernel warning we'll be able to match
it up to a particular test.

Reviewed-by: John Kennedy <john.kennedy@delphix.com>
Signed-off-by: Tony Hutter <hutter2@llnl.gov>
Closes  openzfs#13227
andrewc12 pushed a commit to andrewc12/openzfs that referenced this pull request Sep 23, 2022
@tonyhutter @andrewc12
ZTS: Log test name to /dev/kmsg on Linux
0f7c56a 
Add a -K option to the test suite to log each test name to /dev/kmsg
(on Linux), so if there's a kernel warning we'll be able to match
it up to a particular test.

Reviewed-by: John Kennedy <john.kennedy@delphix.com>
Signed-off-by: Tony Hutter <hutter2@llnl.gov>
Closes  openzfs#13227
andrewc12 pushed a commit to andrewc12/openzfs that referenced this pull request Sep 23, 2022
@tonyhutter @andrewc12
ZTS: Log test name to /dev/kmsg on Linux
5ab5f8e 
Add a -K option to the test suite to log each test name to /dev/kmsg
(on Linux), so if there's a kernel warning we'll be able to match
it up to a particular test.

Reviewed-by: John Kennedy <john.kennedy@delphix.com>
Signed-off-by: Tony Hutter <hutter2@llnl.gov>
Closes  openzfs#13227
andrewc12 pushed a commit to andrewc12/openzfs that referenced this pull request Sep 23, 2022
@tonyhutter @andrewc12
ZTS: Log test name to /dev/kmsg on Linux
5676010 
Add a -K option to the test suite to log each test name to /dev/kmsg
(on Linux), so if there's a kernel warning we'll be able to match
it up to a particular test.

Reviewed-by: John Kennedy <john.kennedy@delphix.com>
Signed-off-by: Tony Hutter <hutter2@llnl.gov>
Closes  openzfs#13227
andrewc12 pushed a commit to andrewc12/openzfs that referenced this pull request Sep 23, 2022
@tonyhutter @andrewc12
ZTS: Log test name to /dev/kmsg on Linux
a5defe9 
Add a -K option to the test suite to log each test name to /dev/kmsg
(on Linux), so if there's a kernel warning we'll be able to match
it up to a particular test.

Reviewed-by: John Kennedy <john.kennedy@delphix.com>
Signed-off-by: Tony Hutter <hutter2@llnl.gov>
Closes  openzfs#13227
andrewc12 pushed a commit to andrewc12/openzfs that referenced this pull request Sep 23, 2022
@tonyhutter @andrewc12
ZTS: Log test name to /dev/kmsg on Linux
55e0b6d 
Add a -K option to the test suite to log each test name to /dev/kmsg
(on Linux), so if there's a kernel warning we'll be able to match
it up to a particular test.

Reviewed-by: John Kennedy <john.kennedy@delphix.com>
Signed-off-by: Tony Hutter <hutter2@llnl.gov>
Closes  openzfs#13227
andrewc12 pushed a commit to andrewc12/openzfs that referenced this pull request Sep 23, 2022
@tonyhutter @andrewc12
ZTS: Log test name to /dev/kmsg on Linux
90420aa 
Add a -K option to the test suite to log each test name to /dev/kmsg
(on Linux), so if there's a kernel warning we'll be able to match
it up to a particular test.

Reviewed-by: John Kennedy <john.kennedy@delphix.com>
Signed-off-by: Tony Hutter <hutter2@llnl.gov>
Closes  openzfs#13227
andrewc12 pushed a commit to andrewc12/openzfs that referenced this pull request Sep 23, 2022
@tonyhutter @andrewc12
ZTS: Log test name to /dev/kmsg on Linux
dc16b67 
Add a -K option to the test suite to log each test name to /dev/kmsg
(on Linux), so if there's a kernel warning we'll be able to match
it up to a particular test.

Reviewed-by: John Kennedy <john.kennedy@delphix.com>
Signed-off-by: Tony Hutter <hutter2@llnl.gov>
Closes  openzfs#13227
andrewc12 pushed a commit to andrewc12/openzfs that referenced this pull request Sep 23, 2022
@tonyhutter @andrewc12
ZTS: Log test name to /dev/kmsg on Linux
67803c2 
Add a -K option to the test suite to log each test name to /dev/kmsg
(on Linux), so if there's a kernel warning we'll be able to match
it up to a particular test.

Reviewed-by: John Kennedy <john.kennedy@delphix.com>
Signed-off-by: Tony Hutter <hutter2@llnl.gov>
Closes  openzfs#13227
andrewc12 pushed a commit to andrewc12/openzfs that referenced this pull request Sep 23, 2022
@tonyhutter @andrewc12
ZTS: Log test name to /dev/kmsg on Linux
77f4cb3 
Add a -K option to the test suite to log each test name to /dev/kmsg
(on Linux), so if there's a kernel warning we'll be able to match
it up to a particular test.

Reviewed-by: John Kennedy <john.kennedy@delphix.com>
Signed-off-by: Tony Hutter <hutter2@llnl.gov>
Closes  openzfs#13227
andrewc12 pushed a commit to andrewc12/openzfs that referenced this pull request Sep 23, 2022
@tonyhutter @andrewc12
ZTS: Log test name to /dev/kmsg on Linux
d12f8bc 
Add a -K option to the test suite to log each test name to /dev/kmsg
(on Linux), so if there's a kernel warning we'll be able to match
it up to a particular test.

Reviewed-by: John Kennedy <john.kennedy@delphix.com>
Signed-off-by: Tony Hutter <hutter2@llnl.gov>
Closes  openzfs#13227
andrewc12 pushed a commit to andrewc12/openzfs that referenced this pull request Sep 23, 2022
@tonyhutter @andrewc12
ZTS: Log test name to /dev/kmsg on Linux
21aa124 
Add a -K option to the test suite to log each test name to /dev/kmsg
(on Linux), so if there's a kernel warning we'll be able to match
it up to a particular test.

Reviewed-by: John Kennedy <john.kennedy@delphix.com>
Signed-off-by: Tony Hutter <hutter2@llnl.gov>
Closes  openzfs#13227
andrewc12 pushed a commit to andrewc12/openzfs that referenced this pull request Sep 23, 2022
@tonyhutter @andrewc12
ZTS: Log test name to /dev/kmsg on Linux
1da86f3 
Add a -K option to the test suite to log each test name to /dev/kmsg
(on Linux), so if there's a kernel warning we'll be able to match
it up to a particular test.

Reviewed-by: John Kennedy <john.kennedy@delphix.com>
Signed-off-by: Tony Hutter <hutter2@llnl.gov>
Closes  openzfs#13227
tonyhutter added a commit to tonyhutter/zfs that referenced this pull request Apr 3, 2023
@tonyhutter
ZTS: Log test name to /dev/kmsg on Linux
fc2c025 
Add a -K option to the test suite to log each test name to /dev/kmsg
(on Linux), so if there's a kernel warning we'll be able to match
it up to a particular test.

Reviewed-by: John Kennedy <john.kennedy@delphix.com>
Signed-off-by: Tony Hutter <hutter2@llnl.gov>
Closes  openzfs#13227
tonyhutter added a commit to tonyhutter/zfs that referenced this pull request Apr 5, 2023
@tonyhutter
ZTS: Log test name to /dev/kmsg on Linux
9505b7e 
Add a -K option to the test suite to log each test name to /dev/kmsg
(on Linux), so if there's a kernel warning we'll be able to match
it up to a particular test.

Reviewed-by: John Kennedy <john.kennedy@delphix.com>
Signed-off-by: Tony Hutter <hutter2@llnl.gov>
Closes  openzfs#13227
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nabijaczleweli nabijaczleweli nabijaczleweli left review comments

@behlendorf behlendorf behlendorf left review comments

@jwk404 jwk404 jwk404 approved these changes

Assignees

@jwk404 jwk404

Labels
Component: Test Suite Indicates an issue with the test framework or a test case Status: Accepted Ready to integrate (reviewed, tested)
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@tonyhutter @behlendorf @jwk404 @nabijaczleweli