Commit Graph

127 Commits

Author SHA1 Message Date
Sander van Harmelen 28543694de Fix refresing an IP when it’s no longer associated 2016-07-12 21:32:56 +02:00
Sander van Harmelen 29ce2df873 Fix refresing ACL rules when the ACL is deleted 2016-07-12 21:11:22 +02:00
Sander van Harmelen b55da4c3e7 Fix creating networks without specifying IP ranges 2016-07-12 21:11:22 +02:00
Sander van Harmelen bdb35006cd Fix several bugs in different resources
- Make sure attaching a disk or a NIC is tried a couple of times as this only works after the OS has fully booted;
- Stop using the device name instead of ID as the names differ depending on the hypervisor that you are using;
- VPC’s do not always have a source NAT IP;
2016-07-12 21:11:22 +02:00
Joakim Sernbrant 1b4c1ab83a provider/cloudstack: do not force a new resource when updating instance user_data 2016-06-30 14:23:43 +02:00
Sander van Harmelen 8bdec15649 Add a few explaining lines to the docs about ICMP types/codes 2016-06-27 11:52:22 +02:00
Sander van Harmelen c14aa6336e Delete all deprecated parameters before the 0.7 release
Updated the docs accordingly and also executed all the acceptance tests
after making the changes…
2016-06-27 11:13:48 +02:00
Mickaël Canévet 318310c019 Remove Computed flag 2016-06-27 10:35:40 +02:00
Mickaël Canévet b4c7a274fa Add provider/cloudstack: add security_group_ids 2016-06-27 10:35:40 +02:00
Mickaël Canévet d7e7a45ec6 provider/cloudstack: add security_group_names 2016-06-27 10:33:25 +02:00
Mickaël Canévet 3d9bdafb97 provider/cloudstack: add affinity_group_names 2016-06-27 09:55:34 +02:00
Sander van Harmelen e51fb92ab5 Merge pull request #7070 from serbaut/root_disk_size
provider/cloudstack: add root_disk_size
2016-06-25 19:47:48 +02:00
Sander van Harmelen b7c71382f6 Make ACL's swappable, unless you want to stop using an ACL
In CloudStack you can dynamically start using an ACL and once you use
an ACL you can dynamically swap ACL’s. But once your using an ACL, you
can no longer stop using an ACL without rebuilding the network.

This change makes the `ForceNew` value dynamic so that it only returns
`true` if you are reverting from using an ACL to not using an ACL
anymore, making this functionally inline with the behaviour CloudStack
offers.
2016-06-24 13:27:05 +02:00
Joakim Sernbrant dd5f121494 provider/cloudstack: add root_disk_size 2016-06-08 10:23:14 +02:00
Sander van Harmelen 23801b5507 Add the option to assign affinity groups to a VM 2016-05-27 09:46:59 +02:00
Sander van Harmelen b20744b133 Merge pull request #6741 from svanharmelen/f-swap-network-acl
provider/cloudstack: make replacing the ACL of a network update the network in place
2016-05-23 21:12:55 +02:00
Sander van Harmelen 6a08e893e3 Use the new option to list ACLs based on the ID and VPC ID 2016-05-18 14:43:21 +02:00
Sander van Harmelen ecb41d478a Make replacing the ACL of a network update the network in place
This prevents having to recreate the whole network and fixes #6630
2016-05-18 12:22:32 +02:00
Sander van Harmelen 5a65cc9cb6 Use the new functionality offered by the `go-cloudstack` package
The updated functionality offers a new means to work with projects.
This commit uses that new functionality for the `cloudstack_network`
resource.
2016-04-22 12:23:47 +02:00
Sander van Harmelen 41b99a4ebc Update to latest `go-cloudstack` package to fix issue #6101 2016-04-19 22:48:57 +02:00
Sander van Harmelen 55d4e3cda7 Merge branch 'cloudstack-group' of https://github.com/jefflaplante/terraform into f-cloustack-group 2016-04-18 22:32:55 +02:00
Jeff LaPlante f840f49fbb Added support to read and update group attribute from existing vm state. 2016-04-11 10:23:19 -07:00
Sander van Harmelen 815c8840a7 Refactor the use of names vs IDs for parameters referencing other TF resources
We have a curtesy function in place allowing you to specify both a
`name` of `ID`. But in order for the graph to be build correctly when
you recreate or taint stuff that other resources depend on, we need to
reference the `ID` and *not* the `name`.

