[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