Merge pull request #123 from xanzy/svh/b-user-data

Unwrap the GetVirtualMachineUserData response

Unwrap the GetVirtualMachineUserData response

Tweak the solution just a little

Merge pull request #119 from cezarsa/detailsencoding

Change details field encoding to match cloudstack expectations

Change details field encoding to match cloudstack expectations

How the details field must be encoded changes depending on the command
implementation. Some commands allow arbitrary keys and some other
commands require only 2 keys to exist ('key' and 'value') and allow a
list of such key value pairs as the details field.

I wasn't able to find anything in the documentation that would make it
clear which encoding was required for each command and had to look at
cloudstack's code to figure this out.

Here is a list of the commands singled out in this commit and references
to the required encoding in cloudstack's code:

addGuestOs
https://github.com/apache/cloudstack/blob/87c43501608a1df72a2f01ed17a522233e6617b0/api/src/main/java/org/apache/cloudstack/api/command/admin/guest/AddGuestOsCmd.java#L88-L89

updateGuestOs
https://github.com/apache/cloudstack/blob/87c43501608a1df72a2f01ed17a522233e6617b0/api/src/main/java/org/apache/cloudstack/api/command/admin/guest/UpdateGuestOsCmd.java#L80-L81

addImageStore
https://github.com/apache/cloudstack/blob/87c43501608a1df72a2f01ed17a522233e6617b0/api/src/main/java/org/apache/cloudstack/api/command/admin/storage/AddImageStoreCmd.java#L90-L91

createSecondaryStagingStore
https://github.com/apache/cloudstack/blob/87c43501608a1df72a2f01ed17a522233e6617b0/api/src/main/java/org/apache/cloudstack/api/command/admin/storage/CreateSecondaryStagingStoreCmd.java#L80

updateCloudToUseObjectStore
https://github.com/apache/cloudstack/blob/87c43501608a1df72a2f01ed17a522233e6617b0/api/src/main/java/org/apache/cloudstack/api/command/admin/storage/UpdateCloudToUseObjectStoreCmd.java#L84-L85

addResourceDetail
https://github.com/apache/cloudstack/blob/87c43501608a1df72a2f01ed17a522233e6617b0/api/src/main/java/org/apache/cloudstack/api/command/user/volume/AddResourceDetailCmd.java#L68-L69

updateZone
https://github.com/apache/cloudstack/blob/87c43501608a1df72a2f01ed17a522233e6617b0/server/src/main/java/com/cloud/configuration/ConfigurationManagerImpl.java#L1866-L1867

Merge pull request #116 from xanzy/svh/f-http-client

Introduce optional client configuration options

Introduce optional client configuration options

Merge pull request #114 from xanzy/svh/b-float64

Fix the type check to test for float64

Fix the type check to test for float64

Merge pull request #113 from xanzy/svh/f-cs-4.12

Update to support CloudStack 4.12.x.x

Update to support CloudStack 4.12.x.x

Merge pull request #112 from xanzy/svh/f-with-domain

Add a `WithDomain` helper to set `domainid` fields

Add a `WithDomain` helper to set `domainid` fields

Merge pull request #108 from xanzy/svh/b-http-client

Fix the transport so connections are managed correctly

Fix the transport so connections are managed correctly

Merge pull request #107 from xanzy/f-4.11.2.0

Update to support CloudStack 4.11

Update to support CloudStack 4.11

And fix a few minor issues caused by PR #106

