[GitHub] vedulasantosh commented on a change in pull request #2240: CLOUDSTACK-10051 Mouse Scrolling is not working in instance VM console

2017-09-22 Thread git
vedulasantosh commented on a change in pull request #2240: CLOUDSTACK-10051 
Mouse Scrolling is not working in instance VM console
URL: https://github.com/apache/cloudstack/pull/2240#discussion_r140425539
 
 

 ##
 File path: systemvm/js/jquery.flot.navigate.js
 ##
 @@ -0,0 +1,336 @@
+/*
 
 Review comment:
   @rhtyd can i place exclude tag for this jquery.flot.navigate.js file?
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] rhtyd commented on a change in pull request #2054: CLOUDSTACK-9886 : After restarting cloudstack-management , It takes time to connect hosts

2017-09-22 Thread git
rhtyd commented on a change in pull request #2054: CLOUDSTACK-9886 : After 
restarting cloudstack-management , It takes time to connect hosts
URL: https://github.com/apache/cloudstack/pull/2054#discussion_r140418406
 
 

 ##
 File path: test/integration/component/test_host.py
 ##
 @@ -0,0 +1,205 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license agreements.  See the NOTICE file
+# distributed with this work for additional information
+# regarding copyright ownership.  The ASF licenses this file
+# to you under the Apache License, Version 2.0 (the
+# "License"); you may not use this file except in compliance
+# with the License.  You may obtain a copy of the License at
+#
+#   http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing,
+# software distributed under the License is distributed on an
+# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+# KIND, either express or implied.  See the License for the
+# specific language governing permissions and limitations
+# under the License.
+from __builtin__ import False
+""" BVT tests for Hosts Test
+"""
+
+# Import Local Modules
+from marvin.codes import FAILED
+from marvin.cloudstackTestCase import *
+from marvin.cloudstackAPI import *
+from marvin.lib.utils import *
+from marvin.lib.base import *
+from marvin.lib.common import *
+from nose.plugins.attrib import attr
+from marvin.sshClient import SshClient
+import time
+from time import sleep
+
+_multiprocess_shared_ = False
+
+
+class TestHostHA(cloudstackTestCase):
 
 Review comment:
   Ping @mrunalinikankariya ?
 
----
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] rhtyd commented on issue #1707: CLOUDSTACK-9397: Add Watchdog timer to KVM Instance

2017-09-21 Thread git
rhtyd commented on issue #1707: CLOUDSTACK-9397: Add Watchdog timer to KVM 
Instance
URL: https://github.com/apache/cloudstack/pull/1707#issuecomment-331349646
 
 
   @wido I think Trillian may be using CentOS 6.8 for buildilng kvm hosts, 
@borisstoyanov @PaulAngus can you comment?
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] rhtyd commented on issue #2269: CLOUDSTACK-10083: Allow ModifySshKeys to be forwarded to the agent in?

2017-09-21 Thread git
rhtyd commented on issue #2269: CLOUDSTACK-10083: Allow ModifySshKeys to be 
forwarded to the agent in?
URL: https://github.com/apache/cloudstack/pull/2269#issuecomment-331349516
 
 
   Test + code LGTM, another review is requested and we can then merge this PR. 
Thanks @marcaurele 
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] DaanHoogland commented on issue #2249: WIP CLOUDSTACK-10007 Isolation method claim

2017-09-21 Thread git
DaanHoogland commented on issue #2249: WIP CLOUDSTACK-10007 Isolation method 
claim
URL: https://github.com/apache/cloudstack/pull/2249#issuecomment-331212929
 
 
   @blueorangutan matrix
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] blueorangutan commented on issue #2269: CLOUDSTACK-10083: Allow ModifySshKeys to be forwarded to the agent in?

2017-09-21 Thread git
blueorangutan commented on issue #2269: CLOUDSTACK-10083: Allow ModifySshKeys 
to be forwarded to the agent in?
URL: https://github.com/apache/cloudstack/pull/2269#issuecomment-331185628
 
 
   Trillian test result (tid-1535)
   Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
   Total time taken: 34937 seconds
   Marvin logs: 
https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr2269-t1535-kvm-centos7.zip
   Intermitten failure detected: /marvin/tests/smoke/test_privategw_acl.py
   Intermitten failure detected: /marvin/tests/smoke/test_vpc_vpn.py
   Test completed. 60 look OK, 2 have error(s)
   
   
   Test | Result | Time (s) | Test File
   --- | --- | --- | ---
   test_01_vpc_remote_access_vpn | `Failure` | 61.06 | test_vpc_vpn.py
   test_04_rvpc_privategw_static_routes | `Failure` | 356.05 | 
test_privategw_acl.py
   test_change_service_offering_for_vm_with_snapshots | Skipped | 0.00 | 
test_vm_snapshots.py
   test_09_copy_delete_template | Skipped | 0.02 | test_templates.py
   test_06_copy_template | Skipped | 0.00 | test_templates.py
   test_static_role_account_acls | Skipped | 0.02 | test_staticroles.py
   test_11_ss_nfs_version_on_ssvm | Skipped | 0.02 | test_ssvm.py
   test_01_scale_vm | Skipped | 0.00 | test_scale_vm.py
   test_01_primary_storage_iscsi | Skipped | 0.04 | test_primary_storage.py
   test_vm_nic_adapter_vmxnet3 | Skipped | 0.00 | test_nic_adapter_type.py
   test_nested_virtualization_vmware | Skipped | 0.00 | 
test_nested_virtualization.py
   test_06_copy_iso | Skipped | 0.00 | test_iso.py
   test_list_ha_for_host_valid | Skipped | 0.02 | test_hostha_simulator.py
   test_list_ha_for_host_invalid | Skipped | 0.03 | test_hostha_simulator.py
   test_list_ha_for_host | Skipped | 0.02 | test_hostha_simulator.py
   test_hostha_enable_feature_without_setting_provider | Skipped | 0.02 | 
test_hostha_simulator.py
   test_hostha_enable_feature_valid | Skipped | 0.02 | test_hostha_simulator.py
   test_hostha_disable_feature_valid | Skipped | 0.02 | test_hostha_simulator.py
   test_hostha_configure_invalid_provider | Skipped | 0.02 | 
test_hostha_simulator.py
   test_hostha_configure_default_driver | Skipped | 0.03 | 
test_hostha_simulator.py
   test_ha_verify_fsm_recovering | Skipped | 0.03 | test_hostha_simulator.py
   test_ha_verify_fsm_fenced | Skipped | 0.02 | test_hostha_simulator.py
   test_ha_verify_fsm_degraded | Skipped | 0.03 | test_hostha_simulator.py
   test_ha_verify_fsm_available | Skipped | 0.05 | test_hostha_simulator.py
   test_ha_multiple_mgmt_server_ownership | Skipped | 0.02 | 
test_hostha_simulator.py
   test_ha_list_providers | Skipped | 0.02 | test_hostha_simulator.py
   test_ha_enable_feature_invalid | Skipped | 0.03 | test_hostha_simulator.py
   test_ha_disable_feature_invalid | Skipped | 0.02 | test_hostha_simulator.py
   test_ha_configure_enabledisable_across_clusterzones | Skipped | 0.02 | 
test_hostha_simulator.py
   test_configure_ha_provider_valid | Skipped | 0.02 | test_hostha_simulator.py
   test_configure_ha_provider_invalid | Skipped | 0.03 | 
test_hostha_simulator.py
   test_deploy_vgpu_enabled_vm | Skipped | 0.03 | test_deploy_vgpu_enabled_vm.py
   test_3d_gpu_support | Skipped | 0.05 | test_deploy_vgpu_enabled_vm.py
   
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] blueorangutan commented on issue #2249: WIP CLOUDSTACK-10007 Isolation method claim

2017-09-21 Thread git
blueorangutan commented on issue #2249: WIP CLOUDSTACK-10007 Isolation method 
claim
URL: https://github.com/apache/cloudstack/pull/2249#issuecomment-331162997
 
 
   Packaging result: ?centos6 ?centos7 ?debian. JID-1109
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] blueorangutan commented on issue #2249: WIP CLOUDSTACK-10007 Isolation method claim

2017-09-21 Thread git
blueorangutan commented on issue #2249: WIP CLOUDSTACK-10007 Isolation method 
claim
URL: https://github.com/apache/cloudstack/pull/2249#issuecomment-331155700
 
 
   @DaanHoogland a Jenkins job has been kicked to build packages. I'll keep you 
posted as I make progress.
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] DaanHoogland commented on issue #2249: WIP CLOUDSTACK-10007 Isolation method claim

2017-09-21 Thread git
DaanHoogland commented on issue #2249: WIP CLOUDSTACK-10007 Isolation method 
claim
URL: https://github.com/apache/cloudstack/pull/2249#issuecomment-331155415
 
 
   @blueorangutan package
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] kiwiflyer commented on issue #1707: CLOUDSTACK-9397: Add Watchdog timer to KVM Instance

2017-09-21 Thread git
kiwiflyer commented on issue #1707: CLOUDSTACK-9397: Add Watchdog timer to KVM 
Instance
URL: https://github.com/apache/cloudstack/pull/1707#issuecomment-331141970
 
 
   According to Redhat, it's supported in RHEL6 U4, so it's quite possible it's 
supported in Centos 6.4 and beyond, but we haven't tested it. All of our 
testing was Centos 7.x.
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] wido commented on issue #1707: CLOUDSTACK-9397: Add Watchdog timer to KVM Instance

2017-09-21 Thread git
wido commented on issue #1707: CLOUDSTACK-9397: Add Watchdog timer to KVM 
Instance
URL: https://github.com/apache/cloudstack/pull/1707#issuecomment-331139737
 
 
   Is virtio-scsi supported in CentOS 6 by libvirt? I doubt it.
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] PranaliM commented on a change in pull request #2258: Cloudstack 10064: Secondary storage Usage for uploadedVolume is not collected

2017-09-21 Thread git
PranaliM commented on a change in pull request #2258: Cloudstack 10064: 
Secondary storage Usage for uploadedVolume is not collected
URL: https://github.com/apache/cloudstack/pull/2258#discussion_r140211884
 
 

 ##
 File path: test/integration/testpaths/Upload_Volume_test_case.py
 ##
 @@ -0,0 +1,223 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license agreements.  See the NOTICE file
+# distributed with this work for additional information
+# regarding copyright ownership.  The ASF licenses this file
+# to you under the Apache License, Version 2.0 (the
+# "License"); you may not use this file except in compliance
+# with the License.  You may obtain a copy of the License at
+#
+#   http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing,
+# software distributed under the License is distributed on an
+# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+# KIND, either express or implied.  See the License for the
+# specific language governing permissions and limitations
+# under the License.
+""" Test cases for checking that the secondary Storage usage is accounted. 
This is verified by making sure that storage_type=4 is added in the 
cloud_usage.usage_storage
+table for a volume in 'Uploaded' state.
+
+This test case:
+1.Creates an account and uploads a volume.
+2.After the volume is uploaded successfully, restarts the usage server.
+3.Connectst ot the datbase
+4.From the database verifies that an entry is added to 
cloud_usage.usage_storage table with storage_type=4 for the uploaded volume.
+5.Clean up resources.
+"""
+
+from marvin.cloudstackTestCase import *
+from marvin.cloudstackAPI import *
+from marvin.lib.utils import *
+from marvin.lib.base import *
+from marvin.lib.common import *
+from nose.plugins.attrib import attr
+from marvin.sshClient import SshClient
+from marvin.codes import (BACKED_UP, PASS, FAIL)
+import time
+
+
+def verify_vm(self, vmid, state):
 
 Review comment:
   @rhtyd: Please check. I have moved the test case to component from testpath
 
----
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] jayakarteek commented on issue #2104: CLOUDSTACK-9908 : Primary Storage allocated capacity goes very high after VM snapshot

2017-09-21 Thread git
jayakarteek commented on issue #2104: CLOUDSTACK-9908 : Primary Storage 
allocated capacity goes very high after VM snapshot
URL: https://github.com/apache/cloudstack/pull/2104#issuecomment-331092401
 
 
   tag:MergeReady
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] niteshsarda commented on issue #2139: CLOUDSTACK-9921: NPE when storage garbage collector is running.

2017-09-21 Thread git
niteshsarda commented on issue #2139: CLOUDSTACK-9921: NPE when storage garbage 
collector is running.
URL: https://github.com/apache/cloudstack/pull/2139#issuecomment-331066581
 
 
   LGTM for code.
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] vedulasantosh commented on a change in pull request #2240: CLOUDSTACK-10051 Mouse Scrolling is not working in instance VM console

2017-09-20 Thread git
vedulasantosh commented on a change in pull request #2240: CLOUDSTACK-10051 
Mouse Scrolling is not working in instance VM console
URL: https://github.com/apache/cloudstack/pull/2240#discussion_r140154390
 
 

 ##
 File path: systemvm/js/jquery.flot.navigate.js
 ##
 @@ -0,0 +1,336 @@
+/*
 
 Review comment:
   @rhtyd for all other jquery files there is no apache license because they 
are placed in exclude tag in pom.xml. while pushing this 
jquery.flot.navigate.js checks are failing at license. That is why i added 
apache license to this file.
   
   pom.xml
   `ui/lib/flot/jquery.flot.js
 ui/lib/flot/jquery.flot.navigate.js
 ui/lib/flot/jquery.flot.pie.js
 ui/lib/flot/jquery.flot.resize.js
 ui/lib/flot/jquery.flot.selection.js
 ui/lib/flot/jquery.flot.stack.js
 ui/lib/flot/jquery.flot.symbol.js
 ui/lib/flot/jquery.flot.threshold.js
 ui/lib/jquery-ui/css/jquery-ui.css
 ui/lib/jquery-ui/index.html
 ui/lib/jquery-ui/js/jquery-ui.js
 ui/lib/jquery.cookies.js
 ui/lib/jquery.easing.js
 ui/lib/jquery.js
 ui/lib/jquery.md5.js
 ui/lib/jquery.validate.js
 ui/lib/jquery.tokeninput.js
 ui/lib/qunit/qunit.css
 ui/lib/qunit/qunit.js
 ui/lib/reset.css
 ui/lib/require.js
 utils/testsmallfileinactive
 systemvm/conf/agent.properties
 systemvm/conf/environment.properties
 systemvm/js/jquery.js`
   
   systemvm/js/jquery.js
   
   `/*
* jQuery JavaScript Library v1.3.2
* http://jquery.com/
*
* Copyright (c) 2009 John Resig
* Dual licensed under the MIT and GPL licenses.
* http://docs.jquery.com/License
*
* Date: 2009-02-19 17:34:21 -0500 (Thu, 19 Feb 2009)
* Revision: 6246
*/
   (function(){var 
l=this,g,y=l.jQuery,p=l.$,o=l.jQuery=l.$=function(E,F){return new 
o.fn.init(E,F)},D=/^[^<]*(<(.|\s)+>)[^>]*$|^#([\w-]+)$/,f=/^.[^:#\[\.,]*$/;o.fn=o.prototype={init:function(E,H){E=E||document;if(E.nodeType)`
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] rhtyd commented on issue #2267: CLOUDSTACK-10077: allow to have different VPN customer gateway configs for same gateway IP

2017-09-20 Thread git
rhtyd commented on issue #2267: CLOUDSTACK-10077: allow to have different VPN 
customer gateway configs for same gateway IP
URL: https://github.com/apache/cloudstack/pull/2267#issuecomment-331055575
 
 
   Thanks @resmo, test LGTM, additional code review is requested after which 
this can be merged.
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] blueorangutan commented on issue #2269: CLOUDSTACK-10083: Allow ModifySshKeys to be forwarded to the agent in?

