Re: [PR] [HEALTH] 4.20 Health Check, please don't merge this! [cloudstack]

2024-12-13 Thread via GitHub
blueorangutan commented on PR #10006: URL: https://github.com/apache/cloudstack/pull/10006#issuecomment-2542655032 [SF] Trillian test result (tid-11895) Environment: vmware-70u3 (x2), Advanced Networking with Mgmt server ol8 Total time taken: 70916 seconds Marvin logs: https://githu

[I] Cannot deploy password-enabled template on a L2 network [cloudstack]

2024-12-13 Thread via GitHub
weizhouapache opened a new issue, #10101: URL: https://github.com/apache/cloudstack/issues/10101 Got an exception: Unable to deploy VM as template 203 is password enabled, but there is no support for UserData service in the default network The template have a default user/password

Re: [PR] kvm-storage: provide isVMMigrate information to storage plugins [cloudstack]

2024-12-13 Thread via GitHub
DaanHoogland commented on PR #10093: URL: https://github.com/apache/cloudstack/pull/10093#issuecomment-2540976600 @blueorangutan test -- 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

Re: [PR] kvm-storage: provide isVMMigrate information to storage plugins [cloudstack]

2024-12-13 Thread via GitHub
blueorangutan commented on PR #10093: URL: https://github.com/apache/cloudstack/pull/10093#issuecomment-2540980673 @DaanHoogland a [SL] Trillian-Jenkins test job (ol8 mgmt + kvm-ol8) has been kicked to run smoke tests -- This is an automated message from the Apache Git Service. To respond

Re: [PR] kvm-storage: provide isVMMigrate information to storage plugins [cloudstack]

2024-12-13 Thread via GitHub
blueorangutan commented on PR #10093: URL: https://github.com/apache/cloudstack/pull/10093#issuecomment-2540969427 Packaging result [SF]: ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 11801 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

Re: [PR] Backroll Plugin [cloudstack]

2024-12-13 Thread via GitHub
DaanHoogland commented on code in PR #8251: URL: https://github.com/apache/cloudstack/pull/8251#discussion_r1883620568 ## plugins/backup/backroll/src/main/java/org/apache/cloudstack/backup/backroll/model/response/archive/BackrollBackupsFromVMResponse.java: ## @@ -1,27 +1,10 @@ -

Re: [PR] Allow config drive deletion of migrated VM, on host maintenance [cloudstack]

2024-12-13 Thread via GitHub
blueorangutan commented on PR #10045: URL: https://github.com/apache/cloudstack/pull/10045#issuecomment-2540969423 Packaging result [SF]: ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 11802 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

Re: [PR] Backroll Plugin [cloudstack]

2024-12-13 Thread via GitHub
DaanHoogland commented on code in PR #8251: URL: https://github.com/apache/cloudstack/pull/8251#discussion_r1883620179 ## plugins/backup/backroll/src/main/java/org/apache/cloudstack/backup/backroll/model/response/backup/BackrollBackupResponse.java: ## @@ -0,0 +1,23 @@ +// Licens

Re: [PR] Backroll Plugin [cloudstack]

2024-12-13 Thread via GitHub
DaanHoogland commented on code in PR #8251: URL: https://github.com/apache/cloudstack/pull/8251#discussion_r1883624046 ## plugins/backup/backroll/src/main/java/org/apache/cloudstack/backup/backroll/BackrollClient.java: ## @@ -52,614 +40,189 @@ import org.apache.cloudstack.back

Re: [PR] Backroll Plugin [cloudstack]

2024-12-13 Thread via GitHub
DaanHoogland commented on PR #8251: URL: https://github.com/apache/cloudstack/pull/8251#issuecomment-2540973962 @PeterBackroll , some lint errors are caused due to these eof markers -- This is an automated message from the Apache Git Service. To respond to the message, please log on to Git

Re: [PR] Backroll Plugin [cloudstack]

2024-12-13 Thread via GitHub
DaanHoogland commented on code in PR #8251: URL: https://github.com/apache/cloudstack/pull/8251#discussion_r1883622457 ## plugins/backup/backroll/src/main/java/org/apache/cloudstack/backup/BackrollBackupProvider.java: ## @@ -105,8 +106,8 @@ public List listBackupOfferings(Long

Re: [PR] kvm-storage: provide isVMMigrate information to storage plugins [cloudstack]

2024-12-13 Thread via GitHub
rp- commented on PR #10093: URL: https://github.com/apache/cloudstack/pull/10093#issuecomment-2540905063 > code LGTM I don't see that the changes could affect the other storage plugins but I still cannot understand what the change does for Linstor In short, it prevents opening a volum

