Re: [PR] Add `acl_id` parameter to the example usage of the `cloudstack_private_gateway` resource [cloudstack-terraform-provider]

2025-08-12 Thread via GitHub
DaanHoogland commented on PR #180: URL: https://github.com/apache/cloudstack-terraform-provider/pull/180#issuecomment-3178429358 > > nothing to do with this PR especially, but I noticed there is no license in the file and no error pertaining to that is reported. > > Is there an exclu

Re: [PR] access: prevent login with username-password when using wrong keys [cloudstack-cloudmonkey]

2025-08-12 Thread via GitHub
DaanHoogland closed pull request #174: access: prevent login with username-password when using wrong keys URL: https://github.com/apache/cloudstack-cloudmonkey/pull/174 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

Re: [PR] github-action: add ci workflow [cloudstack-cloudmonkey]

2025-08-12 Thread via GitHub
shwstppr commented on PR #180: URL: https://github.com/apache/cloudstack-cloudmonkey/pull/180#issuecomment-3178664082 > clgtm, I just wonder if we should invest in creating the zone with cmk as well and leave marvin out of the picture completely. I would not want to do that at the mo

Re: [VOTE] Apache CloudStack 4.21.0.0 (RC3)

2025-08-12 Thread Rohit Yadav
+1 (binding) Tested instance, volume, snapshot, template, iso, network lifecycles in an EL8/KVM environment, along with new features and support for tpm/windows11. Also tested the built-in extensions with Proxmox VE 9.0 and Hyper-V on Windows Server 2019 and custom actions. The setup for both H

Re: [PR] access: prevent login with username-password when using wrong keys [cloudstack-cloudmonkey]

2025-08-12 Thread via GitHub
shwstppr commented on PR #174: URL: https://github.com/apache/cloudstack-cloudmonkey/pull/174#issuecomment-3178655635 @DaanHoogland I think this needs a bit more discussion. I agree that if valid credentials are available, any invalid ones should be ignored. However, the use case @ingox ra

Re: [PR] access: prevent login with username-password when using wrong keys [cloudstack-cloudmonkey]

2025-08-12 Thread via GitHub
DaanHoogland commented on PR #174: URL: https://github.com/apache/cloudstack-cloudmonkey/pull/174#issuecomment-3178553661 @shwstppr work kind of like expected. I have one functional concern though: ``` (randy) 🐱 > sync Discovered 877 APIs (randy) 🐱 > set apikey (randy) 🐱 > sy

Re: [PR] access: prevent login with username-password when using wrong keys [cloudstack-cloudmonkey]

2025-08-12 Thread via GitHub
DaanHoogland commented on PR #174: URL: https://github.com/apache/cloudstack-cloudmonkey/pull/174#issuecomment-3178923237 @ingox can you read https://github.com/apache/cloudstack-cloudmonkey/pull/174#issuecomment-3178553661 and https://github.com/apache/cloudstack-cloudmonkey/pull/174#iss

Re: [PR] Fix usagetype parameter autocomplete [cloudstack-cloudmonkey]

2025-08-12 Thread via GitHub
DaanHoogland commented on PR #156: URL: https://github.com/apache/cloudstack-cloudmonkey/pull/156#issuecomment-3179129938 @hsato03 (cc @winterhazel ) can you update on this? It would be nice to include it in an upcoming release. -- This is an automated message from the Apache Git Servic

Re: [PR] GitHub Action for RAT check [cloudstack-cloudmonkey]

2025-08-12 Thread via GitHub
DaanHoogland commented on code in PR #173: URL: https://github.com/apache/cloudstack-cloudmonkey/pull/173#discussion_r2269566565 ## .github/workflows/rat.yaml: ## @@ -0,0 +1,36 @@ +name: RAT Check + +on: + push: +branches: + - main + pull_request: +branches: +

Re: [PR] GitHub Action for RAT check [cloudstack-cloudmonkey]

2025-08-12 Thread via GitHub
shwstppr commented on PR #173: URL: https://github.com/apache/cloudstack-cloudmonkey/pull/173#issuecomment-3179036529 @DaanHoogland do we have a negative test (like valid go file missing license) somewhere? Otherwise looks good -- This is an automated message from the Apache Git Serv

