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

2025-08-06 Thread via GitHub
DaanHoogland commented on PR #180: URL: https://github.com/apache/cloudstack-terraform-provider/pull/180#issuecomment-3157864075 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 exclusion somewhere

Re: [PR] Add `cloudstack_project` resource [cloudstack-terraform-provider]

2025-08-06 Thread via GitHub
Copilot commented on code in PR #167: URL: https://github.com/apache/cloudstack-terraform-provider/pull/167#discussion_r2256281128 ## cloudstack/resource_cloudstack_project.go: ## @@ -0,0 +1,533 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contr

Re: [PR] Add `cloudstack_limits` data source and resource [cloudstack-terraform-provider]

2025-08-06 Thread via GitHub
Copilot commented on code in PR #197: URL: https://github.com/apache/cloudstack-terraform-provider/pull/197#discussion_r2256282892 ## cloudstack/resource_cloudstack_limits.go: ## @@ -0,0 +1,317 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contri

Re: [PR] chore(deps): bump github.com/cloudflare/circl from 1.3.7 to 1.6.1 [cloudstack-terraform-provider]

2025-08-06 Thread via GitHub
DaanHoogland merged PR #188: URL: https://github.com/apache/cloudstack-terraform-provider/pull/188 -- 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-u

Re: [PR] Update kvm.rst - iptables before rule necessary for Ubuntu 24.04 LTS [cloudstack-documentation]

2025-08-06 Thread via GitHub
DaanHoogland merged PR #510: URL: https://github.com/apache/cloudstack-documentation/pull/510 -- 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-unsubs

Re: [PR] Update usage types in the docs [cloudstack-documentation]

2025-08-06 Thread via GitHub
DaanHoogland commented on PR #504: URL: https://github.com/apache/cloudstack-documentation/pull/504#issuecomment-3158290545 @abh1sar , > > quite frankly I am against this change. there is a list usage types API and it should document the usage types. adding/updating the docs on this

Re: [PR] removed area on documentation page menu left [cloudstack-documentation]

2025-08-06 Thread via GitHub
DaanHoogland merged PR #535: URL: https://github.com/apache/cloudstack-documentation/pull/535 -- 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-unsubs

Re: [PR] note on affinity in combination with vmware DRS [cloudstack-documentation]

2025-08-06 Thread via GitHub
DaanHoogland merged PR #527: URL: https://github.com/apache/cloudstack-documentation/pull/527 -- 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-unsubs

Re: [PR] removed duplicate CloudStack DRS feature from the list [cloudstack-documentation]

2025-08-06 Thread via GitHub
DaanHoogland merged PR #532: URL: https://github.com/apache/cloudstack-documentation/pull/532 -- 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-unsubs

Re: [PR] extention of documentation for flexible tagging [cloudstack-documentation]

2025-08-06 Thread via GitHub
DaanHoogland merged PR #536: URL: https://github.com/apache/cloudstack-documentation/pull/536 -- 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-unsubs

Re: [PR] Update Host LXC Installation page of Installation Guide [cloudstack-documentation]

2025-08-06 Thread via GitHub
DaanHoogland merged PR #528: URL: https://github.com/apache/cloudstack-documentation/pull/528 -- 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-unsubs

Re: [PR] Update Host LXC Installation page of Installation Guide [cloudstack-documentation]

2025-08-06 Thread via GitHub
DaanHoogland commented on PR #528: URL: https://github.com/apache/cloudstack-documentation/pull/528#issuecomment-3158340738 ok, let’s keep in mind that we might deprecate the support if we get a good extension for lxc working with the new extension framework, but for now we can merge. --

[PR] github-action: add pr builder for easier testing [cloudstack-cloudmonkey]

2025-08-06 Thread via GitHub
shwstppr opened a new pull request, #170: URL: https://github.com/apache/cloudstack-cloudmonkey/pull/170 Adds a Github action to build binary for Linux x86-64 for easier testing of PRs. A comment will be added in the PR. Tested here: https://github.com/shwstppr/cloudstack-cloudmonkey/

