[GitHub] [cloudstack] rvalle commented on issue #4270: conserve-mode for Public IPs in VPCs.

2021-01-29 Thread GitBox
rvalle commented on issue #4270: URL: https://github.com/apache/cloudstack/issues/4270#issuecomment-770169746 @DaanHoogland yes, I agree. I am not coding on this issue. I am currently using VPCs with the workaround that I explain above. My coding time is mostly on the

[GitHub] [cloudstack] levindecaro opened a new issue #4632: Problem on selecting "Tier" dropdown and pagination error in "Port forward" UI

2021-01-29 Thread GitBox
levindecaro opened a new issue #4632: URL: https://github.com/apache/cloudstack/issues/4632 # ISSUE TYPE * Bug Report # COMPONENT NAME ~~~ UI ~~~ # CLOUDSTACK VERSION ~~~ 4.15 ~~~ # CONFIGURATION N/A

[GitHub] [cloudstack] weizhouapache commented on a change in pull request #4032: Suspending the VM prior to deleting snapshots to avoid corruption, th…

2021-01-29 Thread GitBox
weizhouapache commented on a change in pull request #4032: URL: https://github.com/apache/cloudstack/pull/4032#discussion_r567104868 ## File path: plugins/hypervisors/kvm/src/main/java/com/cloud/hypervisor/kvm/resource/wrapper/LibvirtDeleteVMSnapshotCommandWrapper.java ## @@

[GitHub] [cloudstack] blueorangutan commented on pull request #4626: server: fix failed to remove template/iso if upload from local fails

2021-01-29 Thread GitBox
blueorangutan commented on pull request #4626: URL: https://github.com/apache/cloudstack/pull/4626#issuecomment-770025172 Trillian test result (tid-3453) Environment: vmware-67u3 (x2), Advanced Networking with Mgmt server 7 Total time taken: 45950 seconds Marvin logs:

[GitHub] [cloudstack] blueorangutan commented on pull request #4624: server: fix wrong error message when create isolated network without SourceNat

2021-01-29 Thread GitBox
blueorangutan commented on pull request #4624: URL: https://github.com/apache/cloudstack/pull/4624#issuecomment-769991086 Trillian test result (tid-3452) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 41917 seconds Marvin logs:

[GitHub] [cloudstack] DaanHoogland commented on issue #4187: Migration not working in ACS 4.14 with Basic Network - VMware Local storage

2021-01-29 Thread GitBox
DaanHoogland commented on issue #4187: URL: https://github.com/apache/cloudstack/issues/4187#issuecomment-769910914 @harikrishna-patnala @nvazquez can you comment? This is an automated message from the Apache Git Service. To

[GitHub] [cloudstack] DaanHoogland commented on issue #4492: Deploy VM from Snapshot

2021-01-29 Thread GitBox
DaanHoogland commented on issue #4492: URL: https://github.com/apache/cloudstack/issues/4492#issuecomment-769910268 @shwstppr @davidjumani what do you think? This is an automated message from the Apache Git Service. To

[GitHub] [cloudstack] blueorangutan commented on pull request #4631: fix: create template with Marvin

2021-01-29 Thread GitBox
blueorangutan commented on pull request #4631: URL: https://github.com/apache/cloudstack/pull/4631#issuecomment-769882464 Packaging result: ✔centos7 ✔centos8 ✔debian. JID-2622 This is an automated message from the Apache Git

[GitHub] [cloudstack] blueorangutan commented on pull request #4630: disable hot add memory and cpu via vm settings

2021-01-29 Thread GitBox
blueorangutan commented on pull request #4630: URL: https://github.com/apache/cloudstack/pull/4630#issuecomment-769877142 Packaging result: ✔centos7 ✔centos8 ✔debian. JID-2621 This is an automated message from the Apache Git

[GitHub] [cloudstack] ccycv commented on issue #4187: Migration not working in ACS 4.14 with Basic Network - VMware Local storage

2021-01-29 Thread GitBox
ccycv commented on issue #4187: URL: https://github.com/apache/cloudstack/issues/4187#issuecomment-769876493 > @ccycv can youtest this with 4.15 please? I still have the same issue, "failed to get message for exception: File

[GitHub] [cloudstack] ccycv removed a comment on issue #4187: Migration not working in ACS 4.14 with Basic Network - VMware Local storage

2021-01-29 Thread GitBox
ccycv removed a comment on issue #4187: URL: https://github.com/apache/cloudstack/issues/4187#issuecomment-769876247 > @ccycv can youtest this with 4.15 please? I still have the same issue, "failed to get message for exception: File

[GitHub] [cloudstack] ccycv commented on issue #4187: Migration not working in ACS 4.14 with Basic Network - VMware Local storage

2021-01-29 Thread GitBox
ccycv commented on issue #4187: URL: https://github.com/apache/cloudstack/issues/4187#issuecomment-769876247 > @ccycv can youtest this with 4.15 please? I still have the same issue, "failed to get message for exception: File

[GitHub] [cloudstack] hstan77 commented on issue #4492: Deploy VM from Snapshot

2021-01-29 Thread GitBox
hstan77 commented on issue #4492: URL: https://github.com/apache/cloudstack/issues/4492#issuecomment-769867044 >> You are asking to unify the createTemplateFromSnapshot and deployTemplate steps into one, correct? Yes, thats right . On Fri, Jan 29, 2021 at 10:16 PM dahn

[GitHub] [cloudstack] hstan77 commented on issue #4391: Allow multiple localstorage based on VolumeGroup

2021-01-29 Thread GitBox
hstan77 commented on issue #4391: URL: https://github.com/apache/cloudstack/issues/4391#issuecomment-769862291 Currently for shared mount point , it need to be run on Clustered Filesystem, such as GFS2 or ocfs2 , OCFS2 is seems support till centos 6 only . GFS2 has not easy to

[GitHub] [cloudstack] ggoodrich-ipp commented on pull request #4137: Adding VPN options for IKE version and IKE split connections.

2021-01-29 Thread GitBox
ggoodrich-ipp commented on pull request #4137: URL: https://github.com/apache/cloudstack/pull/4137#issuecomment-769858118 > @ggoodrich-ipp all the ui conflicts are to files that should now moved to ui/legacy. However I connot see them on github. (cc @rhtyd ?) I believe they've been

[GitHub] [cloudstack] DK101010 commented on a change in pull request #4630: disable hot add memory and cpu via vm settings

2021-01-29 Thread GitBox
DK101010 commented on a change in pull request #4630: URL: https://github.com/apache/cloudstack/pull/4630#discussion_r566875797 ## File path: plugins/hypervisors/vmware/src/main/java/com/cloud/hypervisor/vmware/resource/VmwareResource.java ## @@ -1959,15 +1959,15 @@ protected

[GitHub] [cloudstack] blueorangutan commented on pull request #4631: fix: create template with Marvin

2021-01-29 Thread GitBox
blueorangutan commented on pull request #4631: URL: https://github.com/apache/cloudstack/pull/4631#issuecomment-769850556 @DaanHoogland a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

[GitHub] [cloudstack] DaanHoogland edited a comment on pull request #4631: fix: create template with Marvin

2021-01-29 Thread GitBox
DaanHoogland edited a comment on pull request #4631: URL: https://github.com/apache/cloudstack/pull/4631#issuecomment-769849967 @blueorangutan package This is an automated message from the Apache Git Service. To respond to

[GitHub] [cloudstack] DaanHoogland commented on pull request #4631: fix: create template with Marvin

2021-01-29 Thread GitBox
DaanHoogland commented on pull request #4631: URL: https://github.com/apache/cloudstack/pull/4631#issuecomment-769849967 @blueorangutan This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [cloudstack] DaanHoogland commented on a change in pull request #4630: disable hot add memory and cpu via vm settings

2021-01-29 Thread GitBox
DaanHoogland commented on a change in pull request #4630: URL: https://github.com/apache/cloudstack/pull/4630#discussion_r566868813 ## File path: plugins/hypervisors/vmware/src/main/java/com/cloud/hypervisor/vmware/resource/VmwareResource.java ## @@ -1959,15 +1959,15 @@

[GitHub] [cloudstack] blueorangutan commented on pull request #4630: disable hot add memory and cpu via vm settings

2021-01-29 Thread GitBox
blueorangutan commented on pull request #4630: URL: https://github.com/apache/cloudstack/pull/4630#issuecomment-769844693 @DaanHoogland a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

[GitHub] [cloudstack] DaanHoogland commented on pull request #4630: disable hot add memory and cpu via vm settings

2021-01-29 Thread GitBox
DaanHoogland commented on pull request #4630: URL: https://github.com/apache/cloudstack/pull/4630#issuecomment-769844275 @blueorangutan package This is an automated message from the Apache Git Service. To respond to the

[GitHub] [cloudstack] DaanHoogland commented on issue #3897: Review all list APIs and make then honour listall parameter

