[GitHub] [cloudstack-documentation] rohityadavcloud opened a new pull request, #337: hosts: make it clear what to expect and how to configure balloon feature on KVM

2023-08-04 Thread via GitHub
rohityadavcloud opened a new pull request, #337: URL: https://github.com/apache/cloudstack-documentation/pull/337 (no comment) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment

[GitHub] [cloudstack-documentation] rohityadavcloud commented on pull request #337: hosts: make it clear what to expect and how to configure balloon feature on KVM

2023-08-04 Thread via GitHub
rohityadavcloud commented on PR #337: URL: https://github.com/apache/cloudstack-documentation/pull/337#issuecomment-1665166007 @blueorangutan docbuild -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to g

[GitHub] [cloudstack-documentation] blueorangutan commented on pull request #337: hosts: make it clear what to expect and how to configure balloon feature on KVM

2023-08-04 Thread via GitHub
blueorangutan commented on PR #337: URL: https://github.com/apache/cloudstack-documentation/pull/337#issuecomment-1665166983 @rohityadavcloud a Jenkins job has been kicked to build the document. I'll keep you posted as I make progress. -- This is an automated message from the Apache Git

[GitHub] [cloudstack-documentation] blueorangutan commented on pull request #337: hosts: make it clear what to expect and how to configure balloon feature on KVM

2023-08-04 Thread via GitHub
blueorangutan commented on PR #337: URL: https://github.com/apache/cloudstack-documentation/pull/337#issuecomment-1665168115 QA-Doc build preview: https://qa.cloudstack.cloud/builds/docs-build/pr/337. (QA-JID 105) -- This is an automated message from the Apache Git Service. To respond to

[GitHub] [cloudstack-go] rohityadavcloud merged pull request #59: Add support for unmanaged kubernetes

2023-08-04 Thread via GitHub
rohityadavcloud merged PR #59: URL: https://github.com/apache/cloudstack-go/pull/59 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: dev-unsubscr...@clou

[GitHub] [cloudstack-go] rohityadavcloud merged pull request #65: Link to most current API docs 4.18

2023-08-04 Thread via GitHub
rohityadavcloud merged PR #65: URL: https://github.com/apache/cloudstack-go/pull/65 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: dev-unsubscr...@clou

[GitHub] [cloudstack-go] rohityadavcloud commented on a diff in pull request #64: Fix spelling

2023-08-04 Thread via GitHub
rohityadavcloud commented on code in PR #64: URL: https://github.com/apache/cloudstack-go/pull/64#discussion_r1284113554 ## generate/listApis.json: ## @@ -2059,7 +2059,7 @@ "type": "long" }, { - "description": "the ID of the availablility zo

[GitHub] [cloudstack-documentation] rohityadavcloud merged pull request #337: hosts: make it clear what to expect and how to configure balloon feature on KVM

2023-08-04 Thread via GitHub
rohityadavcloud merged PR #337: URL: https://github.com/apache/cloudstack-documentation/pull/337 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: dev-uns

CloudStack India User Group Meet Up

2023-08-04 Thread Ivet Petrova
Hi all, Reminder that we will be having a user group meetup on August 18th in Bangalore, India. You can register for the event here: https://www.meetup.com/india-cloudstack-user-group/events/294738405/ Also we have an open CFP which will be active until next Friday, If you would like to join wi

[GitHub] [cloudstack-terraform-provider] fabiomatavelli opened a new pull request, #64: Improves service offering resource

2023-08-04 Thread via GitHub
fabiomatavelli opened a new pull request, #64: URL: https://github.com/apache/cloudstack-terraform-provider/pull/64 Hi, Here is some improvement in the `cloudstack_service_offering` resource so we can specify the `cpu_number`, `cpu_speed` and `memory` now. I've added the accept

[GitHub] [cloudstack-terraform-provider] rohityadavcloud commented on a diff in pull request #54: Added contribute document

2023-08-04 Thread via GitHub
rohityadavcloud commented on code in PR #54: URL: https://github.com/apache/cloudstack-terraform-provider/pull/54#discussion_r1284607437 ## CONTRIBUTING.md: ## @@ -0,0 +1,139 @@ +Contributing to Apache CloudStack Terraform Provider (ACS) +===

[GitHub] [cloudstack-terraform-provider] rohityadavcloud merged pull request #54: Added contribute document

2023-08-04 Thread via GitHub
rohityadavcloud merged PR #54: URL: https://github.com/apache/cloudstack-terraform-provider/pull/54 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: dev-

[GitHub] [cloudstack-terraform-provider] rohityadavcloud merged pull request #53: Added correct docker and terraform configuration command

2023-08-04 Thread via GitHub
rohityadavcloud merged PR #53: URL: https://github.com/apache/cloudstack-terraform-provider/pull/53 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: dev-

[GitHub] [cloudstack-terraform-provider] rohityadavcloud commented on pull request #53: Added correct docker and terraform configuration command

2023-08-04 Thread via GitHub
rohityadavcloud commented on PR #53: URL: https://github.com/apache/cloudstack-terraform-provider/pull/53#issuecomment-1665862763 If there are any new changes @kiranchavala pl submit as separate PR cc @harikrishna-patnala -- This is an automated message from the Apache Git Service. To r

[GitHub] [cloudstack-terraform-provider] rohityadavcloud commented on issue #63: project status

2023-08-04 Thread via GitHub
rohityadavcloud commented on issue #63: URL: https://github.com/apache/cloudstack-terraform-provider/issues/63#issuecomment-1665864654 I've created a new milestone to track the next release - https://github.com/apache/cloudstack-terraform-provider/milestone/2 Closing on the remark th

[GitHub] [cloudstack-terraform-provider] rohityadavcloud commented on a diff in pull request #64: Improves service offering resource

2023-08-04 Thread via GitHub
rohityadavcloud commented on code in PR #64: URL: https://github.com/apache/cloudstack-terraform-provider/pull/64#discussion_r1284617395 ## .github/workflows/testacc.yml: ## @@ -0,0 +1,63 @@ +name: Acceptance Test Review Comment: Could you add license header (see other file

[GitHub] [cloudstack-terraform-provider] rohityadavcloud commented on pull request #64: Improves service offering resource

2023-08-04 Thread via GitHub
rohityadavcloud commented on PR #64: URL: https://github.com/apache/cloudstack-terraform-provider/pull/64#issuecomment-1665870084 @fabiomatavelli thank you for the PR. I liked how you've introduce a CI github action. There is a conflict with README, could you help resolve it. -- This is

[GitHub] [cloudstack-terraform-provider] rohityadavcloud closed pull request #57: Adding configuration resource

2023-08-04 Thread via GitHub
rohityadavcloud closed pull request #57: Adding configuration resource URL: https://github.com/apache/cloudstack-terraform-provider/pull/57 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specifi

[GitHub] [cloudstack-terraform-provider] rohityadavcloud commented on pull request #57: Adding configuration resource

2023-08-04 Thread via GitHub
rohityadavcloud commented on PR #57: URL: https://github.com/apache/cloudstack-terraform-provider/pull/57#issuecomment-1665871233 @poddm thanks for the PR, do we need to use the latest go-sdk release (https://github.com/apache/cloudstack-go/releases) ? LGTM havn't tested it though. (closed

[GitHub] [cloudstack-go] rohityadavcloud commented on pull request #37: WIP: (#36) - support async client

2023-08-04 Thread via GitHub
rohityadavcloud commented on PR #37: URL: https://github.com/apache/cloudstack-go/pull/37#issuecomment-1665872177 ping @tetra12 @davidjumani - any update on this? thanks. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[GitHub] [cloudstack-cloudmonkey] rohityadavcloud opened a new issue, #138: In CLI mode, cmk calls sync and doesn't cache it

2023-08-04 Thread via GitHub
rohityadavcloud opened a new issue, #138: URL: https://github.com/apache/cloudstack-cloudmonkey/issues/138 In CLI mode, cmk calls sync command before running the passed arg; this may not be necessary if we can explore the heuristics. Use case; run cmk by passing API url, secret and ap

[GitHub] [cloudstack-terraform-provider] fabiomatavelli commented on a diff in pull request #64: Improves service offering resource

2023-08-04 Thread via GitHub
fabiomatavelli commented on code in PR #64: URL: https://github.com/apache/cloudstack-terraform-provider/pull/64#discussion_r1284630436 ## .github/workflows/testacc.yml: ## @@ -0,0 +1,63 @@ +name: Acceptance Test Review Comment: done -- This is an automated message from

