Re: [PR] Bump tar from 6.1.11 to 6.2.1 in /ui [cloudstack]

2024-04-11 Thread via GitHub
rohityadavcloud closed pull request #8900: Bump tar from 6.1.11 to 6.2.1 in /ui URL: https://github.com/apache/cloudstack/pull/8900 -- 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] linstor: Do not pretend handling disconnect paths that are non Linstor [cloudstack]

2024-04-11 Thread via GitHub
rohityadavcloud commented on PR #8897: URL: https://github.com/apache/cloudstack/pull/8897#issuecomment-2051067036 Rekicked CI by close/reopen. @blueorangutan package -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub an

Re: [PR] linstor: Do not pretend handling disconnect paths that are non Linstor [cloudstack]

2024-04-11 Thread via GitHub
blueorangutan commented on PR #8897: URL: https://github.com/apache/cloudstack/pull/8897#issuecomment-2051068850 @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) branch dependabot/npm_and_yarn/ui/tar-6.2.1 deleted (was 0ffc92bb1fd)

2024-04-11 Thread github-bot
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to branch dependabot/npm_and_yarn/ui/tar-6.2.1 in repository https://gitbox.apache.org/repos/asf/cloudstack.git was 0ffc92bb1fd Bump tar from 6.1.11 to 6.2.1 in /ui The revisions that were on thi

Re: [PR] Bump tar from 6.1.11 to 6.2.1 in /ui [cloudstack]

2024-04-11 Thread via GitHub
dependabot[bot] commented on PR #8900: URL: https://github.com/apache/cloudstack/pull/8900#issuecomment-2051067559 OK, I won't notify you again about this release, but will get in touch when a new version is available. If you'd rather skip all updates until the next major or minor version,

Re: [PR] linstor: Do not pretend handling disconnect paths that are non Linstor [cloudstack]

2024-04-11 Thread via GitHub
rohityadavcloud closed pull request #8897: linstor: Do not pretend handling disconnect paths that are non Linstor URL: https://github.com/apache/cloudstack/pull/8897 -- 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: [PR] Change storage pool scope from Cluster to Zone and vise versa [cloudstack]

2024-04-11 Thread via GitHub
abh1sar commented on code in PR #8875: URL: https://github.com/apache/cloudstack/pull/8875#discussion_r1562038355 ## server/src/main/java/com/cloud/storage/StorageManagerImpl.java: ## @@ -1148,6 +1151,93 @@ public PrimaryDataStoreInfo updateStoragePool(UpdateStoragePoolCmd cmd)

[I] CKS Cluster goes into a alert state when a vm part of the [cloudstack]

2024-04-11 Thread via GitHub
kiranchavala opened a new issue, #8902: URL: https://github.com/apache/cloudstack/issues/8902 ISSUE TYPE Improvement request/BUG COMPONENT NAME Component: CKS CLOUDSTACK VERSION Cloudstack version 4.18 4.19 SUMMARY CKS Cluster goes into an aler

Re: [I] Option to set ACL Rule to be the very bottom by default [cloudstack]

2024-04-11 Thread via GitHub
btzq commented on issue #8865: URL: https://github.com/apache/cloudstack/issues/8865#issuecomment-2050842134 @DaanHoogland i wanst sure that the 'number' parameter was working. Because i was intuitively expecting that the row would automatically be moved/sorted to the bottom of the page. I

Re: [I] Support Destination CIDR [cloudstack]

2024-04-11 Thread via GitHub
btzq commented on issue #8864: URL: https://github.com/apache/cloudstack/issues/8864#issuecomment-2050840253 Yes maybe we can limit it to Private Adresses for the time being and see how it goes. This is also to limit the blast radius of bugs due to the big change. -- This is an automated

Re: [I] Support ACL on Public IP [cloudstack]

2024-04-11 Thread via GitHub
btzq commented on issue #8863: URL: https://github.com/apache/cloudstack/issues/8863#issuecomment-2050840516 @DaanHoogland when you mean topopology, do you mean that the users will need to know the existance of a the Virtual Router? -- This is an automated message from the Apache Git Serv

Re: [I] Export ACL List is missing some data [cloudstack]

