[GitHub] blueorangutan commented on issue #2839: Enable DPDK support on KVM

2018-11-05 Thread GitBox
blueorangutan commented on issue #2839: Enable DPDK support on KVM URL: https://github.com/apache/cloudstack/pull/2839#issuecomment-436148371 Trillian test result (tid-3143) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 21856 seconds Marvin

[GitHub] div8cn opened a new issue #3008: Runing cloudstack failed in docker

2018-11-05 Thread GitBox
div8cn opened a new issue #3008: Runing cloudstack failed in docker URL: https://github.com/apache/cloudstack/issues/3008 If docker does not enable IPV6, cloudstack-management will fail after startup; Server.properties default IPV6, but docker does not start IPV6, so the startup

[GitHub] nvazquez opened a new pull request #3007: Add missing ConfigDrive entries on existing zones after upgrade

2018-11-05 Thread GitBox
nvazquez opened a new pull request #3007: Add missing ConfigDrive entries on existing zones after upgrade URL: https://github.com/apache/cloudstack/pull/3007 ## Description After upgrade existing environments to 4.11, ConfigDrive cannot be enabled for existing zones due to missing entry

[GitHub] blueorangutan commented on issue #2839: Enable DPDK support on KVM

2018-11-05 Thread GitBox
blueorangutan commented on issue #2839: Enable DPDK support on KVM URL: https://github.com/apache/cloudstack/pull/2839#issuecomment-436085042 @nvazquez a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests --

[GitHub] nvazquez commented on issue #2839: Enable DPDK support on KVM

2018-11-05 Thread GitBox
nvazquez commented on issue #2839: Enable DPDK support on KVM URL: https://github.com/apache/cloudstack/pull/2839#issuecomment-436084941 @blueorangutan test This is an automated message from the Apache Git Service. To respond

[GitHub] blueorangutan commented on issue #2839: Enable DPDK support on KVM

2018-11-05 Thread GitBox
blueorangutan commented on issue #2839: Enable DPDK support on KVM URL: https://github.com/apache/cloudstack/pull/2839#issuecomment-436083540 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-2405 This is an automated message f

[GitHub] ccheline opened a new issue #3006: Secondary Storage server fails if multiple Management servers are defined

2018-11-05 Thread GitBox
ccheline opened a new issue #3006: Secondary Storage server fails if multiple Management servers are defined URL: https://github.com/apache/cloudstack/issues/3006 # ISSUE TYPE * Bug Report # COMPONENT NAME ~~~ systemvm agent (secondary storage vm)

[GitHub] blueorangutan commented on issue #2839: Enable DPDK support on KVM

2018-11-05 Thread GitBox
blueorangutan commented on issue #2839: Enable DPDK support on KVM URL: https://github.com/apache/cloudstack/pull/2839#issuecomment-436076552 @nvazquez a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

[GitHub] nvazquez commented on issue #2839: Enable DPDK support on KVM

2018-11-05 Thread GitBox
nvazquez commented on issue #2839: Enable DPDK support on KVM URL: https://github.com/apache/cloudstack/pull/2839#issuecomment-436076348 @blueorangutan package This is an automated message from the Apache Git Service. To respo

[GitHub] DennisKonrad commented on issue #2820: template downloading fails

2018-11-05 Thread GitBox
DennisKonrad commented on issue #2820: template downloading fails URL: https://github.com/apache/cloudstack/issues/2820#issuecomment-436007728 Ok, the logs we get in the SSVM look like this when trying to upload a template: ![image](https://user-images.githubusercontent.com/33484272

[GitHub] bwsw commented on issue #3000: Added two more device names (lo* and dummy*)

2018-11-05 Thread GitBox
bwsw commented on issue #3000: Added two more device names (lo* and dummy*) URL: https://github.com/apache/cloudstack/pull/3000#issuecomment-436004298 @wido I addressed your comments by refactoring the code even further. Changed "+" to Paths.get(...) which does it safely. Please, review. -

[GitHub] csquire opened a new pull request #3005: Discover tags field on superclass of API responses

2018-11-05 Thread GitBox
csquire opened a new pull request #3005: Discover tags field on superclass of API responses URL: https://github.com/apache/cloudstack/pull/3005 ## Description Updated `ApiServiceDiscoveryImpl` to check superclasses of API responses for fields. Fixes: #

