[GitHub] blueorangutan commented on issue #2766: kvm: Agent should not check if remaining memory on host is sufficient

2018-07-26 Thread GitBox
blueorangutan commented on issue #2766: kvm: Agent should not check if remaining memory on host is sufficient URL: https://github.com/apache/cloudstack/pull/2766#issuecomment-408319053 @borisstoyanov a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke test

[GitHub] borisstoyanov commented on issue #2766: kvm: Agent should not check if remaining memory on host is sufficient

2018-07-26 Thread GitBox
borisstoyanov commented on issue #2766: kvm: Agent should not check if remaining memory on host is sufficient URL: https://github.com/apache/cloudstack/pull/2766#issuecomment-408318941 @blueorangutan test This is an automated

[GitHub] bennysp commented on issue #2777: Cannot add new host with bonded interface.

2018-07-26 Thread GitBox
bennysp commented on issue #2777: Cannot add new host with bonded interface. URL: https://github.com/apache/cloudstack/issues/2777#issuecomment-408301929 Well, I reinstalled the OS and did this all from scratch... Doesn't look like a "bond" issue, since I continue to get the same issue rega

[GitHub] mike-tutkowski commented on a change in pull request #2761: Add managed storage pool constraints to MigrateWithVolume API method

2018-07-26 Thread GitBox
mike-tutkowski commented on a change in pull request #2761: Add managed storage pool constraints to MigrateWithVolume API method URL: https://github.com/apache/cloudstack/pull/2761#discussion_r205656884 ## File path: engine/orchestration/src/main/java/com/cloud/vm/VirtualMachineMan

[GitHub] mike-tutkowski commented on a change in pull request #2761: Add managed storage pool constraints to MigrateWithVolume API method

2018-07-26 Thread GitBox
mike-tutkowski commented on a change in pull request #2761: Add managed storage pool constraints to MigrateWithVolume API method URL: https://github.com/apache/cloudstack/pull/2761#discussion_r205656884 ## File path: engine/orchestration/src/main/java/com/cloud/vm/VirtualMachineMan

[GitHub] bennysp commented on issue #2777: Cannot add new host with bonded interface.

2018-07-26 Thread GitBox
bennysp commented on issue #2777: Cannot add new host with bonded interface. URL: https://github.com/apache/cloudstack/issues/2777#issuecomment-408281916 I changed to a single interface and I still see issues... ``` DEBUG:root:execute:setenforce 1 DEBUG:root:execute:hostname -f

[GitHub] rafaelweingartner commented on a change in pull request #2761: Add managed storage pool constraints to MigrateWithVolume API method

2018-07-26 Thread GitBox
rafaelweingartner commented on a change in pull request #2761: Add managed storage pool constraints to MigrateWithVolume API method URL: https://github.com/apache/cloudstack/pull/2761#discussion_r205643152 ## File path: engine/orchestration/src/main/java/com/cloud/vm/VirtualMachine

[GitHub] rafaelweingartner commented on issue #2761: Add managed storage pool constraints to MigrateWithVolume API method

2018-07-26 Thread GitBox
rafaelweingartner commented on issue #2761: Add managed storage pool constraints to MigrateWithVolume API method URL: https://github.com/apache/cloudstack/pull/2761#issuecomment-408278876 Ok, no problem, I got the gist. Those changes created a confusion for me though. -

[GitHub] bennysp opened a new issue #2777: Cannot add new host with bonded interface.

2018-07-26 Thread GitBox
bennysp opened a new issue #2777: Cannot add new host with bonded interface. URL: https://github.com/apache/cloudstack/issues/2777 # ISSUE TYPE * Bug Report # COMPONENT NAME ~~~ cloudstack-agent ~~~ # CLOUDSTACK VERSION ~~~

[GitHub] mike-tutkowski commented on issue #2761: Add managed storage pool constraints to MigrateWithVolume API method

2018-07-26 Thread GitBox
mike-tutkowski commented on issue #2761: Add managed storage pool constraints to MigrateWithVolume API method URL: https://github.com/apache/cloudstack/pull/2761#issuecomment-408271570 By the way, I didn't change the test for https://github.com/mike-tutkowski/cloudstack/commit/bea1dad5b84b

[GitHub] mike-tutkowski commented on issue #2761: Add managed storage pool constraints to MigrateWithVolume API method

2018-07-26 Thread GitBox
mike-tutkowski commented on issue #2761: Add managed storage pool constraints to MigrateWithVolume API method URL: https://github.com/apache/cloudstack/pull/2761#issuecomment-408270678 @rafaelweingartner Let's say the user provided a (target) managed storage pool for the volume in question