2024-04-11 Thread via GitHub
btzq commented on issue #8862: URL: https://github.com/apache/cloudstack/issues/8862#issuecomment-2050838856 @DaanHoogland when i meant 'Export', i meant the existing export function. Exporting via CMK is not required (For us at least). My thoughts are that if a ACL rule doesnt have

Re: [I] Allow user to create an ACL Rule for Multiple Ports [cloudstack]

2024-04-11 Thread via GitHub
btzq commented on issue #8861: URL: https://github.com/apache/cloudstack/issues/8861#issuecomment-2050836740 @DaanHoogland , bad :P but maybe the implementation doesnt need to be sophisticated. What i think can be done is that behind the scenes, it could be created as multiple rules

Re: [I] Allow users to import ACL from a CSV File [cloudstack]

2024-04-11 Thread via GitHub
btzq commented on issue #8858: URL: https://github.com/apache/cloudstack/issues/8858#issuecomment-2050832369 I just thought of something. If the 'Import CSV' is going to be supported, it may also be necessary to support 'Override Existing ACL Rules'. Just so that it doesnt create duplicate

Re: [I] Display ACL Rules in the form of a excel like table [cloudstack]

2024-04-11 Thread via GitHub
btzq commented on issue #8857: URL: https://github.com/apache/cloudstack/issues/8857#issuecomment-2050831192 @DaanHoogland the ListView in VM or Volume lists would work perfectly actually. The only issue i can think of is the scenario that there would be too many colums. In that scen

Re: [I] User Experience: Allow users to add Description to Private Gateway and Static Routes [cloudstack]

2024-04-11 Thread via GitHub
btzq commented on issue #8837: URL: https://github.com/apache/cloudstack/issues/8837#issuecomment-2050827680 @DaanHoogland , no audit trail is needed... yet... Its a good to have but not big for us right now yet. Better to get pick the lowest fruits and work on the more complicated st

[I] Wrong mountpoint generated on secondary storage for ISO on live migration [cloudstack]

2024-04-11 Thread via GitHub
loth opened a new issue, #8901: URL: https://github.com/apache/cloudstack/issues/8901 # ISSUE TYPE * Bug Report # COMPONENT NAME Secondary Storage ~~~ ~~~ # CLOUDSTACK VERSION 4.18.1 ~~~ ~~~ # CONFIGURATION KVM using 2x

Re: [PR] Mark libvirt events experimental, add properties flag [cloudstack]

2024-04-11 Thread via GitHub
blueorangutan commented on PR #8825: URL: https://github.com/apache/cloudstack/pull/8825#issuecomment-2050779094 [SF] Trillian test result (tid-9799) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 46148 seconds Marvin logs: https://github.com

Re: [PR] Change storage pool scope from Cluster to Zone and vise versa [cloudstack]

2024-04-11 Thread via GitHub
weizhouapache commented on code in PR #8875: URL: https://github.com/apache/cloudstack/pull/8875#discussion_r1561803442 ## plugins/storage/volume/datera/src/main/java/org/apache/cloudstack/storage/datastore/lifecycle/DateraPrimaryDataStoreLifeCycle.java: ## @@ -395,6 +395,15 @@

Re: [PR] Quota tariff order [cloudstack]

2024-04-11 Thread via GitHub
hsato03 commented on code in PR #8347: URL: https://github.com/apache/cloudstack/pull/8347#discussion_r1561549494 ## framework/quota/src/main/java/org/apache/cloudstack/quota/QuotaManagerImpl.java: ## @@ -367,9 +369,22 @@ protected BigDecimal aggregateQuotaTariffsValues(UsageVO

Re: [PR] Quota tariff order [cloudstack]

2024-04-11 Thread via GitHub
BryanMLima commented on code in PR #8347: URL: https://github.com/apache/cloudstack/pull/8347#discussion_r1561536028 ## engine/schema/src/main/resources/META-INF/db/schema-41900to42000.sql: ## @@ -79,3 +79,7 @@ CREATE TABLE IF NOT EXISTS `cloud_usage`.`quota_email_configuration

Re: [PR] Quota tariff order [cloudstack]

2024-04-11 Thread via GitHub
JoaoJandre commented on code in PR #8347: URL: https://github.com/apache/cloudstack/pull/8347#discussion_r1561533659 ## framework/quota/src/main/java/org/apache/cloudstack/quota/QuotaManagerImpl.java: ## @@ -367,9 +369,22 @@ protected BigDecimal aggregateQuotaTariffsValues(Usage