2017-09-20 Thread git
blueorangutan commented on issue #2269: CLOUDSTACK-10083: Allow ModifySshKeys 
to be forwarded to the agent in?
URL: https://github.com/apache/cloudstack/pull/2269#issuecomment-331054301
 
 
   @rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been 
kicked to run smoke tests
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] rhtyd commented on issue #2269: CLOUDSTACK-10083: Allow ModifySshKeys to be forwarded to the agent in?

2017-09-20 Thread git
rhtyd commented on issue #2269: CLOUDSTACK-10083: Allow ModifySshKeys to be 
forwarded to the agent in?
URL: https://github.com/apache/cloudstack/pull/2269#issuecomment-331054196
 
 
   @blueorangutan test
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] rhtyd closed pull request #876: CLOUDSTACK-8865: Adding SR doesn't create Storage_pool_host_ref entry?

2017-09-20 Thread git
rhtyd closed pull request #876: CLOUDSTACK-8865: Adding SR doesn't create 
Storage_pool_host_ref entry?
URL: https://github.com/apache/cloudstack/pull/876
 
 
   
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] rhtyd commented on issue #876: CLOUDSTACK-8865: Adding SR doesn't create Storage_pool_host_ref entry?

2017-09-20 Thread git
rhtyd commented on issue #876: CLOUDSTACK-8865: Adding SR doesn't create 
Storage_pool_host_ref entry?
URL: https://github.com/apache/cloudstack/pull/876#issuecomment-331053657
 
 
   Okay test LGTM, give this has enough code review/LGTM from others I'll merge 
this. Thanks.
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] SudharmaJain opened a new pull request #2271: CLOUDSTACK-10087 Template registration errors out when template URL i?

2017-09-20 Thread git
SudharmaJain opened a new pull request #2271: CLOUDSTACK-10087 Template 
registration errors out when template URL i?
URL: https://github.com/apache/cloudstack/pull/2271
 
 
   ?s https
   
   ca certificates were missing in realhostip.keystore.
   
   Before adding the fix while registering the template, it raises an exception 
"unable to find valid certification path to requested target".
   
   
![image](https://user-images.githubusercontent.com/12229259/30679204-da21685e-9e65-11e7-87c3-8515523b7b3d.png)
   
   After the fix register template works fine.
   
   
![image](https://user-images.githubusercontent.com/12229259/30679212-e5a53714-9e65-11e7-94c2-4147b30be0ba.png)
   
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] blueorangutan commented on issue #2269: CLOUDSTACK-10083: Allow ModifySshKeys to be forwarded to the agent in?

2017-09-20 Thread git
s_clusterzones | Skipped | 0.02 | 
test_hostha_simulator.py
   test_configure_ha_provider_valid | Skipped | 0.02 | test_hostha_simulator.py
   test_configure_ha_provider_invalid | Skipped | 0.02 | 
test_hostha_simulator.py
   test_deploy_vgpu_enabled_vm | Skipped | 0.03 | test_deploy_vgpu_enabled_vm.py
   test_3d_gpu_support | Skipped | 0.04 | test_deploy_vgpu_enabled_vm.py
   
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] blueorangutan commented on issue #876: CLOUDSTACK-8865: Adding SR doesn't create Storage_pool_host_ref entry?

2017-09-20 Thread git
blueorangutan commented on issue #876: CLOUDSTACK-8865: Adding SR doesn't 
create Storage_pool_host_ref entry?
URL: https://github.com/apache/cloudstack/pull/876#issuecomment-330977316
 
 
   Trillian test result (tid-1533)
   Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
   Total time taken: 33526 seconds
   Marvin logs: 
https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr876-t1533-kvm-centos7.zip
   Intermitten failure detected: /marvin/tests/smoke/test_privategw_acl.py
   Intermitten failure detected: /marvin/tests/smoke/test_ssvm.py
   Intermitten failure detected: /marvin/tests/smoke/test_vpc_vpn.py
   Test completed. 60 look OK, 2 have error(s)
   
   
   Test | Result | Time (s) | Test File
   --- | --- | --- | ---
   test_01_vpc_remote_access_vpn | `Failure` | 55.87 | test_vpc_vpn.py
   test_04_rvpc_privategw_static_routes | `Failure` | 375.77 | 
test_privategw_acl.py
   test_change_service_offering_for_vm_with_snapshots | Skipped | 0.00 | 
test_vm_snapshots.py
   test_09_copy_delete_template | Skipped | 0.01 | test_templates.py
   test_06_copy_template | Skipped | 0.00 | test_templates.py
   test_static_role_account_acls | Skipped | 0.03 | test_staticroles.py
   test_11_ss_nfs_version_on_ssvm | Skipped | 0.02 | test_ssvm.py
   test_01_scale_vm | Skipped | 0.00 | test_scale_vm.py
   test_01_primary_storage_iscsi | Skipped | 0.10 | test_primary_storage.py
   test_vm_nic_adapter_vmxnet3 | Skipped | 0.00 | test_nic_adapter_type.py
   test_nested_virtualization_vmware | Skipped | 0.00 | 
test_nested_virtualization.py
   test_06_copy_iso | Skipped | 0.00 | test_iso.py
   test_list_ha_for_host_valid | Skipped | 0.02 | test_hostha_simulator.py
   test_list_ha_for_host_invalid | Skipped | 0.03 | test_hostha_simulator.py
   test_list_ha_for_host | Skipped | 0.03 | test_hostha_simulator.py
   test_hostha_enable_feature_without_setting_provider | Skipped | 0.02 | 
test_hostha_simulator.py
   test_hostha_enable_feature_valid | Skipped | 0.02 | test_hostha_simulator.py
   test_hostha_disable_feature_valid | Skipped | 0.02 | test_hostha_simulator.py
   test_hostha_configure_invalid_provider | Skipped | 0.02 | 
test_hostha_simulator.py
   test_hostha_configure_default_driver | Skipped | 0.02 | 
test_hostha_simulator.py
   test_ha_verify_fsm_recovering | Skipped | 0.02 | test_hostha_simulator.py
   test_ha_verify_fsm_fenced | Skipped | 0.02 | test_hostha_simulator.py
   test_ha_verify_fsm_degraded | Skipped | 0.02 | test_hostha_simulator.py
   test_ha_verify_fsm_available | Skipped | 0.02 | test_hostha_simulator.py
   test_ha_multiple_mgmt_server_ownership | Skipped | 0.02 | 
