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
minikube: 1.2.0 -> 1.6.2 #66578
minikube: 1.2.0 -> 1.6.2 #66578
Conversation
@GrahamcOfBorg build minikube docker-machine-hyperkit @GrahamcOfBorg test docker |
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 🐯
This uses a more recent version than upstream nixpkgs. It also builds docker-machine-hyperkit separately from minikube. This is based on Atkins Chang's pull request: NixOS/nixpkgs#66578 Note: the docker-machine-hyperkit minikube plugin is currently disabled, because minikube on macOS needs to run it as root via setuid, and this is incompatible with the Nix store. I don't know of a way to wrap this appropriately on nix-darwin yet. At any rate, minikube will download a compatible version automatically, and it should work as long as hyperkit is in the minikube path, which this minikube derivation does on macOS.
Can we push this forward. The nix pkgs |
I just wonder why there was a |
@AtkinsChang sorry about that 😓 it's open-source, sometimes a newer PR gets merged before an older one and causes conflicts, it happens 😅 Can you rebase yours 👼 |
I was able to update 1.2.0 -> 1.5.2 by bumping the minikube version in master (and updating the modSha256): diff --git a/pkgs/applications/networking/cluster/minikube/default.nix b/pkgs/applications/networking/cluster/minikube/default.nix
index 5637e486ef1..e107cb98212 100644
--- a/pkgs/applications/networking/cluster/minikube/default.nix
+++ b/pkgs/applications/networking/cluster/minikube/default.nix
@@ -13,7 +13,7 @@ let
in buildGoModule rec {
pname = "minikube";
- version = "1.2.0";
+ version = "1.5.2";
kubernetesVersion = "1.15.0";
@@ -26,7 +26,7 @@ in buildGoModule rec {
sha256 = "0l9znrp49877cp1bkwx84c8lv282ga5a946rjbxi8gznkf3kwaw7";
};
- modSha256 = "1cp63n0x2lgbqvvymx9byx48r42qw6w224x5x4iiarc2nryfdhn0";
+ modSha256 = "1spyc71jfa05dyf30q30hjk2lx5w7929jsm3p03gbm6ssvmclfrh";
buildInputs = [ go-bindata makeWrapper gpgme ] ++ stdenv.lib.optional stdenv.hostPlatform.isDarwin vmnet;
subPackages = [ "cmd/minikube" ] ++ stdenv.lib.optional stdenv.hostPlatform.isDarwin "cmd/drivers/hyperkit"; |
@vdemeester although it works on my machine, it would be great that you can help to double verify the new version |
@AtkinsChang hum, interesting, I've got a different error that the CI :
|
@vdemeester it should be good now |
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 🐯
Can we drop the minikube-beta commit? There has been some new releases already, and it might make sense to just bump to the latest minikube release in a followup PR. |
b0e0e77
to
a0ce66e
Compare
@flokli virtualbox driver works well but kvm driver is not working because of |
@AtkinsChang sorry for the delay! I can't really test the virtualbox backend anymore. Did you get a chance to fix the libvirt backend? Missed a reference here. |
Unfortunately I couldn't find time to test it properly so far. Thanks @AtkinsChang for the PR, and thanks @Mic92 for merging it! 👍 |
Motivation for this change
Upgrade to
v1.6.2
and fix bugs in previous version:buildGoModule
fordocker-machine-kvm2
minikube
bugs running onnixos
cause byexec /bin/bash
commanddocker-machine-hyperkit
fromminikube
like whatdocker-machine-kvm2
didThings done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)Notify maintainers
cc @