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

2024-04-29 Thread via GitHub
blueorangutan commented on PR #8875: URL: https://github.com/apache/cloudstack/pull/8875#issuecomment-2084422951 @abh1sar 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] Change storage pool scope from Cluster to Zone and vise versa [cloudstack]

2024-04-29 Thread via GitHub
abh1sar commented on PR #8875: URL: https://github.com/apache/cloudstack/pull/8875#issuecomment-2084421819 @blueorangutan pakage -- 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] Ability to specify NFS mount options while adding a primary storage and modify them on a pre-existing primary storage [cloudstack]

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

Re: [PR] feature: webhooks [cloudstack]

2024-04-29 Thread via GitHub
blueorangutan commented on PR #8674: URL: https://github.com/apache/cloudstack/pull/8674#issuecomment-2084392667 @shwstppr 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] feature: webhooks [cloudstack]

2024-04-29 Thread via GitHub
shwstppr commented on PR #8674: URL: https://github.com/apache/cloudstack/pull/8674#issuecomment-2084391438 @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

Re: [PR] feature: webhooks [cloudstack]

2024-04-29 Thread via GitHub
shwstppr commented on code in PR #8674: URL: https://github.com/apache/cloudstack/pull/8674#discussion_r1584139253 ## engine/schema/src/main/resources/META-INF/db/schema-41900to42000.sql: ## @@ -79,3 +78,46 @@ CREATE TABLE IF NOT EXISTS

Re: [PR] feature: webhooks [cloudstack]

2024-04-29 Thread via GitHub
blueorangutan commented on PR #8674: URL: https://github.com/apache/cloudstack/pull/8674#issuecomment-2084386793 [LL] Trillian Build Failed (tid-6921) -- 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

Re: [PR] api,server: purge expunged resources [cloudstack]

2024-04-29 Thread via GitHub
blueorangutan commented on PR #8999: URL: https://github.com/apache/cloudstack/pull/8999#issuecomment-2084385779 @shwstppr 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] api,server: purge expunged resources [cloudstack]

2024-04-29 Thread via GitHub
shwstppr commented on PR #8999: URL: https://github.com/apache/cloudstack/pull/8999#issuecomment-2084384323 @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

Re: [PR] Ability to specify NFS mount options while adding a primary storage and modify them on a pre-existing primary storage [cloudstack]

2024-04-29 Thread via GitHub
blueorangutan commented on PR #8947: URL: https://github.com/apache/cloudstack/pull/8947#issuecomment-2084355476 @abh1sar 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] Ability to specify NFS mount options while adding a primary storage and modify them on a pre-existing primary storage [cloudstack]

2024-04-29 Thread via GitHub
abh1sar commented on PR #8947: URL: https://github.com/apache/cloudstack/pull/8947#issuecomment-2084353571 @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

Re: [PR] Ability to specify NFS mount options while adding a primary storage and modify them on a pre-existing primary storage [cloudstack]

2024-04-29 Thread via GitHub
abh1sar commented on code in PR #8947: URL: https://github.com/apache/cloudstack/pull/8947#discussion_r1584111489 ## server/src/main/java/com/cloud/storage/StorageManagerImpl.java: ## @@ -839,6 +839,21 @@ protected String createLocalStoragePoolName(Host host, StoragePoolInfo

Re: [PR] Ability to specify NFS mount options while adding a primary storage and modify them on a pre-existing primary storage [cloudstack]

2024-04-29 Thread via GitHub
abh1sar commented on code in PR #8947: URL: https://github.com/apache/cloudstack/pull/8947#discussion_r1584106723 ## server/src/main/java/com/cloud/storage/StorageManagerImpl.java: ## @@ -839,6 +839,21 @@ protected String createLocalStoragePoolName(Host host, StoragePoolInfo

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

2024-04-29 Thread via GitHub
blueorangutan commented on PR #8602: URL: https://github.com/apache/cloudstack/pull/8602#issuecomment-2084035107 [SF] Trillian test result (tid-10073) Environment: vmware-67u3 (x2), Advanced Networking with Mgmt server r8 Total time taken: 57736 seconds Marvin logs:

Re: [PR] Fix message publish in transaction [cloudstack]

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

Re: [PR] Use parameter dcId as wrapper to prevent NPE [cloudstack]

2024-04-29 Thread via GitHub
blueorangutan commented on PR #8986: URL: https://github.com/apache/cloudstack/pull/8986#issuecomment-2083806058 [SF] Trillian test result (tid-10071) Environment: vmware-70u3 (x2), Advanced Networking with Mgmt server a9 Total time taken: 49703 seconds Marvin logs:

Re: [PR] VNF: add API listVnfAppliances [cloudstack]

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

Re: [PR] Normalize dates in Usage and Quota APIs [cloudstack]

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

Re: [I] Unable to add the host: Cannot find the server resources [cloudstack]

2024-04-29 Thread via GitHub
printfdebugging commented on issue #8851: URL: https://github.com/apache/cloudstack/issues/8851#issuecomment-2083779299 I was getting the same error. Turns out that reading the log files helps, as a log file is cloudstack's way of communicating back what's the issue. I searched for "server

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

2024-04-29 Thread via GitHub
loth commented on issue #8901: URL: https://github.com/apache/cloudstack/issues/8901#issuecomment-2083778098 > @loth there are some PRs out for similar issues, #8952. Will you have the chance to test with that patch? Hello, I tried the mentioned patch and it diddnt affect the

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

2024-04-29 Thread via GitHub
blueorangutan commented on PR #8602: URL: https://github.com/apache/cloudstack/pull/8602#issuecomment-2083759057 [SF] Trillian test result (tid-10072) Environment: xenserver-71 (x2), Advanced Networking with Mgmt server 7 Total time taken: 46603 seconds Marvin logs:

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

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

Re: [PR] Add logs to CPVM connection process [cloudstack]

2024-04-29 Thread via GitHub
codecov-commenter commented on PR #8924: URL: https://github.com/apache/cloudstack/pull/8924#issuecomment-2083629288 ## [Codecov](https://app.codecov.io/gh/apache/cloudstack/pull/8924?dropdown=coverage=pr=h1_medium=referral_source=github_content=comment_campaign=pr+comments_term=apache)

Re: [PR] Add logs to CPVM connection process [cloudstack]

2024-04-29 Thread via GitHub
bernardodemarco commented on code in PR #8924: URL: https://github.com/apache/cloudstack/pull/8924#discussion_r1583716662 ## services/console-proxy/server/src/main/java/com/cloud/consoleproxy/ConsoleProxyNoVNCHandler.java: ## @@ -104,20 +105,20 @@ public void onConnect(final

Re: [PR] Updates to HPE-Primera and Pure FlashArray Drivers to use Host-based VLUN Assignments [cloudstack]

2024-04-29 Thread via GitHub
rg9975 commented on code in PR #8889: URL: https://github.com/apache/cloudstack/pull/8889#discussion_r1583561237 ## plugins/hypervisors/kvm/src/main/java/com/cloud/hypervisor/kvm/storage/MultipathSCSIAdapterBase.java: ## @@ -223,15 +228,19 @@ public boolean

Re: [I] OVS Netplan yaml config Ubuntu 22.04 documentation [cloudstack]

2024-04-29 Thread via GitHub
Agility0493 commented on issue #8983: URL: https://github.com/apache/cloudstack/issues/8983#issuecomment-2083318127 We were able to remove all netplan errors with the following revisions to etc/netplan/01-netcfg.yaml, but networking is still not working after rebooting. These changes were

Re: [I] External DNS servers provided with use.external.dns disabled [cloudstack]

2024-04-29 Thread via GitHub
kriegsmanj commented on issue #9003: URL: https://github.com/apache/cloudstack/issues/9003#issuecomment-2083270720 In case of an isolated network, the resolvers configured are the Virtual Router IP and external dns1/dns2 in zone setting. In our environment we have no internal dns1/dns2

Re: [I] External DNS servers provided with use.external.dns disabled [cloudstack]

2024-04-29 Thread via GitHub
weizhouapache commented on issue #9003: URL: https://github.com/apache/cloudstack/issues/9003#issuecomment-2083259406 > > @kriegsmanj the description of the global setting is: `Bypass internal dns, use external dns1 and dns2` it looks like the global setting is used to determine if

Re: [PR] Fix null host tags while creating a zone [cloudstack]

2024-04-29 Thread via GitHub
weizhouapache commented on PR #8976: URL: https://github.com/apache/cloudstack/pull/8976#issuecomment-2083254280 > > good catch @GaOrtiga > > I wonder if it is better to consider all the null params > > ``` > > > > +Object.keys(args).forEach((key) => (args[key] == null)

Re: [PR] Ability to specify NFS mount options while adding a primary storage and modify them on a pre-existing primary storage [cloudstack]

2024-04-29 Thread via GitHub
abh1sar commented on code in PR #8947: URL: https://github.com/apache/cloudstack/pull/8947#discussion_r1583432991 ## server/src/main/java/com/cloud/storage/StoragePoolAutomationImpl.java: ## @@ -318,9 +323,12 @@ public boolean cancelMaintain(DataStore store) { if

Re: [I] External DNS servers provided with use.external.dns disabled [cloudstack]

2024-04-29 Thread via GitHub
kriegsmanj commented on issue #9003: URL: https://github.com/apache/cloudstack/issues/9003#issuecomment-2083227113 > @kriegsmanj the description of the global setting is: `Bypass internal dns, use external dns1 and dns2` it looks like the global setting is used to determine if internal dns

Re: [I] External DNS servers provided with use.external.dns disabled [cloudstack]

2024-04-29 Thread via GitHub
weizhouapache commented on issue #9003: URL: https://github.com/apache/cloudstack/issues/9003#issuecomment-2083218220 @kriegsmanj the description of the global setting is: `Bypass internal dns, use external dns1 and dns2` it looks like the global setting is used to determine if