[GitHub] nvazquez opened a new pull request #3004: [Placeholder] Add new data motion strategy for KVM non managed storage live migrations

2018-11-05 Thread GitBox
nvazquez opened a new pull request #3004: [Placeholder] Add new data motion strategy for KVM non managed storage live migrations URL: https://github.com/apache/cloudstack/pull/3004 ## Description Add a new data motion strategy as a placeholder for new KVM live storage migration features

[GitHub] somejfn commented on issue #2989: Unable to create network post upgrade to 4.11

2018-11-05 Thread GitBox
somejfn commented on issue #2989: Unable to create network post upgrade to 4.11 URL: https://github.com/apache/cloudstack/issues/2989#issuecomment-435953495 Hi Paul, I have 2 custom neworkofferings created as shared networks, one with config drive (post 4.11.2 upgrade) enabled and

[GitHub] PaulAngus commented on issue #2989: Unable to create network post upgrade to 4.11

2018-11-05 Thread GitBox
PaulAngus commented on issue #2989: Unable to create network post upgrade to 4.11 URL: https://github.com/apache/cloudstack/issues/2989#issuecomment-435949178 so to clarify - does the shared network with no-services deploy in 4.11.2rc4 if you don't use config drive ? (the 'details' shou

[GitHub] somejfn commented on issue #2989: Unable to create network post upgrade to 4.11

2018-11-05 Thread GitBox
somejfn commented on issue #2989: Unable to create network post upgrade to 4.11 URL: https://github.com/apache/cloudstack/issues/2989#issuecomment-435946141 Yep I did had to enable it there to get it working while testing with new L2 networks. On Mon, Nov 5, 2018 at 11:36 AM Paul A

[GitHub] PaulAngus commented on issue #2989: Unable to create network post upgrade to 4.11

2018-11-05 Thread GitBox
PaulAngus commented on issue #2989: Unable to create network post upgrade to 4.11 URL: https://github.com/apache/cloudstack/issues/2989#issuecomment-435942213 @somejfn i'm not sure how well config drive is supported in shared networks, can you check that it is enabled for the guest network

[GitHub] DaanHoogland edited a comment on issue #3003: make sure no time skew can interfere with comparison

2018-11-05 Thread GitBox
DaanHoogland edited a comment on issue #3003: make sure no time skew can interfere with comparison URL: https://github.com/apache/cloudstack/pull/3003#issuecomment-435941591 makes sense, if the contents is covoured. I didn't give that any attention, @rafaelweingartner. --

[GitHub] DaanHoogland edited a comment on issue #3003: make sure no time skew can interfere with comparison

2018-11-05 Thread GitBox
DaanHoogland edited a comment on issue #3003: make sure no time skew can interfere with comparison URL: https://github.com/apache/cloudstack/pull/3003#issuecomment-435941591 makes sense, if the contents is covoured. I didn't give that any attention @rafaelweingartner

[GitHub] DaanHoogland commented on issue #3003: make sure no time skew can interfere with comparison

2018-11-05 Thread GitBox
DaanHoogland commented on issue #3003: make sure no time skew can interfere with comparison URL: https://github.com/apache/cloudstack/pull/3003#issuecomment-435941591 makes sense, if the cantants is covoured. I didn't give that any attention @rafaelweingartner ---

[GitHub] rafaelweingartner commented on issue #3003: make sure no time skew can interfere with comparison

2018-11-05 Thread GitBox
rafaelweingartner commented on issue #3003: make sure no time skew can interfere with comparison URL: https://github.com/apache/cloudstack/pull/3003#issuecomment-435941306 @DaanHoogland just one remark. What about removing that `main` method there? -

[GitHub] DaanHoogland commented on issue #3003: make sure no time skew can interfere with comparison

2018-11-05 Thread GitBox
DaanHoogland commented on issue #3003: make sure no time skew can interfere with comparison URL: https://github.com/apache/cloudstack/pull/3003#issuecomment-435941288 I'll take that as 👍 😉 This is an automated message from t

[GitHub] bwsw commented on issue #3003: make sure no time skew can interfere with comparison

