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

update terraform-providers.{helm,kubernetes} to 2.0.2 #115064

Merged
merged 3 commits into from Mar 4, 2021

Conversation

andir
Copy link
Member

@andir andir commented Mar 4, 2021

Motivation for this change

I had a requirement for a newer version of both of these providers.

Things done

@andir andir requested review from zimbatm and timstott March 4, 2021 11:34
I wasn't able to figure out for which unstable version of Nix this is
supposed to work so I added nix itself to the package list. Nix will now
be the same stable version that is also provided in this nixpkgs
checkout.

To make it work with a stable Nix version I also had to trim the build
error output as otherwise it would contain the sha256 prefix & a newline
followed by "error:".
Copy link
Member

@zimbatm zimbatm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@zimbatm
Copy link
Member

zimbatm commented Mar 4, 2021

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

3 packages built:
  • terraform-full
  • terraform-providers.helm
  • terraform-providers.kubernetes

@zimbatm zimbatm merged commit 0722dda into NixOS:master Mar 4, 2021
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.

None yet

2 participants