Re: [PR] github-action: add ci workflow [cloudstack-cloudmonkey]

2025-08-12 Thread via GitHub
DaanHoogland commented on PR #180: URL: https://github.com/apache/cloudstack-cloudmonkey/pull/180#issuecomment-3178901896 > > clgtm, I just wonder if we should invest in creating the zone with cmk as well and leave marvin out of the picture completely. > > I would not want to do that

Re: [VOTE] Apache CloudStack 4.21.0.0 (RC3)

2025-08-12 Thread Rohit Yadav
Tata - the extensions framework and built-in extensions features ship with the upcoming ACS 4.21.0.0, and not 4.20.1.0 (4.20.1.0 is already GA). Regards. From: Chi vediamo Sent: Tuesday, August 12, 2025 16:46 To: [email protected] Cc: dev Subje

Re: [PR] GitHub Action for RAT check [cloudstack-cloudmonkey]

2025-08-12 Thread via GitHub
DaanHoogland commented on code in PR #173: URL: https://github.com/apache/cloudstack-cloudmonkey/pull/173#discussion_r2269564271 ## .github/workflows/rat.yaml: ## @@ -0,0 +1,36 @@ +name: RAT Check + +on: + push: +branches: + - main + pull_request: +branches: +

[PR] compat: clarify compatibility matrix for ACS 4.20+ [cloudstack-documentation]

2025-08-12 Thread via GitHub
rohityadavcloud opened a new pull request, #537: URL: https://github.com/apache/cloudstack-documentation/pull/537 (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

Re: [PR] Update rat.yaml [cloudstack-cloudmonkey]

2025-08-12 Thread via GitHub
github-actions[bot] commented on PR #181: URL: https://github.com/apache/cloudstack-cloudmonkey/pull/181#issuecomment-3179259278 ✅ Build complete for PR #181. 🔗 Download the [cmk binaries](https://github.com/apache/cloudstack-cloudmonkey/actions/runs/16909736174/artifacts/3744840

Re: [PR] rat cli check and exclude list [cloudstack-go]

2025-08-12 Thread via GitHub
DaanHoogland commented on code in PR #116: URL: https://github.com/apache/cloudstack-go/pull/116#discussion_r2269789783 ## .rat-excludes: ## @@ -0,0 +1,3 @@ +.rat-excludes +go.mod +go.sum Review Comment: ```suggestion .rat-excludes go.mod go.sum rat-report.txt

Re: [PR] GitHub Action for RAT check [cloudstack-cloudmonkey]

2025-08-12 Thread via GitHub
DaanHoogland commented on PR #173: URL: https://github.com/apache/cloudstack-cloudmonkey/pull/173#issuecomment-3178964011 @weizhouapache @shwstppr , I finally got my head around the excludes syntax. can you guys have a look? If alright I will create similar PRs for other cloudstack side pr

Re: [I] Credential fall back [cloudstack-cloudmonkey]

2025-08-12 Thread via GitHub
shwstppr commented on issue #168: URL: https://github.com/apache/cloudstack-cloudmonkey/issues/168#issuecomment-3179090489 Based on the discussion with @DaanHoogland and @ingox the behaviour can be tweaked to the following: - When credentials are passed as command-line parameters, th

[PR] rat cli check and exclude list [cloudstack-go]

2025-08-12 Thread via GitHub
DaanHoogland opened a new pull request, #116: URL: https://github.com/apache/cloudstack-go/pull/116 (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 unsubscr

[PR] Update rat.yaml [cloudstack-cloudmonkey]

2025-08-12 Thread via GitHub
DaanHoogland opened a new pull request, #181: URL: https://github.com/apache/cloudstack-cloudmonkey/pull/181 eat your own ... -- 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.

[PR] rat + excludes and add licenses to other files [cloudstack-terraform-provider]

2025-08-12 Thread via GitHub
DaanHoogland opened a new pull request, #200: URL: https://github.com/apache/cloudstack-terraform-provider/pull/200 (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 comme

Re: [PR] rat + excludes and add licenses to other files [cloudstack-terraform-provider]