Re: [PR] linstor: Do not pretend handling disconnect paths that are non Linstor [cloudstack]

2024-04-11 Thread via GitHub
blueorangutan commented on PR #8897: URL: https://github.com/apache/cloudstack/pull/8897#issuecomment-2050312892 [SF] Trillian test result (tid-9795) Environment: kvm-rocky8 (x2), Advanced Networking with Mgmt server r8 Total time taken: 43358 seconds Marvin logs: https://github.com

Re: [PR] Quota tariff order [cloudstack]

2024-04-11 Thread via GitHub
hsato03 commented on code in PR #8347: URL: https://github.com/apache/cloudstack/pull/8347#discussion_r1561383592 ## framework/quota/src/main/java/org/apache/cloudstack/quota/QuotaManagerImpl.java: ## @@ -367,9 +369,22 @@ protected BigDecimal aggregateQuotaTariffsValues(UsageVO

Re: [PR] Change storage pool scope from Cluster to Zone and vise versa [cloudstack]

2024-04-11 Thread via GitHub
abh1sar commented on code in PR #8875: URL: https://github.com/apache/cloudstack/pull/8875#discussion_r1561357090 ## plugins/storage/volume/datera/src/main/java/org/apache/cloudstack/storage/datastore/lifecycle/DateraPrimaryDataStoreLifeCycle.java: ## @@ -395,6 +395,15 @@ public

(cloudstack-documentation) branch main updated: replace download.cloud.com with download.cloudstack.org

2024-04-11 Thread rohit
This is an automated email from the ASF dual-hosted git repository. rohit pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/cloudstack-documentation.git The following commit(s) were added to refs/heads/main by this push: new fd50cb8 replace download.cloud.com

(cloudstack-documentation) branch main updated: Update building_from_source.po

2024-04-11 Thread rohit
This is an automated email from the ASF dual-hosted git repository. rohit pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/cloudstack-documentation.git The following commit(s) were added to refs/heads/main by this push: new ccc387e Update building_from_sourc

(cloudstack-documentation) branch main updated: Update installation.po

2024-04-11 Thread rohit
This is an automated email from the ASF dual-hosted git repository. rohit pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/cloudstack-documentation.git The following commit(s) were added to refs/heads/main by this push: new 342ce40 Update installation.po 342

(cloudstack-documentation) branch main updated: Update building_from_source.pot

2024-04-11 Thread rohit
This is an automated email from the ASF dual-hosted git repository. rohit pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/cloudstack-documentation.git The following commit(s) were added to refs/heads/main by this push: new 9aeb891 Update building_from_sourc

(cloudstack-documentation) branch main updated: Update installation.pot

2024-04-11 Thread rohit
This is an automated email from the ASF dual-hosted git repository. rohit pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/cloudstack-documentation.git The following commit(s) were added to refs/heads/main by this push: new d8f469f Update installation.pot d8

Re: [PR] linstor: Do not pretend handling disconnect paths that are non Linstor [cloudstack]

2024-04-11 Thread via GitHub
JoaoJandre commented on PR #8897: URL: https://github.com/apache/cloudstack/pull/8897#issuecomment-2050043451 @sureshanaparti @rohityadavcloud @shwstppr could we run the CI again? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHu

Re: [PR] Temporarily backup StorPool volume before expunge [cloudstack]

2024-04-11 Thread via GitHub
slavkap commented on code in PR #8843: URL: https://github.com/apache/cloudstack/pull/8843#discussion_r1561040808 ## plugins/storage/volume/storpool/src/main/java/org/apache/cloudstack/storage/datastore/driver/StorPoolStatsCollector.java: ## @@ -70,7 +86,7 @@ enum StorPoolObject

Re: [PR] Temporarily backup StorPool volume before expunge [cloudstack]

2024-04-11 Thread via GitHub
slavkap commented on code in PR #8843: URL: https://github.com/apache/cloudstack/pull/8843#discussion_r1561037449 ## plugins/storage/volume/storpool/src/main/java/org/apache/cloudstack/storage/datastore/driver/StorPoolPrimaryDataStoreDriver.java: ## @@ -431,6 +439,54 @@ public v

Re: [PR] Change storage pool scope from Cluster to Zone and vise versa [cloudstack]

