Skip to content
This repository has been archived by the owner on Jul 26, 2021. It is now read-only.

Guest ostypeid considered as int64 instead of string #117

Closed
rohityadavcloud opened this issue Jun 14, 2019 · 4 comments
Closed

Guest ostypeid considered as int64 instead of string #117

rohityadavcloud opened this issue Jun 14, 2019 · 4 comments

Comments

@rohityadavcloud
Copy link

rohityadavcloud commented Jun 14, 2019

This is used as dependency for packer, and the following issue was hit while using packer:

cloudstack: Error creating new instance packer-5d0367bd-d45c-0760-b976-6f9a7358c9d0: json: cannot unmarshal string into Go struct field DeployVirtualMachineResponse.ostypeid of type int64

The issue finds its root in recent changes where the guest os type ID was correclty changed from integer to string (so we export UUID and not internal cloudstack ids). Moving the usage throughout go-cloudstack may be acceptable:

Issue reference:
apache/cloudstack#2528

Commit reference:
apache/cloudstack@44bc516

@rohityadavcloud
Copy link
Author

cc @svanharmelen @PaulAngus @andrijapanic @DaanHoogland @shwstppr @anuragaw

@svanharmelen
Copy link
Member

svanharmelen commented Jun 14, 2019

Not sure why you opened this issue here? I already updated and fixed this in this code base and a PR to update this in Packer is also merged, but not yet released in a new version.

@rohityadavcloud
Copy link
Author

@svanharmelen - I'm not aware of the issues you've already fixed, this came from a support issue. I'll close the issue on your remark. Thanks.

@svanharmelen
Copy link
Member

Al good, thanks!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants