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.ncclient: refactor + init pythonPackages.selectors2 #46186
Conversation
@GrahamcOfBorg build python27Packages.ncclient python36Packages.ncclient |
Success on x86_64-linux (full log) Attempted: python27Packages.ncclient The following builds were skipped because they don't evaluate on x86_64-linux: python36Packages.ncclient Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: python27Packages.ncclient The following builds were skipped because they don't evaluate on aarch64-linux: python36Packages.ncclient Partial log (click to expand)
|
@GrahamcOfBorg build python36Packages.selectors2 python27Packages.selectors2 |
Success on x86_64-linux (full log) Attempted: python36Packages.selectors2, python27Packages.selectors2 Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: python36Packages.selectors2, python27Packages.selectors2 Partial log (click to expand)
|
Failure on x86_64-darwin (full log) Attempted: python27Packages.ncclient The following builds were skipped because they don't evaluate on x86_64-darwin: python36Packages.ncclient Partial log (click to expand)
|
Success on x86_64-darwin (full log) Attempted: python36Packages.selectors2, python27Packages.selectors2 Partial log (click to expand)
|
Darwin failure looks like due to rounding errors. Maybe you can try to increase test tolerances? |
Looks like this could be fixed with a I don't think that it is changeable via the command line. Do you think it is appropriate to use a |
These are timing tests, this is very machine dependent and non-deterministic and will be pretty useless on Hydra. So I think we should disable them completely. Upstream does that for Travis CI too: https://github.com/SethMichaelLarson/selectors2/blob/master/tests/support.py#L135 Looks like an easy way for us to do the same without patching is to set a |
Added package becuase now dependency of ncclient.
adding selectors2 as dependency of ncclient. Added recently 20 days ago in August. ncclient/ncclient@8ef20f2
0e3a3e7
to
a726956
Compare
Great catch. I have fixed this with the latest commit by setting a TRAVIS environment variable within the tests. |
@GrahamcOfBorg build python36Packages.selectors2 python27Packages.selectors2 python27Packages.ncclient |
Success on aarch64-linux (full log) Attempted: python36Packages.selectors2, python27Packages.selectors2, python27Packages.ncclient Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: python36Packages.selectors2, python27Packages.selectors2, python27Packages.ncclient Partial log (click to expand)
|
Success on x86_64-darwin (full log) Attempted: python36Packages.selectors2, python27Packages.selectors2, python27Packages.ncclient Partial log (click to expand)
|
Thank you! |
Motivation for this change
18.09 Zero Hydra Failures #45960
Things done
ncclient fails to build due to new dependency. adding selectors2 as dependency of ncclient. Added recently 20 days ago in August. ncclient/ncclient@8ef20f2
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)