2024-04-11 Thread via GitHub
weizhouapache commented on code in PR #8875: URL: https://github.com/apache/cloudstack/pull/8875#discussion_r1560975816 ## plugins/storage/volume/datera/src/main/java/org/apache/cloudstack/storage/datastore/lifecycle/DateraPrimaryDataStoreLifeCycle.java: ## @@ -395,6 +395,15 @@

Re: [I] User Experience: Allow users to add Description to Private Gateway and Static Routes [cloudstack]

2024-04-11 Thread via GitHub
DaanHoogland commented on issue #8837: URL: https://github.com/apache/cloudstack/issues/8837#issuecomment-2049537257 just a new DB field or two, seems like a simple issue, provided no audit trail is needed. If there is a comments trail needs to be added. -- This is an automated message fr

Re: [PR] linstor: Do not pretend handling disconnect paths that are non Linstor [cloudstack]

2024-04-11 Thread via GitHub
JoaoJandre commented on PR #8897: URL: https://github.com/apache/cloudstack/pull/8897#issuecomment-2049530142 @DaanHoogland From what I've looked at the errors from https://github.com/apache/cloudstack/pull/8897#issuecomment-2048642812, them look like an environment instability; could we ru

Re: [PR] Change iops on offering change [cloudstack]

2024-04-11 Thread via GitHub
blueorangutan commented on PR #8872: URL: https://github.com/apache/cloudstack/pull/8872#issuecomment-2049529691 [SF] Trillian Build Failed (tid-9798) -- 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

Re: [I] Display ACL Rules in the form of a excel like table [cloudstack]

2024-04-11 Thread via GitHub
DaanHoogland commented on issue #8857: URL: https://github.com/apache/cloudstack/issues/8857#issuecomment-2049511564 Would a ListView do @btzq ? Like the one for `vm` or `volume` lists. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

Re: [PR] Improve migration of external VMware VMs into KVM cluster [cloudstack]

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

Re: [PR] Mark libvirt events experimental, add properties flag [cloudstack]

2024-04-11 Thread via GitHub
blueorangutan commented on PR #8825: URL: https://github.com/apache/cloudstack/pull/8825#issuecomment-2049505022 @rohityadavcloud a [SL] Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests -- This is an automated message from the Apache Git Service. T

Re: [PR] Mark libvirt events experimental, add properties flag [cloudstack]

2024-04-11 Thread via GitHub
rohityadavcloud commented on PR #8825: URL: https://github.com/apache/cloudstack/pull/8825#issuecomment-2049501747 @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