test_hostha_simulator.py
   test_ha_list_providers | Skipped | 0.02 | test_hostha_simulator.py
   test_ha_enable_feature_invalid | Skipped | 0.02 | test_hostha_simulator.py
   test_ha_disable_feature_invalid | Skipped | 0.02 | test_hostha_simulator.py
   test_ha_configure_enabledisable_across_clusterzones | Skipped | 0.02 | 
test_hostha_simulator.py
   test_configure_ha_provider_valid | Skipped | 0.02 | test_hostha_simulator.py
   test_configure_ha_provider_invalid | Skipped | 0.02 | 
test_hostha_simulator.py
   test_deploy_vgpu_enabled_vm | Skipped | 0.03 | test_deploy_vgpu_enabled_vm.py
   test_3d_gpu_support | Skipped | 0.03 | test_deploy_vgpu_enabled_vm.py
   
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] niteshsarda opened a new pull request #2270: CLOUDSTACK-10085 : Upload volume from local fails when global config max.account.seconday.storage is set to -1

2017-09-20 Thread git
niteshsarda opened a new pull request #2270: CLOUDSTACK-10085 : Upload volume 
from local fails when global config max.account.seconday.storage is set to -1
URL: https://github.com/apache/cloudstack/pull/2270
 
 
   **ISSUE**
   
   Upload volume from local fails when global config 
max.account.seconday.storage is set to -1
   
   **STEPS TO REPRODUCE**
   
   1. Download any volume on your local machine.
   2. Change value of global configuration max.account.seconday.storage to -1.
   3. Restart Management server.
   4. Try to upload a volume from UI using "Upload from Local" option. Upload 
will fail.
   
   **FIX**
   
   Implementation of -1 option for unlimited secondary storage usage, which is 
absent in current code.
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] jayakarteek commented on a change in pull request #2058: CLOUDSTACK-9888: Duplicate usage while processing usage events

2017-09-20 Thread git
jayakarteek commented on a change in pull request #2058: CLOUDSTACK-9888: 
Duplicate usage while processing usage events 
URL: https://github.com/apache/cloudstack/pull/2058#discussion_r139943969
 
 

 ##
 File path: usage/src/com/cloud/usage/UsageManagerImpl.java
 ##
 @@ -1054,8 +1054,6 @@ private void createVMHelperEvent(UsageEventVO event) {
 
 if (EventTypes.EVENT_VM_START.equals(event.getType())) {
 // create a new usage_VM_instance row for this VM
-try {
 
 Review comment:
   Created duplicate record in cloud.usage_event table
   and tested the scenario with out the fix and posted the results in above 
comments 
   Also @niteshsarda  tested the fix and gave test LGTM.
   Are you looking for marvin test case
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] resmo commented on issue #2267: CLOUDSTACK-10077: allow to have different VPN customer gateway configs for same gateway IP

2017-09-20 Thread git
resmo commented on issue #2267: CLOUDSTACK-10077: allow to have different VPN 
customer gateway configs for same gateway IP
URL: https://github.com/apache/cloudstack/pull/2267#issuecomment-330830509
 
 
   travis build error looks unrelated to me. Trilian test failure, AFAICS, are 
also unrelated.
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] jayakarteek commented on a change in pull request #2058: CLOUDSTACK-9888: Duplicate usage while processing usage events

2017-09-20 Thread git
jayakarteek commented on a change in pull request #2058: CLOUDSTACK-9888: 
Duplicate usage while processing usage events 
URL: https://github.com/apache/cloudstack/pull/2058#discussion_r139943969
 
 

 ##
 File path: usage/src/com/cloud/usage/UsageManagerImpl.java
 ##
 @@ -1054,8 +1054,6 @@ private void createVMHelperEvent(UsageEventVO event) {
 
 if (EventTypes.EVENT_VM_START.equals(event.getType())) {
 // create a new usage_VM_instance row for this VM
-try {
 
 Review comment:
   I already tested the scenario with out the fix and posted the results in 
above comments 
   Also @niteshsarda  tested the fix and gave test LGTM.
   Are you looking for marvin test case
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] rhtyd commented on issue #876: CLOUDSTACK-8865: Adding SR doesn't create Storage_pool_host_ref entry?

2017-09-20 Thread git
rhtyd commented on issue #876: CLOUDSTACK-8865: Adding SR doesn't create 
Storage_pool_host_ref entry?
URL: https://github.com/apache/cloudstack/pull/876#issuecomment-330824520
 
 
   @blueorangutan test
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] rhtyd commented on issue #2269: CLOUDSTACK-10083: Allow ModifySshKeys to be forwarded to the agent in?

2017-09-20 Thread git
rhtyd commented on issue #2269: CLOUDSTACK-10083: Allow ModifySshKeys to be 
forwarded to the agent in?
URL: https://github.com/apache/cloudstack/pull/2269#issuecomment-330824426
 
 
   @blueorangutan test
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] blueorangutan commented on issue #2269: CLOUDSTACK-10083: Allow ModifySshKeys to be forwarded to the agent in?

2017-09-20 Thread git
blueorangutan commented on issue #2269: CLOUDSTACK-10083: Allow ModifySshKeys 
to be forwarded to the agent in?
URL: https://github.com/apache/cloudstack/pull/2269#issuecomment-330823347
 
 
   Packaging result: ?centos6 ?centos7 ?debian. JID-1108
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] blueorangutan commented on issue #876: CLOUDSTACK-8865: Adding SR doesn't create Storage_pool_host_ref entry?

2017-09-20 Thread git
blueorangutan commented on issue #876: CLOUDSTACK-8865: Adding SR doesn't 
create Storage_pool_host_ref entry?
URL: https://github.com/apache/cloudstack/pull/876#issuecomment-330822931
 
 
   Packaging result: ?centos6 ?centos7 ?debian. JID-1107
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] rhtyd commented on issue #2269: CLOUDSTACK-10083: Allow ModifySshKeys to be forwarded to the agent in?

2017-09-20 Thread git
rhtyd commented on issue #2269: CLOUDSTACK-10083: Allow ModifySshKeys to be 
forwarded to the agent in?
URL: https://github.com/apache/cloudstack/pull/2269#issuecomment-330817016
 
 
   LGTM
   @blueorangutan package
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] blueorangutan commented on issue #2269: CLOUDSTACK-10083: Allow ModifySshKeys to be forwarded to the agent in?

2017-09-20 Thread git
blueorangutan commented on issue #2269: CLOUDSTACK-10083: Allow ModifySshKeys 
to be forwarded to the agent in?
URL: https://github.com/apache/cloudstack/pull/2269#issuecomment-330817093
 
 
   @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted 
as I make progress.
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] rhtyd closed pull request #1252: CLOUDSTACK-9182: Some running VMs turned off on manual migration when auto migration failed while host preparing for maintenance.

2017-09-20 Thread git
rhtyd closed pull request #1252: CLOUDSTACK-9182: Some running VMs turned off 
on manual migration when auto migration failed while host preparing for 
maintenance.
URL: https://github.com/apache/cloudstack/pull/1252
 
 
   
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] rhtyd commented on issue #1252: CLOUDSTACK-9182: Some running VMs turned off on manual migration when auto migration failed while host preparing for maintenance.

2017-09-20 Thread git
rhtyd commented on issue #1252: CLOUDSTACK-9182: Some running VMs turned off on 
manual migration when auto migration failed while host preparing for 
maintenance.
URL: https://github.com/apache/cloudstack/pull/1252#issuecomment-330816775
 
 
   Tests LGTM, merging.
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] rhtyd commented on a change in pull request #2258: Cloudstack 10064: Secondary storage Usage for uploadedVolume is not collected

2017-09-20 Thread git
rhtyd commented on a change in pull request #2258: Cloudstack 10064: Secondary 
storage Usage for uploadedVolume is not collected
URL: https://github.com/apache/cloudstack/pull/2258#discussion_r139936130
 
 

 ##
 File path: test/integration/testpaths/Upload_Volume_test_case.py
 ##
 @@ -0,0 +1,223 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license agreements.  See the NOTICE file
+# distributed with this work for additional information
+# regarding copyright ownership.  The ASF licenses this file
+# to you under the Apache License, Version 2.0 (the
+# "License"); you may not use this file except in compliance
+# with the License.  You may obtain a copy of the License at
+#
+#   http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing,
+# software distributed under the License is distributed on an
+# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+# KIND, either express or implied.  See the License for the
+# specific language governing permissions and limitations
+# under the License.
+""" Test cases for checking that the secondary Storage usage is accounted. 
This is verified by making sure that storage_type=4 is added in the 
cloud_usage.usage_storage
+table for a volume in 'Uploaded' state.
+
+This test case:
+1.Creates an account and uploads a volume.
+2.After the volume is uploaded successfully, restarts the usage server.
+3.Connectst ot the datbase
+4.From the database verifies that an entry is added to 
cloud_usage.usage_storage table with storage_type=4 for the uploaded volume.
+5.Clean up resources.
+"""
+
+from marvin.cloudstackTestCase import *
+from marvin.cloudstackAPI import *
+from marvin.lib.utils import *
+from marvin.lib.base import *
+from marvin.lib.common import *
+from nose.plugins.attrib import attr
+from marvin.sshClient import SshClient
+from marvin.codes import (BACKED_UP, PASS, FAIL)
+import time
+
+
+def verify_vm(self, vmid, state):
 
 Review comment:
   Ping, test LGTM. If you can answer outstanding issues - we can merge this 
PR. Thanks.
 
----
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] rhtyd commented on issue #2246: CLOUDSTACK-10046 checksum validation for any java supported Digests-type

2017-09-20 Thread git
rhtyd commented on issue #2246: CLOUDSTACK-10046 checksum validation for any 
java supported Digests-type
URL: https://github.com/apache/cloudstack/pull/2246#issuecomment-330816426
 
 
   LGTM, @DaanHoogland @borisstoyanov advise if we're happy with the PR. I'll 
merge on your confirmation.
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] blueorangutan commented on issue #876: CLOUDSTACK-8865: Adding SR doesn't create Storage_pool_host_ref entry?

2017-09-20 Thread git
blueorangutan commented on issue #876: CLOUDSTACK-8865: Adding SR doesn't 
create Storage_pool_host_ref entry?
URL: https://github.com/apache/cloudstack/pull/876#issuecomment-330816255
 
 
   @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted 
as I make progress.
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] rhtyd commented on issue #876: CLOUDSTACK-8865: Adding SR doesn't create Storage_pool_host_ref entry?

2017-09-20 Thread git
rhtyd commented on issue #876: CLOUDSTACK-8865: Adding SR doesn't create 
Storage_pool_host_ref entry?
URL: https://github.com/apache/cloudstack/pull/876#issuecomment-330816085
 
 
   @SudharmaJain I can rekick the tests, will rebuild packages as well
   @blueorangutan package
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] rhtyd commented on issue #1707: CLOUDSTACK-9397: Add Watchdog timer to KVM Instance