[GitHub] rafaelweingartner edited a comment on issue #2761: Add managed storage pool constraints to MigrateWithVolume API method

2018-07-26 Thread GitBox
rafaelweingartner edited a comment on issue #2761: Add managed storage pool constraints to MigrateWithVolume API method URL: https://github.com/apache/cloudstack/pull/2761#issuecomment-408264657 Thanks mike. I applied your suggestions. I am only a little confused with the use of `executeMa

[GitHub] rafaelweingartner commented on issue #2761: Add managed storage pool constraints to MigrateWithVolume API method

2018-07-26 Thread GitBox
rafaelweingartner commented on issue #2761: Add managed storage pool constraints to MigrateWithVolume API method URL: https://github.com/apache/cloudstack/pull/2761#issuecomment-408264657 Thanks mike. I applied your suggestions. I am only a little confused with the use of `executeManagedSt

[GitHub] mike-tutkowski commented on issue #2761: Add managed storage pool constraints to MigrateWithVolume API method

2018-07-26 Thread GitBox
mike-tutkowski commented on issue #2761: Add managed storage pool constraints to MigrateWithVolume API method URL: https://github.com/apache/cloudstack/pull/2761#issuecomment-408255118 @rafaelweingartner I think we are very close. Take a look at what I did here: https://github.com/mike-tu

[GitHub] GabrielBrascher commented on issue #2773: CLOUDSTACK-10328: Add Secondary IPv6 address through API

2018-07-26 Thread GitBox
GabrielBrascher commented on issue #2773: CLOUDSTACK-10328: Add Secondary IPv6 address through API URL: https://github.com/apache/cloudstack/pull/2773#issuecomment-408215215 @rafaelweingartner @rhtyd the Travis is ok now, fixed the missing Apache license. -

[GitHub] wido commented on issue #2766: kvm: Agent should not check if remaining memory on host is sufficient

2018-07-26 Thread GitBox
wido commented on issue #2766: kvm: Agent should not check if remaining memory on host is sufficient URL: https://github.com/apache/cloudstack/pull/2766#issuecomment-408214723 @rafaelweingartner Ah, yes, I didn't notice the tests yet :) Just wondering when we merge PRs into master right no

[GitHub] rafaelweingartner commented on issue #2766: kvm: Agent should not check if remaining memory on host is sufficient

2018-07-26 Thread GitBox
rafaelweingartner commented on issue #2766: kvm: Agent should not check if remaining memory on host is sufficient URL: https://github.com/apache/cloudstack/pull/2766#issuecomment-408210369 Yes, I approved it already two days ago. Shouldn't we wait for the test results though? --

[GitHub] wido commented on issue #2766: kvm: Agent should not check if remaining memory on host is sufficient

2018-07-26 Thread GitBox
wido commented on issue #2766: kvm: Agent should not check if remaining memory on host is sufficient URL: https://github.com/apache/cloudstack/pull/2766#issuecomment-408209932 @rafaelweingartner Are you OK with me merging this PR as we have the approvals? -

[GitHub] khos2ow commented on issue #2775: Fix typo in ISO url

2018-07-26 Thread GitBox
khos2ow commented on issue #2775: Fix typo in ISO url URL: https://github.com/apache/cloudstack/pull/2775#issuecomment-408206255 I updated the checksum type to `sha1` and double checked that the has is actually valid, as well as the URL.

[GitHub] rafaelweingartner commented on a change in pull request #2776: Issue 2774: Changed the implementation of isVolumeOnManagedStorage(VolumeInfo) to…

2018-07-26 Thread GitBox
rafaelweingartner commented on a change in pull request #2776: Issue 2774: Changed the implementation of isVolumeOnManagedStorage(VolumeInfo) to… URL: https://github.com/apache/cloudstack/pull/2776#discussion_r205571779 ## File path: engine/storage/datamotion/src/org/apache/cloudst

[GitHub] mike-tutkowski opened a new pull request #2776: Issue 2774: Changed the implementation of isVolumeOnManagedStorage(VolumeInfo) to…

2018-07-26 Thread GitBox
mike-tutkowski opened a new pull request #2776: Issue 2774: Changed the implementation of isVolumeOnManagedStorage(VolumeInfo) to… URL: https://github.com/apache/cloudstack/pull/2776 … check if the data store in question is for primary storage ## Description In the StorageSystemDa

[GitHub] blueorangutan commented on issue #2769: Fix config drive iso path on Vmware

2018-07-26 Thread GitBox
blueorangutan commented on issue #2769: Fix config drive iso path on Vmware URL: https://github.com/apache/cloudstack/pull/2769#issuecomment-408197757 Trillian test result (tid-2889) Environment: vmware-65 (x2), Advanced Networking with Mgmt server 7 Total time taken: 36675 seconds