So in order to enforce this and by that help people to not make this
mistake unknowingly, I deprecated all the parameters this allies to and
changed the logic, docs and tests accordingly.
2016-04-11 17:14:28 +02:00
Hany Fahim 3135706d1f Update to use setProjectid helper function 2016-04-11 08:22:01 -04:00
Hany Fahim 44fc1b5d80 Add project parameter to cloudstack_port_forward.
- Add parameter to resource.
- Modify read operation to pass in projectid if defined.
2016-04-09 11:58:57 -04:00
Jeff LaPlante ba4ec0097a fixed formatting 2016-04-05 09:21:23 -07:00
Jeff LaPlante cf607e8a58 Added Group attribute to cloudstack instance resource 2016-04-05 09:12:45 -07:00
Sander van Harmelen fddf3eccc6 Make the CloudStack provider more inline with the other provider
It turns out all other providers use `ip_address` where the CloudStack
provider uses `ipaddress`. To make this more consistent this PR
deprecates `ipaddress` and adds `ip_address` where needed…
2016-04-04 22:16:15 +02:00
Sander van Harmelen e5fecca7cc Add `cloudstack_static_nat` resource 2016-04-04 15:48:14 +02:00
Sander van Harmelen 5c1f410ebe Some minor updates and added a proper test 2016-03-25 16:59:03 +01:00
James Nugent 63008b857f Make name optional on cloudstack_instance resource
This commit matches the behaviour of the cloudstack_instance resource to
the documentation at [1], which indicates that the virtual machine name
is optional. This also changes the update logic to allow it to be
updated as described in [2].

[1] http://cloudstack.apache.org/api/apidocs-4.8/domain_admin/deployVirtualMachine.html
[2] http://cloudstack.apache.org/api/apidocs-4.8/domain_admin/updateVirtualMachine.html
2016-03-11 15:59:40 +00:00
Sander van Harmelen 5525772a74 provider/cloudstack: small doc update 2016-02-26 23:44:53 +01:00
Joakim Sernbrant 7d30423a61 provider/cloudstack: Improve ssh keypair handling
- adds support for projects

- adds support for public_key strings as well as filenames
2016-02-26 23:06:16 +01:00
Sander van Harmelen c0f1962f51 Policing up the new `tags` code from @Carles-Figuerola
Just some cosmetics and some cleaning up to make the code fit in a
little better with the existing code. Functionally no changes are made
and the existing tests still pass.
2016-01-22 12:13:26 +01:00
Carles Figuerola 755917db07 Add tests for Network Tags for provider/cloudstack and applied tips from PR#4742 2016-01-21 22:45:51 -06:00
Carles Figuerola 0983ca4c2a Merge remote-tracking branch 'upstream/master' into add-tags-plus-networktags 2016-01-21 22:37:23 -06:00
Sander van Harmelen 3385100f56 Make the concurrence for applying rules configurable 2016-01-21 21:30:54 +01:00
Sander van Harmelen e3d530aaa2 Merge pull request #4772 from svanharmelen/b-fix-panic
provider/cloudstack: add a check to see if the port value is valid
2016-01-21 17:05:03 +01:00
Carles Figuerola e2d6f7c5e9 Specify that the vlan attribute in cloudstack is only usable for ROOT admins and stop reading it back 2016-01-21 09:52:27 -06:00
Sander van Harmelen 20901a6478 Add a check to see if the port value is valid
Without this additional check it could happen that one of the firewall
resources would panic is given an unexpected port value.
2016-01-21 09:38:14 +01:00
Carles Figuerola b18afaa5db Add taging shared module for cloudstack and implement it in resource_network 2016-01-19 11:41:18 -06:00
Carles Figuerola fab9b16de5 Move tests to ParseCIDR in resource_cloudstack_network and add items in read section 2016-01-15 14:24:18 -06:00
Carles Figuerola c80ddccc12 Add vlan, startip, endip and gateway for network creation 2016-01-14 22:59:04 -06:00
Sander van Harmelen b8f3417e79 Change all firewall related resources to take a cidr_list
Also some additional tweaks to improve performance and add in a little
concurrency to speed things up a little.
2015-12-03 15:34:53 +01:00
Sander van Harmelen 84645bd8b5 More tweaks to improve performance 2015-12-01 00:36:33 +01:00
Sander van Harmelen d4ce2b87fb Modified executable bit 2015-11-27 14:49:28 +01:00
Sander van Harmelen a3eae45b32 Improve performance all firewall related resources 2015-11-24 18:27:52 +01:00
Sander van Harmelen aa5d02104c Add `computed` flag to the `network_domain` parameter
Without this flag you will get a diff based on the `network_domain`
parameter, if you not specify the parameter.
2015-10-23 09:54:54 +02:00
Panagiotis Moustafellos e4845f75cc removed extra parentheses 2015-10-08 15:48:04 +03:00
Sander van Harmelen cc921b0bc7 Small refactor for better readability and updated the docs 2015-10-08 10:09:11 +02:00