2017-09-20 Thread git
rhtyd commented on issue #1707: CLOUDSTACK-9397: Add Watchdog timer to KVM 
Instance
URL: https://github.com/apache/cloudstack/pull/1707#issuecomment-330815965
 
 
   @borisstoyanov can you check failures related to libvirt tests on 
kvm/centos6, however those failures are not seen in kvm/centos7? /cc @wido 
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] rhtyd commented on issue #2267: CLOUDSTACK-10077: allow to have different VPN customer gateway configs for same gateway IP

2017-09-20 Thread git
rhtyd commented on issue #2267: CLOUDSTACK-10077: allow to have different VPN 
customer gateway configs for same gateway IP
URL: https://github.com/apache/cloudstack/pull/2267#issuecomment-330815830
 
 
   Additional code review requested and please also see Travis failures.
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] rhtyd commented on a change in pull request #2058: CLOUDSTACK-9888: Duplicate usage while processing usage events

2017-09-20 Thread git
rhtyd commented on a change in pull request #2058: CLOUDSTACK-9888: Duplicate 
usage while processing usage events 
URL: https://github.com/apache/cloudstack/pull/2058#discussion_r139935400
 
 

 ##
 File path: usage/src/com/cloud/usage/UsageManagerImpl.java
 ##
 @@ -1054,8 +1054,6 @@ private void createVMHelperEvent(UsageEventVO event) {
 
 if (EventTypes.EVENT_VM_START.equals(event.getType())) {
 // create a new usage_VM_instance row for this VM
-try {
 
 Review comment:
   @jayakarteek please add a test case
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] jayakarteek commented on a change in pull request #2058: CLOUDSTACK-9888: Duplicate usage while processing usage events

2017-09-20 Thread git
jayakarteek commented on a change in pull request #2058: CLOUDSTACK-9888: 
Duplicate usage while processing usage events 
URL: https://github.com/apache/cloudstack/pull/2058#discussion_r139921404
 
 

 ##
 File path: usage/src/com/cloud/usage/UsageManagerImpl.java
 ##
 @@ -1054,8 +1054,6 @@ private void createVMHelperEvent(UsageEventVO event) {
 
 if (EventTypes.EVENT_VM_START.equals(event.getType())) {
 // create a new usage_VM_instance row for this VM
-try {
 
 Review comment:
   marvin test case is not there.
   Manually tested the scenario with and without the fix for duplicate entries.
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] marcaurele opened a new pull request #2269: CLOUDSTACK-10083: Allow ModifySshKeys to be forwarded to the agent in?

2017-09-20 Thread git
marcaurele opened a new pull request #2269: CLOUDSTACK-10083: Allow 
ModifySshKeys to be forwarded to the agent in?
URL: https://github.com/apache/cloudstack/pull/2269
 
 
   ? maintenance mode
   
   Signed-off-by: Marc-Aur?le Brothier <m...@brothier.org>
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] blueorangutan commented on issue #2267: CLOUDSTACK-10077: allow to have different VPN customer gateway configs for same gateway IP

2017-09-19 Thread git
blueorangutan commented on issue #2267: CLOUDSTACK-10077: allow to have 
different VPN customer gateway configs for same gateway IP
URL: https://github.com/apache/cloudstack/pull/2267#issuecomment-330748617
 
 
   Trillian test result (tid-1531)
   Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
   Total time taken: 39761 seconds
   Marvin logs: 
https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr2267-t1531-kvm-centos7.zip
   Intermitten failure detected: /marvin/tests/smoke/test_privategw_acl.py
   Intermitten failure detected: /marvin/tests/smoke/test_vpc_redundant.py
   Intermitten failure detected: /marvin/tests/smoke/test_vpc_vpn.py
   Test completed. 59 look OK, 3 have error(s)
   
   
   Test | Result | Time (s) | Test File
   --- | --- | --- | ---
   test_01_vpc_remote_access_vpn | `Failure` | 65.98 | test_vpc_vpn.py
   test_03_create_redundant_VPC_1tier_2VMs_2IPs_2PF_ACL_reboot_routers | 
`Failure` | 364.69 | test_vpc_redundant.py
   test_04_rvpc_privategw_static_routes | `Failure` | 421.48 | 
test_privategw_acl.py
   test_change_service_offering_for_vm_with_snapshots | Skipped | 0.00 | 
test_vm_snapshots.py
   test_09_copy_delete_template | Skipped | 0.02 | test_templates.py
   test_06_copy_template | Skipped | 0.00 | test_templates.py
   test_static_role_account_acls | Skipped | 0.02 | test_staticroles.py
   test_11_ss_nfs_version_on_ssvm | Skipped | 0.02 | test_ssvm.py
   test_01_scale_vm | Skipped | 0.00 | test_scale_vm.py
   test_01_primary_storage_iscsi | Skipped | 0.05 | test_primary_storage.py
   test_vm_nic_adapter_vmxnet3 | Skipped | 0.00 | test_nic_adapter_type.py
   test_nested_virtualization_vmware | Skipped | 0.00 | 
test_nested_virtualization.py
   test_06_copy_iso | Skipped | 0.00 | test_iso.py
   test_list_ha_for_host_valid | Skipped | 0.03 | test_hostha_simulator.py
   test_list_ha_for_host_invalid | Skipped | 0.03 | test_hostha_simulator.py
   test_list_ha_for_host | Skipped | 0.03 | test_hostha_simulator.py
   test_hostha_enable_feature_without_setting_provider | Skipped | 0.02 | 
test_hostha_simulator.py
   test_hostha_enable_feature_valid | Skipped | 0.02 | test_hostha_simulator.py
   test_hostha_disable_feature_valid | Skipped | 0.03 | test_hostha_simulator.py
   test_hostha_configure_invalid_provider | Skipped | 0.03 | 
test_hostha_simulator.py
   test_hostha_configure_default_driver | Skipped | 0.03 | 
test_hostha_simulator.py
   test_ha_verify_fsm_recovering | Skipped | 0.03 | test_hostha_simulator.py
   test_ha_verify_fsm_fenced | Skipped | 0.03 | test_hostha_simulator.py
   test_ha_verify_fsm_degraded | Skipped | 0.03 | test_hostha_simulator.py
   test_ha_verify_fsm_available | Skipped | 0.03 | test_hostha_simulator.py
   test_ha_multiple_mgmt_server_ownership | Skipped | 0.02 | 
test_hostha_simulator.py
   test_ha_list_providers | Skipped | 0.03 | test_hostha_simulator.py
   test_ha_enable_feature_invalid | Skipped | 0.04 | test_hostha_simulator.py
   test_ha_disable_feature_invalid | Skipped | 0.03 | test_hostha_simulator.py
   test_ha_configure_enabledisable_across_clusterzones | Skipped | 0.03 | 
test_hostha_simulator.py
   test_configure_ha_provider_valid | Skipped | 0.03 | test_hostha_simulator.py
   test_configure_ha_provider_invalid | Skipped | 0.03 | 
test_hostha_simulator.py
   test_deploy_vgpu_enabled_vm | Skipped | 0.04 | test_deploy_vgpu_enabled_vm.py
   test_3d_gpu_support | Skipped | 0.06 | test_deploy_vgpu_enabled_vm.py
   
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] blueorangutan commented on issue #2263: CLOUDSTACK-10070: Fixing some component tests and adding them in travis

2017-09-19 Thread git
blueorangutan commented on issue #2263: CLOUDSTACK-10070: Fixing some component 
tests and adding them in travis
URL: https://github.com/apache/cloudstack/pull/2263#issuecomment-330747639
 
 
   Trillian test result (tid-1532)
   Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
   Total time taken: 36910 seconds
   Marvin logs: 
https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr2263-t1532-kvm-centos7.zip
   Intermitten failure detected: /marvin/tests/smoke/test_privategw_acl.py
   Intermitten failure detected: /marvin/tests/smoke/test_vpc_vpn.py
   Test completed. 60 look OK, 2 have error(s)
   
   
   Test | Result | Time (s) | Test File
   --- | --- | --- | ---
   test_01_vpc_remote_access_vpn | `Failure` | 55.68 | test_vpc_vpn.py
   test_04_rvpc_privategw_static_routes | `Failure` | 354.29 | 
test_privategw_acl.py
   test_change_service_offering_for_vm_with_snapshots | Skipped | 0.00 | 
test_vm_snapshots.py
   test_09_copy_delete_template | Skipped | 0.01 | test_templates.py
   test_06_copy_template | Skipped | 0.00 | test_templates.py
   test_static_role_account_acls | Skipped | 0.02 | test_staticroles.py
   test_11_ss_nfs_version_on_ssvm | Skipped | 0.02 | test_ssvm.py
   test_01_scale_vm | Skipped | 0.00 | test_scale_vm.py
   test_01_primary_storage_iscsi | Skipped | 0.03 | test_primary_storage.py
   test_vm_nic_adapter_vmxnet3 | Skipped | 0.00 | test_nic_adapter_type.py
   test_nested_virtualization_vmware | Skipped | 0.00 | 
test_nested_virtualization.py
   test_06_copy_iso | Skipped | 0.00 | test_iso.py
   test_list_ha_for_host_valid | Skipped | 0.02 | test_hostha_simulator.py
   test_list_ha_for_host_invalid | Skipped | 0.02 | test_hostha_simulator.py
   test_list_ha_for_host | Skipped | 0.02 | test_hostha_simulator.py
   test_hostha_enable_feature_without_setting_provider | Skipped | 0.02 | 
test_hostha_simulator.py
   test_hostha_enable_feature_valid | Skipped | 0.02 | test_hostha_simulator.py
   test_hostha_disable_feature_valid | Skipped | 0.02 | test_hostha_simulator.py
   test_hostha_configure_invalid_provider | Skipped | 0.02 | 
test_hostha_simulator.py
   test_hostha_configure_default_driver | Skipped | 0.02 | 
test_hostha_simulator.py
   test_ha_verify_fsm_recovering | Skipped | 0.02 | test_hostha_simulator.py
   test_ha_verify_fsm_fenced | Skipped | 0.02 | test_hostha_simulator.py
   test_ha_verify_fsm_degraded | Skipped | 0.02 | test_hostha_simulator.py
   test_ha_verify_fsm_available | Skipped | 0.02 | test_hostha_simulator.py
   test_ha_multiple_mgmt_server_ownership | Skipped | 0.03 | 
test_hostha_simulator.py
   test_ha_list_providers | Skipped | 0.02 | test_hostha_simulator.py
   test_ha_enable_feature_invalid | Skipped | 0.03 | test_hostha_simulator.py
   test_ha_disable_feature_invalid | Skipped | 0.02 | test_hostha_simulator.py
   test_ha_configure_enabledisable_across_clusterzones | Skipped | 0.02 | 
test_hostha_simulator.py
   test_configure_ha_provider_valid | Skipped | 0.02 | test_hostha_simulator.py
   test_configure_ha_provider_invalid | Skipped | 0.02 | 
test_hostha_simulator.py
   test_deploy_vgpu_enabled_vm | Skipped | 0.02 | test_deploy_vgpu_enabled_vm.py
   test_3d_gpu_support | Skipped | 0.03 | test_deploy_vgpu_enabled_vm.py
   
   
   Additional tests completed. 1 look ok, 0 have error(s)
   
   
   Test | Result | Time (s) | Test File
   --- | --- | --- | ---
   
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] blueorangutan commented on issue #1707: CLOUDSTACK-9397: Add Watchdog timer to KVM Instance

2017-09-19 Thread git
blueorangutan commented on issue #1707: CLOUDSTACK-9397: Add Watchdog timer to 
KVM Instance
URL: https://github.com/apache/cloudstack/pull/1707#issuecomment-330711812
 
 
   Trillian test result (tid-1529)
   Environment: kvm-centos6 (x2), Advanced Networking with Mgmt server 7
   Total time taken: 49877 seconds
   Marvin logs: 
https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr1707-t1529-kvm-centos6.zip
   Intermitten failure detected: 
/marvin/tests/smoke/test_deploy_virtio_scsi_vm.py
   Intermitten failure detected: /marvin/tests/smoke/test_internal_lb.py
   Intermitten failure detected: /marvin/tests/smoke/test_iso.py
   Intermitten failure detected: /marvin/tests/smoke/test_list_ids_parameter.py
   Intermitten failure detected: /marvin/tests/smoke/test_privategw_acl.py
   Intermitten failure detected: /marvin/tests/smoke/test_router_dnsservice.py
   Intermitten failure detected: /marvin/tests/smoke/test_routers_network_ops.py
   Intermitten failure detected: /marvin/tests/smoke/test_ssvm.py
   Intermitten failure detected: /marvin/tests/smoke/test_vpc_vpn.py
   Test completed. 55 look OK, 7 have error(s)
   
   
   Test | Result | Time (s) | Test File
   --- | --- | --- | ---
   test_01_vpc_remote_access_vpn | `Failure` | 66.00 | test_vpc_vpn.py
   test_04_rvpc_privategw_static_routes | `Failure` | 451.12 | 
test_privategw_acl.py
   test_01_create_iso | `Failure` | 1513.55 | test_iso.py
   test_05_change_vm_ostype_restart | `Failure` | 21.35 | 
test_deploy_virtio_scsi_vm.py
   test_03_verify_libvirt_attach_disk | `Failure` | 11.04 | 
test_deploy_virtio_scsi_vm.py
   test_02_verify_libvirt_after_restart | `Failure` | 141.87 | 
test_deploy_virtio_scsi_vm.py
   test_01_verify_libvirt | `Failure` | 0.77 | test_deploy_virtio_scsi_vm.py
   ContextSuite context=TestRedundantIsolateNetworks>:setup | `Error` | 1815.07 
| test_routers_network_ops.py
   ContextSuite context=TestListIdsParams>:setup | `Error` | 0.00 | 
test_list_ids_parameter.py
   ContextSuite context=TestISO>:setup | `Error` | 3032.19 | test_iso.py
   ContextSuite context=TestInternalLb>:setup | `Error` | 0.00 | 
test_internal_lb.py
   test_change_service_offering_for_vm_with_snapshots | Skipped | 0.00 | 
test_vm_snapshots.py
   test_09_copy_delete_template | Skipped | 0.03 | test_templates.py
   test_06_copy_template | Skipped | 0.00 | test_templates.py
   test_static_role_account_acls | Skipped | 0.02 | test_staticroles.py
   test_11_ss_nfs_version_on_ssvm | Skipped | 0.02 | test_ssvm.py
   test_02_list_snapshots_with_removed_data_store | Skipped | 0.00 | 
test_snapshots.py
   test_01_snapshot_root_disk | Skipped | 0.00 | test_snapshots.py
   test_01_scale_vm | Skipped | 0.00 | test_scale_vm.py
   test_01_primary_storage_iscsi | Skipped | 0.03 | test_primary_storage.py
   test_vm_nic_adapter_vmxnet3 | Skipped | 0.00 | test_nic_adapter_type.py
   test_nested_virtualization_vmware | Skipped | 0.00 | 
test_nested_virtualization.py
   test_list_ha_for_host_valid | Skipped | 0.03 | test_hostha_simulator.py
   test_list_ha_for_host_invalid | Skipped | 0.03 | test_hostha_simulator.py
   test_list_ha_for_host | Skipped | 0.03 | test_hostha_simulator.py
   test_hostha_enable_feature_without_setting_provider | Skipped | 0.03 | 
test_hostha_simulator.py
   test_hostha_enable_feature_valid | Skipped | 0.03 | test_hostha_simulator.py
   test_hostha_disable_feature_valid | Skipped | 0.04 | test_hostha_simulator.py
   test_hostha_configure_invalid_provider | Skipped | 0.03 | 
test_hostha_simulator.py
   test_hostha_configure_default_driver | Skipped | 0.04 | 
test_hostha_simulator.py
   test_ha_verify_fsm_recovering | Skipped | 0.03 | test_hostha_simulator.py
   test_ha_verify_fsm_fenced | Skipped | 0.02 | test_hostha_simulator.py
   test_ha_verify_fsm_degraded | Skipped | 0.03 | test_hostha_simulator.py
   test_ha_verify_fsm_available | Skipped | 0.03 | test_hostha_simulator.py
   test_ha_multiple_mgmt_server_ownership | Skipped | 0.02 | 
test_hostha_simulator.py
   test_ha_list_providers | Skipped | 0.04 | test_hostha_simulator.py
   test_ha_enable_feature_invalid | Skipped | 0.06 | test_hostha_simulator.py
   test_ha_disable_feature_invalid | Skipped | 0.05 | test_hostha_simulator.py
   test_ha_configure_enabledisable_across_clusterzones | Skipped | 0.06 | 
test_hostha_simulator.py
   test_configure_ha_provider_valid | Skipped | 0.02 | test_hostha_simulator.py
   test_configure_ha_provider_invalid | Skipped | 0.04 | 
test_hostha_simulator.py
   test_deploy_vgpu_enabled_vm | Skipped | 0.05 | test_deploy_vgpu_enabled_vm.py
   test_3d_gpu_support | Skipped | 0.04 | test_deploy_vgpu_enabled_vm.py
   
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With reg

[GitHub] blueorangutan commented on issue #2258: Cloudstack 10064: Secondary storage Usage for uploadedVolume is not collected

2017-09-19 Thread git
blueorangutan commented on issue #2258: Cloudstack 10064: Secondary storage 
Usage for uploadedVolume is not collected
URL: https://github.com/apache/cloudstack/pull/2258#issuecomment-330667959
 
 
   Trillian test result (tid-1530)
   Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
   Total time taken: 33737 seconds
   Marvin logs: 
https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr2258-t1530-kvm-centos7.zip
   Intermitten failure detected: /marvin/tests/smoke/test_hostha_kvm.py
   Intermitten failure detected: /marvin/tests/smoke/test_privategw_acl.py
   Intermitten failure detected: /marvin/tests/smoke/test_vpc_vpn.py
   Test completed. 60 look OK, 2 have error(s)
   
   
   Test | Result | Time (s) | Test File
   --- | --- | --- | ---
   test_01_vpc_remote_access_vpn | `Failure` | 60.72 | test_vpc_vpn.py
   test_04_rvpc_privategw_static_routes | `Failure` | 384.62 | 
test_privategw_acl.py
   test_change_service_offering_for_vm_with_snapshots | Skipped | 0.00 | 
test_vm_snapshots.py
   test_09_copy_delete_template | Skipped | 0.01 | test_templates.py
   test_06_copy_template | Skipped | 0.00 | test_templates.py
   test_static_role_account_acls | Skipped | 0.01 | test_staticroles.py
   test_11_ss_nfs_version_on_ssvm | Skipped | 0.02 | test_ssvm.py
   test_01_scale_vm | Skipped | 0.00 | test_scale_vm.py
   test_01_primary_storage_iscsi | Skipped | 0.03 | test_primary_storage.py
   test_vm_nic_adapter_vmxnet3 | Skipped | 0.00 | test_nic_adapter_type.py
   test_nested_virtualization_vmware | Skipped | 0.00 | 
test_nested_virtualization.py
   test_06_copy_iso | Skipped | 0.00 | test_iso.py
   test_list_ha_for_host_valid | Skipped | 0.02 | test_hostha_simulator.py
   test_list_ha_for_host_invalid | Skipped | 0.02 | test_hostha_simulator.py
   test_list_ha_for_host | Skipped | 0.02 | test_hostha_simulator.py
   test_hostha_enable_feature_without_setting_provider | Skipped | 0.02 | 
test_hostha_simulator.py
   test_hostha_enable_feature_valid | Skipped | 0.02 | test_hostha_simulator.py
   test_hostha_disable_feature_valid | Skipped | 0.02 | test_hostha_simulator.py
   test_hostha_configure_invalid_provider | Skipped | 0.02 | 
test_hostha_simulator.py
   test_hostha_configure_default_driver | Skipped | 0.05 | 
test_hostha_simulator.py
   test_ha_verify_fsm_recovering | Skipped | 0.07 | test_hostha_simulator.py
   test_ha_verify_fsm_fenced | Skipped | 0.07 | test_hostha_simulator.py
   test_ha_verify_fsm_degraded | Skipped | 0.07 | test_hostha_simulator.py
   test_ha_verify_fsm_available | Skipped | 0.07 | test_hostha_simulator.py
   test_ha_multiple_mgmt_server_ownership | Skipped | 0.04 | 
test_hostha_simulator.py
   test_ha_list_providers | Skipped | 0.07 | test_hostha_simulator.py
   test_ha_enable_feature_invalid | Skipped | 0.07 | test_hostha_simulator.py
   test_ha_disable_feature_invalid | Skipped | 0.03 | test_hostha_simulator.py
   test_ha_configure_enabledisable_across_clusterzones | Skipped | 0.02 | 
test_hostha_simulator.py
   test_configure_ha_provider_valid | Skipped | 0.02 | test_hostha_simulator.py
   test_configure_ha_provider_invalid | Skipped | 0.02 | 
test_hostha_simulator.py
   test_hostha_kvm_host_degraded | Skipped | 30.98 | test_hostha_kvm.py
   test_deploy_vgpu_enabled_vm | Skipped | 0.04 | test_deploy_vgpu_enabled_vm.py
   test_3d_gpu_support | Skipped | 0.04 | test_deploy_vgpu_enabled_vm.py
   
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] SudharmaJain commented on issue #876: CLOUDSTACK-8865: Adding SR doesn't create Storage_pool_host_ref entry?