[GitHub] rafaelweingartner commented on a change in pull request #2775: Fix typo in ISO url

2018-07-26 Thread GitBox
rafaelweingartner commented on a change in pull request #2775: Fix typo in ISO url URL: https://github.com/apache/cloudstack/pull/2775#discussion_r205509652 ## File path: tools/appliance/builtin/template.json ## @@ -23,7 +23,7 @@ "disk_interface": "virtio", "

[GitHub] khos2ow commented on a change in pull request #2747: systemvm: Update ISO URLs to the latest

2018-07-26 Thread GitBox
khos2ow commented on a change in pull request #2747: systemvm: Update ISO URLs to the latest URL: https://github.com/apache/cloudstack/pull/2747#discussion_r205508736 ## File path: tools/appliance/builtin/template.json ## @@ -23,8 +23,8 @@ "disk_interface": "virtio"

[GitHub] khos2ow commented on a change in pull request #2775: Fix typo in ISO url

2018-07-26 Thread GitBox
khos2ow commented on a change in pull request #2775: Fix typo in ISO url URL: https://github.com/apache/cloudstack/pull/2775#discussion_r205505855 ## File path: tools/appliance/builtin/template.json ## @@ -23,7 +23,7 @@ "disk_interface": "virtio", "net_device"

[GitHub] khos2ow commented on a change in pull request #2775: Fix typo in ISO url

2018-07-26 Thread GitBox
khos2ow commented on a change in pull request #2775: Fix typo in ISO url URL: https://github.com/apache/cloudstack/pull/2775#discussion_r205505855 ## File path: tools/appliance/builtin/template.json ## @@ -23,7 +23,7 @@ "disk_interface": "virtio", "net_device"

[GitHub] rafaelweingartner commented on a change in pull request #2775: Fix typo in ISO url

2018-07-26 Thread GitBox
rafaelweingartner commented on a change in pull request #2775: Fix typo in ISO url URL: https://github.com/apache/cloudstack/pull/2775#discussion_r205503599 ## File path: tools/appliance/builtin/template.json ## @@ -23,7 +23,7 @@ "disk_interface": "virtio", "

[GitHub] DaanHoogland commented on a change in pull request #2747: systemvm: Update ISO URLs to the latest

2018-07-26 Thread GitBox
DaanHoogland commented on a change in pull request #2747: systemvm: Update ISO URLs to the latest URL: https://github.com/apache/cloudstack/pull/2747#discussion_r205500628 ## File path: tools/appliance/builtin/template.json ## @@ -23,8 +23,8 @@ "disk_interface": "vi

[GitHub] khos2ow commented on issue #2775: Fix typo in ISO url

2018-07-26 Thread GitBox
khos2ow commented on issue #2775: Fix typo in ISO url URL: https://github.com/apache/cloudstack/pull/2775#issuecomment-408124262 @rhtyd @rafaelweingartner fixed typo. This is an automated message from the Apache Git Service. T

[GitHub] khos2ow opened a new pull request #2775: Fix typo in ISO url

2018-07-26 Thread GitBox
khos2ow opened a new pull request #2775: Fix typo in ISO url URL: https://github.com/apache/cloudstack/pull/2775 ## Description The correct ISO url is http://mirror.nbrc.ac.in/centos/7/isos/x86_64/CentOS-7-x86_64-Minimal-1804.iso and not ends with `1808`. ## Types

[GitHub] khos2ow commented on a change in pull request #2747: systemvm: Update ISO URLs to the latest

2018-07-26 Thread GitBox
khos2ow commented on a change in pull request #2747: systemvm: Update ISO URLs to the latest URL: https://github.com/apache/cloudstack/pull/2747#discussion_r205486364 ## File path: tools/appliance/builtin/template.json ## @@ -23,8 +23,8 @@ "disk_interface": "virtio"

[GitHub] rafaelweingartner commented on issue #2761: Add managed storage pool constraints to MigrateWithVolume API method

2018-07-26 Thread GitBox
rafaelweingartner commented on issue #2761: Add managed storage pool constraints to MigrateWithVolume API method URL: https://github.com/apache/cloudstack/pull/2761#issuecomment-408113305 @mike-tutkowski I just pushed a new commit with the changes you suggested. ---

[GitHub] rafaelweingartner commented on a change in pull request #2761: Add managed storage pool constraints to MigrateWithVolume API method

2018-07-26 Thread GitBox
rafaelweingartner commented on a change in pull request #2761: Add managed storage pool constraints to MigrateWithVolume API method URL: https://github.com/apache/cloudstack/pull/2761#discussion_r205432339 ## File path: engine/orchestration/src/main/java/com/cloud/vm/VirtualMachine