[GitHub] [cloudstack-terraform-provider] fabiomatavelli commented on pull request #64: Improves service offering resource

2023-08-04 Thread via GitHub
fabiomatavelli commented on PR #64: URL: https://github.com/apache/cloudstack-terraform-provider/pull/64#issuecomment-1665884527 changes applied @rohityadavcloud -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the UR

[GitHub] [cloudstack-go] poddm opened a new issue, #66: CreateZoneResponse empty

2023-08-04 Thread via GitHub
poddm opened a new issue, #66: URL: https://github.com/apache/cloudstack-go/issues/66 CreateZone always returns an empty response. The struct should be nested with the "zone" key. ```go // Create zone r, err := cs.Zone.CreateZone(p) ``` Example response from API

[GitHub] [cloudstack-go] poddm commented on issue #66: CreateZoneResponse empty

2023-08-04 Thread via GitHub
poddm commented on issue #66: URL: https://github.com/apache/cloudstack-go/issues/66#issuecomment-1665897322 I believe the CreateZoneResponse should be updated to this. I'm trying to understand the generator to submit a PR. ```go type CreateZoneResponse struct { Zone struc

[GitHub] [cloudstack-go] jbampton opened a new pull request, #67: README: change latest version to 4.18

2023-08-04 Thread via GitHub
jbampton opened a new pull request, #67: URL: https://github.com/apache/cloudstack-go/pull/67 (no comment) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e

[GitHub] [cloudstack-go] rohityadavcloud merged pull request #67: README: change latest version to 4.18

2023-08-04 Thread via GitHub
rohityadavcloud merged PR #67: URL: https://github.com/apache/cloudstack-go/pull/67 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: dev-unsubscr...@clou

[GitHub] [cloudstack-go] poddm commented on issue #66: CreateZoneResponse empty

2023-08-04 Thread via GitHub
poddm commented on issue #66: URL: https://github.com/apache/cloudstack-go/issues/66#issuecomment-1666004967 @Pearl1594, that resolves the issue. Thanks. ```go // Generic function to get the first non-count raw value from a response as json.RawMessage func getRawValue(b jso

[GitHub] [cloudstack-go] soreana commented on pull request #64: Fix spelling

2023-08-04 Thread via GitHub
soreana commented on PR #64: URL: https://github.com/apache/cloudstack-go/pull/64#issuecomment-1666013772 @rohityadavcloud I will take care of this. I'm updating the domainid description, I would add these as well. Checkout the following pr please: https://github.com/apache/cloudst

[GitHub] [cloudstack-go] soreana commented on pull request #64: Fix spelling

2023-08-04 Thread via GitHub
soreana commented on PR #64: URL: https://github.com/apache/cloudstack-go/pull/64#issuecomment-1666017593 Btw, I don't have a permission to assign PRs in this repository, feel free to assign this PR to me. -- This is an automated message from the Apache Git Service. To respond to the mess

[GitHub] [cloudstack-go] rohityadavcloud merged pull request #68: Adding getRawValue to CreateZone api

2023-08-04 Thread via GitHub
rohityadavcloud merged PR #68: URL: https://github.com/apache/cloudstack-go/pull/68 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: dev-unsubscr...@clou

[GitHub] [cloudstack-go] rohityadavcloud closed issue #66: CreateZoneResponse empty

2023-08-04 Thread via GitHub
rohityadavcloud closed issue #66: CreateZoneResponse empty URL: https://github.com/apache/cloudstack-go/issues/66 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe

[GitHub] [cloudstack-terraform-provider] poddm opened a new pull request, #65: Enhancing zone resource

2023-08-04 Thread via GitHub
poddm opened a new pull request, #65: URL: https://github.com/apache/cloudstack-terraform-provider/pull/65 Updating Zone resource - Added all fields - set terraform id as zone id instead of zone name. - added updated Note: this requires the latest cloudstack-go client.

[GitHub] [cloudstack-terraform-provider] poddm opened a new issue, #66: Terraform plugin SDK update

2023-08-04 Thread via GitHub
poddm opened a new issue, #66: URL: https://github.com/apache/cloudstack-terraform-provider/issues/66 Looks like cloudstack is using the "legacy Terraform plugin SDK". To ensure terraform core compatibility we should look at upgrading to SDK v2. Current: `github.com/hashicorp/terraf