2018-11-05 Thread GitBox
bwsw commented on issue #3003: make sure no time skew can interfere with comparison URL: https://github.com/apache/cloudstack/pull/3003#issuecomment-435940277 @DaanHoogland LGTM, but I'm not a skilled java developer. Don't rely on my review. ---

[GitHub] DaanHoogland commented on issue #2893: ApiServer: signature v3 to accept more formats

2018-11-05 Thread GitBox
DaanHoogland commented on issue #2893: ApiServer: signature v3 to accept more formats URL: https://github.com/apache/cloudstack/pull/2893#issuecomment-435939906 @bwsw #3003 contains what I think is enough, let's discuss further overthere

[GitHub] bwsw commented on issue #2893: ApiServer: signature v3 to accept more formats

2018-11-05 Thread GitBox
bwsw commented on issue #2893: ApiServer: signature v3 to accept more formats URL: https://github.com/apache/cloudstack/pull/2893#issuecomment-435937832 @DaanHoogland @nvazquez There are five tests like the one above in that unit test module. All of them must be reworked. I think it

[GitHub] DaanHoogland commented on issue #3003: make sure no time skew can interfere with comparison

2018-11-05 Thread GitBox
DaanHoogland commented on issue #3003: make sure no time skew can interfere with comparison URL: https://github.com/apache/cloudstack/pull/3003#issuecomment-435937456 @bwsw @nvazquez have a look This is an automated message f

[GitHub] GabrielBrascher commented on a change in pull request #2997: Allow KVM VM live migration with ROOT volume on file storage type

2018-11-05 Thread GitBox
GabrielBrascher commented on a change in pull request #2997: Allow KVM VM live migration with ROOT volume on file storage type URL: https://github.com/apache/cloudstack/pull/2997#discussion_r230814607 ## File path: plugins/hypervisors/kvm/src/main/java/com/cloud/hypervisor/kvm/reso

[GitHub] wido commented on issue #2985: kvm/ceph: When create a new RBD image use modern flags/features for t…

2018-11-05 Thread GitBox
wido commented on issue #2985: kvm/ceph: When create a new RBD image use modern flags/features for t… URL: https://github.com/apache/cloudstack/pull/2985#issuecomment-435936752 Yes, I trusted my IDE to help with this, but fixed it now! Compiles :) --

[GitHub] wido commented on a change in pull request #2997: Allow KVM VM live migration with ROOT volume on file storage type

2018-11-05 Thread GitBox
wido commented on a change in pull request #2997: Allow KVM VM live migration with ROOT volume on file storage type URL: https://github.com/apache/cloudstack/pull/2997#discussion_r230814211 ## File path: engine/storage/datamotion/src/main/java/org/apache/cloudstack/storage/motion/K

[GitHub] DaanHoogland opened a new pull request #3003: make sure no time skew can interfere with comparison

2018-11-05 Thread GitBox
DaanHoogland opened a new pull request #3003: make sure no time skew can interfere with comparison URL: https://github.com/apache/cloudstack/pull/3003 ## Description ## Types of changes Test fix ## Screenshots (if appropriate): ##

[GitHub] csquire opened a new issue #3002: api: listApis command missing 'tags' field for many responses

2018-11-05 Thread GitBox
csquire opened a new issue #3002: api: listApis command missing 'tags' field for many responses URL: https://github.com/apache/cloudstack/issues/3002 # ISSUE TYPE * Bug Report # COMPONENT NAME ~~~ API ~~~ # CLOUDSTACK VERSION

[GitHub] GabrielBrascher commented on a change in pull request #2997: Allow KVM VM live migration with ROOT volume on file storage type

2018-11-05 Thread GitBox
GabrielBrascher commented on a change in pull request #2997: Allow KVM VM live migration with ROOT volume on file storage type URL: https://github.com/apache/cloudstack/pull/2997#discussion_r230799869 ## File path: engine/storage/datamotion/src/main/java/org/apache/cloudstack/stora

[GitHub] nvazquez commented on issue #2893: ApiServer: signature v3 to accept more formats

2018-11-05 Thread GitBox
nvazquez commented on issue #2893: ApiServer: signature v3 to accept more formats URL: https://github.com/apache/cloudstack/pull/2893#issuecomment-435912401 Tested on latest master: first tested running every test and passed successfully. Then replaced the last one for the code @DaanHoogla

[GitHub] DaanHoogland commented on a change in pull request #2995: Improvements on upload direct download certificates