[GitHub] rafaelweingartner commented on a change in pull request #2761: Add managed storage pool constraints to MigrateWithVolume API method

2018-07-26 Thread GitBox
rafaelweingartner commented on a change in pull request #2761: Add managed storage pool constraints to MigrateWithVolume API method URL: https://github.com/apache/cloudstack/pull/2761#discussion_r205469018 ## File path: engine/orchestration/src/main/java/com/cloud/vm/VirtualMachine

[GitHub] rafaelweingartner commented on a change in pull request #2761: Add managed storage pool constraints to MigrateWithVolume API method

2018-07-26 Thread GitBox
rafaelweingartner commented on a change in pull request #2761: Add managed storage pool constraints to MigrateWithVolume API method URL: https://github.com/apache/cloudstack/pull/2761#discussion_r205432501 ## File path: engine/orchestration/src/main/java/com/cloud/vm/VirtualMachine

[GitHub] rafaelweingartner commented on a change in pull request #2761: Add managed storage pool constraints to MigrateWithVolume API method

2018-07-26 Thread GitBox
rafaelweingartner commented on a change in pull request #2761: Add managed storage pool constraints to MigrateWithVolume API method URL: https://github.com/apache/cloudstack/pull/2761#discussion_r205432339 ## File path: engine/orchestration/src/main/java/com/cloud/vm/VirtualMachine

[GitHub] rafaelweingartner commented on a change in pull request #2761: Add managed storage pool constraints to MigrateWithVolume API method

2018-07-26 Thread GitBox
rafaelweingartner commented on a change in pull request #2761: Add managed storage pool constraints to MigrateWithVolume API method URL: https://github.com/apache/cloudstack/pull/2761#discussion_r205419513 ## File path: engine/orchestration/src/main/java/com/cloud/vm/VirtualMachine

[GitHub] rafaelweingartner commented on a change in pull request #2773: CLOUDSTACK-10328: Add Secondary IPv6 address through API

2018-07-26 Thread GitBox
rafaelweingartner commented on a change in pull request #2773: CLOUDSTACK-10328: Add Secondary IPv6 address through API URL: https://github.com/apache/cloudstack/pull/2773#discussion_r205415478 ## File path: api/src/main/java/com/cloud/network/NetworkService.java ## @@ -18

[GitHub] rafaelweingartner commented on a change in pull request #2773: CLOUDSTACK-10328: Add Secondary IPv6 address through API

2018-07-26 Thread GitBox
rafaelweingartner commented on a change in pull request #2773: CLOUDSTACK-10328: Add Secondary IPv6 address through API URL: https://github.com/apache/cloudstack/pull/2773#discussion_r205417670 ## File path: api/src/main/java/org/apache/cloudstack/api/command/user/vm/AddIpToVmNicCm

[GitHub] rafaelweingartner commented on a change in pull request #2773: CLOUDSTACK-10328: Add Secondary IPv6 address through API

2018-07-26 Thread GitBox
rafaelweingartner commented on a change in pull request #2773: CLOUDSTACK-10328: Add Secondary IPv6 address through API URL: https://github.com/apache/cloudstack/pull/2773#discussion_r205415983 ## File path: api/src/main/java/org/apache/cloudstack/api/command/user/vm/AddIpToVmNicCm

[GitHub] rafaelweingartner commented on a change in pull request #2773: CLOUDSTACK-10328: Add Secondary IPv6 address through API

2018-07-26 Thread GitBox
rafaelweingartner commented on a change in pull request #2773: CLOUDSTACK-10328: Add Secondary IPv6 address through API URL: https://github.com/apache/cloudstack/pull/2773#discussion_r205416258 ## File path: api/src/main/java/org/apache/cloudstack/api/command/user/vm/RemoveIpFromVm

[GitHub] rafaelweingartner commented on a change in pull request #2773: CLOUDSTACK-10328: Add Secondary IPv6 address through API

2018-07-26 Thread GitBox
rafaelweingartner commented on a change in pull request #2773: CLOUDSTACK-10328: Add Secondary IPv6 address through API URL: https://github.com/apache/cloudstack/pull/2773#discussion_r205416950 ## File path: server/src/main/java/com/cloud/network/Ipv6AddressManagerImpl.java ###

[GitHub] rafaelweingartner commented on a change in pull request #2773: CLOUDSTACK-10328: Add Secondary IPv6 address through API

2018-07-26 Thread GitBox
rafaelweingartner commented on a change in pull request #2773: CLOUDSTACK-10328: Add Secondary IPv6 address through API URL: https://github.com/apache/cloudstack/pull/2773#discussion_r205415515 ## File path: api/src/main/java/com/cloud/network/NetworkService.java ## @@ -18

