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.dask-gateway: init at 0.8.0 #94693
Conversation
Result of 4 packages built:- python37Packages.dask-gateway - python37Packages.dask-gateway-server - python38Packages.dask-gateway - python38Packages.dask-gateway-server |
@GrahamcOfBorg build python3Packages.dask-gateway python3Packages.dask-gateway-server |
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.
if the server is just meant to be an application (E.g: dask-server ...
) I would prefer for it to be packaged as an application. Otherwise the use of go needs to be patched into the call sites if it's a package.
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.
In case they're needed to be updated in lockstep, consider adding the other package to passthru.tests
. At least ofborg will test that then.
5ca997c
to
ed2696a
Compare
@jonringer needs to be a python package due to how it is used. Often times the @FRidh both of these packages come from the same git repo https://github.com/dask/dask-gateway. So me saying that they need to be updated together is because they are released together. Should I switch to using the git repo and build from that? Should I set |
@GrahamcOfBorg build python3Packages.dask-gateway python3Packages.dask-gateway-server |
pkgs/development/python-modules/dask-gateway-server/default.nix
Outdated
Show resolved
Hide resolved
ed2696a
to
ad61fbf
Compare
Result of 4 packages built:- python37Packages.dask-gateway - python37Packages.dask-gateway-server - python38Packages.dask-gateway - python38Packages.dask-gateway-server |
@GrahamcOfBorg build python3Packages.dask-gateway python3Packages.dask-gateway-server |
do you mind reviewing #94894 |
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
Result of nixpkgs-review pr 94693 1
Motivation for this change
Looking at creating dask-gateway nixos service for hpc computations. Need package to exist first.
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)