2018-11-05 Thread GitBox
DaanHoogland commented on a change in pull request #2995: Improvements on upload direct download certificates URL: https://github.com/apache/cloudstack/pull/2995#discussion_r230784363 ## File path: agent/src/com/cloud/agent/Agent.java ## @@ -697,16 +697,34 @@ private Answe

[GitHub] DaanHoogland commented on a change in pull request #2995: Improvements on upload direct download certificates

2018-11-05 Thread GitBox
DaanHoogland commented on a change in pull request #2995: Improvements on upload direct download certificates URL: https://github.com/apache/cloudstack/pull/2995#discussion_r230784522 ## File path: agent/src/com/cloud/agent/Agent.java ## @@ -697,16 +697,34 @@ private Answe

[GitHub] DaanHoogland commented on a change in pull request #2995: Improvements on upload direct download certificates

2018-11-05 Thread GitBox
DaanHoogland commented on a change in pull request #2995: Improvements on upload direct download certificates URL: https://github.com/apache/cloudstack/pull/2995#discussion_r230784764 ## File path: agent/src/com/cloud/agent/Agent.java ## @@ -697,16 +697,34 @@ private Answe

[GitHub] DaanHoogland commented on a change in pull request #2997: Allow KVM VM live migration with ROOT volume on file storage type

2018-11-05 Thread GitBox
DaanHoogland commented on a change in pull request #2997: Allow KVM VM live migration with ROOT volume on file storage type URL: https://github.com/apache/cloudstack/pull/2997#discussion_r230782702 ## File path: plugins/hypervisors/kvm/src/main/java/com/cloud/hypervisor/kvm/resourc

[GitHub] DaanHoogland commented on a change in pull request #2997: Allow KVM VM live migration with ROOT volume on file storage type

2018-11-05 Thread GitBox
DaanHoogland commented on a change in pull request #2997: Allow KVM VM live migration with ROOT volume on file storage type URL: https://github.com/apache/cloudstack/pull/2997#discussion_r230780662 ## File path: core/src/main/java/com/cloud/agent/api/MigrateCommand.java ##

[GitHub] DaanHoogland commented on a change in pull request #2997: Allow KVM VM live migration with ROOT volume on file storage type

2018-11-05 Thread GitBox
DaanHoogland commented on a change in pull request #2997: Allow KVM VM live migration with ROOT volume on file storage type URL: https://github.com/apache/cloudstack/pull/2997#discussion_r230781281 ## File path: engine/storage/datamotion/src/main/java/org/apache/cloudstack/storage/

[GitHub] somejfn commented on issue #2989: Unable to create network post upgrade to 4.11

2018-11-05 Thread GitBox
somejfn commented on issue #2989: Unable to create network post upgrade to 4.11 URL: https://github.com/apache/cloudstack/issues/2989#issuecomment-435904443 Same error on RC4 If that matters, I created a new network offering on 4.11.2 similarly to the 4.9.3 one with the only diffe

[GitHub] DaanHoogland commented on a change in pull request #2985: kvm/ceph: When create a new RBD image use modern flags/features for t…

2018-11-05 Thread GitBox
DaanHoogland commented on a change in pull request #2985: kvm/ceph: When create a new RBD image use modern flags/features for t… URL: https://github.com/apache/cloudstack/pull/2985#discussion_r230779467 ## File path: plugins/hypervisors/kvm/src/main/java/com/cloud/hypervisor/kvm/st

[GitHub] DaanHoogland commented on issue #2893: ApiServer: signature v3 to accept more formats

2018-11-05 Thread GitBox
DaanHoogland commented on issue #2893: ApiServer: signature v3 to accept more formats URL: https://github.com/apache/cloudstack/pull/2893#issuecomment-435901833 strangely this works: ``` @Test public void zonedTimeFormatIsoNoColonZMs() throws ParseException { Dat

[GitHub] somejfn commented on issue #2989: Unable to create network post upgrade to 4.11

2018-11-05 Thread GitBox
somejfn commented on issue #2989: Unable to create network post upgrade to 4.11 URL: https://github.com/apache/cloudstack/issues/2989#issuecomment-435901342 I'll upgrade to RC4 right now and see On Mon, Nov 5, 2018 at 9:41 AM Paul Angus wrote: > I have also tried a clean insta