[GitHub] rafaelweingartner commented on a change in pull request #2773: CLOUDSTACK-10328: Add Secondary IPv6 address through API

2018-07-26 Thread GitBox
rafaelweingartner commented on a change in pull request #2773: CLOUDSTACK-10328: Add Secondary IPv6 address through API URL: https://github.com/apache/cloudstack/pull/2773#discussion_r205416708 ## File path: server/src/main/java/com/cloud/network/IpAddressManagerImpl.java #

[GitHub] rafaelweingartner commented on issue #2773: CLOUDSTACK-10328: Add Secondary IPv6 address through API

2018-07-26 Thread GitBox
rafaelweingartner commented on issue #2773: CLOUDSTACK-10328: Add Secondary IPv6 address through API URL: https://github.com/apache/cloudstack/pull/2773#issuecomment-408060351 @GabrielBrascher: ``` Unapproved licenses: server/src/test/java/com/cloud/network/Ipv6AddressManagerTest

[GitHub] DaanHoogland commented on issue #2660: Macaddresses

2018-07-26 Thread GitBox
DaanHoogland commented on issue #2660: Macaddresses URL: https://github.com/apache/cloudstack/pull/2660#issuecomment-408058575 surely 'squash and merge' @wido This is an automated message from the Apache Git Service. To respo

[GitHub] DaanHoogland commented on issue #2593: api: move default response from XML to JSON

2018-07-26 Thread GitBox
DaanHoogland commented on issue #2593: api: move default response from XML to JSON URL: https://github.com/apache/cloudstack/pull/2593#issuecomment-408057489 let's do the 5.0 dance This is an automated message from the Apache

[GitHub] blueorangutan commented on issue #2755: Fix migrate vol xen vmware test

2018-07-26 Thread GitBox
blueorangutan commented on issue #2755: Fix migrate vol xen vmware test URL: https://github.com/apache/cloudstack/pull/2755#issuecomment-408043464 @borisstoyanov a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

[GitHub] borisstoyanov commented on issue #2755: Fix migrate vol xen vmware test

2018-07-26 Thread GitBox
borisstoyanov commented on issue #2755: Fix migrate vol xen vmware test URL: https://github.com/apache/cloudstack/pull/2755#issuecomment-408043295 @blueorangutan test This is an automated message from the Apache Git Service. T

[GitHub] wido commented on issue #2593: api: move default response from XML to JSON

2018-07-26 Thread GitBox
wido commented on issue #2593: api: move default response from XML to JSON URL: https://github.com/apache/cloudstack/pull/2593#issuecomment-408043084 I agree with @rafaelweingartner This would be a rather big change and I think we should discuss it properly. I understand the change,

[GitHub] blueorangutan commented on issue #2755: Fix migrate vol xen vmware test

2018-07-26 Thread GitBox
blueorangutan commented on issue #2755: Fix migrate vol xen vmware test URL: https://github.com/apache/cloudstack/pull/2755#issuecomment-408041738 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-2211 This is an automated mess

[GitHub] marcaurele commented on issue #2578: api: add command to list management servers

2018-07-26 Thread GitBox
marcaurele commented on issue #2578: api: add command to list management servers URL: https://github.com/apache/cloudstack/pull/2578#issuecomment-408041817 @rhtyd it was about the sprites, but I found the one I was looking for `infrastructure-icons.png` which does not have anything for a ma

[GitHub] blueorangutan commented on issue #2755: Fix migrate vol xen vmware test

2018-07-26 Thread GitBox
blueorangutan commented on issue #2755: Fix migrate vol xen vmware test URL: https://github.com/apache/cloudstack/pull/2755#issuecomment-408033055 @borisstoyanov a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. --

[GitHub] borisstoyanov commented on issue #2755: Fix migrate vol xen vmware test

2018-07-26 Thread GitBox
borisstoyanov commented on issue #2755: Fix migrate vol xen vmware test URL: https://github.com/apache/cloudstack/pull/2755#issuecomment-408032999 @blueorangutan package This is an automated message from the Apache Git Service

[GitHub] DaanHoogland opened a new issue #2774: NPE in isOnVolumeManagedStorage() leads to not being able to extract volumes

2018-07-26 Thread GitBox
DaanHoogland opened a new issue #2774: NPE in isOnVolumeManagedStorage() leads to not being able to extract volumes URL: https://github.com/apache/cloudstack/issues/2774 # ISSUE TYPE * Bug Report # COMPONENT NAME ~~~ Storage Motion ~~~