2021-01-29 Thread GitBox
DaanHoogland commented on issue #3897: URL: https://github.com/apache/cloudstack/issues/3897#issuecomment-769832484 @GabrielBrascher I don't mind progressively solving this but I have two worries: 1. this issue drags on for the longest while 2. consistant implementation is going to

[GitHub] [cloudstack] sureshanaparti commented on a change in pull request #4561: [WIP] Support for persistence mode in L2 networks

2021-01-29 Thread GitBox
sureshanaparti commented on a change in pull request #4561: URL: https://github.com/apache/cloudstack/pull/4561#discussion_r566850541 ## File path: plugins/hypervisors/vmware/src/main/java/com/cloud/hypervisor/vmware/resource/VmwareResource.java ## @@ -617,6 +621,21 @@ public

[GitHub] [cloudstack] DaanHoogland commented on issue #4492: Deploy VM from Snapshot

2021-01-29 Thread GitBox
DaanHoogland commented on issue #4492: URL: https://github.com/apache/cloudstack/issues/4492#issuecomment-769830295 ok, but as @andrijapanicsb described, the undelying workflow should still involve a template creation. so this is really a UI feature than; You are asking to unify the

[GitHub] [cloudstack] sureshanaparti commented on a change in pull request #4561: [WIP] Support for persistence mode in L2 networks

2021-01-29 Thread GitBox
sureshanaparti commented on a change in pull request #4561: URL: https://github.com/apache/cloudstack/pull/4561#discussion_r566849808 ## File path: plugins/hypervisors/kvm/src/main/java/com/cloud/hypervisor/kvm/resource/wrapper/LibvirtUnPlugNicCommandWrapper.java ## @@ -82,4

[GitHub] [cloudstack] GabrielBrascher commented on issue #3897: Review all list APIs and make then honour listall parameter

2021-01-29 Thread GitBox
GabrielBrascher commented on issue #3897: URL: https://github.com/apache/cloudstack/issues/3897#issuecomment-769829424 @rhtyd @DaanHoogland do you think that PR #4276 should extend its goals to more APIs? Or is it ok to do it "progressively" with different PRs addressing this

[GitHub] [cloudstack] DaanHoogland commented on issue #4391: Allow multiple localstorage based on VolumeGroup

2021-01-29 Thread GitBox
DaanHoogland commented on issue #4391: URL: https://github.com/apache/cloudstack/issues/4391#issuecomment-769828792 @hstan77 local is local filesystem based not file based any LGLVM that has a filesystem on it could be used. How filesystems are mounted locally on the hypervisor would not

[GitHub] [cloudstack] DaanHoogland commented on a change in pull request #4625: server: throw exception when update vm nic on L2 network

2021-01-29 Thread GitBox
DaanHoogland commented on a change in pull request #4625: URL: https://github.com/apache/cloudstack/pull/4625#discussion_r566845488 ## File path: server/src/main/java/com/cloud/vm/UserVmManagerImpl.java ## @@ -1648,8 +1648,7 @@ public void

[GitHub] [cloudstack] sureshanaparti commented on a change in pull request #4561: [WIP] Support for persistence mode in L2 networks

2021-01-29 Thread GitBox
sureshanaparti commented on a change in pull request #4561: URL: https://github.com/apache/cloudstack/pull/4561#discussion_r566845457 ## File path: engine/schema/src/main/java/com/cloud/network/dao/NetworkDaoImpl.java ## @@ -265,6 +276,8 @@ protected void init() {

[GitHub] [cloudstack] sureshanaparti commented on a change in pull request #4561: [WIP] Support for persistence mode in L2 networks

2021-01-29 Thread GitBox
sureshanaparti commented on a change in pull request #4561: URL: https://github.com/apache/cloudstack/pull/4561#discussion_r566844799 ## File path: engine/orchestration/src/main/java/org/apache/cloudstack/engine/orchestration/NetworkOrchestrator.java ## @@ -1187,6 +1202,86 @@

[GitHub] [cloudstack] sureshanaparti commented on a change in pull request #4561: [WIP] Support for persistence mode in L2 networks

2021-01-29 Thread GitBox
sureshanaparti commented on a change in pull request #4561: URL: https://github.com/apache/cloudstack/pull/4561#discussion_r566842841 ## File path: engine/orchestration/src/main/java/org/apache/cloudstack/engine/orchestration/NetworkOrchestrator.java ## @@ -2864,6 +2962,33 @@