[GitHub] somejfn commented on issue #2989: Unable to create network post upgrade to 4.11

2018-11-05 Thread GitBox
somejfn commented on issue #2989: Unable to create network post upgrade to 4.11 URL: https://github.com/apache/cloudstack/issues/2989#issuecomment-435900306 Hmm I don't see the difference by the look of it. The network offering on 4.9.3 was created this way: cs createNetworkO

[GitHub] PaulAngus commented on issue #2989: Unable to create network post upgrade to 4.11

2018-11-05 Thread GitBox
PaulAngus commented on issue #2989: Unable to create network post upgrade to 4.11 URL: https://github.com/apache/cloudstack/issues/2989#issuecomment-435898783 I have also tried a clean install and an upgrade from 4.9.3, i can't reproduce this issue. Have you retried on **RC4** ? -

[GitHub] DennisKonrad edited a comment on issue #2820: template downloading fails

2018-11-05 Thread GitBox
DennisKonrad edited a comment on issue #2820: template downloading fails URL: https://github.com/apache/cloudstack/issues/2820#issuecomment-435894726 Can someone confirm that [HttpTemplateDownloader](https://github.com/apache/cloudstack/blob/1d05fead49f5c856257a741b07122f5633d2e359/core/src

[GitHub] DennisKonrad commented on issue #2820: template downloading fails

2018-11-05 Thread GitBox
DennisKonrad commented on issue #2820: template downloading fails URL: https://github.com/apache/cloudstack/issues/2820#issuecomment-435894726 Can someone confirm that "HttpTemplateDownloader.java" is used for this? This is an

[GitHub] DaanHoogland commented on issue #2893: ApiServer: signature v3 to accept more formats

2018-11-05 Thread GitBox
DaanHoogland commented on issue #2893: ApiServer: signature v3 to accept more formats URL: https://github.com/apache/cloudstack/pull/2893#issuecomment-435894161 well @bwsw the code needs debugging still. This is an automated

[GitHub] bwsw commented on issue #2893: ApiServer: signature v3 to accept more formats

2018-11-05 Thread GitBox
bwsw commented on issue #2893: ApiServer: signature v3 to accept more formats URL: https://github.com/apache/cloudstack/pull/2893#issuecomment-435892213 @DaanHoogland Looks good to me. May be imperfect, but should work, at least. -

[GitHub] DaanHoogland commented on issue #2893: ApiServer: signature v3 to accept more formats

2018-11-05 Thread GitBox
DaanHoogland commented on issue #2893: ApiServer: signature v3 to accept more formats URL: https://github.com/apache/cloudstack/pull/2893#issuecomment-435891242 @bwsw a quick hack of mine: ``` @Test public void zonedTimeFormatIsoNoColonZMs() throws ParseException {

[GitHub] borisstoyanov commented on issue #2989: Unable to create network post upgrade to 4.11

2018-11-05 Thread GitBox
borisstoyanov commented on issue #2989: Unable to create network post upgrade to 4.11 URL: https://github.com/apache/cloudstack/issues/2989#issuecomment-435889695 Hi @somejfn, I've tried reproducing this on an upgraded env from LTS 4.9.3 to 4.11.2 RC4 Using this offering: a91d1c0e-4

[GitHub] somejfn commented on issue #2989: Unable to create network post upgrade to 4.11

2018-11-05 Thread GitBox
somejfn commented on issue #2989: Unable to create network post upgrade to 4.11 URL: https://github.com/apache/cloudstack/issues/2989#issuecomment-435881410 L2 did not exist in 4.9.3 and unless we have a way to convert them to L2 live on 4.11.2 (I'd like that) well the existing netwo

[GitHub] DennisKonrad commented on issue #3001: storage upload -> "custom disk offerings" not displayed correctly (ui bug)

2018-11-05 Thread GitBox
DennisKonrad commented on issue #3001: storage upload -> "custom disk offerings" not displayed correctly (ui bug) URL: https://github.com/apache/cloudstack/issues/3001#issuecomment-435880681 https://github.com/apache/cloudstack/blob/1d05fead49f5c856257a741b07122f5633d2e359/ui/scripts/stora

[GitHub] borisstoyanov commented on issue #2989: Unable to create network post upgrade to 4.11