[GitHub] blueorangutan commented on issue #2755: Fix migrate vol xen vmware test

2018-07-26 Thread GitBox
blueorangutan commented on issue #2755: Fix migrate vol xen vmware test URL: https://github.com/apache/cloudstack/pull/2755#issuecomment-408019790 Packaging result: ✔centos6 ✖centos7 ✔debian. JID-2210 This is an automated mess

[GitHub] wido commented on issue #2766: kvm: Agent should not check if remaining memory on host is sufficient

2018-07-26 Thread GitBox
wido commented on issue #2766: kvm: Agent should not check if remaining memory on host is sufficient URL: https://github.com/apache/cloudstack/pull/2766#issuecomment-408017277 @rhtyd Thanks! As I stated, the Agent shouldn't worry about Memory Allocation is that's up to the Management Serve

[GitHub] wido commented on issue #2660: Macaddresses

2018-07-26 Thread GitBox
wido commented on issue #2660: Macaddresses URL: https://github.com/apache/cloudstack/pull/2660#issuecomment-408015501 A lot of commits for a fairly small change? Maybe squash and then merge? It seems like a code cleanup. Thi

[GitHub] wido commented on issue #2769: Fix config drive iso path on Vmware

2018-07-26 Thread GitBox
wido commented on issue #2769: Fix config drive iso path on Vmware URL: https://github.com/apache/cloudstack/pull/2769#issuecomment-408014920 In which part of the code is the lowercase 'configdrive' set? This is an automated m

[GitHub] blueorangutan commented on issue #2766: kvm: Agent should not check if remaining memory on host is sufficient

2018-07-26 Thread GitBox
blueorangutan commented on issue #2766: kvm: Agent should not check if remaining memory on host is sufficient URL: https://github.com/apache/cloudstack/pull/2766#issuecomment-408014240 @rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests -

[GitHub] blueorangutan commented on issue #2769: Fix config drive iso path on Vmware

2018-07-26 Thread GitBox
blueorangutan commented on issue #2769: Fix config drive iso path on Vmware URL: https://github.com/apache/cloudstack/pull/2769#issuecomment-408014232 @rhtyd a Trillian-Jenkins test job (centos7 mgmt + vmware-65) has been kicked to run smoke tests --

[GitHub] rhtyd commented on issue #2769: Fix config drive iso path on Vmware

2018-07-26 Thread GitBox
rhtyd commented on issue #2769: Fix config drive iso path on Vmware URL: https://github.com/apache/cloudstack/pull/2769#issuecomment-408014198 @blueorangutan test centos7 vmware-65 This is an automated message from the Apache

[GitHub] rhtyd commented on issue #2766: kvm: Agent should not check if remaining memory on host is sufficient

2018-07-26 Thread GitBox
rhtyd commented on issue #2766: kvm: Agent should not check if remaining memory on host is sufficient URL: https://github.com/apache/cloudstack/pull/2766#issuecomment-408014090 @blueorangutan test This is an automated messag

[GitHub] blueorangutan commented on issue #2766: kvm: Agent should not check if remaining memory on host is sufficient

2018-07-26 Thread GitBox
blueorangutan commented on issue #2766: kvm: Agent should not check if remaining memory on host is sufficient URL: https://github.com/apache/cloudstack/pull/2766#issuecomment-408013985 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-2209 ---

[GitHub] blueorangutan commented on issue #2769: Fix config drive iso path on Vmware

2018-07-26 Thread GitBox
blueorangutan commented on issue #2769: Fix config drive iso path on Vmware URL: https://github.com/apache/cloudstack/pull/2769#issuecomment-408013982 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-2208 This is an automated

[GitHub] blueorangutan commented on issue #2755: Fix migrate vol xen vmware test

2018-07-26 Thread GitBox
blueorangutan commented on issue #2755: Fix migrate vol xen vmware test URL: https://github.com/apache/cloudstack/pull/2755#issuecomment-408010366 @borisstoyanov a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. --

[GitHub] borisstoyanov commented on issue #2755: Fix migrate vol xen vmware test

2018-07-26 Thread GitBox
borisstoyanov commented on issue #2755: Fix migrate vol xen vmware test URL: https://github.com/apache/cloudstack/pull/2755#issuecomment-408010160 @blueorangutan package This is an automated message from the Apache Git Service

[GitHub] borisstoyanov commented on issue #2755: Fix migrate vol xen vmware test

2018-07-26 Thread GitBox
borisstoyanov commented on issue #2755: Fix migrate vol xen vmware test URL: https://github.com/apache/cloudstack/pull/2755#issuecomment-408009476 @rhtyd let me rebase and run tests again to see that we've actually fixed the issue. --