(cloudstack) branch 4.19 updated: Fix: Update rootdisksize detail on restore VM (#8981)

2024-04-29 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 027149487b5 Fix: Update rootdisksize detail on

Re: [PR] Fix: Update rootdisksize detail on restore VM [cloudstack]

2024-04-29 Thread via GitHub
rohityadavcloud merged PR #8981: URL: https://github.com/apache/cloudstack/pull/8981 -- 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:

Re: [I] VMs not loading kvm and kvm-intel module at boot once if useing custom cpu [cloudstack]

2024-04-29 Thread via GitHub
weizhouapache commented on issue #9005: URL: https://github.com/apache/cloudstack/issues/9005#issuecomment-2083106401 @yashi4engg you need to add the following to agent.properties ``` guest.cpu.features=vmx ``` -- This is an automated message from the Apache Git Service.

Re: [PR] Updates to HPE-Primera and Pure FlashArray Drivers to use Host-based VLUN Assignments [cloudstack]

2024-04-29 Thread via GitHub
rg9975 commented on code in PR #8889: URL: https://github.com/apache/cloudstack/pull/8889#discussion_r1583298637 ## plugins/hypervisors/kvm/src/main/java/com/cloud/hypervisor/kvm/storage/MultipathSCSIAdapterBase.java: ## @@ -223,15 +228,19 @@ public boolean

[I] Webgui SSL import does not check on root certificate [cloudstack]

2024-04-29 Thread via GitHub
pcfriek1987 opened a new issue, #9004: URL: https://github.com/apache/cloudstack/issues/9004 # ISSUE TYPE * Bug Report # COMPONENT NAME ~~~ UI ~~~ # CLOUDSTACK VERSION ~~~ 4.19.0.1 ~~~ # CONFIGURATION

Re: [I] System VM / Agent on KVM host does not support NFSv3 on Secondary Storage [cloudstack]

2024-04-29 Thread via GitHub
Jayd603 commented on issue #8996: URL: https://github.com/apache/cloudstack/issues/8996#issuecomment-2082935486 It turns out, the NFS client returns protocol not supported when you try to have it mount a directory that does not exist, which makes no sense but ok. So this is not a bug in

Re: [I] System VM / Agent on KVM host does not support NFSv3 on Secondary Storage [cloudstack]

2024-04-29 Thread via GitHub
Jayd603 closed issue #8996: System VM / Agent on KVM host does not support NFSv3 on Secondary Storage URL: https://github.com/apache/cloudstack/issues/8996 -- 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

Re: [I] NFS mounting on VM [cloudstack]

2024-04-29 Thread via GitHub
alexandremattioli commented on issue #8990: URL: https://github.com/apache/cloudstack/issues/8990#issuecomment-2082854907 @Noelantogerorge the only sensible way to achieve this would be to add an interface to your NFS server in the same VLAN and subnet as your ACS network. In general you

[I] External DNS servers provided with use.external.dns disabled [cloudstack]

2024-04-29 Thread via GitHub
kriegsmanj opened a new issue, #9003: URL: https://github.com/apache/cloudstack/issues/9003 # ISSUE TYPE * Bug Report # COMPONENT NAME ~~~ VR / Isolated network ~~~ # CLOUDSTACK VERSION ~~~ 4.18.1.1 ~~~ #

Re: [PR] feature: webhooks [cloudstack]

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

Re: [I] (VirtualRouter) Provider is not ready, cannot Enable the provider, please configure the provider first [cloudstack]

2024-04-29 Thread via GitHub
Luskan777 commented on issue #8997: URL: https://github.com/apache/cloudstack/issues/8997#issuecomment-2082764350 HI @kiranchavala , thank you very much for your answer As it was an environment that was not yet in production, I reinstalled the management server, after reinstalling

Re: [I] (VirtualRouter) Provider is not ready, cannot Enable the provider, please configure the provider first [cloudstack]

2024-04-29 Thread via GitHub
Luskan777 closed issue #8997: (VirtualRouter) Provider is not ready, cannot Enable the provider, please configure the provider first URL: https://github.com/apache/cloudstack/issues/8997 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

Re: [PR] Fix null host tags while creating a zone [cloudstack]

2024-04-29 Thread via GitHub
GaOrtiga commented on PR #8976: URL: https://github.com/apache/cloudstack/pull/8976#issuecomment-2082654654 > good catch @GaOrtiga > > I wonder if it is better to consider all the null params > > ``` > > +Object.keys(args).forEach((key) => (args[key] == null) &&

Re: [PR] api,server: purge expunged resources [cloudstack]

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

Re: [PR] feature: webhooks [cloudstack]

2024-04-29 Thread via GitHub
blueorangutan commented on PR #8674: URL: https://github.com/apache/cloudstack/pull/8674#issuecomment-2082580063 @shwstppr 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] feature: webhooks [cloudstack]

2024-04-29 Thread via GitHub
shwstppr commented on PR #8674: URL: https://github.com/apache/cloudstack/pull/8674#issuecomment-2082575017 @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

Re: [PR] api,server: purge expunged resources [cloudstack]

2024-04-29 Thread via GitHub
blueorangutan commented on PR #8999: URL: https://github.com/apache/cloudstack/pull/8999#issuecomment-2082552171 @shwstppr 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. To

Re: [PR] api,server: purge expunged resources [cloudstack]

2024-04-29 Thread via GitHub
shwstppr commented on PR #8999: URL: https://github.com/apache/cloudstack/pull/8999#issuecomment-2082547969 @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] feature: webhooks [cloudstack]

2024-04-29 Thread via GitHub
blueorangutan commented on PR #8674: URL: https://github.com/apache/cloudstack/pull/8674#issuecomment-2082546701 [LL] Trillian Build Failed (tid-6920) -- 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

Re: [PR] api,server: purge expunged resources [cloudstack]

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

Re: [PR] api: improve listVM API handling and response speed [cloudstack]

2024-04-29 Thread via GitHub
GutoVeronezi commented on PR #8985: URL: https://github.com/apache/cloudstack/pull/8985#issuecomment-2082487250 > At the time I couldn't co-relate what made the API slow only to later find out through community and customer issues who are facing degraded performance. That is the main

Re: [PR] New feature: Implicit host tags [cloudstack]

2024-04-29 Thread via GitHub
weizhouapache commented on PR #8929: URL: https://github.com/apache/cloudstack/pull/8929#issuecomment-2082483387 @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

Re: [PR] feature: webhooks [cloudstack]

2024-04-29 Thread via GitHub
blueorangutan commented on PR #8674: URL: https://github.com/apache/cloudstack/pull/8674#issuecomment-2082475422 [LL] Trillian Build Failed (tid-6919) -- 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

[I] Default systemVM template is not downloaded to S3 secondary storage [cloudstack]

2024-04-29 Thread via GitHub
rajujith opened a new issue, #9002: URL: https://github.com/apache/cloudstack/issues/9002 # ISSUE TYPE * Bug Report # COMPONENT NAME ~~~ S3 Secondary storage, system VM template ~~~ # CLOUDSTACK VERSION ~~~ 4.19 with

Re: [PR] api,server: purge expunged resources [cloudstack]

2024-04-29 Thread via GitHub
blueorangutan commented on PR #8999: URL: https://github.com/apache/cloudstack/pull/8999#issuecomment-2082413547 @shwstppr 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] api,server: purge expunged resources [cloudstack]

2024-04-29 Thread via GitHub
shwstppr commented on PR #8999: URL: https://github.com/apache/cloudstack/pull/8999#issuecomment-2082410439 @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

(cloudstack) branch main updated: Fix build error with MigrateKVMAsync

2024-04-29 Thread weizhou
This is an automated email from the ASF dual-hosted git repository. weizhou pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/cloudstack.git The following commit(s) were added to refs/heads/main by this push: new f41c934c94a Fix build error with

Re: [I] apacheCloudStack证书 [cloudstack]

2024-04-29 Thread via GitHub
boring-cyborg[bot] commented on issue #9001: URL: https://github.com/apache/cloudstack/issues/9001#issuecomment-2082371075 Thanks for opening your first issue here! Be sure to follow the issue template! -- This is an automated message from the Apache Git Service. To respond to the

[I] apacheCloudStack证书 [cloudstack]

2024-04-29 Thread via GitHub
123qeqe45 opened a new issue, #9001: URL: https://github.com/apache/cloudstack/issues/9001 # ISSUE TYPE * Bug Report * Improvement Request * Enhancement Request * Feature Idea * Documentation Report * Other # COMPONENT NAME ~~~

Re: [PR] UI: Keyboard Shortcut(s) [cloudstack]

2024-04-29 Thread via GitHub
vladimirpetrov commented on PR #8250: URL: https://github.com/apache/cloudstack/pull/8250#issuecomment-2082353265 Well, I think we should be consistent - wherever there is a 'Refresh' button, the shortcut must work, so yes, @DaanHoogland -- This is an automated message from the Apache

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

2024-04-29 Thread via GitHub
abh1sar commented on PR #8875: URL: https://github.com/apache/cloudstack/pull/8875#issuecomment-2082322949 @shwstppr @weizhouapache I have added some UT to increase coverage. Please check. -- This is an automated message from the Apache Git Service. To respond to the message, please

Re: [PR] VNF: add API listVnfAppliances [cloudstack]

2024-04-29 Thread via GitHub
blueorangutan commented on PR #8994: URL: https://github.com/apache/cloudstack/pull/8994#issuecomment-2082317850 @weizhouapache 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. To

Re: [PR] VNF: add API listVnfAppliances [cloudstack]

2024-04-29 Thread via GitHub
weizhouapache commented on PR #8994: URL: https://github.com/apache/cloudstack/pull/8994#issuecomment-2082316295 @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] Normalizing volume consolidation in live migration on KVM [cloudstack]

2024-04-29 Thread via GitHub
github-actions[bot] commented on PR #8911: URL: https://github.com/apache/cloudstack/pull/8911#issuecomment-2082309367 This pull request has merge conflicts. Dear author, please fix the conflicts and sync your branch with the base branch. -- This is an automated message from the Apache

Re: [PR] VNF: add API listVnfAppliances [cloudstack]

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

(cloudstack) branch main updated (54926c324ff -> e61f3bae4da)

2024-04-29 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.git from 54926c324ff sanatise log in case multiple passwordsa have been passed (#8891) add e409c6d870a Fixup listing of

(cloudstack) 01/01: Merge branch '4.19'

2024-04-29 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.git commit e61f3bae4daf752b3d4cf23d1c4685e074d07c4d Merge: 54926c324ff 3de1f8b4baf Author: Daan Hoogland AuthorDate: Mon Apr 29

Re: [PR] linstor: cleanup diskless nodes on disconnect [cloudstack]

2024-04-29 Thread via GitHub
DaanHoogland commented on PR #8790: URL: https://github.com/apache/cloudstack/pull/8790#issuecomment-2082231182 I think I got it right and running a test compile now. if that passes , I'll push. -- This is an automated message from the Apache Git Service. To respond to the message,

Re: [PR] linstor: cleanup diskless nodes on disconnect [cloudstack]

2024-04-29 Thread via GitHub
rp- commented on PR #8790: URL: https://github.com/apache/cloudstack/pull/8790#issuecomment-2082221255 > @rp- , this is causing some conflicts in merge forward. Can you advise how `tryDisconnectLinstor` should be implemented in main? > > I think I know but please review in main once

Re: [PR] Fix message publish in transaction [cloudstack]

2024-04-29 Thread via GitHub
blueorangutan commented on PR #8980: URL: https://github.com/apache/cloudstack/pull/8980#issuecomment-2082220230 @borisstoyanov 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. To

Re: [PR] Fix message publish in transaction [cloudstack]

2024-04-29 Thread via GitHub
borisstoyanov commented on PR #8980: URL: https://github.com/apache/cloudstack/pull/8980#issuecomment-2082216656 @blueorangutan test keepEnv -- 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

Re: [PR] api: improve listVM API handling and response speed [cloudstack]

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

Re: [PR] UI: hide Cleanup/Expunge when delete a CloudManaged CKS cluster [cloudstack]

2024-04-29 Thread via GitHub
weizhouapache commented on PR #8593: URL: https://github.com/apache/cloudstack/pull/8593#issuecomment-2082191087 Merging thanks @DaanHoogland @vishesh92 for review -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

(cloudstack) branch 4.19 updated: UI: hide Cleanup/Expunge when delete a CloudManaged CKS cluster (#8593)

2024-04-29 Thread weizhou
This is an automated email from the ASF dual-hosted git repository. weizhou 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 f7b95cd8f06 UI: hide Cleanup/Expunge when

Re: [PR] linstor: cleanup diskless nodes on disconnect [cloudstack]

2024-04-29 Thread via GitHub
DaanHoogland commented on PR #8790: URL: https://github.com/apache/cloudstack/pull/8790#issuecomment-2082191504 @rp- , this is causing some conflicts in merge forward. Can you advise how `tryDisconnectLinstor` should be implemented in main? -- This is an automated message from the Apache

Re: [PR] UI: hide Cleanup/Expunge when delete a CloudManaged CKS cluster [cloudstack]

2024-04-29 Thread via GitHub
weizhouapache merged PR #8593: URL: https://github.com/apache/cloudstack/pull/8593 -- 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:

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

2024-04-29 Thread via GitHub
DaanHoogland commented on PR #8602: URL: https://github.com/apache/cloudstack/pull/8602#issuecomment-2082157752 @blueorangutan test matrix -- 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

(cloudstack) 01/01: Merge remote-tracking branch 'origin/4.18' into 4.19

2024-04-29 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 commit 3de1f8b4baf2af1b1cd5387aff4024a1fdd5a8ec Merge: b23ceefc7ea 9d5d4e55648 Author: Rohit Yadav AuthorDate: Mon Apr 29

(cloudstack) branch 4.19 updated (b23ceefc7ea -> 3de1f8b4baf)

2024-04-29 Thread rohit
This is an automated email from the ASF dual-hosted git repository. rohit pushed a change to branch 4.19 in repository https://gitbox.apache.org/repos/asf/cloudstack.git from b23ceefc7ea utils: cleanup MacAddress and MacAddressTest (#8988) add 0e08a126dfd systemvm: add

Re: [PR] VNF: add API listVnfAppliances [cloudstack]

2024-04-29 Thread via GitHub
blueorangutan commented on PR #8994: URL: https://github.com/apache/cloudstack/pull/8994#issuecomment-2082154575 @weizhouapache 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

Re: [PR] VNF: add API listVnfAppliances [cloudstack]

2024-04-29 Thread via GitHub
weizhouapache commented on PR #8994: URL: https://github.com/apache/cloudstack/pull/8994#issuecomment-2082151640 @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

Re: [PR] framework/db: introduce a new MySQL table based distributed lock [cloudstack]

2024-04-29 Thread via GitHub
github-actions[bot] commented on PR #8968: URL: https://github.com/apache/cloudstack/pull/8968#issuecomment-2082151345 This pull request has merge conflicts. Dear author, please fix the conflicts and sync your branch with the base branch. -- This is an automated message from the Apache

Re: [PR] VNF: add API listVnfAppliances [cloudstack]

2024-04-29 Thread via GitHub
weizhouapache commented on code in PR #8994: URL: https://github.com/apache/cloudstack/pull/8994#discussion_r1582702301 ## server/src/main/java/com/cloud/api/query/QueryManagerImpl.java: ## @@ -1409,6 +1410,12 @@ private Pair, Integer> searchForUserVMIdsAndCount(ListVMsCmd

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

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

Re: [PR] Use parameter dcId as wrapper to prevent NPE [cloudstack]

2024-04-29 Thread via GitHub
blueorangutan commented on PR #8986: URL: https://github.com/apache/cloudstack/pull/8986#issuecomment-2082131162 @DaanHoogland a [SL] Trillian-Jenkins test job (alma9 mgmt + vmware-70u3) has been kicked to run smoke tests -- This is an automated message from the Apache Git Service. To

Re: [PR] Use parameter dcId as wrapper to prevent NPE [cloudstack]

2024-04-29 Thread via GitHub
DaanHoogland commented on PR #8986: URL: https://github.com/apache/cloudstack/pull/8986#issuecomment-2082127555 @blueorangutan test alma9 vmware-70u3 -- 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

(cloudstack) branch 4.19 updated: utils: cleanup MacAddress and MacAddressTest (#8988)

2024-04-29 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 b23ceefc7ea utils: cleanup MacAddress and

Re: [I] cloudstack-sysvmadm - /etc/rc.d/init.d/functions: No such file or directory [cloudstack]

2024-04-29 Thread via GitHub
dR3b commented on issue #9000: URL: https://github.com/apache/cloudstack/issues/9000#issuecomment-2082122111 I think `/etc/rc.d` was only available on Redhat and with Systemd it is obsolete. -- This is an automated message from the Apache Git Service. To respond to the message, please

Re: [PR] utils: cleanup MacAddress and MacAddressTest [cloudstack]

2024-04-29 Thread via GitHub
rohityadavcloud merged PR #8988: URL: https://github.com/apache/cloudstack/pull/8988 -- 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:

Re: [PR] utils: cleanup MacAddress and MacAddressTest [cloudstack]

2024-04-29 Thread via GitHub
weizhouapache commented on PR #8988: URL: https://github.com/apache/cloudstack/pull/8988#issuecomment-2082119686 code lgtm deployed it on an existing environment, there is no new management server added to `mshost` table -- This is an automated message from the Apache Git

Re: [PR] feature: webhooks [cloudstack]

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

Re: [PR] VNF: add API listVnfAppliances [cloudstack]

2024-04-29 Thread via GitHub
weizhouapache commented on code in PR #8994: URL: https://github.com/apache/cloudstack/pull/8994#discussion_r1582679193 ## server/src/main/java/com/cloud/api/query/QueryManagerImpl.java: ## @@ -1409,6 +1410,12 @@ private Pair, Integer> searchForUserVMIdsAndCount(ListVMsCmd

Re: [PR] VNF: add API listVnfAppliances [cloudstack]

2024-04-29 Thread via GitHub
DaanHoogland commented on code in PR #8994: URL: https://github.com/apache/cloudstack/pull/8994#discussion_r1582677680 ## server/src/main/java/com/cloud/api/query/QueryManagerImpl.java: ## @@ -1409,6 +1410,12 @@ private Pair, Integer> searchForUserVMIdsAndCount(ListVMsCmd cmd)

Re: [I] cloudstack-sysvmadm - /etc/rc.d/init.d/functions: No such file or directory [cloudstack]

2024-04-29 Thread via GitHub
DaanHoogland commented on issue #9000: URL: https://github.com/apache/cloudstack/issues/9000#issuecomment-2082099495 @dR3b sounds like a dependency is missing, or is this file phased out in a maintenance release of ubuntu? -- This is an automated message from the Apache Git Service. To

Re: [PR] Fix spelling, word case, duplicate words in code comments only [cloudstack]

2024-04-29 Thread via GitHub
DaanHoogland commented on PR #8987: URL: https://github.com/apache/cloudstack/pull/8987#issuecomment-2082094586 are you still working on this @jbampton ? -- 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

Re: [PR] api: improve listVM API handling and response speed [cloudstack]

2024-04-29 Thread via GitHub
blueorangutan commented on PR #8985: URL: https://github.com/apache/cloudstack/pull/8985#issuecomment-2082091391 @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

Re: [PR] api: improve listVM API handling and response speed [cloudstack]

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

Re: [PR] Use parameter dcId as wrapper to prevent NPE [cloudstack]

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

  1   2   >