+1 thanks Giles.
For the API we could also update the API docs descriptions for methods,
parameters, and response fields (even though we can end up with: parameter:
virtualmachineid and description: ‘Instance ID’ for example)
Regards,
Nicolas Vazquez
From: Marco Sinhoreli
Date: Friday, 9 Jun
rohityadavcloud commented on PR #62:
URL:
https://github.com/apache/cloudstack-terraform-provider/pull/62#issuecomment-1584903704
@Z-eeshan can you check the build failures? (probably a go fmt fix?)
--
This is an automated message from the Apache Git Service.
To respond to the message, pl
+1 to use “Instance” in the UI and docs. Everyone knows what " Instance " is,
in my view, just a label to refer to an object in ACS. As Rohit said, it is
under Compute, then it refers to a Compute Instance.
From: Giles Sirett
Date: Thursday, 8 June 2023 at 16:46
To: dev@cloudstack.apache.org
S
rohityadavcloud commented on PR #62:
URL:
https://github.com/apache/cloudstack-terraform-provider/pull/62#issuecomment-1584759497
Thanks for the PR @Z-eeshan I've pinged a few contributors to review.
--
This is an automated message from the Apache Git Service.
To respond to the message, p
rohityadavcloud commented on PR #60:
URL:
https://github.com/apache/cloudstack-terraform-provider/pull/60#issuecomment-1584759050
Thanks for the PR @FarnazBGH I still see the error from the CI:
```==> Checking that code complies with gofmt requirements...
make: *** [GNUmakefile:32: fmt
Giles, just about point one as the others follow from it.
On Fri, Jun 9, 2023 at 10:17 AM Giles Sirett wrote:
>
> Hi Daan - thanks for your input. Some comments inline below
>
>
>
> Kind Regards
> Giles
>
>
>
>
> -Original Message-
> From: Daan Hoogland
> Sent: Thursday, June 8, 2023 4:1
DaanHoogland commented on code in PR #315:
URL:
https://github.com/apache/cloudstack-documentation/pull/315#discussion_r1224018203
##
source/plugins/cloudstack-kubernetes-service.rst:
##
@@ -185,22 +187,24 @@ New Kubernetes clusters can be created using the API or
via the UI.
Hi Daan - thanks for your input. Some comments inline below
Kind Regards
Giles
-Original Message-
From: Daan Hoogland
Sent: Thursday, June 8, 2023 4:17 PM
To: dev@cloudstack.apache.org
Subject: Re: [proposal] Consistency of naming in Cloudstack
Giles, the principle of what you ar
+1
Agree with the approach, Vishesh.
-Original Message-
From: Wei ZHOU
Sent: Tuesday, June 6, 2023 8:11 PM
To: dev@cloudstack.apache.org
Subject: Re: [DISCUSS] Upgrading Mockito & phasing out powermock
lgtm. go ahead Vishesh.
-Wei
On Tue, 6 Jun 2023 at 14:17, Vishesh Jindal
wrot