[PR] lint: fix failures [cloudstack-cloudmonkey]

2025-08-06 Thread via GitHub
shwstppr opened a new pull request, #171: URL: https://github.com/apache/cloudstack-cloudmonkey/pull/171 Fixes `make lint` -- 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. T

Re: [PR] Create new Instance from VM backup documentation [cloudstack-documentation]

2025-08-06 Thread via GitHub
DaanHoogland merged PR #474: URL: https://github.com/apache/cloudstack-documentation/pull/474 -- 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-unsubs

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

2025-08-06 Thread via GitHub
DaanHoogland commented on code in PR #173: URL: https://github.com/apache/cloudstack-cloudmonkey/pull/173#discussion_r2257031432 ## .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-06 Thread via GitHub
DaanHoogland commented on PR #173: URL: https://github.com/apache/cloudstack-cloudmonkey/pull/173#issuecomment-3160026406 I guess a make target that calls the rat is better... -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub a

Re: [PR] github-action: add pr builder for easier testing [cloudstack-cloudmonkey]

2025-08-06 Thread via GitHub
DaanHoogland commented on PR #170: URL: https://github.com/apache/cloudstack-cloudmonkey/pull/170#issuecomment-3160048446 @shwstppr how about adding `make dist` and attach the contents of the dist dir to the pr? in that way arm/windows users will have binaries to test as well. -- This is

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

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

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

2025-08-06 Thread via GitHub
shwstppr opened a new pull request, #174: URL: https://github.com/apache/cloudstack-cloudmonkey/pull/174 Fixes #168 -- 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 unsub

[PR] login: allow 2fa code input if mandated [cloudstack-cloudmonkey]

2025-08-06 Thread via GitHub
shwstppr opened a new pull request, #175: URL: https://github.com/apache/cloudstack-cloudmonkey/pull/175 Fixes #145 -- 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 unsub

Re: Issue while making fedora 38 and 41 template

2025-08-06 Thread Sanjay Kumar
Hi Nux, Thanks for quick response I follow below link of cloudstack and chatGpt instruction as well. https://docs.cloudstack.apache.org/en/latest/adminguide/templates/_cloud_init.html#linux-with-cloud-init On Wed, Aug 6, 2025 at 3:26 PM Nux wrote: > What instructions are you following? > >

Re: Issue while making fedora 38 and 41 template

2025-08-06 Thread Nux
What instructions are you following? On 2025-08-06 10:17, Sanjay Kumar wrote: Hello!, I was trying to make the fedora 38 and 41 template but cloud-init not working for root password generated by cloudstack. I follow the instruction like as centos steam 9 as it is working fine. How to make the

Re: [PR] config: fix access for config and history file [cloudstack-cloudmonkey]

2025-08-06 Thread via GitHub
shwstppr commented on code in PR #166: URL: https://github.com/apache/cloudstack-cloudmonkey/pull/166#discussion_r2256938071 ## config/config.go: ## @@ -391,6 +396,12 @@ func (c *Config) UpdateConfig(key string, value string, update bool) { } } +func makeFileGroupPr

Re: [PR] output: allow filtering and honour output format for non-list apis [cloudstack-cloudmonkey]

2025-08-06 Thread via GitHub
DaanHoogland commented on PR #164: URL: https://github.com/apache/cloudstack-cloudmonkey/pull/164#issuecomment-3159895332 retested: ``` (randy) 🐱 > destroy virtualmachine id=6983f65d-002f-4b39-abcd-101a87ed87bd filter=name,account,hostid,zonename, virtualmachine: +-

Re: Issue while making fedora 38 and 41 template

2025-08-06 Thread Nux
And what exactly isn't working? What did you try and has failed? On 2025-08-06 11:03, Sanjay Kumar wrote: Hi Nux, Thanks for quick response I follow below link of cloudstack and chatGpt instruction as well. https://docs.cloudstack.apache.org/en/latest/adminguide/templates/_cloud_init.html#l