Avoid embedded structs by generating new struct types (#106)

These changes are based on PR #104 contributed by @cezarsa! The result it the same, but the implementation is slightly different.

Thank you very much for your PR @cezarsa!

Closes #104

Fixed typo within README.md (#102)

Update secondarystoragetotal field to float64 for Cloudstack 4.11.1.0 (#101)

After Cloudstack 4.11.1.0 the secondarystoragetotal field was changed
from Long to Float in Cloudstack in apache/cloudstack#2439.
Changing this in the generated client seems safe and backward compatible
because long values will still be parsed correctly as a float64.

Change synchronous boolean response to bool (#99)

The `success` fields in newer versions of CloudStack will use a `bool` value instead of a `string`, matching the behavious of asynchronious responses.

To make sure this change is backwards compatible the related types now have a custom `UnmarshalJSON` method that unmarshals the data (string or bool) into the new bool field.

Make sure we treat `OutOfBandManagementResponse` correctly (#97)

Add support for the login command

Update the generated code using the correct layout

Update layout to match CS 4.9.3

Update the generated code

Add option to generate code from a custom input file

Generation updates:

- Allowing generation based on local "listApis" response
- Adding flag to prevent ",omitempty" on generated response models
- Changing "set" type from "[]string" to "[]interface{}"
- Adding "massageServices" function to facilitate manual cleanup of
  somewhat...loose listApis response from cloudstack.
- Updating "cloudstack" package based on 4.8 api response

Update to support CloudStack 4.9.3 (#91)

Update encoding of 'details' maps (#86)

Update encoding of 'details' maps

Allow setting default options (#85)

Using it looks like this:

```
// Create a new API client
cs := cloudstack.NewAsyncClient("https://cloudstack.company.com", "your-api-key", "your-api-secret", false)

// Set an option
cs.DefaultOptions(WithProject(“project”))

// Or set multiple options
cs.DefautlOptions(WithProject(“project”), WithVPCID(“xxxx”))
```

Tweak the CustomService (#83)

adds support for custom services (#82)

Fixed responses for create user and register user keys (#81)

Correctly unmarshal the `CreateAccountResponse` (#80)

Fixes #78

Add an example of how to use the package (#74)

Make sure the FirewallService works with old CS version (#72)

Fix the authorizeSecurityGroup[In|E]gress responses (#69)

Fix/improve #66 (#68)

Fixing it this way makes it cleaner to use/set the `usersecuritygrouplist` in clients using this, and keeps it simular in use compared to working with `serviceproviderlist` and `tags`.

Fixes #67

Fix usersecuritygrouplist parameters (#66)

Fix a bug in the createSecurityGroup call (#62) (#63)

Fix a bug in the listAffinityGroup call (#61)

It reports the number of VirtualMachines in the groups as being the number of groups found.

Fix a bug that created bad parameter names (#60)

When the parameter is called `type` we have to convert this to prevent using the reserved keyword `type` as a variable name. But we were doing it in a few other locations as well. That’s fixed now.

Add a copy of the license file to the cloudstack package

Fix the ListLoadBalancerRuleInstances response (#59)

This one can have two possible response types, depending on the query values used. By adding this, both types are supported.

Improver helper function to retrieve ID (#58)

Merge pull request #56 from svanharmelen/f-list-with-vpcid-option

Add an option to search with VPC ID

Add an option to search with VPC ID

Merge pull request #55 from svanharmelen/b-loadbalancerruleinstance-response

Fix issue #54

Fix issue #54

This fixes issue #54 by adding some specific tweaks needed to handle
the `listLoadBalancerRuleInstances` response correctly.

Unfortunately the CloudStack API has a few strange corner
cases/inconsistencies that requires specific handling of certain calls.

Merge pull request #53 from svanharmelen/f-project-support

Add better support for using projects with the helper functions

Add better support for using projects with the helper functions

Merge pull request #51 from svanharmelen/f-v50

Change repo layout and updated to latest version

Make sure the FirewallService remains backwards compatible

Generate packages based on latest API output

Also cleanup the old packages and make sure the the current package is
backwards compatible.

Merge pull request #50 from Carles-Figuerola/f-createnetworkoffering

Add a special use case for CreateNetworkOffering

Add a special use case for CreateNetworkOffering

Merge pull request #49 from svanharmelen/dev

Introduce a way to have specific key/value names

Introduce a way to have specific key/value names

This is an alternative solution for PR #47.

By using this approach we don’t have to make any changes to the raw
output generated by the `listApis` API call (which is saved in the
`v43.go` and `v44.go` files).

So when we now want to generate code based on a new version, we only
have to paste the output of the `listApis` call info a new `vxx.go`
file and were good to go, without having to manually alter the file.

Next to all that’ it simple feels a little cleaner and more robust
:wink:

Merge pull request #48 from svanharmelen/dev

Add a special use case for CreateServiceOffering

Add a special use case for CreateServiceOffering

I tried to make this a more generic fix as it seems more API’s have
this pattern, but it seems CloudStack is just too inconsistent for this.

Fixes #44

Merge pull request #45 from svanharmelen/Dev

Fix listAsyncJobs

Fix listAsyncJobs

Merge pull request #43 from svanharmelen/dev

Tweak the async logic

Add a simple kind of exponential backoff feature for async clients

Improve async job handling and increase default timeout

Merge pull request #42 from svanharmelen/dev

Extend the ISO helper functions

Extend the ISO helper functions

Merge pull request #41 from svanharmelen/dev

Generate new code form files instead of API calls

Generate new code form files instead of API calls

This release adds the possibility to generate updated packages based on
the files in this repo instead of needing actual access to a certain
CloudStack environment.

Merge pull request #40 from svanharmelen/dev

Make closing the body a little more robust

Make closing the body a little more robust

And with that also a little more idiomatic

Merge pull request #39 from benjvi/http-timeout-and-retry

Added http request timeout & queryAsyncJobResult retry

http request timeout & queryasynjob retry

Merge pull request #38 from benjvi/list-apis-v43

add listApis response for v43

add listApis response for v43

Merge pull request #37 from svanharmelen/dev

Add listApis response for v44

Add listApis response for v44

Merge pull request #36 from glenjamin/master

Capitalise the k in apiKey, as some api handlers require this

Capitalise the k in apiKey, as some api handlers require this

Merge pull request #34 from svanharmelen/dev

Adding a special case for secondary IPs

Adding a special case for secondary IPs

Merge pull request #33 from svanharmelen/dev

Export the HTTPGETOnly field for direct access

Export the HTTPGETOnly field for direct access

As you would need both read and write access, it makes sense to just
export the field instead of writing a getter and setter.

Merge pull request #32 from svanharmelen/dev

Add option to only use HTTP GET calls

Add option to only use HTTP GET calls

Merge pull request #29 from benjvi/update-cloudstack-43

Generate v4.3 code for getID projects update

gen. v4.3 code for id projects update

Merge pull request #28 from svanharmelen/dev

Make the GetXXXID also search projects

Make the GetXXXID also search projects

Merge pull request #27 from svanharmelen/dev

Generated packages based on latests fixes...

Generated packages based on latests fixes...

Merge pull request #26 from benjvi/listNetworks

Fixes for API requests that contain maps and lists

dont encrypt param keys when making cloudstack requests

Generate cloudstack43 code

Remove extraneous space breaking list values in requests

Merge pull request #25 from svanharmelen/dev

Generated the packages using on the last updates

Generated the packages using on the last updates

Merge pull request #24 from benjvi/postuserdataupdate

updateVirtualMachine should use POST

Merge pull request #23 from benjvi/project-generate-cloudstack43

Generate cloudstack43 services including latest updates

Generate cloudstack43 services including latest updates