[GitHub] [cloudstack] DaanHoogland commented on pull request #4032: Suspending the VM prior to deleting snapshots to avoid corruption, th…

2021-01-29 Thread GitBox
DaanHoogland commented on pull request #4032: URL: https://github.com/apache/cloudstack/pull/4032#issuecomment-769822337 re-opening to re-kick travis This is an automated message from the Apache Git Service. To respond to

[GitHub] [cloudstack] ggoodrich-ipp opened a new pull request #4032: Suspending the VM prior to deleting snapshots to avoid corruption, th…

2021-01-29 Thread GitBox
ggoodrich-ipp opened a new pull request #4032: URL: https://github.com/apache/cloudstack/pull/4032 …en resuming ## Description These changes are related to PR #3194, but include suspending/resuming the VM when doing a VM snapshot as well, when deleting a VM snapshot, as it is

[GitHub] [cloudstack] DaanHoogland closed pull request #4032: Suspending the VM prior to deleting snapshots to avoid corruption, th…

2021-01-29 Thread GitBox
DaanHoogland closed pull request #4032: URL: https://github.com/apache/cloudstack/pull/4032 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

[GitHub] [cloudstack] DaanHoogland commented on pull request #4032: Suspending the VM prior to deleting snapshots to avoid corruption, th…

2021-01-29 Thread GitBox
DaanHoogland commented on pull request #4032: URL: https://github.com/apache/cloudstack/pull/4032#issuecomment-769822134 travis is reporting a checkstyle error but i cannot find it closing This is an automated message from

[GitHub] [cloudstack] hstan77 commented on issue #4492: Deploy VM from Snapshot

2021-01-29 Thread GitBox
hstan77 commented on issue #4492: URL: https://github.com/apache/cloudstack/issues/4492#issuecomment-769821514 Yes, this is the alternative is work for me. But my intention is only to make the UI more easy for end user where end user can finish the job in one step instead of two.

[GitHub] [cloudstack] hstan77 commented on issue #4391: Allow multiple localstorage based on VolumeGroup

2021-01-29 Thread GitBox
hstan77 commented on issue #4391: URL: https://github.com/apache/cloudstack/issues/4391#issuecomment-769818996 Hi I understand Andrija said . Just I hope there is opton of LVM as localstorage insteaod of file based. On Fri, Jan 29, 2021 at 9:01 PM dahn wrote: >

[GitHub] [cloudstack] weizhouapache commented on pull request #4627: VR: fix expunging vm will remove dhcp entries of another vm in VR

2021-01-29 Thread GitBox
weizhouapache commented on pull request #4627: URL: https://github.com/apache/cloudstack/pull/4627#issuecomment-769813924 > @weizhouapache in final step. After expunging second VM, VR still had an entry for the first VM. > Also just for info, I was testing with an isolated network.

[GitHub] [cloudstack] DaanHoogland commented on issue #3717: Host HA is configre fail

2021-01-29 Thread GitBox
DaanHoogland commented on issue #3717: URL: https://github.com/apache/cloudstack/issues/3717#issuecomment-769809374 @div8cn is this still an issue in 4.15? This is an automated message from the Apache Git Service. To respond

[GitHub] [cloudstack] DaanHoogland commented on issue #3689: Add LDAP account button is not being shown when ldap configuration is added at domain level

2021-01-29 Thread GitBox
DaanHoogland commented on issue #3689: URL: https://github.com/apache/cloudstack/issues/3689#issuecomment-769808999 @andrijapanicsb -dont-use-this-one do you have ideas on use cases for this? This is an automated message

[GitHub] [cloudstack] DaanHoogland commented on issue #3677: linkAccountToLDAP must either use role or account type

2021-01-29 Thread GitBox
DaanHoogland commented on issue #3677: URL: https://github.com/apache/cloudstack/issues/3677#issuecomment-769805978 To be clear @borisstoyanov you want the API linkAccountToLdap to take a role instead of an accounttype, am I right? so this account must not existe yet or already have the

[GitHub] [cloudstack] DaanHoogland commented on issue #3664: Instances status out of sync with hypervisor VM's

2021-01-29 Thread GitBox
DaanHoogland commented on issue #3664: URL: https://github.com/apache/cloudstack/issues/3664#issuecomment-769802677 @AndreaGHG as 4.15 is out and you didn't reply I'm assuming this can be closed, please re-open if i'm in err

[GitHub] [cloudstack] DaanHoogland closed issue #3664: Instances status out of sync with hypervisor VM's