(cloudstack) branch 4.19 updated: Mark libvirt events experimental, add properties flag (#8825)

2024-04-11 Thread rohit
This is an automated email from the ASF dual-hosted git repository. rohit 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 d3e020a5452 Mark libvirt events experimental, add

Re: [PR] Mark libvirt events experimental, add properties flag [cloudstack]

2024-04-11 Thread via GitHub
rohityadavcloud merged PR #8825: URL: https://github.com/apache/cloudstack/pull/8825 -- 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] Change iops on offering change [cloudstack]

2024-04-11 Thread via GitHub
blueorangutan commented on PR #8872: URL: https://github.com/apache/cloudstack/pull/8872#issuecomment-2049489807 @rohityadavcloud a [SL] Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests -- This is an automated message from the Apache Git Service. T

Re: [PR] Change iops on offering change [cloudstack]

2024-04-11 Thread via GitHub
rohityadavcloud commented on PR #8872: URL: https://github.com/apache/cloudstack/pull/8872#issuecomment-2049489709 @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

(cloudstack) branch 4.19 updated: Change iops on offering change (#8872)

2024-04-11 Thread rohit
This is an automated email from the ASF dual-hosted git repository. rohit 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 730cc5d5b8f Change iops on offering change (#8872

Re: [PR] Change iops on offering change [cloudstack]

2024-04-11 Thread via GitHub
rohityadavcloud merged PR #8872: URL: https://github.com/apache/cloudstack/pull/8872 -- 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] Change iops on offering change [cloudstack]

2024-04-11 Thread via GitHub
rohityadavcloud commented on PR #8872: URL: https://github.com/apache/cloudstack/pull/8872#issuecomment-2049486872 @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

(cloudstack-documentation) branch main updated: Update building_from_source.rst

2024-04-11 Thread rohit
This is an automated email from the ASF dual-hosted git repository. rohit pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/cloudstack-documentation.git The following commit(s) were added to refs/heads/main by this push: new b6a4a09 Update building_from_sourc

Re: [I] Allow users to import ACL from a CSV File [cloudstack]

2024-04-11 Thread via GitHub
DaanHoogland commented on issue #8858: URL: https://github.com/apache/cloudstack/issues/8858#issuecomment-2049424733 valid request @btzq , I think this can be partly implemented in the UI. There is an example for roles already (for whomever plans to implement this), ImportRole. In th

Re: [PR] Improve migration of external VMware VMs into KVM cluster [cloudstack]

2024-04-11 Thread via GitHub
blueorangutan commented on PR #8815: URL: https://github.com/apache/cloudstack/pull/8815#issuecomment-2049406531 @vladimirpetrov 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 progre

Re: [PR] Improve migration of external VMware VMs into KVM cluster [cloudstack]

2024-04-11 Thread via GitHub
vladimirpetrov commented on PR #8815: URL: https://github.com/apache/cloudstack/pull/8815#issuecomment-2049403480 @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 specif

Re: [PR] Fix volume snapshot of encrypted NFS/StorPool volume [cloudstack]

2024-04-11 Thread via GitHub
slavkap commented on code in PR #8873: URL: https://github.com/apache/cloudstack/pull/8873#discussion_r1560787113 ## plugins/hypervisors/kvm/src/main/java/com/cloud/hypervisor/kvm/storage/KVMStorageProcessor.java: ## @@ -1936,26 +1938,38 @@ protected void manuallyDeleteUnusedSna

Re: [I] Allow users to Search ACL Rule [cloudstack]

2024-04-11 Thread via GitHub
DaanHoogland commented on issue #8859: URL: https://github.com/apache/cloudstack/issues/8859#issuecomment-2049353871 There is no generic `keyword` search (working!) as for other list APIs. There is no search on the field `reason`. There is no search by order/`number`. There is no sea

Re: [I] Allow user to create an ACL Rule for Multiple Ports [cloudstack]

2024-04-11 Thread via GitHub
DaanHoogland commented on issue #8861: URL: https://github.com/apache/cloudstack/issues/8861#issuecomment-2049331133 @btzq the only way multiple ports are supported is through a range (start-port/end-port) I think you need to create multiple rules. You can implement a convenience API but it

Re: [I] Export ACL List is missing some data [cloudstack]

2024-04-11 Thread via GitHub
DaanHoogland commented on issue #8862: URL: https://github.com/apache/cloudstack/issues/8862#issuecomment-2049310190 @btzq , with "export", do you mean the output of a tool like cmk? This seems a UI bug; if the first rule does not have start and end port these wont be in the exported CVS.

Re: [I] Support Destination CIDR [cloudstack]

2024-04-11 Thread via GitHub
DaanHoogland commented on issue #8864: URL: https://github.com/apache/cloudstack/issues/8864#issuecomment-2049260607 This is a bit complicated but a valid request. For this we need to add changes through the whole system: in the - VR scripts - backend command(s) - DB - virtual

Re: [I] Option to set ACL Rule to be the very bottom by default [cloudstack]

2024-04-11 Thread via GitHub
DaanHoogland commented on issue #8865: URL: https://github.com/apache/cloudstack/issues/8865#issuecomment-2049234863 I think the createNetworkAcl should be renamed to createNetworkAclItem to be more consistent with the other commands. Same for deleteNetworkACL(Item)! -- This is an automat

Re: [I] Option to set ACL Rule to be the very bottom by default [cloudstack]

2024-04-11 Thread via GitHub
DaanHoogland commented on issue #8865: URL: https://github.com/apache/cloudstack/issues/8865#issuecomment-2049231754 @btzq , you can use the parameter `number` to set the order of an acl item in the acl list. Does this fullfill this functionality? We may need to improve the documentation fo

(cloudstack-terraform-provider) branch main updated: Update README.md

2024-04-11 Thread rohit
This is an automated email from the ASF dual-hosted git repository. rohit pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/cloudstack-terraform-provider.git The following commit(s) were added to refs/heads/main by this push: new 5b2829d Update README.md 5b2