2018-11-05 Thread GitBox
borisstoyanov commented on issue #2989: Unable to create network post upgrade to 4.11 URL: https://github.com/apache/cloudstack/issues/2989#issuecomment-435876249 Hi @somejfn, shouldn't the guest type of the network offering be L2, instead of Shared? -

[GitHub] somejfn edited a comment on issue #2989: Unable to create network post upgrade to 4.11

2018-11-05 Thread GitBox
somejfn edited a comment on issue #2989: Unable to create network post upgrade to 4.11 URL: https://github.com/apache/cloudstack/issues/2989#issuecomment-435872009 Also, the same error happens the same way when the network is scoped to projects or is "global". --

[GitHub] somejfn commented on issue #2989: Unable to create network post upgrade to 4.11

2018-11-05 Thread GitBox
somejfn commented on issue #2989: Unable to create network post upgrade to 4.11 URL: https://github.com/apache/cloudstack/issues/2989#issuecomment-435872009 Also, the same error happens the same way when the network scoped to projects or is "global". -

[GitHub] bwsw commented on a change in pull request #3000: Added two more device names (lo* and dummy*)

2018-11-05 Thread GitBox
bwsw commented on a change in pull request #3000: Added two more device names (lo* and dummy*) URL: https://github.com/apache/cloudstack/pull/3000#discussion_r230745145 ## File path: plugins/hypervisors/kvm/src/main/java/com/cloud/hypervisor/kvm/resource/LibvirtComputingResource.ja

[GitHub] bwsw commented on a change in pull request #3000: Added two more device names (lo* and dummy*)

2018-11-05 Thread GitBox
bwsw commented on a change in pull request #3000: Added two more device names (lo* and dummy*) URL: https://github.com/apache/cloudstack/pull/3000#discussion_r230745074 ## File path: plugins/hypervisors/kvm/src/main/java/com/cloud/hypervisor/kvm/resource/LibvirtComputingResource.ja

[GitHub] nvazquez commented on a change in pull request #2997: Allow KVM VM live migration with ROOT volume on file storage type

2018-11-05 Thread GitBox
nvazquez commented on a change in pull request #2997: Allow KVM VM live migration with ROOT volume on file storage type URL: https://github.com/apache/cloudstack/pull/2997#discussion_r230742583 ## File path: plugins/hypervisors/kvm/src/main/java/com/cloud/hypervisor/kvm/storage/KVM

[GitHub] nvazquez commented on a change in pull request #2997: Allow KVM VM live migration with ROOT volume on file storage type

2018-11-05 Thread GitBox
nvazquez commented on a change in pull request #2997: Allow KVM VM live migration with ROOT volume on file storage type URL: https://github.com/apache/cloudstack/pull/2997#discussion_r230740662 ## File path: plugins/hypervisors/kvm/src/main/java/com/cloud/hypervisor/kvm/resource/Mi

[GitHub] nvazquez commented on a change in pull request #2997: Allow KVM VM live migration with ROOT volume on file storage type

2018-11-05 Thread GitBox
nvazquez commented on a change in pull request #2997: Allow KVM VM live migration with ROOT volume on file storage type URL: https://github.com/apache/cloudstack/pull/2997#discussion_r230740662 ## File path: plugins/hypervisors/kvm/src/main/java/com/cloud/hypervisor/kvm/resource/Mi

[GitHub] nvazquez commented on a change in pull request #2997: Allow KVM VM live migration with ROOT volume on file storage type

2018-11-05 Thread GitBox
nvazquez commented on a change in pull request #2997: Allow KVM VM live migration with ROOT volume on file storage type URL: https://github.com/apache/cloudstack/pull/2997#discussion_r230739394 ## File path: engine/storage/datamotion/src/main/java/org/apache/cloudstack/storage/moti

[GitHub] DennisKonrad edited a comment on issue #2820: template downloading fails

2018-11-05 Thread GitBox
DennisKonrad edited a comment on issue #2820: template downloading fails URL: https://github.com/apache/cloudstack/issues/2820#issuecomment-435843927 Could this have to do with this hardcored max size defined here: [DownloadManagerImpl.java](https://github.com/apache/cloudstack/blob/

[GitHub] DennisKonrad edited a comment on issue #2820: template downloading fails

