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
OpenZFS 6642 - testrunner output can be displayed in the wrong order #5777
Conversation
6643 zfstest should enforce the required privileges before running. Reviewed by: George Wilson <george.wilson@delphix.com> Reviewed by: Jonathan Mackenzie <jonathan.mackenzie@delphix.com> Reviewed by: Yuri Pankov <yuri.pankov@nexenta.com> Approved by: Robert Mustacchi <rm@joyent.com> Ported-by: Brian Behlendorf <behlendorf1@llnl.gov> Porting notes: - The 6643 changes were dropped a different version of this script is used to configure the environment under Linux.
@behlendorf, thanks for your PR! By analyzing the history of the files in this pull request, we identified @dinatale2 and @jwk404 to be potential reviewers. |
@gmelikov I've added the zfs-tests OpenZFS path to the OpenZFS Tracking results so we can identify and pull in any missing test suite changes. There's a good chance some of these changes are already applied in the initial ZTS commit 6bb24f4. I've identified some and updated the exceptions list accordingly, but there are probably more. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing the issue link and the openzfs commit link. Other than that, LGTM.
@behlendorf thanks, that's great! I'll work on them too later. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
6643 zfstest should enforce the required privileges before running. Reviewed by: George Wilson <george.wilson@delphix.com> Reviewed by: Jonathan Mackenzie <jonathan.mackenzie@delphix.com> Reviewed by: Yuri Pankov <yuri.pankov@nexenta.com> Approved by: Robert Mustacchi <rm@joyent.com> Reviewed-by: George Melikov <mail@gmelikov.ru> Reviewed-by: Giuseppe Di Natale <dinatale2@llnl.gov> Reviewed by: John Kennedy <john.kennedy@delphix.com> Ported-by: Brian Behlendorf <behlendorf1@llnl.gov OpenZFS-issue: https://www.illumos.org/issues/6642 OpenZFS-commit: openzfs/openzfs@412db4e Closes openzfs#5777 Porting notes: - The 6643 changes were dropped a different version of this script is used to configure the environment under Linux.
6643 zfstest should enforce the required privileges before running. Reviewed by: George Wilson <george.wilson@delphix.com> Reviewed by: Jonathan Mackenzie <jonathan.mackenzie@delphix.com> Reviewed by: Yuri Pankov <yuri.pankov@nexenta.com> Approved by: Robert Mustacchi <rm@joyent.com> Reviewed-by: George Melikov <mail@gmelikov.ru> Reviewed-by: Giuseppe Di Natale <dinatale2@llnl.gov> Reviewed by: John Kennedy <john.kennedy@delphix.com> Ported-by: Brian Behlendorf <behlendorf1@llnl.gov OpenZFS-issue: https://www.illumos.org/issues/6642 OpenZFS-commit: openzfs/openzfs@412db4e Closes openzfs#5777 Porting notes: - The 6643 changes were dropped a different version of this script is used to configure the environment under Linux.
6643 zfstest should enforce the required privileges before running.
Reviewed by: George Wilson george.wilson@delphix.com
Reviewed by: Jonathan Mackenzie jonathan.mackenzie@delphix.com
Reviewed by: Yuri Pankov yuri.pankov@nexenta.com
Approved by: Robert Mustacchi rm@joyent.com
Ported-by: Brian Behlendorf behlendorf1@llnl.gov
Porting notes:
is used to configure the environment under Linux.