2021-01-29 Thread GitBox
DaanHoogland closed issue #3664: URL: https://github.com/apache/cloudstack/issues/3664 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

[GitHub] [cloudstack] shwstppr commented on pull request #4627: VR: fix expunging vm will remove dhcp entries of another vm in VR

2021-01-29 Thread GitBox
shwstppr commented on pull request #4627: URL: https://github.com/apache/cloudstack/pull/4627#issuecomment-769795449 @weizhouapache in final step. After expunging second VM, VR still had an entry for the first VM. Also just for info, I was testing with an isolated network.

[GitHub] [cloudstack] DaanHoogland commented on issue #4270: Multiple purposes for Public IPs in VPCs.

2021-01-29 Thread GitBox
DaanHoogland commented on issue #4270: URL: https://github.com/apache/cloudstack/issues/4270#issuecomment-769795036 @rvalle this seems related to #4454 would you agree? I'll rename this to "conserve-mode in vpc" as it describes the issue correctly. btw, are you working on this

[GitHub] [cloudstack] DaanHoogland commented on issue #4391: Allow multiple localstorage based on VolumeGroup

2021-01-29 Thread GitBox
DaanHoogland commented on issue #4391: URL: https://github.com/apache/cloudstack/issues/4391#issuecomment-769791713 @hstan77 I'm afraid this issue is still not well defined. Do you have an update? This is an automated

[GitHub] [cloudstack] DaanHoogland closed issue #4504: storage.template.OVAProcessor running on KVM templates

2021-01-29 Thread GitBox
DaanHoogland closed issue #4504: URL: https://github.com/apache/cloudstack/issues/4504 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

[GitHub] [cloudstack] DaanHoogland commented on issue #4504: storage.template.OVAProcessor running on KVM templates

2021-01-29 Thread GitBox
DaanHoogland commented on issue #4504: URL: https://github.com/apache/cloudstack/issues/4504#issuecomment-769789776 closing this as it is an architectural feature; for several processing units lists of backend specific code is presented the option to process. This is then assuming only

[GitHub] [cloudstack] DaanHoogland commented on issue #4286: Management network on VLAN

2021-01-29 Thread GitBox
DaanHoogland commented on issue #4286: URL: https://github.com/apache/cloudstack/issues/4286#issuecomment-769788212 @shwstppr @davidjumani do you think this is feasible in a near version? @felipegabriel could you try to make your feature request more prozaic?

[GitHub] [cloudstack] slavkap opened a new pull request #4631: fix: create template with Marvin

2021-01-29 Thread GitBox
slavkap opened a new pull request #4631: URL: https://github.com/apache/cloudstack/pull/4631 Fixes problems with: * undefined params * if we define those params when invoking the functions create_from_volume/create_from_snapshot will be created two templates * according to the

[GitHub] [cloudstack] DaanHoogland commented on issue #4492: Deploy VM from Snapshot

2021-01-29 Thread GitBox
DaanHoogland commented on issue #4492: URL: https://github.com/apache/cloudstack/issues/4492#issuecomment-769784371 @hstan77 is the scenario where you first have to create a template and deploy from that good enough to you, and a proper alternative to your request?

[GitHub] [cloudstack] DaanHoogland closed issue #4347: CS simulator docker hub image issues

2021-01-29 Thread GitBox
DaanHoogland closed issue #4347: URL: https://github.com/apache/cloudstack/issues/4347 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

[GitHub] [cloudstack] DaanHoogland commented on issue #4347: CS simulator docker hub image issues

2021-01-29 Thread GitBox
DaanHoogland commented on issue #4347: URL: https://github.com/apache/cloudstack/issues/4347#issuecomment-769775219 @lmhetz either dev@ or us...@cloudstack.apache.org This is an automated message from the Apache Git Service.

[GitHub] [cloudstack] weizhouapache commented on pull request #4627: VR: fix expunging vm will remove dhcp entries of another vm in VR

2021-01-29 Thread GitBox
weizhouapache commented on pull request #4627: URL: https://github.com/apache/cloudstack/pull/4627#issuecomment-769764378 > I'm not able to reproduce the issue on a KVM and a VMware environment. Can someone else verify @vladimirpetrov @nvazquez @rhtyd @shwstppr did you follow the

[GitHub] [cloudstack] DaanHoogland commented on pull request #4137: Adding VPN options for IKE version and IKE split connections.

2021-01-29 Thread GitBox
DaanHoogland commented on pull request #4137: URL: https://github.com/apache/cloudstack/pull/4137#issuecomment-769760426 @ggoodrich-ipp all the ui conflicts are to files that should now moved to ui/legacy. However I connot see them on github. (cc @rhtyd ?)

