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
terraform: 0.9.1 -> 0.9.2 #24505
terraform: 0.9.1 -> 0.9.2 #24505
Conversation
@zimbatm, thanks for your PR! By analyzing the history of the files in this pull request, we identified @peterhoeg, @kamilchm and @siddharthist to be potential reviewers. |
Please don't merge yet, there is a flakey test in the kubenetes provider that needs fixing too.
|
* makes the generic builder open for extension * upgrades terraform and set the 0_9 series by default
b989d87
to
1afd3ce
Compare
all good now! |
sha256 = "1yj5x1d10028fm3v3gjyjdn128ps0as345hr50y8x3vn86n70lxl"; | ||
|
||
patches = [ | ||
(fetchurl { |
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.
Maybe use fetchPatch
here?
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.
Do you mean extract it into a function? The only fetchPatch
I could find in nixpkgs was at pkgs/applications/networking/irc/bip/default.nix
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.
I think @NeQuissimus is referring to this:
pkgs/top-level/all-packages.nix
154: fetchpatch = callPackage ../build-support/fetchpatch { };
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.
My bad, case-sensitive grep :) Fixed in ae03402
use fetchpatch to improve patch download hash stability
I don't think this is working, at least on linux: https://hydra.nixos.org/job/nixpkgs/trunk/terraform_0_9_2.x86_64-linux/all It seems to fail some test locally too. |
Weird, I couldn't find the build logs either. Thanks for pointing it out @bdimcheff, I'll check it tonight when I have access to my Linux box |
https://cache.nixos.org/log/am62rb1gf9w70xnk82vl9j11xhl0qjip-terraform-0.9.2.drv A failure I'd expect, due to sandboxing:
|
Ah right, I suppose |
Motivation for this change
The terraform 0.9.1 tests where failing for me.
Things done
(nix.useSandbox on NixOS,
or option
build-use-sandbox
innix.conf
on non-NixOS)
nix-shell -p nox --run "nox-review wip"
./result/bin/
)