2025-08-12 Thread via GitHub
DaanHoogland commented on code in PR #200: URL: https://github.com/apache/cloudstack-terraform-provider/pull/200#discussion_r2269956604 ## .github/workflows/rat.yaml: ## @@ -0,0 +1,36 @@ +name: RAT Check + +on: + push: +branches: + - main + pull_request: +branche

Re: [I] feature/featch all pages from api request [cloudstack-cloudmonkey]

2025-08-12 Thread via GitHub
DaanHoogland commented on issue #153: URL: https://github.com/apache/cloudstack-cloudmonkey/issues/153#issuecomment-3179193662 @shwstppr would this require service changes for some APIs ? I recall the cap of 500 being an issue in the UI as well. -- This is an automated message from the A

Re: [I] feature/featch all pages from api request [cloudstack-cloudmonkey]

2025-08-12 Thread via GitHub
shwstppr commented on issue #153: URL: https://github.com/apache/cloudstack-cloudmonkey/issues/153#issuecomment-3179353923 @DaanHoogland I don't think so. I think one way to it can be done at cmk level itself. We keep fetching and storing in memory (not print) till count returned by API re

Re: [PR] github-action: add ci workflow [cloudstack-cloudmonkey]

2025-08-12 Thread via GitHub
DaanHoogland merged PR #180: URL: https://github.com/apache/cloudstack-cloudmonkey/pull/180 -- 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

Re: [PR] GitHub Action for RAT check [cloudstack-cloudmonkey]

2025-08-12 Thread via GitHub
DaanHoogland merged PR #173: URL: https://github.com/apache/cloudstack-cloudmonkey/pull/173 -- 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

Re: CMK 6.5.0 milestone/release

2025-08-12 Thread Abhishek Kumar
Update on the milestone/release: Currently, we've 5 open items (3 PRs, 2 issues) in the milestone [1] and we can aim to close or merge them by the end of this week. Depending on the CloudStack 4.21 release, we can aim to cut an RC next week (18-22 Aug). In the past two weeks, we’ve introduced som

Re: [PR] Update rat.yaml [cloudstack-cloudmonkey]

2025-08-12 Thread via GitHub
DaanHoogland merged PR #181: URL: https://github.com/apache/cloudstack-cloudmonkey/pull/181 -- 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

Re: [PR] Fix usagetype parameter autocomplete [cloudstack-cloudmonkey]

2025-08-12 Thread via GitHub
winterhazel commented on PR #156: URL: https://github.com/apache/cloudstack-cloudmonkey/pull/156#issuecomment-3181599452 > I have other usage and quota enabled but do not get any quota commands except for quota isenabled. What else might I be forgetting? @DaanHoogland did you restart

[I] Feature request: Add dedicated command to switch between existing profiles (avoid accidental creation) [cloudstack-cloudmonkey]

2025-08-12 Thread via GitHub
luganofer opened a new issue, #182: URL: https://github.com/apache/cloudstack-cloudmonkey/issues/182 Hi! I’ve been using CloudMonkey with multiple profiles and noticed a usability issue with the current set profile command. Right now, set profile serves both to create a new profi

[PR] Update GPU related docs [cloudstack-documentation]

2025-08-12 Thread via GitHub
vishesh92 opened a new pull request, #538: URL: https://github.com/apache/cloudstack-documentation/pull/538 (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

Re: [PR] feature: allow file upload for getuploadparams* apis [cloudstack-cloudmonkey]

2025-08-12 Thread via GitHub
shwstppr commented on PR #177: URL: https://github.com/apache/cloudstack-cloudmonkey/pull/177#issuecomment-3182227714 > @shwstppr can we add a file= or filepath= parameter for only cmk to: > > * check that file exists & is readable (mod bits...) > > * move this in the f

Re: [PR] Fix usagetype parameter autocomplete [cloudstack-cloudmonkey]

2025-08-12 Thread via GitHub
DaanHoogland commented on PR #156: URL: https://github.com/apache/cloudstack-cloudmonkey/pull/156#issuecomment-3182404156 right , > @hsato03 @DaanHoogland regarding the completion for `type` in `listUsageRecords`, I think that we can address it separately, as this issue is not exclu