2017-09-19 Thread git
SudharmaJain commented on issue #876: CLOUDSTACK-8865: Adding SR doesn't create 
Storage_pool_host_ref entry?
URL: https://github.com/apache/cloudstack/pull/876#issuecomment-330660489
 
 
   @rhtyd All the tests failed here, except the skipped ones. Even the 
listSystemVms command returned response None. Looks like this was a setup 
issue. 
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] kiwiflyer commented on issue #2083: Iptables speedup

2017-09-19 Thread git
kiwiflyer commented on issue #2083: Iptables speedup
URL: https://github.com/apache/cloudstack/pull/2083#issuecomment-330659335
 
 
   @DaanHoogland What can we do to help with this one? It looks really worth 
while. 
   
   @dmabry 
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] blueorangutan commented on issue #1252: CLOUDSTACK-9182: Some running VMs turned off on manual migration when auto migration failed while host preparing for maintenance.

2017-09-19 Thread git
blueorangutan commented on issue #1252: CLOUDSTACK-9182: Some running VMs 
turned off on manual migration when auto migration failed while host preparing 
for maintenance.
URL: https://github.com/apache/cloudstack/pull/1252#issuecomment-330638199
 
 
   Trillian test result (tid-1527)
   Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
   Total time taken: 39218 seconds
   Marvin logs: 
