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

pythonPackages.python-prctl: disabled tests causing build failure #106875

Merged

Conversation

kira-bruneau
Copy link
Contributor

@kira-bruneau kira-bruneau commented Dec 14, 2020

Motivation for this change

Fixes #106871

Now disables tests using pytestCheckHook + disabledTests instead of patch file.

Disables:

  • test_speculation_ctrl
  • test_mpx

Enables:

  • test_mce_kill
  • test_proctitle
Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
  • Built on platform(s)
    • NixOS
    • macOS
    • other Linux distributions
  • Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
    No binary files, package is library
  • Determined the impact on package closure size (by running nix path-info -S before and after)
  • Ensured that relevant documentation is up to date
  • Fits CONTRIBUTING.md.

@catern

@catern
Copy link
Contributor

catern commented Dec 14, 2020

Looks good besides the comment nitpick.

@kira-bruneau
Copy link
Contributor Author

@catern Oh sorry about that, I'll update that. Thank you 😄

@kira-bruneau kira-bruneau changed the title pythonPackages.python-prctl: disable test_speculation_ctrl pythonPackages.python-prctl: disable test_speculation_ctrl to fix build Dec 19, 2020
@rmcgibbo
Copy link
Contributor

Result of nixpkgs-review pr 106875 run on x86_64-linux 1

3 packages built:
  • python37Packages.python-prctl
  • python38Packages.python-prctl
  • python39Packages.python-prctl

Now disables tests using pytestCheckHook + disabledTests instead of
patch file.

Disables:
- test_speculation_ctrl
- test_mpx

Enables:
- test_mce_kill
- test_proctitle
@kira-bruneau kira-bruneau changed the title pythonPackages.python-prctl: disable test_speculation_ctrl to fix build pythonPackages.python-prctl: disabled tests causing build failure Jan 4, 2021
@kira-bruneau
Copy link
Contributor Author

@catern test_mpx also started failing since #107783, so I added it as a disabled test.

@SuperSandro2000
Copy link
Member

@ofborg eval

@SuperSandro2000
Copy link
Member

Those tests no longer fail in 1.8.1.

@kira-bruneau
Copy link
Contributor Author

kira-bruneau commented Mar 21, 2021

@SuperSandro2000 This commit was based on failures found in version 1.8.1. I just tried the latest version of master, and they're still failing for me:

======================================================================
ERROR: test_mpx (test_prctl.PrctlTest)
Test MPX enabling/disabling
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/build/python-prctl-1.8.1/test_prctl.py", line 207, in test_mpx
    self.assertEqual(prctl.mpx_enable_management(), None)
  File "/build/python-prctl-1.8.1/build/lib.linux-x86_64-3.8/prctl.py", line 16, in call_prctl
    return _prctl.prctl(option)
OSError: [Errno 22] Invalid argument

======================================================================
ERROR: test_speculation_ctrl (test_prctl.PrctlTest)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/build/python-prctl-1.8.1/test_prctl.py", line 335, in test_speculation_ctrl
    prctl.set_speculation_ctrl(prctl.SPEC_STORE_BYPASS, prctl.SPEC_ENABLE)
  File "/build/python-prctl-1.8.1/build/lib.linux-x86_64-3.8/prctl.py", line 19, in call_prctl
    return _prctl.prctl(option, arg, arg2)
PermissionError: [Errno 1] Operation not permitted

----------------------------------------------------------------------
Ran 33 tests in 0.005s

FAILED (errors=2, skipped=4)
Test failed: <unittest.runner.TextTestResult run=33 errors=2 failures=0>
error: Test failed: <unittest.runner.TextTestResult run=33 errors=2 failures=0>

@kira-bruneau
Copy link
Contributor Author

I'm reopening this because this is still broken on hydra, and has been broken since 2020-12-06: https://hydra.nixos.org/build/139366147.

@kira-bruneau kira-bruneau reopened this Mar 21, 2021
@SuperSandro2000
Copy link
Member

This is a semi-automatic executed nixpkgs-review with nixpkgs-review-checks extension. It is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.

Result of nixpkgs-review pr 106875 run on x86_64-linux 1

3 packages built:
  • python37Packages.python-prctl
  • python38Packages.python-prctl
  • python39Packages.python-prctl

The following issues got detected with the above build packages.
Please fix at least the ones listed with your changed packages:

python37Packages.python-prctl:

warning: unclear-gpl
gpl3 is a deprecated license, check if project uses gpl3Plus or gpl3Only and change meta.license accordingly.

Near pkgs/development/python-modules/python-prctl/default.nix:39:5:

   |
39 |     license = lib.licenses.gpl3;
   |     ^

See: https://github.com/jtojnar/nixpkgs-hammering/blob/master/explanations/unclear-gpl.md

@SuperSandro2000 SuperSandro2000 merged commit 6be0d00 into NixOS:master Mar 21, 2021
@kira-bruneau kira-bruneau deleted the pythonPackages.python-prctl branch December 6, 2023 21:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

pythonPackages.python-prctl fails to build
4 participants