[GitHub] [cloudstack] soreana commented on issue #4093: Centralised logging via rsyslog

2021-01-29 Thread GitBox
soreana commented on issue #4093: URL: https://github.com/apache/cloudstack/issues/4093#issuecomment-769758089 No, they are legitimate requirements :D This is an automated message from the Apache Git Service. To respond to

[GitHub] [cloudstack] nvazquez commented on issue #4344: Changing disk controller type is not working for root disk or data disk

2021-01-29 Thread GitBox
nvazquez commented on issue #4344: URL: https://github.com/apache/cloudstack/issues/4344#issuecomment-769757349 @harikrishna-patnala @rhtyd this affects 4.14 and starting from 4.15 only to VMs deployed from older templates (not using the deploy-as-is logic)

[GitHub] [cloudstack] DaanHoogland commented on issue #4093: Centralised logging via rsyslog

2021-01-29 Thread GitBox
DaanHoogland commented on issue #4093: URL: https://github.com/apache/cloudstack/issues/4093#issuecomment-769757151 > @DaanHoogland Honestly, it took a lot of time to apply the changes you required to merging this pr. I discussed it with my colleagues, and we concluded that it isn't our

[GitHub] [cloudstack] DaanHoogland commented on issue #4093: Centralised logging via rsyslog

2021-01-29 Thread GitBox
DaanHoogland commented on issue #4093: URL: https://github.com/apache/cloudstack/issues/4093#issuecomment-769756854 tnx for your efforts @soreana , i'm marking it as unplanned for now, feel free to come back at any time of your convenience.

[GitHub] [cloudstack] DaanHoogland commented on issue #3742: migratewait does not seem to be honored in KVM

2021-01-29 Thread GitBox
DaanHoogland commented on issue #3742: URL: https://github.com/apache/cloudstack/issues/3742#issuecomment-769755359 ok, so we can plan this for coming versions. I'll put it to 4.16 for now, let's change if needed. This is

[GitHub] [cloudstack] DaanHoogland commented on issue #4454: VPC SourceNat with Port Forwarding

2021-01-29 Thread GitBox
DaanHoogland commented on issue #4454: URL: https://github.com/apache/cloudstack/issues/4454#issuecomment-769754068 @alexandru-bagu are you still planning work on this (for 4.16) or should we archive the issue/mark it as unplanned?

[GitHub] [cloudstack] alitheg closed issue #4277: Use database name other than "cloud"

2021-01-29 Thread GitBox
alitheg closed issue #4277: URL: https://github.com/apache/cloudstack/issues/4277 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

[GitHub] [cloudstack] alitheg commented on issue #4277: Use database name other than "cloud"

2021-01-29 Thread GitBox
alitheg commented on issue #4277: URL: https://github.com/apache/cloudstack/issues/4277#issuecomment-769742357 That's fine - I was only testing the waters of cloudstack and haven't the time to dig into it further. Closing the issue.

[GitHub] [cloudstack] DaanHoogland commented on issue #4277: Use database name other than "cloud"

2021-01-29 Thread GitBox
DaanHoogland commented on issue #4277: URL: https://github.com/apache/cloudstack/issues/4277#issuecomment-769740279 @alitheg if you are planning to do more investigation we can keep this issue open, otherwise we'd close it. care to comment?

[GitHub] [cloudstack] soreana commented on issue #4093: Centralised logging via rsyslog

2021-01-29 Thread GitBox
soreana commented on issue #4093: URL: https://github.com/apache/cloudstack/issues/4093#issuecomment-769733352 @DaanHoogland Honestly, it took a lot of time to apply the changes you required to merging this pr. I discussed it with my colleagues, and we concluded that it isn't our priority

[GitHub] [cloudstack] DaanHoogland commented on issue #3897: Review all list APIs and make then honour listall parameter

2021-01-29 Thread GitBox
DaanHoogland commented on issue #3897: URL: https://github.com/apache/cloudstack/issues/3897#issuecomment-769730298 in view of the changes in #3894 it would seem that no further work needs to be done, @rhtyd can you comment? @wido @weizhouapache did you have anymore issues as a result

[GitHub] [cloudstack] shwstppr commented on pull request #4627: VR: fix expunging vm will remove dhcp entries of another vm in VR