[GitHub] rhtyd commented on issue #2204: [CLOUDSTACK-10025] Adding Support for NoVNC Console for KVM and XENSERVER

2018-07-26 Thread GitBox
rhtyd commented on issue #2204: [CLOUDSTACK-10025] Adding Support for NoVNC Console for KVM and XENSERVER URL: https://github.com/apache/cloudstack/pull/2204#issuecomment-408006346 Looking forward to it @syed ! This is an aut

[cloudstack] 01/01: Merge branch '4.11'

2018-07-26 Thread rohit
This is an automated email from the ASF dual-hosted git repository. rohit pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/cloudstack.git commit c8a980b9cfd1361633d1e48a087c23cca3b8a261 Merge: d64f787 52fb426 Author: Rohit Yadav AuthorDate: Thu Jul 26 13:08:41 2

[cloudstack] branch master updated (d64f787 -> c8a980b)

2018-07-26 Thread rohit
This is an automated email from the ASF dual-hosted git repository. rohit pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/cloudstack.git. from d64f787 Merge branch '4.11' add 52fb426 systemvm: Update ISO URLs to the latest (#2747) new c8a980b M

[GitHub] rhtyd commented on a change in pull request #2747: systemvm: Update ISO URLs to the latest

2018-07-26 Thread GitBox
rhtyd commented on a change in pull request #2747: systemvm: Update ISO URLs to the latest URL: https://github.com/apache/cloudstack/pull/2747#discussion_r205357787 ## File path: tools/appliance/builtin/template.json ## @@ -23,8 +23,8 @@ "disk_interface": "virtio",

[GitHub] rhtyd closed pull request #2747: systemvm: Update ISO URLs to the latest

2018-07-26 Thread GitBox
rhtyd closed pull request #2747: systemvm: Update ISO URLs to the latest URL: https://github.com/apache/cloudstack/pull/2747 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pul

[cloudstack] branch 4.11 updated: systemvm: Update ISO URLs to the latest (#2747)

2018-07-26 Thread rohit
This is an automated email from the ASF dual-hosted git repository. rohit pushed a commit to branch 4.11 in repository https://gitbox.apache.org/repos/asf/cloudstack.git The following commit(s) were added to refs/heads/4.11 by this push: new 52fb426 systemvm: Update ISO URLs to the latest

[GitHub] rhtyd commented on issue #2747: systemvm: Update ISO URLs to the latest

2018-07-26 Thread GitBox
rhtyd commented on issue #2747: systemvm: Update ISO URLs to the latest URL: https://github.com/apache/cloudstack/pull/2747#issuecomment-408005444 Changes are only in systemvmtemplate appliance build code. I'll merge this based on reviews. --

[GitHub] asfgit closed issue #2733: invalid consoleproxy domain after upgrade from 4.5 to 4.11.1

2018-07-26 Thread GitBox
asfgit closed issue #2733: invalid consoleproxy domain after upgrade from 4.5 to 4.11.1 URL: https://github.com/apache/cloudstack/issues/2733 This is an automated message from the Apache Git Service. To respond to the messag

[cloudstack] 01/01: Merge branch '4.11'

2018-07-26 Thread rohit
This is an automated email from the ASF dual-hosted git repository. rohit pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/cloudstack.git commit d64f787b6674cf72aa39a75a1ec7c49177f29909 Merge: 1960124 f88d606 Author: Rohit Yadav AuthorDate: Thu Jul 26 13:03:46 2

[cloudstack] branch master updated (1960124 -> d64f787)

2018-07-26 Thread rohit
This is an automated email from the ASF dual-hosted git repository. rohit pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/cloudstack.git. from 1960124 server: Make HTTP handler of embedded server to be configured (#2762) add 4c66908 ui: Fix clicking

[GitHub] rhtyd closed pull request #2734: Fix invalid consoleproxy url after upgrade

2018-07-26 Thread GitBox
rhtyd closed pull request #2734: Fix invalid consoleproxy url after upgrade URL: https://github.com/apache/cloudstack/pull/2734 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign

[cloudstack] branch 4.11 updated: consoleproxy: Fixes #2733 invalid consoleproxy url after upgrade (#2734)

2018-07-26 Thread rohit
This is an automated email from the ASF dual-hosted git repository. rohit pushed a commit to branch 4.11 in repository https://gitbox.apache.org/repos/asf/cloudstack.git The following commit(s) were added to refs/heads/4.11 by this push: new f88d606 consoleproxy: Fixes #2733 invalid consol

[GitHub] rhtyd commented on issue #2755: Fix migrate vol xen vmware test

2018-07-26 Thread GitBox
rhtyd commented on issue #2755: Fix migrate vol xen vmware test URL: https://github.com/apache/cloudstack/pull/2755#issuecomment-408004193 @borisstoyanov can you comment if this is ready to be merged? This is an automated mess

[GitHub] rhtyd closed pull request #2762: Make HTTP handler of embedded server to be configured

2018-07-26 Thread GitBox
rhtyd closed pull request #2762: Make HTTP handler of embedded server to be configured URL: https://github.com/apache/cloudstack/pull/2762 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this i

[cloudstack] branch master updated: server: Make HTTP handler of embedded server to be configured (#2762)

2018-07-26 Thread rohit
This is an automated email from the ASF dual-hosted git repository. rohit pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/cloudstack.git The following commit(s) were added to refs/heads/master by this push: new 1960124 server: Make HTTP handler of embedde

[cloudstack] branch master updated: test: Fix test_deploy_virtio_scsi_vm.py smoke test failures (#2752)

2018-07-26 Thread rohit
This is an automated email from the ASF dual-hosted git repository. rohit pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/cloudstack.git The following commit(s) were added to refs/heads/master by this push: new 6156b44 test: Fix test_deploy_virtio_scsi_vm

[GitHub] rhtyd closed pull request #2752: Fix test_deploy_virtio_scsi_vm.py smoke test failures

2018-07-26 Thread GitBox
rhtyd closed pull request #2752: Fix test_deploy_virtio_scsi_vm.py smoke test failures URL: https://github.com/apache/cloudstack/pull/2752 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this i

[GitHub] rhtyd commented on issue #2773: CLOUDSTACK-10328: Add Secondary IPv6 address through API

2018-07-26 Thread GitBox
rhtyd commented on issue #2773: CLOUDSTACK-10328: Add Secondary IPv6 address through API URL: https://github.com/apache/cloudstack/pull/2773#issuecomment-408003265 @GabrielBrascher can you have a look at travis failure This i

[GitHub] rhtyd commented on issue #2578: api: add command to list management servers

2018-07-26 Thread GitBox
rhtyd commented on issue #2578: api: add command to list management servers URL: https://github.com/apache/cloudstack/pull/2578#issuecomment-408002852 @marcaurele sorry did not get the question, if you're looking for link/url where systemvmtemplates are hosted you can use the http://downlo

[cloudstack] branch master updated: kvm: Fixes #2763 move post-renewal libvirt restart class suitably (#2764)

2018-07-26 Thread rohit
This is an automated email from the ASF dual-hosted git repository. rohit pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/cloudstack.git The following commit(s) were added to refs/heads/master by this push: new 7667846 kvm: Fixes #2763 move post-renewal l

[GitHub] rhtyd closed pull request #2764: kvm: Fixes #2763 move post-renewal libvirt restart class suitably

2018-07-26 Thread GitBox
rhtyd closed pull request #2764: kvm: Fixes #2763 move post-renewal libvirt restart class suitably URL: https://github.com/apache/cloudstack/pull/2764 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance

[GitHub] rhtyd closed issue #2763: Host is still unsecured when provisionCertificate api is called

2018-07-26 Thread GitBox
rhtyd closed issue #2763: Host is still unsecured when provisionCertificate api is called URL: https://github.com/apache/cloudstack/issues/2763 This is an automated message from the Apache Git Service. To respond to the mess

[GitHub] blueorangutan commented on issue #2766: kvm: Agent should not check if remaining memory on host is sufficient

2018-07-26 Thread GitBox
blueorangutan commented on issue #2766: kvm: Agent should not check if remaining memory on host is sufficient URL: https://github.com/apache/cloudstack/pull/2766#issuecomment-408001606 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. ---

[GitHub] blueorangutan commented on issue #2769: Fix config drive iso path on Vmware

2018-07-26 Thread GitBox
blueorangutan commented on issue #2769: Fix config drive iso path on Vmware URL: https://github.com/apache/cloudstack/pull/2769#issuecomment-408001360 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. --

[GitHub] rhtyd commented on issue #2766: kvm: Agent should not check if remaining memory on host is sufficient

2018-07-26 Thread GitBox
rhtyd commented on issue #2766: kvm: Agent should not check if remaining memory on host is sufficient URL: https://github.com/apache/cloudstack/pull/2766#issuecomment-408001413 @blueorangutan package This is an automa

[GitHub] rhtyd commented on issue #2769: Fix config drive iso path on Vmware

2018-07-26 Thread GitBox
rhtyd commented on issue #2769: Fix config drive iso path on Vmware URL: https://github.com/apache/cloudstack/pull/2769#issuecomment-408001236 @blueorangutan package This is an automated message from the Apache Git Ser