2018-11-05 Thread GitBox
DennisKonrad edited a comment on issue #2820: template downloading fails URL: https://github.com/apache/cloudstack/issues/2820#issuecomment-435843927 Could this have to do with this hardcored max size defined here: [DownloadManagerImpl.java](https://github.com/apache/cloudstack/blob/

[GitHub] nvazquez commented on a change in pull request #2983: KVM live storage migration intra cluster from NFS source and destination

2018-11-05 Thread GitBox
nvazquez commented on a change in pull request #2983: KVM live storage migration intra cluster from NFS source and destination URL: https://github.com/apache/cloudstack/pull/2983#discussion_r230736042 ## File path: plugins/hypervisors/kvm/src/main/java/org/apache/cloudstack/utils/q

[GitHub] nvazquez commented on a change in pull request #2983: KVM live storage migration intra cluster from NFS source and destination

2018-11-05 Thread GitBox
nvazquez commented on a change in pull request #2983: KVM live storage migration intra cluster from NFS source and destination URL: https://github.com/apache/cloudstack/pull/2983#discussion_r230734799 ## File path: core/src/main/java/com/cloud/agent/api/storage/CheckStorageAvailabi

[GitHub] nvazquez commented on a change in pull request #2983: KVM live storage migration intra cluster from NFS source and destination

2018-11-05 Thread GitBox
nvazquez commented on a change in pull request #2983: KVM live storage migration intra cluster from NFS source and destination URL: https://github.com/apache/cloudstack/pull/2983#discussion_r230733469 ## File path: plugins/hypervisors/kvm/src/test/java/com/cloud/hypervisor/kvm/reso

[GitHub] DennisKonrad edited a comment on issue #2820: template downloading fails

2018-11-05 Thread GitBox
DennisKonrad edited a comment on issue #2820: template downloading fails URL: https://github.com/apache/cloudstack/issues/2820#issuecomment-435843927 Could this have to do with this hardcored max size defined here: [DownloadManagerImpl.java](https://github.com/apache/cloudstack/blob/

[GitHub] DennisKonrad edited a comment on issue #2820: template downloading fails

2018-11-05 Thread GitBox
DennisKonrad edited a comment on issue #2820: template downloading fails URL: https://github.com/apache/cloudstack/issues/2820#issuecomment-435843927 Could this have to do with this hardcored max size defined here: [DownloadManagerImpl.java](https://github.com/apache/cloudstack/blob/

[GitHub] DennisKonrad commented on issue #2820: template downloading fails

2018-11-05 Thread GitBox
DennisKonrad commented on issue #2820: template downloading fails URL: https://github.com/apache/cloudstack/issues/2820#issuecomment-435843927 Could this have to do with this hardcored max size defined here: [DownloadManagerImpl.java](https://github.com/apache/cloudstack/blob/1d05fea

[GitHub] svenvogel commented on issue #3001: storage upload -> "custom disk offerings" not displayed correctly (ui bug)

2018-11-05 Thread GitBox
svenvogel commented on issue #3001: storage upload -> "custom disk offerings" not displayed correctly (ui bug) URL: https://github.com/apache/cloudstack/issues/3001#issuecomment-435840175 @mike-tutkowski what do you think? Th

[GitHub] svenvogel opened a new issue #3001: storage upload -> "custom disk offerings" not displayed correctly (ui bug)

2018-11-05 Thread GitBox
svenvogel opened a new issue #3001: storage upload -> "custom disk offerings" not displayed correctly (ui bug) URL: https://github.com/apache/cloudstack/issues/3001 # ISSUE TYPE * Bug Report # COMPONENT NAME ~~~ UI ~~~ # CLOUDSTACK VERS

[GitHub] DennisKonrad edited a comment on issue #2820: template downloading fails

2018-11-05 Thread GitBox
DennisKonrad edited a comment on issue #2820: template downloading fails URL: https://github.com/apache/cloudstack/issues/2820#issuecomment-435833556 Hi, we also obseve the same problem. It happens for us (for example) when uploading a 20GB qcow image. We tried to upload fro

[GitHub] DennisKonrad edited a comment on issue #2820: template downloading fails

