Skip to content

Commit

Permalink
terraform: build all known 0.10 plugins in Hydra
Browse files Browse the repository at this point in the history
  • Loading branch information
@copumpkin
copumpkin committed Aug 31, 2017
1 parent 0383026 commit e8b8864
Showing 1 changed file with 4 additions and 0 deletions.
4 changes: 4 additions & 0 deletions pkgs/top-level/all-packages.nix
View file
Original file line number Diff line number Diff line change
Expand Up @@ -19013,6 +19013,10 @@ with pkgs;
terraform_0_10
;

# Terraform with all the plugins, both to get Hydra to build all plugins for us and for
# convenience if someone doesn't want to have to think about which plugins to use.
terraform_0_10-full = terraform_0_10.withPlugins lib.attrValues;

terraform = terraform_0_9;

terraform-inventory = callPackage ../applications/networking/cluster/terraform-inventory {};
Expand Down

8 comments on commit e8b8864

@copumpkin
Copy link
Member Author

Choose a reason for hiding this comment

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

cc @zimbatm @shlevy

@zimbatm
Copy link
Member

Choose a reason for hiding this comment

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

@copumpkin sounds good but aren't you using the other providers? Some other plugins I'm using: terraform, template, random, local, vault, mailgun

@zimbatm
Copy link
Member

Choose a reason for hiding this comment

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

I will add them later otherwise

@copumpkin
Copy link
Member Author

Choose a reason for hiding this comment

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

Ah, I didn't realize template was a separate provider, so I'll add that. I don't use the others, but I welcome additions.

@copumpkin
Copy link
Member Author

Choose a reason for hiding this comment

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

Since all the plugins are just copied & pasted code right now, I might make a function for building one of the standard TF providers.

@zimbatm
Copy link
Member

Choose a reason for hiding this comment

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

I've got a script to auto-generate all the plugins. I'm just waiting for the github rate-limit to calm down before testing it proper: https://github.com/zimbatm/nixpkgs/tree/terraform-update-all

@copumpkin
Copy link
Member Author

Choose a reason for hiding this comment

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

Cool! Rather than generate repetitive Nix code though, I'd rather factor out the common stuff. The updater would still be good since the hashes and versions would need to be determined, but it always makes me uneasy to codegen repetitive code in a language that can express the repetition natively.

@zimbatm
Copy link
Member

@zimbatm zimbatm commented on e8b8864 Sep 1, 2017

Choose a reason for hiding this comment

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

yeah good idea, making the change

Please sign in to comment.