2021-01-29 Thread GitBox
shwstppr commented on pull request #4627: URL: https://github.com/apache/cloudstack/pull/4627#issuecomment-769729261 I'm not able to reproduce the issue on a KVM and a VMware environment. Can someone else verify @vladimirpetrov @nvazquez @rhtyd

[GitHub] [cloudstack] DaanHoogland commented on issue #3719: VMware: SystemVMs not cleared after forcing them to stop

2021-01-29 Thread GitBox
DaanHoogland commented on issue #3719: URL: https://github.com/apache/cloudstack/issues/3719#issuecomment-769726843 I can confirm this still happens in 4.15. I wonder if it is on purpose as we are talking about the last host in the zone, might it be thought to be usefull to keep the

[GitHub] [cloudstack] weizhouapache commented on issue #3742: migratewait does not seem to be honored in KVM

2021-01-29 Thread GitBox
weizhouapache commented on issue #3742: URL: https://github.com/apache/cloudstack/issues/3742#issuecomment-769719743 > are you saying we connot solve this because of the lack of an API on KVM for stopping an ongoing job? @DaanHoogland I checked this again. actually there is

[GitHub] [cloudstack] DaanHoogland commented on issue #4093: Centralised logging via rsyslog

2021-01-29 Thread GitBox
DaanHoogland commented on issue #4093: URL: https://github.com/apache/cloudstack/issues/4093#issuecomment-769713695 @soreana are you still planning for this? This is an automated message from the Apache Git Service. To

[GitHub] [cloudstack] DaanHoogland commented on issue #3802: OVS Provider not visible under Network Service Providers for guest network

2021-01-29 Thread GitBox
DaanHoogland commented on issue #3802: URL: https://github.com/apache/cloudstack/issues/3802#issuecomment-769712313 @shoaibjdev are you still working on this issue? As suggested implicitly, can you try a cli tool like cloudmonkey to find the provider and does that work?

[GitHub] [cloudstack] DaanHoogland commented on issue #3792: Register iso template get "Stream closed" error. Can not register self build iso template.

2021-01-29 Thread GitBox
DaanHoogland commented on issue #3792: URL: https://github.com/apache/cloudstack/issues/3792#issuecomment-769709700 @div8cn @wqmeng , can you try this in 4.15? This is an automated message from the Apache Git Service. To

[GitHub] [cloudstack] DaanHoogland commented on issue #3742: migratewait does not seem to be honored in KVM

2021-01-29 Thread GitBox
DaanHoogland commented on issue #3742: URL: https://github.com/apache/cloudstack/issues/3742#issuecomment-769706715 @weizhouapache are you saying we connot solve this because of the lack of an API on KVM for stopping an ongoing job?

[GitHub] [cloudstack] sureshanaparti commented on a change in pull request #4561: [WIP] Support for persistence mode in L2 networks

2021-01-29 Thread GitBox
sureshanaparti commented on a change in pull request #4561: URL: https://github.com/apache/cloudstack/pull/4561#discussion_r566692617 ## File path: engine/orchestration/src/main/java/org/apache/cloudstack/engine/orchestration/NetworkOrchestrator.java ## @@ -1187,6 +1202,86 @@

[GitHub] [cloudstack] DaanHoogland commented on issue #4024: qemu 2.10+: consider adding --force-share flag

2021-01-29 Thread GitBox
DaanHoogland commented on issue #4024: URL: https://github.com/apache/cloudstack/issues/4024#issuecomment-769690505 @andrijapanicsb -dont-use-this-one is this covored by the -u patch by @GutoVeronezi - #4567 ? This is an

[GitHub] [cloudstack] sureshanaparti commented on a change in pull request #4561: [WIP] Support for persistence mode in L2 networks

2021-01-29 Thread GitBox
sureshanaparti commented on a change in pull request #4561: URL: https://github.com/apache/cloudstack/pull/4561#discussion_r566687342 ## File path: engine/orchestration/src/main/java/com/cloud/vm/VirtualMachineManagerImpl.java ## @@ -1827,6 +1848,63 @@ private void

[GitHub] [cloudstack] sureshanaparti commented on a change in pull request #4561: [WIP] Support for persistence mode in L2 networks

2021-01-29 Thread GitBox
sureshanaparti commented on a change in pull request #4561: URL: https://github.com/apache/cloudstack/pull/4561#discussion_r566687342 ## File path: engine/orchestration/src/main/java/com/cloud/vm/VirtualMachineManagerImpl.java ## @@ -1827,6 +1848,63 @@ private void