https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr1252-t1527-kvm-centos7.zip
   Intermitten failure detected: 
/marvin/tests/smoke/test_deploy_virtio_scsi_vm.py
   Intermitten failure detected: /marvin/tests/smoke/test_internal_lb.py
   Intermitten failure detected: /marvin/tests/smoke/test_privategw_acl.py
   Intermitten failure detected: /marvin/tests/smoke/test_vpc_vpn.py
   Test completed. 58 look OK, 4 have error(s)
   
   
   Test | Result | Time (s) | Test File
   --- | --- | --- | ---
   test_01_vpc_remote_access_vpn | `Failure` | 65.98 | test_vpc_vpn.py
   test_04_rvpc_privategw_static_routes | `Failure` | 381.03 | 
test_privategw_acl.py
   ContextSuite context=TestInternalLb>:setup | `Error` | 0.00 | 
test_internal_lb.py
   ContextSuite context=TestDeployVirtioSCSIVM>:setup | `Error` | 0.00 | 
test_deploy_virtio_scsi_vm.py
   test_change_service_offering_for_vm_with_snapshots | Skipped | 0.00 | 
test_vm_snapshots.py
   test_09_copy_delete_template | Skipped | 0.01 | test_templates.py
   test_06_copy_template | Skipped | 0.00 | test_templates.py
   test_static_role_account_acls | Skipped | 0.02 | test_staticroles.py
   test_11_ss_nfs_version_on_ssvm | Skipped | 0.02 | test_ssvm.py
   test_01_scale_vm | Skipped | 0.00 | test_scale_vm.py
   test_01_primary_storage_iscsi | Skipped | 0.03 | test_primary_storage.py
   test_vm_nic_adapter_vmxnet3 | Skipped | 0.00 | test_nic_adapter_type.py
   test_nested_virtualization_vmware | Skipped | 0.00 | 
test_nested_virtualization.py
   test_06_copy_iso | Skipped | 0.00 | test_iso.py
   test_list_ha_for_host_valid | Skipped | 0.02 | test_hostha_simulator.py
   test_list_ha_for_host_invalid | Skipped | 0.02 | test_hostha_simulator.py
   test_list_ha_for_host | Skipped | 0.02 | test_hostha_simulator.py
   test_hostha_enable_feature_without_setting_provider | Skipped | 0.02 | 
test_hostha_simulator.py
   test_hostha_enable_feature_valid | Skipped | 0.02 | test_hostha_simulator.py
   test_hostha_disable_feature_valid | Skipped | 0.02 | test_hostha_simulator.py
   test_hostha_configure_invalid_provider | Skipped | 0.02 | 
test_hostha_simulator.py
   test_hostha_configure_default_driver | Skipped | 0.02 | 
test_hostha_simulator.py
   test_ha_verify_fsm_recovering | Skipped | 0.02 | test_hostha_simulator.py
   test_ha_verify_fsm_fenced | Skipped | 0.03 | test_hostha_simulator.py
   test_ha_verify_fsm_degraded | Skipped | 0.02 | test_hostha_simulator.py
   test_ha_verify_fsm_available | Skipped | 0.02 | test_hostha_simulator.py
   test_ha_multiple_mgmt_server_ownership | Skipped | 0.02 | 
test_hostha_simulator.py
   test_ha_list_providers | Skipped | 0.02 | test_hostha_simulator.py
   test_ha_enable_feature_invalid | Skipped | 0.02 | test_hostha_simulator.py
   test_ha_disable_feature_invalid | Skipped | 0.02 | test_hostha_simulator.py
   test_ha_configure_enabledisable_across_clusterzones | Skipped | 0.02 | 
test_hostha_simulator.py
   test_configure_ha_provider_valid | Skipped | 0.03 | test_hostha_simulator.py
   test_configure_ha_provider_invalid | Skipped | 0.02 | 
test_hostha_simulator.py
   test_deploy_vgpu_enabled_vm | Skipped | 0.03 | test_deploy_vgpu_enabled_vm.py
   test_3d_gpu_support | Skipped | 0.03 | test_deploy_vgpu_enabled_vm.py
   
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] blueorangutan commented on issue #2246: CLOUDSTACK-10046 checksum validation for any java supported Digests-type

2017-09-19 Thread git
blueorangutan commented on issue #2246: CLOUDSTACK-10046 checksum validation 
for any java supported Digests-type
URL: https://github.com/apache/cloudstack/pull/2246#issuecomment-330626931
 
 
   Trillian test result (tid-1528)
   Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
   Total time taken: 36490 seconds
   Marvin logs: 
https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr2246-t1528-kvm-centos7.zip
   Intermitten failure detected: /marvin/tests/smoke/test_privategw_acl.py
   Intermitten failure detected: /marvin/tests/smoke/test_vpc_redundant.py
   Intermitten failure detected: /marvin/tests/smoke/test_vpc_vpn.py
   Test completed. 60 look OK, 2 have error(s)
   
   
   Test | Result | Time (s) | Test File
   --- | --- | --- | ---
   test_01_vpc_remote_access_vpn | `Failure` | 55.92 | test_vpc_vpn.py
   test_04_rvpc_privategw_static_routes | `Failure` | 300.78 | 
test_privategw_acl.py
   test_change_service_offering_for_vm_with_snapshots | Skipped | 0.00 | 
test_vm_snapshots.py
   test_09_copy_delete_template | Skipped | 0.02 | test_templates.py
   test_06_copy_template | Skipped | 0.00 | test_templates.py
   test_static_role_account_acls | Skipped | 0.03 | test_staticroles.py
   test_11_ss_nfs_version_on_ssvm | Skipped | 0.03 | test_ssvm.py
   test_01_scale_vm | Skipped | 0.00 | test_scale_vm.py
   test_01_primary_storage_iscsi | Skipped | 0.04 | test_primary_storage.py
   test_vm_nic_adapter_vmxnet3 | Skipped | 0.00 | test_nic_adapter_type.py
   test_nested_virtualization_vmware | Skipped | 0.00 | 
test_nested_virtualization.py
   test_06_copy_iso | Skipped | 0.00 | test_iso.py
   test_list_ha_for_host_valid | Skipped | 0.02 | test_hostha_simulator.py
   test_list_ha_for_host_invalid | Skipped | 0.02 | test_hostha_simulator.py
   test_list_ha_for_host | Skipped | 0.02 | test_hostha_simulator.py
   test_hostha_enable_feature_without_setting_provider | Skipped | 0.02 | 
test_hostha_simulator.py
   test_hostha_enable_feature_valid | Skipped | 0.03 | test_hostha_simulator.py
   test_hostha_disable_feature_valid | Skipped | 0.03 | test_hostha_simulator.py
   test_hostha_configure_invalid_provider | Skipped | 0.02 | 
test_hostha_simulator.py
   test_hostha_configure_default_driver | Skipped | 0.02 | 
test_hostha_simulator.py
   test_ha_verify_fsm_recovering | Skipped | 0.02 | test_hostha_simulator.py
   test_ha_verify_fsm_fenced | Skipped | 0.03 | test_hostha_simulator.py
   test_ha_verify_fsm_degraded | Skipped | 0.02 | test_hostha_simulator.py
   test_ha_verify_fsm_available | Skipped | 0.02 | test_hostha_simulator.py
   test_ha_multiple_mgmt_server_ownership | Skipped | 0.02 | 
test_hostha_simulator.py
   test_ha_list_providers | Skipped | 0.03 | test_hostha_simulator.py
   test_ha_enable_feature_invalid | Skipped | 0.04 | test_hostha_simulator.py
   test_ha_disable_feature_invalid | Skipped | 0.03 | test_hostha_simulator.py
   test_ha_configure_enabledisable_across_clusterzones | Skipped | 0.02 | 
test_hostha_simulator.py
   test_configure_ha_provider_valid | Skipped | 0.02 | test_hostha_simulator.py
   test_configure_ha_provider_invalid | Skipped | 0.03 | 
test_hostha_simulator.py
   test_deploy_vgpu_enabled_vm | Skipped | 0.04 | test_deploy_vgpu_enabled_vm.py
   test_3d_gpu_support | Skipped | 0.04 | test_deploy_vgpu_enabled_vm.py
   
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] GabrielBrascher commented on issue #2185: CLOUDSTACK-9976: Redirect saml2 failed login message to a configurable URL

2017-09-19 Thread git
GabrielBrascher commented on issue #2185: CLOUDSTACK-9976: Redirect saml2 
failed login message to a configurable URL
URL: https://github.com/apache/cloudstack/pull/2185#issuecomment-330608678
 
 
   Sorry for the delay @rhtyd. I will work on that when possible.
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] sgoeminn opened a new pull request #2268: CLOUDSTACK-10081

2017-09-19 Thread git
sgoeminn opened a new pull request #2268: CLOUDSTACK-10081
URL: https://github.com/apache/cloudstack/pull/2268
 
 
   CloudUtils getDevInfo function will now return "bridge" instead of "dev" 
when the name of a ovs bridge is passed.
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] blueorangutan commented on issue #2263: CLOUDSTACK-10070: Fixing some component tests and adding them in travis

2017-09-19 Thread git
blueorangutan commented on issue #2263: CLOUDSTACK-10070: Fixing some component 
tests and adding them in travis
URL: https://github.com/apache/cloudstack/pull/2263#issuecomment-330525186
 
 
   @borisstoyanov a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has 
been kicked to run smoke tests
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] borisstoyanov commented on issue #2263: CLOUDSTACK-10070: Fixing some component tests and adding them in travis

2017-09-19 Thread git
borisstoyanov commented on issue #2263: CLOUDSTACK-10070: Fixing some component 
tests and adding them in travis
URL: https://github.com/apache/cloudstack/pull/2263#issuecomment-330524943
 
 
   @blueorangutan test centos7 kvm-centos7 
component/test_affinity_groups_projects.py, 
component/test_cpu_domain_limits.py, component/test_organization_states.py, 
component/test_project_resources.py, component/test_vpc.py
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] blueorangutan commented on issue #2263: CLOUDSTACK-10070: Fixing some component tests and adding them in travis