Re: [PR] Added health checks and monitoring to optional installation [cloudstack-documentation]

2025-08-06 Thread via GitHub
Pearl1594 commented on code in PR #496: URL: https://github.com/apache/cloudstack-documentation/pull/496#discussion_r2256809985 ## source/installguide/optional_installation.rst: ## @@ -114,6 +114,39 @@ For storing certificates, admins can create and configure a java keystore f

Re: [I] test and lint fail [cloudstack-cloudmonkey]

2025-08-06 Thread via GitHub
shwstppr closed issue #169: test and lint fail URL: https://github.com/apache/cloudstack-cloudmonkey/issues/169 -- 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,

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

2025-08-06 Thread via GitHub
shwstppr commented on issue #168: URL: https://github.com/apache/cloudstack-cloudmonkey/issues/168#issuecomment-3159907563 @ingox I think this can be prevented if you can use different profiles with `set profile`, ie, before setting apikey/secrekey, set a new profile -- This is an automa

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

2025-08-06 Thread via GitHub
bernardodemarco commented on PR #180: URL: https://github.com/apache/cloudstack-terraform-provider/pull/180#issuecomment-3160266983 > 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. @DaanHoogland, yes

Re: [PR] github-action: add pr builder for easier testing [cloudstack-cloudmonkey]

2025-08-06 Thread via GitHub
shwstppr commented on PR #170: URL: https://github.com/apache/cloudstack-cloudmonkey/pull/170#issuecomment-3160262992 > @shwstppr how about adding `make dist` and attach the contents of the dist dir to the pr? in that way arm/windows users will have binaries to test as well. done -

Re: [PR] lint: fix failures [cloudstack-cloudmonkey]

2025-08-06 Thread via GitHub
shwstppr merged PR #171: URL: https://github.com/apache/cloudstack-cloudmonkey/pull/171 -- 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...@

Issue while making fedora 38 and 41 template

2025-08-06 Thread Sanjay Kumar
Hello!, I was trying to make the fedora 38 and 41 template but cloud-init not working for root password generated by cloudstack. I follow the instruction like as centos steam 9 as it is working fine. How to make the fedora template work for me. Any help would be really appreciated. Thank you

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

2025-08-06 Thread via GitHub
shwstppr commented on issue #168: URL: https://github.com/apache/cloudstack-cloudmonkey/issues/168#issuecomment-3160160128 Thanks @ingox for the clarification. I understand your issue now. I'll check and update -- This is an automated message from the Apache Git Service. To respond to t

[PR] githib-action: lint runner [cloudstack-cloudmonkey]

2025-08-06 Thread via GitHub
shwstppr opened a new pull request, #172: URL: https://github.com/apache/cloudstack-cloudmonkey/pull/172 Adds a runner for `make lint` with Ubuntu 24.04 Currently, it needs #171 to fix make lint -- This is an automated message from the Apache Git Service. To respond to the message,

Re: [PR] github-action: add pr builder for easier testing [cloudstack-cloudmonkey]

2025-08-06 Thread via GitHub
shwstppr commented on PR #170: URL: https://github.com/apache/cloudstack-cloudmonkey/pull/170#issuecomment-3159879389 This needs whitelisting peter-evans/find-comment@v3 and peter-evans/create-or-update-comment@v4 If that can't be done I can try to modify workflow to not use them cc @

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

2025-08-06 Thread via GitHub
DaanHoogland commented on issue #168: URL: https://github.com/apache/cloudstack-cloudmonkey/issues/168#issuecomment-3159882038 @ingox , this kind of sounds like a bash scripting issue and not a cmk issue. Can you explain your setup a bit more? It sounds like you should have two profi

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

2025-08-06 Thread via GitHub
ingox commented on issue #168: URL: https://github.com/apache/cloudstack-cloudmonkey/issues/168#issuecomment-3160131808 Leave that bash thing out for now. Here are the steps: - I have a single profile in my config file with username (admin) and password. - Once I login to cmk (command