2018-11-05 Thread GitBox
DennisKonrad edited a comment on issue #2820: template downloading fails URL: https://github.com/apache/cloudstack/issues/2820#issuecomment-435833556 Hi, we also obseve the same problem. It happens for us (for example) when uploading a 20GB qcow image. We tried to upload fro

[GitHub] DennisKonrad commented on issue #2820: template downloading fails

2018-11-05 Thread GitBox
DennisKonrad commented on issue #2820: template downloading fails URL: https://github.com/apache/cloudstack/issues/2820#issuecomment-435833865 @rhtyd Is there a workaround for this? This is an automated message from the Apache

[GitHub] DennisKonrad commented on issue #2820: template downloading fails

2018-11-05 Thread GitBox
DennisKonrad commented on issue #2820: template downloading fails URL: https://github.com/apache/cloudstack/issues/2820#issuecomment-435833556 Hi, we also obseve the same problem. It happens for us when uploading a 20GB qcow image. We tried to upload from http via: Templa

[GitHub] wido commented on issue #2985: kvm/ceph: When create a new RBD image use modern flags/features for t…

2018-11-05 Thread GitBox
wido commented on issue #2985: kvm/ceph: When create a new RBD image use modern flags/features for t… URL: https://github.com/apache/cloudstack/pull/2985#issuecomment-435827311 Thanks for the feedback! See the updated code Th

[GitHub] DaanHoogland commented on issue #2893: ApiServer: signature v3 to accept more formats

2018-11-05 Thread GitBox
DaanHoogland commented on issue #2893: ApiServer: signature v3 to accept more formats URL: https://github.com/apache/cloudstack/pull/2893#issuecomment-435825341 @bwsw You are right that these tests depend on hwclocktime and are not reliable. Any sugestions on how to fix without losing test

[GitHub] wido commented on a change in pull request #3000: Added two more device names (lo* and dummy*)

2018-11-05 Thread GitBox
wido commented on a change in pull request #3000: Added two more device names (lo* and dummy*) URL: https://github.com/apache/cloudstack/pull/3000#discussion_r230693871 ## File path: plugins/hypervisors/kvm/src/main/java/com/cloud/hypervisor/kvm/resource/LibvirtComputingResource.ja

[GitHub] wido commented on a change in pull request #3000: Added two more device names (lo* and dummy*)

2018-11-05 Thread GitBox
wido commented on a change in pull request #3000: Added two more device names (lo* and dummy*) URL: https://github.com/apache/cloudstack/pull/3000#discussion_r230693919 ## File path: plugins/hypervisors/kvm/src/main/java/com/cloud/hypervisor/kvm/resource/LibvirtComputingResource.ja

[GitHub] wido commented on a change in pull request #2997: Allow KVM VM live migration with ROOT volume on file storage type

2018-11-05 Thread GitBox
wido commented on a change in pull request #2997: Allow KVM VM live migration with ROOT volume on file storage type URL: https://github.com/apache/cloudstack/pull/2997#discussion_r230681797 ## File path: engine/storage/datamotion/src/main/java/org/apache/cloudstack/storage/motion/K

[GitHub] wido commented on a change in pull request #2997: Allow KVM VM live migration with ROOT volume on file storage type

2018-11-05 Thread GitBox
wido commented on a change in pull request #2997: Allow KVM VM live migration with ROOT volume on file storage type URL: https://github.com/apache/cloudstack/pull/2997#discussion_r230687385 ## File path: plugins/hypervisors/kvm/src/main/java/com/cloud/hypervisor/kvm/resource/wrappe

[GitHub] wido commented on a change in pull request #2997: Allow KVM VM live migration with ROOT volume on file storage type

2018-11-05 Thread GitBox
wido commented on a change in pull request #2997: Allow KVM VM live migration with ROOT volume on file storage type URL: https://github.com/apache/cloudstack/pull/2997#discussion_r230680473 ## File path: engine/storage/datamotion/src/main/java/org/apache/cloudstack/storage/motion/K

[GitHub] DennisKonrad commented on issue #2930: HA for Management Server - roundrobin: Certificate ownership

2018-11-05 Thread GitBox
DennisKonrad commented on issue #2930: HA for Management Server - roundrobin: Certificate ownership URL: https://github.com/apache/cloudstack/issues/2930#issuecomment-435810872 Hi @rhtyd, so I don't really get where this issue comes from. Some questions: 1. What did we do to g