2017-09-19 Thread git
blueorangutan commented on issue #2263: CLOUDSTACK-10070: Fixing some component 
tests and adding them in travis
URL: https://github.com/apache/cloudstack/pull/2263#issuecomment-330523983
 
 
   @borisstoyanov I understand these words: "help", "hello", "thanks", 
"package", "test"
   Test command usage: test [mgmt os] [hypervisor] [additional tests]
   Mgmt OS options: ['centos6', 'centos7', 'ubuntu']
   Hypervisor options: ['kvm-centos6', 'kvm-centos7', 'kvm-ubuntu', 
'xenserver-65sp1', 'xenserver-62sp1', 'vmware-60u2', 'vmware-55u3', 
'vmware-51u1', 'vmware-50u1']
   Additional tests: list of comma separated tests with paths relative to the 
`test/integration` directory, for example: component/test_acl_listvm.py, 
component/test_volumes.py
   Note: when additional tests are passed, you need to specify mgmt server os 
and hypervisor or use the `matrix` command.
   
   Blessed contributors for kicking Trillian test jobs: ['rhtyd', 'nvazquez', 
'PaulAngus', 'borisstoyanov', 'DaanHoogland']
 
----
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] borisstoyanov commented on issue #2263: CLOUDSTACK-10070: Fixing some component tests and adding them in travis

2017-09-19 Thread git
borisstoyanov commented on issue #2263: CLOUDSTACK-10070: Fixing some component 
tests and adding them in travis
URL: https://github.com/apache/cloudstack/pull/2263#issuecomment-330523775
 
 
   @blueorangutan help
   
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] rhtyd commented on issue #2263: CLOUDSTACK-10070: Fixing some component tests and adding them in travis

2017-09-19 Thread git
rhtyd commented on issue #2263: CLOUDSTACK-10070: Fixing some component tests 
and adding them in travis
URL: https://github.com/apache/cloudstack/pull/2263#issuecomment-330521737
 
 
   @borisstoyanov yes, since we're changing test code, let's validate them 
against one of the hypervisors; kvm would be a good candidate.
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] blueorangutan commented on issue #2267: CLOUDSTACK-10077: allow to have different VPN customer gateway configs for same gateway IP

2017-09-19 Thread git
blueorangutan commented on issue #2267: CLOUDSTACK-10077: allow to have 
different VPN customer gateway configs for same gateway IP
URL: https://github.com/apache/cloudstack/pull/2267#issuecomment-330518581
 
 
   @DaanHoogland a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has 
been kicked to run smoke tests
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] DaanHoogland commented on issue #2267: CLOUDSTACK-10077: allow to have different VPN customer gateway configs for same gateway IP

2017-09-19 Thread git
DaanHoogland commented on issue #2267: CLOUDSTACK-10077: allow to have 
different VPN customer gateway configs for same gateway IP
URL: https://github.com/apache/cloudstack/pull/2267#issuecomment-330518438
 
 
   @blueorangutan test
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] borisstoyanov commented on issue #2263: CLOUDSTACK-10070: Fixing some component tests and adding them in travis

2017-09-19 Thread git
borisstoyanov commented on issue #2263: CLOUDSTACK-10070: Fixing some component 
tests and adding them in travis
URL: https://github.com/apache/cloudstack/pull/2263#issuecomment-330512052
 
 
   @rhtyd those tests pass in travis do you want explicit run on KVM env? 
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] rhtyd commented on issue #2263: CLOUDSTACK-10070: Fixing some component tests and adding them in travis

2017-09-19 Thread git
rhtyd commented on issue #2263: CLOUDSTACK-10070: Fixing some component tests 
and adding them in travis
URL: https://github.com/apache/cloudstack/pull/2263#issuecomment-330508802
 
 
   @borisstoyanov good point, can you run a component test run to confirm 
regression testings?
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] jayakarteek commented on a change in pull request #2122: CLOUDSTACK-9909: Logrotate config for tomcat needs to be updated for ?

2017-09-19 Thread git
jayakarteek commented on a change in pull request #2122: CLOUDSTACK-9909: 
Logrotate config for tomcat needs to be updated for ?
URL: https://github.com/apache/cloudstack/pull/2122#discussion_r139658445
 
 

 ##
 File path: server/conf/cloudstack-catalina7.logrotate
 ##
 @@ -0,0 +1,27 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license agreements.  See the NOTICE file
+# distributed with this work for additional information
+# regarding copyright ownership.  The ASF licenses this file
+# to you under the Apache License, Version 2.0 (the
+# "License"); you may not use this file except in compliance
+# with the License.  You may obtain a copy of the License at
+#
+#   http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing,
+# software distributed under the License is distributed on an
+# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+# KIND, either express or implied.  See the License for the
+# specific language governing permissions and limitations
+# under the License.
+
+
+/var/log/cloudstack/management/catalina.out {
+copytruncate
+daily
+rotate 14
+compress
+missingok
+create 0644 cloud cloud
+su cloud cloud
 
 Review comment:
   any update @rhtyd 
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] borisstoyanov commented on issue #2263: CLOUDSTACK-10070: Fixing some component tests and adding them in travis

2017-09-19 Thread git
borisstoyanov commented on issue #2263: CLOUDSTACK-10070: Fixing some component 
tests and adding them in travis
URL: https://github.com/apache/cloudstack/pull/2263#issuecomment-330502525
 
 
   @rhtyd the changes are in /component/.. do you think it's required to run a 
smoketest?
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] blueorangutan commented on issue #2258: Cloudstack 10064: Secondary storage Usage for uploadedVolume is not collected

2017-09-19 Thread git
blueorangutan commented on issue #2258: Cloudstack 10064: Secondary storage 
Usage for uploadedVolume is not collected
URL: https://github.com/apache/cloudstack/pull/2258#issuecomment-330501929
 
 
   @rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been 
kicked to run smoke tests
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] rhtyd commented on issue #2258: Cloudstack 10064: Secondary storage Usage for uploadedVolume is not collected

2017-09-19 Thread git
rhtyd commented on issue #2258: Cloudstack 10064: Secondary storage Usage for 
uploadedVolume is not collected
URL: https://github.com/apache/cloudstack/pull/2258#issuecomment-330501756
 
 
   @blueorangutan test
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] blueorangutan commented on issue #2258: Cloudstack 10064: Secondary storage Usage for uploadedVolume is not collected

2017-09-19 Thread git
blueorangutan commented on issue #2258: Cloudstack 10064: Secondary storage 
Usage for uploadedVolume is not collected
URL: https://github.com/apache/cloudstack/pull/2258#issuecomment-330501396
 
 
   Packaging result: ?centos6 ?centos7 ?debian. JID-1105
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] blueorangutan commented on issue #2263: CLOUDSTACK-10070: Fixing some component tests and adding them in travis

2017-09-19 Thread git
blueorangutan commented on issue #2263: CLOUDSTACK-10070: Fixing some component 
tests and adding them in travis
URL: https://github.com/apache/cloudstack/pull/2263#issuecomment-330501412
 
 
   Packaging result: ?centos6 ?centos7 ?debian. JID-1106
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] blueorangutan commented on issue #1707: CLOUDSTACK-9397: Add Watchdog timer to KVM Instance

2017-09-19 Thread git
blueorangutan commented on issue #1707: CLOUDSTACK-9397: Add Watchdog timer to 
KVM Instance
URL: https://github.com/apache/cloudstack/pull/1707#issuecomment-330501286
 
 
   @rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos6) has been 
kicked to run smoke tests
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] rhtyd commented on issue #1707: CLOUDSTACK-9397: Add Watchdog timer to KVM Instance

2017-09-19 Thread git
rhtyd commented on issue #1707: CLOUDSTACK-9397: Add Watchdog timer to KVM 
Instance
URL: https://github.com/apache/cloudstack/pull/1707#issuecomment-330501194
 
 
   @blueorangutan test centos7 kvm-centos6
   
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] blueorangutan commented on issue #2267: CLOUDSTACK-10077: allow to have different VPN customer gateway configs for same gateway IP

2017-09-19 Thread git
blueorangutan commented on issue #2267: CLOUDSTACK-10077: allow to have 
different VPN customer gateway configs for same gateway IP
URL: https://github.com/apache/cloudstack/pull/2267#issuecomment-330497849
 
 
   Packaging result: ?centos6 ?centos7 ?debian. JID-1104
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] blueorangutan commented on issue #2257: CLOUDSTACK-10060:ListUsage API always displays the Virtual size as '0' for Usage type=9 (snapshot)

2017-09-19 Thread git
blueorangutan commented on issue #2257: CLOUDSTACK-10060:ListUsage API always 
displays the Virtual size as '0' for Usage type=9 (snapshot)
URL: https://github.com/apache/cloudstack/pull/2257#issuecomment-330496618
 
 
   Packaging result: ?centos6 ?centos7 ?debian. JID-1102
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] blueorangutan commented on issue #1707: CLOUDSTACK-9397: Add Watchdog timer to KVM Instance

2017-09-19 Thread git
blueorangutan commented on issue #1707: CLOUDSTACK-9397: Add Watchdog timer to 
KVM Instance
URL: https://github.com/apache/cloudstack/pull/1707#issuecomment-330496645
 
 
   Packaging result: ?centos6 ?centos7 ?debian. JID-1103
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] blueorangutan commented on issue #2263: CLOUDSTACK-10070: Fixing some component tests and adding them in travis

2017-09-19 Thread git
blueorangutan commented on issue #2263: CLOUDSTACK-10070: Fixing some component 
tests and adding them in travis
URL: https://github.com/apache/cloudstack/pull/2263#issuecomment-330487855
 
 
   @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted 
as I make progress.
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] rhtyd commented on issue #2263: CLOUDSTACK-10070: Fixing some component tests and adding them in travis

2017-09-19 Thread git
rhtyd commented on issue #2263: CLOUDSTACK-10070: Fixing some component tests 
and adding them in travis
URL: https://github.com/apache/cloudstack/pull/2263#issuecomment-330487639
 
 
   @borisstoyanov I've rekicked one of the failing travis jobs, in general LGTM.
   @blueorangutan package
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] blueorangutan commented on issue #2267: CLOUDSTACK-10077: allow to have different VPN customer gateway configs for same gateway IP

2017-09-19 Thread git
blueorangutan commented on issue #2267: CLOUDSTACK-10077: allow to have 
different VPN customer gateway configs for same gateway IP
URL: https://github.com/apache/cloudstack/pull/2267#issuecomment-330487344
 
 
   @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted 
as I make progress.
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] blueorangutan commented on issue #2258: Cloudstack 10064: Secondary storage Usage for uploadedVolume is not collected

2017-09-19 Thread git
blueorangutan commented on issue #2258: Cloudstack 10064: Secondary storage 
Usage for uploadedVolume is not collected
URL: https://github.com/apache/cloudstack/pull/2258#issuecomment-330487359
 
 
   @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted 
as I make progress.
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] rhtyd commented on issue #2267: CLOUDSTACK-10077: allow to have different VPN customer gateway configs for same gateway IP

