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

terraform-providers: bump versions #72782

Closed
wants to merge 2 commits into from

Conversation

@DzmitrySudnik
Copy link
Contributor

DzmitrySudnik commented Nov 4, 2019

Motivation for this change

Upgrade TF providers, in particular AWS one

Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
  • Built on platform(s)
    • NixOS
    • macOS
    • other Linux distributions
  • Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Determined the impact on package closure size (by running nix path-info -S before and after)
  • Ensured that relevant documentation is up to date
  • Fits CONTRIBUTING.md.
Notify maintainers

cc @

@d-goldin

This comment has been minimized.

Copy link
Contributor

d-goldin commented Nov 8, 2019

Something seems to be wrong with terraform-providers.scaleway.

nix-review log:

[104 built (1 failed), 78 copied (2045.6 MiB), 458.3 MiB DL]
error: build of '/nix/store/c5p4pfx24a5s03v8wix9ivwm6fj5hmaa-env.drv' failed
https://github.com/NixOS/nixpkgs/pull/72782
4 package failed to build:
terraform-full terraform-providers.scaleway terraform_0_11-full terragrunt

50 package were build:
terraform-providers.acme terraform-providers.alicloud terraform-providers.archive terraform-providers.aws terraform-providers.azuread terraform-providers.azurerm terraform-providers.azurestack terraform-providers.bigip terraform-providers.cloudflare terraform-providers.datadog terraform-providers.digitalocean terraform-providers.docker terraform-providers.fastly terraform-providers.flexibleengine terraform-providers.github terraform-providers.google terraform-providers.google-beta terraform-providers.hcloud terraform-providers.heroku terraform-providers.huaweicloud terraform-providers.kubernetes terraform-providers.local terraform-providers.mailgun terraform-providers.mysql terraform-providers.nomad terraform-providers.ns1 terraform-providers.nutanix terraform-providers.oci terraform-providers.openstack terraform-providers.opentelekomcloud terraform-providers.opsgenie terraform-providers.oraclepaas terraform-providers.packet terraform-providers.pagerduty terraform-providers.panos terraform-providers.postgresql terraform-providers.powerdns terraform-providers.random terraform-providers.secret terraform-providers.selectel terraform-providers.spotinst terraform-providers.tencentcloud terraform-providers.tfe terraform-providers.tls terraform-providers.triton terraform-providers.ucloud terraform-providers.vault terraform-providers.vsphere terraform-providers.yandex terraform_plugins_test
the previous commit got stale, need to update
@DzmitrySudnik

This comment has been minimized.

Copy link
Contributor Author

DzmitrySudnik commented Nov 8, 2019

Thanks @d-goldin , it got stale and the old version of scaleway is not compatible with a newer version of go in the master. I've bumped them all again and now nix-review passes for me.

@DzmitrySudnik

This comment has been minimized.

Copy link
Contributor Author

DzmitrySudnik commented Nov 11, 2019

With #72939 merged, that one can be abandoned now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.