[GitHub] [cloudstack] sureshanaparti commented on a change in pull request #4561: [WIP] Support for persistence mode in L2 networks

2021-01-29 Thread GitBox
sureshanaparti commented on a change in pull request #4561: URL: https://github.com/apache/cloudstack/pull/4561#discussion_r566687342 ## File path: engine/orchestration/src/main/java/com/cloud/vm/VirtualMachineManagerImpl.java ## @@ -1827,6 +1848,63 @@ private void

[GitHub] [cloudstack] sureshanaparti commented on a change in pull request #4561: [WIP] Support for persistence mode in L2 networks

2021-01-29 Thread GitBox
sureshanaparti commented on a change in pull request #4561: URL: https://github.com/apache/cloudstack/pull/4561#discussion_r566683597 ## File path: engine/orchestration/src/main/java/com/cloud/vm/VirtualMachineManagerImpl.java ## @@ -1827,6 +1848,63 @@ private void

[GitHub] [cloudstack] sureshanaparti commented on a change in pull request #4561: [WIP] Support for persistence mode in L2 networks

2021-01-29 Thread GitBox
sureshanaparti commented on a change in pull request #4561: URL: https://github.com/apache/cloudstack/pull/4561#discussion_r566683141 ## File path: engine/orchestration/src/main/java/com/cloud/vm/VirtualMachineManagerImpl.java ## @@ -1827,6 +1848,63 @@ private void

[GitHub] [cloudstack] DaanHoogland commented on issue #4227: confused about User Permission

2021-01-29 Thread GitBox
DaanHoogland commented on issue #4227: URL: https://github.com/apache/cloudstack/issues/4227#issuecomment-769676656 @gaaray2k this question is stale as well, Are you still on it? This is an automated message from the Apache

[GitHub] [cloudstack] DaanHoogland commented on issue #4209: use X-forward-for address instead of the load balancer ip.

2021-01-29 Thread GitBox
DaanHoogland commented on issue #4209: URL: https://github.com/apache/cloudstack/issues/4209#issuecomment-769675446 @gaaray2k did you get any progres on this or is it still an issue/feature request? This is an automated

[GitHub] [cloudstack] weizhouapache commented on pull request #4630: disable hot add memory and cpu via vm settings

2021-01-29 Thread GitBox
weizhouapache commented on pull request #4630: URL: https://github.com/apache/cloudstack/pull/4630#issuecomment-769669465 code lgtm This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [cloudstack] weizhouapache edited a comment on pull request #4137: Adding VPN options for IKE version and IKE split connections.

2021-01-29 Thread GitBox
weizhouapache edited a comment on pull request #4137: URL: https://github.com/apache/cloudstack/pull/4137#issuecomment-769668458 > > @ggoodrich-ipp > > thanks for pr. > > could you fix the conflicts and make primate ui change ? > > Should this be re-targeted to either 4.15 or

[GitHub] [cloudstack] weizhouapache commented on pull request #4137: Adding VPN options for IKE version and IKE split connections.

2021-01-29 Thread GitBox
weizhouapache commented on pull request #4137: URL: https://github.com/apache/cloudstack/pull/4137#issuecomment-769668458 > > @ggoodrich-ipp > > thanks for pr. > > could you fix the conflicts and make primate ui change ? > > Should this be re-targeted to either 4.15 or 4.16

[GitHub] [cloudstack] weizhouapache commented on pull request #4629: server: prevent update vm read-only details

2021-01-29 Thread GitBox
weizhouapache commented on pull request #4629: URL: https://github.com/apache/cloudstack/pull/4629#issuecomment-769666240 @shwstppr good, thanks. should the detail name be changed from 'user.vm.readonly.ui.details' to 'user.vm.readonly.details' ?

[GitHub] [cloudstack] weizhouapache commented on pull request #3915: Incorporate VR OOB start checks to properly HA the VR

2021-01-29 Thread GitBox
weizhouapache commented on pull request #3915: URL: https://github.com/apache/cloudstack/pull/3915#issuecomment-769664798 > > @DaanHoogland starting VR OOB looks not like a normal behavior > > @weizhouapache We aren't starting it OOB, we are just running checks when it is believed

[GitHub] [cloudstack] DK101010 opened a new pull request #4630: disable hot add memory and cpu via vm settings

2021-01-29 Thread GitBox
DK101010 opened a new pull request #4630: URL: https://github.com/apache/cloudstack/pull/4630 ### Description Currently hot add memory and cpu is always enabled when it supported. In some situation it is necessary to disable this to features for a specific vm. With this PR