2017-09-19 Thread git
rhtyd commented on issue #2267: CLOUDSTACK-10077: allow to have different VPN 
customer gateway configs for same gateway IP
URL: https://github.com/apache/cloudstack/pull/2267#issuecomment-330487314
 
 
   @blueorangutan package
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] rhtyd commented on issue #2258: Cloudstack 10064: Secondary storage Usage for uploadedVolume is not collected

2017-09-19 Thread git
rhtyd commented on issue #2258: Cloudstack 10064: Secondary storage Usage for 
uploadedVolume is not collected
URL: https://github.com/apache/cloudstack/pull/2258#issuecomment-330487165
 
 
   @blueorangutan package
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] rhtyd commented on issue #2260: CLOUDSTACK-10065: Optimize SQL queries in listTemplate API to improve performance

2017-09-19 Thread git
rhtyd commented on issue #2260: CLOUDSTACK-10065: Optimize SQL queries in 
listTemplate API to improve performance
URL: https://github.com/apache/cloudstack/pull/2260#issuecomment-330487064
 
 
   Test LGTM (did not do code review), we'll need one more code review after 
that if there are no outstanding issues we can merge this. 
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] rhtyd closed pull request #2153: CLOUDSTACK-9956 File search on the vmware datastore may select wrong file if there are multiple files with same name.

2017-09-19 Thread git
rhtyd closed pull request #2153: CLOUDSTACK-9956 File search on the vmware 
datastore may select wrong file if there are multiple files with same name.
URL: https://github.com/apache/cloudstack/pull/2153
 
 
   
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] rhtyd commented on issue #2153: CLOUDSTACK-9956 File search on the vmware datastore may select wrong file if there are multiple files with same name.

2017-09-19 Thread git
rhtyd commented on issue #2153: CLOUDSTACK-9956 File search on the vmware 
datastore may select wrong file if there are multiple files with same name.
URL: https://github.com/apache/cloudstack/pull/2153#issuecomment-330486621
 
 
   Test LGTM, merging this.
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] blueorangutan commented on issue #1707: CLOUDSTACK-9397: Add Watchdog timer to KVM Instance

2017-09-19 Thread git
blueorangutan commented on issue #1707: CLOUDSTACK-9397: Add Watchdog timer to 
KVM Instance
URL: https://github.com/apache/cloudstack/pull/1707#issuecomment-330486577
 
 
   @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted 
as I make progress.
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] rhtyd commented on issue #1707: CLOUDSTACK-9397: Add Watchdog timer to KVM Instance

2017-09-19 Thread git
rhtyd commented on issue #1707: CLOUDSTACK-9397: Add Watchdog timer to KVM 
Instance
URL: https://github.com/apache/cloudstack/pull/1707#issuecomment-330486375
 
 
   @wido I can run another round of testing, for comparison I'm using the #2225 
test results as baseline for comparison
   @blueorangutan package
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] blueorangutan commented on issue #2257: CLOUDSTACK-10060:ListUsage API always displays the Virtual size as '0' for Usage type=9 (snapshot)

2017-09-19 Thread git
blueorangutan commented on issue #2257: CLOUDSTACK-10060:ListUsage API always 
displays the Virtual size as '0' for Usage type=9 (snapshot)
URL: https://github.com/apache/cloudstack/pull/2257#issuecomment-330486317
 
 
   @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted 
as I make progress.
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] rhtyd commented on issue #2257: CLOUDSTACK-10060:ListUsage API always displays the Virtual size as '0' for Usage type=9 (snapshot)

2017-09-19 Thread git
rhtyd commented on issue #2257: CLOUDSTACK-10060:ListUsage API always displays 
the Virtual size as '0' for Usage type=9 (snapshot)
URL: https://github.com/apache/cloudstack/pull/2257#issuecomment-330486146
 
 
   @blueorangutan package
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] rhtyd closed pull request #2078: CLOUDSTACK-9902: add consoleproxy.sslEnable toggle

2017-09-19 Thread git
rhtyd closed pull request #2078: CLOUDSTACK-9902: add consoleproxy.sslEnable 
toggle
URL: https://github.com/apache/cloudstack/pull/2078
 
 
   
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] rhtyd commented on issue #2078: CLOUDSTACK-9902: add consoleproxy.sslEnable toggle

2017-09-19 Thread git
rhtyd commented on issue #2078: CLOUDSTACK-9902: add consoleproxy.sslEnable 
toggle
URL: https://github.com/apache/cloudstack/pull/2078#issuecomment-330485772
 
 
   LGTM, this is a schema only change and evaluation/testing with Travis is 
enough to qualify merging.
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] rhtyd commented on a change in pull request #2258: Cloudstack 10064: Secondary storage Usage for uploadedVolume is not collected

2017-09-19 Thread git
rhtyd commented on a change in pull request #2258: Cloudstack 10064: Secondary 
storage Usage for uploadedVolume is not collected
URL: https://github.com/apache/cloudstack/pull/2258#discussion_r139642828
 
 

 ##
 File path: test/integration/testpaths/Upload_Volume_test_case.py
 ##
 @@ -0,0 +1,223 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license agreements.  See the NOTICE file
+# distributed with this work for additional information
+# regarding copyright ownership.  The ASF licenses this file
+# to you under the Apache License, Version 2.0 (the
+# "License"); you may not use this file except in compliance
+# with the License.  You may obtain a copy of the License at
+#
+#   http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing,
+# software distributed under the License is distributed on an
+# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+# KIND, either express or implied.  See the License for the
+# specific language governing permissions and limitations
+# under the License.
+""" Test cases for checking that the secondary Storage usage is accounted. 
This is verified by making sure that storage_type=4 is added in the 
cloud_usage.usage_storage
+table for a volume in 'Uploaded' state.
+
+This test case:
+1.Creates an account and uploads a volume.
+2.After the volume is uploaded successfully, restarts the usage server.
+3.Connectst ot the datbase
+4.From the database verifies that an entry is added to 
cloud_usage.usage_storage table with storage_type=4 for the uploaded volume.
+5.Clean up resources.
+"""
+
+from marvin.cloudstackTestCase import *
+from marvin.cloudstackAPI import *
+from marvin.lib.utils import *
+from marvin.lib.base import *
+from marvin.lib.common import *
+from nose.plugins.attrib import attr
+from marvin.sshClient import SshClient
+from marvin.codes import (BACKED_UP, PASS, FAIL)
+import time
+
+
+def verify_vm(self, vmid, state):
 
 Review comment:
   @PranaliM can this test be moved to smoke or component test? What is the 
significant/usage of putting some tests in `integration/testpaths`?
 
----
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] rhtyd commented on a change in pull request #2240: CLOUDSTACK-10051 Mouse Scrolling is not working in instance VM console

2017-09-19 Thread git
rhtyd commented on a change in pull request #2240: CLOUDSTACK-10051 Mouse 
Scrolling is not working in instance VM console
URL: https://github.com/apache/cloudstack/pull/2240#discussion_r139642434
 
 

 ##
 File path: systemvm/js/jquery.flot.navigate.js
 ##
 @@ -0,0 +1,336 @@
+/*
 
 Review comment:
   @vedulasantosh if the license used in this specific library are same as 
those used by other jquery files then I suppose we should be okay, however can 
you start a discussion on dev@?
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] blueorangutan commented on issue #2246: CLOUDSTACK-10046 checksum validation for any java supported Digests-type

2017-09-19 Thread git
blueorangutan commented on issue #2246: CLOUDSTACK-10046 checksum validation 
for any java supported Digests-type
URL: https://github.com/apache/cloudstack/pull/2246#issuecomment-330463131
 
 
   @borisstoyanov a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has 
been kicked to run smoke tests
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] borisstoyanov commented on issue #2246: CLOUDSTACK-10046 checksum validation for any java supported Digests-type

2017-09-19 Thread git
borisstoyanov commented on issue #2246: CLOUDSTACK-10046 checksum validation 
for any java supported Digests-type
URL: https://github.com/apache/cloudstack/pull/2246#issuecomment-330463000
 
 
   I've re-run the tests and they are passing... 
   @blueorangutan test
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] borisstoyanov commented on issue #1252: CLOUDSTACK-9182: Some running VMs turned off on manual migration when auto migration failed while host preparing for maintenance.

2017-09-19 Thread git
borisstoyanov commented on issue #1252: CLOUDSTACK-9182: Some running VMs 
turned off on manual migration when auto migration failed while host preparing 
for maintenance.
URL: https://github.com/apache/cloudstack/pull/1252#issuecomment-330462365
 
 
   I've just re-run the iso tests and they are passing.. 
   @blueorangutan test
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] blueorangutan commented on issue #1252: CLOUDSTACK-9182: Some running VMs turned off on manual migration when auto migration failed while host preparing for maintenance.

2017-09-19 Thread git
blueorangutan commented on issue #1252: CLOUDSTACK-9182: Some running VMs 
turned off on manual migration when auto migration failed while host preparing 
for maintenance.
URL: https://github.com/apache/cloudstack/pull/1252#issuecomment-330462469
 
 
   @borisstoyanov a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has 
been kicked to run smoke tests
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] borisstoyanov commented on issue #1252: CLOUDSTACK-9182: Some running VMs turned off on manual migration when auto migration failed while host preparing for maintenance.

2017-09-19 Thread git
borisstoyanov commented on issue #1252: CLOUDSTACK-9182: Some running VMs 
turned off on manual migration when auto migration failed while host preparing 
for maintenance.
URL: https://github.com/apache/cloudstack/pull/1252#issuecomment-330462365
 
 
   I've just re-run the iso tests and they are passing.. 
   @blueorangutan test
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] PranaliM commented on issue #2258: Cloudstack 10064: Secondary storage Usage for uploadedVolume is not collected

2017-09-19 Thread git
PranaliM commented on issue #2258: Cloudstack 10064: Secondary storage Usage 
for uploadedVolume is not collected
URL: https://github.com/apache/cloudstack/pull/2258#issuecomment-330447354
 
 
   tag:mergeready
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] yvsubhash commented on issue #2258: Cloudstack 10064: Secondary storage Usage for uploadedVolume is not collected

2017-09-19 Thread git
yvsubhash commented on issue #2258: Cloudstack 10064: Secondary storage Usage 
for uploadedVolume is not collected
URL: https://github.com/apache/cloudstack/pull/2258#issuecomment-330439828
 
 
   LGTM for code
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] vedulasantosh commented on a change in pull request #2240: CLOUDSTACK-10051 Mouse Scrolling is not working in instance VM console

2017-09-18 Thread git
vedulasantosh commented on a change in pull request #2240: CLOUDSTACK-10051 
Mouse Scrolling is not working in instance VM console
URL: https://github.com/apache/cloudstack/pull/2240#discussion_r139596104
 
 

 ##
 File path: systemvm/js/jquery.flot.navigate.js
 ##
 @@ -0,0 +1,336 @@
+/*
 
 Review comment:
   @rhtyd can you review this license
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] blueorangutan commented on issue #2246: CLOUDSTACK-10046 checksum validation for any java supported Digests-type

2017-09-18 Thread git
m the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


<    3   4   5   6   7   8   9   10   11   12   >