Re: [PR] Backroll Plugin [cloudstack]

2024-12-13 Thread via GitHub
PeterBackroll commented on code in PR #8251: URL: https://github.com/apache/cloudstack/pull/8251#discussion_r1883639003 ## plugins/backup/backroll/src/main/java/org/apache/cloudstack/backup/backroll/BackrollClient.java: ## @@ -52,614 +40,189 @@ import org.apache.cloudstack.bac

Re: [PR] server: fix typo removeaccessvpn in VirtualRouterElement [cloudstack]

2024-12-13 Thread via GitHub
DaanHoogland merged PR #10086: URL: https://github.com/apache/cloudstack/pull/10086 -- 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: commits-unsubscr...@

(cloudstack) branch 4.19 updated: server: fix typo removeaccessvpn in VirtualRouterElement (#10086)

2024-12-13 Thread dahn
This is an automated email from the ASF dual-hosted git repository. dahn pushed a commit to branch 4.19 in repository https://gitbox.apache.org/repos/asf/cloudstack.git The following commit(s) were added to refs/heads/4.19 by this push: new a278849507f server: fix typo removeaccessvpn in Vi

Re: [PR] Support for Management Server Maintenance Mode [cloudstack]

2024-12-13 Thread via GitHub
rohityadavcloud closed pull request #9854: Support for Management Server Maintenance Mode URL: https://github.com/apache/cloudstack/pull/9854 -- 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 spec

Re: [PR] Support for Management Server Maintenance Mode [cloudstack]

2024-12-13 Thread via GitHub
rohityadavcloud commented on PR #9854: URL: https://github.com/apache/cloudstack/pull/9854#issuecomment-2541074080 @blueorangutan package -- 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 speci

Re: [PR] Support for Management Server Maintenance Mode [cloudstack]

2024-12-13 Thread via GitHub
blueorangutan commented on PR #9854: URL: https://github.com/apache/cloudstack/pull/9854#issuecomment-2541077650 @rohityadavcloud a [SL] Jenkins job has been kicked to build packages. It will be bundled with KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progr

(cloudstack-go) branch main updated (4f01bd9 -> b9fa85f)

2024-12-13 Thread dahn
This is an automated email from the ASF dual-hosted git repository. dahn pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/cloudstack-go.git from 4f01bd9 Merge pull request #89 from suecodelabs/feature_add_more_apis add d7cc5d0 Update go version new

Re: [I] Cannot add new host in ACS [cloudstack]

2024-12-13 Thread via GitHub
tdtmusic2 commented on issue #10094: URL: https://github.com/apache/cloudstack/issues/10094#issuecomment-2541189597 > > these entries seem to be consequential, as the host is already removed from the DB. Can you try adding the host again and keep the log from the start of the API ca

[I] When removing an account, not everything is cleaned up [cloudstack]

2024-12-13 Thread via GitHub
jmkleijer opened a new issue, #10103: URL: https://github.com/apache/cloudstack/issues/10103 # ISSUE TYPE * Bug Report # COMPONENT NAME ~~~ UI ~~~ # CLOUDSTACK VERSION ~~~ 4.19.1.2 ~~~ # CONFIGURATION Networks that are shar

Re: [PR] Support for Management Server Maintenance Mode [cloudstack]

2024-12-13 Thread via GitHub
blueorangutan commented on PR #9854: URL: https://github.com/apache/cloudstack/pull/9854#issuecomment-2541216788 Packaging result [SF]: ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 11803 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to G

(cloudstack-go) branch update-goversion deleted (was d7cc5d0)

2024-12-13 Thread dahn
This is an automated email from the ASF dual-hosted git repository. dahn pushed a change to branch update-goversion in repository https://gitbox.apache.org/repos/asf/cloudstack-go.git was d7cc5d0 Update go version The revisions that were on this branch are still contained in other referen

(cloudstack-go) branch main updated (2b21240 -> 4f01bd9)

2024-12-13 Thread dahn
This is an automated email from the ASF dual-hosted git repository. dahn pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/cloudstack-go.git from 2b21240 Merge pull request #86 from apache/post add 9994d40 add +30 more apis new 4f01bd9 Merge pull r

(cloudstack-go) 01/01: Merge pull request #89 from suecodelabs/feature_add_more_apis

2024-12-13 Thread dahn
This is an automated email from the ASF dual-hosted git repository. dahn pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/cloudstack-go.git commit 4f01bd9209556a1f2742f3c359faae80cd8bbd21 Merge: 2b21240 9994d40 Author: dahn AuthorDate: Fri Dec 13 11:28:42 2024 +01

(cloudstack-go) 01/01: Update go version #94

2024-12-13 Thread dahn
This is an automated email from the ASF dual-hosted git repository. dahn pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/cloudstack-go.git commit b9fa85f6d0e8195222f070c4a241b9e6cae25338 Merge: 4f01bd9 d7cc5d0 Author: dahn AuthorDate: Fri Dec 13 11:41:17 2024 +01

(cloudstack-go) 01/01: Merge pull request #93 from apache/sdk-4.20

2024-12-13 Thread dahn
This is an automated email from the ASF dual-hosted git repository. dahn pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/cloudstack-go.git commit 75d368f7738cf188234cc88ddcbf63311704e1e2 Merge: b9fa85f d4d7f16 Author: dahn AuthorDate: Fri Dec 13 11:45:49 2024 +01

(cloudstack-go) branch sdk-4.20 deleted (was d4d7f16)

2024-12-13 Thread dahn
This is an automated email from the ASF dual-hosted git repository. dahn pushed a change to branch sdk-4.20 in repository https://gitbox.apache.org/repos/asf/cloudstack-go.git was d4d7f16 Update SDK with 4.20 APIs The revisions that were on this branch are still contained in other referen

(cloudstack-go) branch main updated (b9fa85f -> 75d368f)

2024-12-13 Thread dahn
This is an automated email from the ASF dual-hosted git repository. dahn pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/cloudstack-go.git from b9fa85f Update go version #94 add d4d7f16 Update SDK with 4.20 APIs new 75d368f Merge pull request #93

Re: [PR] kvm-storage: provide isVMMigrate information to storage plugins [cloudstack]

2024-12-13 Thread via GitHub
blueorangutan commented on PR #10093: URL: https://github.com/apache/cloudstack/pull/10093#issuecomment-2540841442 @rp- a [SL] Jenkins job has been kicked to build packages. It will be bundled with KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress. --

Re: [PR] kvm-storage: provide isVMMigrate information to storage plugins [cloudstack]

2024-12-13 Thread via GitHub
rp- commented on PR #10093: URL: https://github.com/apache/cloudstack/pull/10093#issuecomment-2540840599 @blueorangutan package -- 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 commen

Re: [PR] Allow config drive deletion of migrated VM, on host maintenance [cloudstack]

2024-12-13 Thread via GitHub
sureshanaparti commented on PR #10045: URL: https://github.com/apache/cloudstack/pull/10045#issuecomment-2540849170 @blueorangutan package -- 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 spec

Re: [PR] enhancement: add password to configdrive vendor_data.json [cloudstack]

2024-12-13 Thread via GitHub
weizhouapache commented on PR #10061: URL: https://github.com/apache/cloudstack/pull/10061#issuecomment-2540847843 I know some users use the cloudstack script in the vm template https://github.com/apache/cloudstack/blob/main/setup/bindir/cloud-set-guest-sshkey-password-userdata-configdriv

Re: [PR] Allow config drive deletion of migrated VM, on host maintenance [cloudstack]

2024-12-13 Thread via GitHub
blueorangutan commented on PR #10045: URL: https://github.com/apache/cloudstack/pull/10045#issuecomment-2540850388 @sureshanaparti a [SL] Jenkins job has been kicked to build packages. It will be bundled with KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make prog

Re: [PR] enhancement: add password to configdrive vendor_data.json [cloudstack]

2024-12-13 Thread via GitHub
phsm commented on PR #10061: URL: https://github.com/apache/cloudstack/pull/10061#issuecomment-2540857852 I specifically left the old functionality, so the users can choose whether to use the script you mentioned, or the cloud-init's native functionality. So this code will not break the scr

Re: [PR] kvm-storage: provide isVMMigrate information to storage plugins [cloudstack]

2024-12-13 Thread via GitHub
slavkap commented on PR #10093: URL: https://github.com/apache/cloudstack/pull/10093#issuecomment-2540933620 Thanks @rp- for the explanation! That makes sense. Btw, for the Host HA, I've added the option for the storage to disconnect the volume from all hosts before the start of a VM on ano

Re: [I] Cannot add new host in ACS [cloudstack]

2024-12-13 Thread via GitHub
tdtmusic2 commented on issue #10094: URL: https://github.com/apache/cloudstack/issues/10094#issuecomment-2541384620 > > these entries seem to be consequential, as the host is already removed from the DB. Can you try adding the host again and keep the log from the start of the API call t

Re: [PR] [HEALTH] 4.19 LTS Health Check, please don't merge this! [cloudstack]

2024-12-13 Thread via GitHub
blueorangutan commented on PR #9315: URL: https://github.com/apache/cloudstack/pull/9315#issuecomment-2542411248 [SF] Trillian test result (tid-11896) Environment: kvm-ol8 (x2), Advanced Networking with Mgmt server ol8 Total time taken: 45458 seconds Marvin logs: https://github.com/

Re: [I] Ceph RGW Object Storage couldn't be added (Invalid credentials or URL) [cloudstack]

2024-12-13 Thread via GitHub
hanisirfan commented on issue #10044: URL: https://github.com/apache/cloudstack/issues/10044#issuecomment-2542820113 > Just to cover all bases @hanisirfan can you please verify these : > > 1. url contains the correct radosgw port? > > 2. the user should be a System User.

Re: [PR] Support for Management Server Maintenance Mode [cloudstack]

2024-12-13 Thread via GitHub
blueorangutan commented on PR #9854: URL: https://github.com/apache/cloudstack/pull/9854#issuecomment-2542920940 [SF] Trillian test result (tid-11900) Environment: kvm-ol8 (x2), Advanced Networking with Mgmt server ol8 Total time taken: 51752 seconds Marvin logs: https://github.com/

Re: [PR] [Draft] VR: allow outgoing traffic from Remote access VPN clients [cloudstack]

2024-12-13 Thread via GitHub
blueorangutan commented on PR #10083: URL: https://github.com/apache/cloudstack/pull/10083#issuecomment-2541855703 Packaging result [SF]: ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 11804 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

Re: [PR] linstor: improve heartbeat check with also asking linstor [cloudstack]

2024-12-13 Thread via GitHub
blueorangutan commented on PR #10105: URL: https://github.com/apache/cloudstack/pull/10105#issuecomment-2541861564 Packaging result [SF]: ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 11805 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

Re: [PR] server: do not allocate nic on public network for NSX VPC VR [cloudstack]

2024-12-13 Thread via GitHub
weizhouapache commented on PR #10081: URL: https://github.com/apache/cloudstack/pull/10081#issuecomment-2541603849 cc @nvazquez @Pearl1594 @alexandremattioli -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

Re: [I] Cannot deploy password-enabled template on a L2 network [cloudstack]

2024-12-13 Thread via GitHub
DaanHoogland commented on issue #10101: URL: https://github.com/apache/cloudstack/issues/10101#issuecomment-2541713955 Hey @weizhouapache , this is actually a feature I had wanted before , because of testing. I never got around entering it. So we want to not provision a password on a passwo

[I] Quiesce Instance not an option in "Take Instance volume Snapshot" window [cloudstack]

2024-12-13 Thread via GitHub
Rubueno opened a new issue, #10106: URL: https://github.com/apache/cloudstack/issues/10106 # ISSUE TYPE * Bug Report # COMPONENT NAME ~~~ UI ~~~ # CLOUDSTACK VERSION ~~~ 4.19.1 ~~~ # CONFIGURATION N/A # OS / ENVI

Re: [PR] fix typo in SnapshotManager.java [cloudstack]

2024-12-13 Thread via GitHub
DaanHoogland commented on PR #10104: URL: https://github.com/apache/cloudstack/pull/10104#issuecomment-2541739051 config key description only, let's wait for github actions and merge -- This is an automated message from the Apache Git Service. To respond to the message, please log on to Gi

Re: [PR] [Draft] server: do not allocate nic on public network for NSX VPC VR [cloudstack]

2024-12-13 Thread via GitHub
weizhouapache commented on PR #10081: URL: https://github.com/apache/cloudstack/pull/10081#issuecomment-2541663926 as discussed offline, we need to evaluate the impact of the changes and test heavily. moved to draft, we will revisit in the future -- This is an automated message from th

Re: [PR] Support for Management Server Maintenance Mode [cloudstack]

2024-12-13 Thread via GitHub
blueorangutan commented on PR #9854: URL: https://github.com/apache/cloudstack/pull/9854#issuecomment-2541788173 @DaanHoogland a [SL] Trillian-Jenkins test job (ol8 mgmt + kvm-ol8) has been kicked to run smoke tests -- This is an automated message from the Apache Git Service. To respond t

Re: [I] Cannot deploy password-enabled template on a L2 network [cloudstack]

2024-12-13 Thread via GitHub
weizhouapache commented on issue #10101: URL: https://github.com/apache/cloudstack/issues/10101#issuecomment-2541799811 > Hey @weizhouapache , this is actually a feature I had wanted before , because of testing. I never got around entering it. So we want to not provision a password on a pas

Re: [I] Cannot add new host in ACS [cloudstack]

2024-12-13 Thread via GitHub
weizhouapache commented on issue #10094: URL: https://github.com/apache/cloudstack/issues/10094#issuecomment-2541419954 @tdtmusic2 you mentioned that the rbd storage pool is used by several hosts, is the new host same as the other hosts ? (same hardware, same OS, in the same subnet ) -

Re: [I] Cannot add new host in ACS [cloudstack]

2024-12-13 Thread via GitHub
tdtmusic2 commented on issue #10094: URL: https://github.com/apache/cloudstack/issues/10094#issuecomment-2541427018 > @tdtmusic2 you mentioned that the rbd storage pool is used by several hosts, is the new host same as the other hosts ? (same hardware, same OS, in the same subnet ) T

Re: [PR] fix typo in SnapshotManager.java [cloudstack]

2024-12-13 Thread via GitHub
codecov[bot] commented on PR #10104: URL: https://github.com/apache/cloudstack/pull/10104#issuecomment-2541828419 ## [Codecov](https://app.codecov.io/gh/apache/cloudstack/pull/10104?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comme

Re: [PR] linstor: improve heartbeat check with also asking linstor [cloudstack]

2024-12-13 Thread via GitHub
codecov[bot] commented on PR #10105: URL: https://github.com/apache/cloudstack/pull/10105#issuecomment-2541514595 ## [Codecov](https://app.codecov.io/gh/apache/cloudstack/pull/10105?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comme

(cloudstack) branch 4.20 updated: UI: create VPC network offering with conserve mode (#10082)

2024-12-13 Thread dahn
This is an automated email from the ASF dual-hosted git repository. dahn pushed a commit to branch 4.20 in repository https://gitbox.apache.org/repos/asf/cloudstack.git The following commit(s) were added to refs/heads/4.20 by this push: new ac19379c967 UI: create VPC network offering with c

Re: [PR] UI: create VPC network offering with conserve mode [cloudstack]

2024-12-13 Thread via GitHub
DaanHoogland merged PR #10082: URL: https://github.com/apache/cloudstack/pull/10082 -- 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: commits-unsubscr...@

Re: [PR] Support for Management Server Maintenance Mode [cloudstack]

2024-12-13 Thread via GitHub
DaanHoogland commented on PR #9854: URL: https://github.com/apache/cloudstack/pull/9854#issuecomment-2541784388 @blueorangutan test -- 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 co

[PR] fix typo in SnapshotManager.java [cloudstack]

2024-12-13 Thread via GitHub
Rubueno opened a new pull request, #10104: URL: https://github.com/apache/cloudstack/pull/10104 ### Description This PR fixes a typo ### Types of changes - [x] Cleanup (Code refactoring and cleanup, that may add test cases) ### How Has This Been Tested? No t

[PR] linstor: improve heartbeat check with also asking linstor [cloudstack]

2024-12-13 Thread via GitHub
rp- opened a new pull request, #10105: URL: https://github.com/apache/cloudstack/pull/10105 ### Description This improves the heartbeat check done from Linstor primary storage. If a node doesn't have a DRBD connection to another node, additionally ask Linstor-Controller if the node

Re: [PR] Add maximum results number to retrieval of unregistered VMs on vmware [cloudstack]

2024-12-13 Thread via GitHub
blueorangutan commented on PR #9925: URL: https://github.com/apache/cloudstack/pull/9925#issuecomment-2541879605 Packaging result [SF]: ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 11806 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to G

Re: [PR] [HEALTH] 4.19 LTS Health Check, please don't merge this! [cloudstack]

2024-12-13 Thread via GitHub
blueorangutan commented on PR #9315: URL: https://github.com/apache/cloudstack/pull/9315#issuecomment-2542524662 [SF] Trillian test result (tid-11897) Environment: kvm-ubuntu22 (x2), Advanced Networking with Mgmt server u22 Total time taken: 51710 seconds Marvin logs: https://github

Re: [PR] kvm-storage: provide isVMMigrate information to storage plugins [cloudstack]

2024-12-13 Thread via GitHub
blueorangutan commented on PR #10093: URL: https://github.com/apache/cloudstack/pull/10093#issuecomment-2542581477 [SF] Trillian test result (tid-11898) Environment: kvm-ol8 (x2), Advanced Networking with Mgmt server ol8 Total time taken: 51068 seconds Marvin logs: https://github.co