[GitHub] cloudmonger commented on issue #2004: [4.11/Future] CLOUDSTACK-9832: Do not assign public IP NIC to the VPC VR when the VPC offering does not contain VpcVirtualRouter as a SourceNat provider

2017-08-29 Thread git
cloudmonger commented on issue #2004: [4.11/Future] CLOUDSTACK-9832: Do not 
assign public IP NIC to the VPC VR when the VPC offering does not contain 
VpcVirtualRouter as a SourceNat provider
URL: https://github.com/apache/cloudstack/pull/2004#issuecomment-305546325
 
 
   ### ACS CI BVT Run
**Sumarry:**
Build Number 781
Hypervisor xenserver
NetworkType Advanced
Passed=108
Failed=1
Skipped=12
   
   _Link to logs Folder (search by build_no):_ 
https://www.dropbox.com/sh/r2si930m8xxzavs/AAAzNrnoF1fC3auFrvsKo_8-a?dl=0
   
   **Failed tests:**
   * test_router_dns.py
   
* test_router_dns_guestipquery Failed
   
   
   **Skipped tests:**
   test_vm_nic_adapter_vmxnet3
   test_01_verify_libvirt
   test_02_verify_libvirt_after_restart
   test_03_verify_libvirt_attach_disk
   test_04_verify_guest_lspci
   test_05_change_vm_ostype_restart
   test_06_verify_guest_lspci_again
   test_static_role_account_acls
   test_11_ss_nfs_version_on_ssvm
   test_nested_virtualization_vmware
   test_3d_gpu_support
   test_deploy_vgpu_enabled_vm
   
   **Passed test suits:**
   test_deploy_vm_with_userdata.py
   test_affinity_groups_projects.py
   test_portable_publicip.py
   test_vm_snapshots.py
   test_over_provisioning.py
   test_global_settings.py
   test_scale_vm.py
   test_service_offerings.py
   test_routers_iptables_default_policy.py
   test_loadbalance.py
   test_routers.py
   test_reset_vm_on_reboot.py
   test_deploy_vms_with_varied_deploymentplanners.py
   test_network.py
   test_non_contigiousvlan.py
   test_login.py
   test_deploy_vm_iso.py
   test_list_ids_parameter.py
   test_public_ip_range.py
   test_multipleips_per_nic.py
   test_metrics_api.py
   test_regions.py
   test_affinity_groups.py
   test_network_acl.py
   test_pvlan.py
   test_volumes.py
   test_nic.py
   test_deploy_vm_root_resize.py
   test_resource_detail.py
   test_secondary_storage.py
   test_routers_network_ops.py
   test_disk_offerings.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] cloudmonger commented on issue #2004: [4.11/Future] CLOUDSTACK-9832: Do not assign public IP NIC to the VPC VR when the VPC offering does not contain VpcVirtualRouter as a SourceNat provider

2017-08-29 Thread git
cloudmonger commented on issue #2004: [4.11/Future] CLOUDSTACK-9832: Do not 
assign public IP NIC to the VPC VR when the VPC offering does not contain 
VpcVirtualRouter as a SourceNat provider
URL: https://github.com/apache/cloudstack/pull/2004#issuecomment-325876881
 
 
   ### ACS CI BVT Run
**Sumarry:**
Build Number 1165
Hypervisor xenserver
NetworkType Advanced
Passed=105
Failed=7
Skipped=12
   
   _Link to logs Folder (search by build_no):_ 
https://www.dropbox.com/sh/r2si930m8xxzavs/AAAzNrnoF1fC3auFrvsKo_8-a?dl=0
   
   **Failed tests:**
   * test_non_contigiousvlan.py
   
* test_extendPhysicalNetworkVlan Failing since 2 runs
   
   * test_list_ids_parameter.py
   
* ContextSuite context=TestListIdsParams>:setup Failing since 58 runs
   
   * test_volumes.py
   
* test_06_download_detached_volume Failing since 2 runs
   
   * test_routers_network_ops.py
   
* test_01_isolate_network_FW_PF_default_routes_egress_true Failing since 29 
runs
   
* test_02_isolate_network_FW_PF_default_routes_egress_false Failing since 
156 runs
   
* test_01_RVR_Network_FW_PF_SSH_default_routes_egress_true Failing since 
151 runs
   
* test_02_RVR_Network_FW_PF_SSH_default_routes_egress_false Failing since 
151 runs
   
   
   **Skipped tests:**
   test_vm_nic_adapter_vmxnet3
   test_01_verify_libvirt
   test_02_verify_libvirt_after_restart
   test_03_verify_libvirt_attach_disk
   test_04_verify_guest_lspci
   test_05_change_vm_ostype_restart
   test_06_verify_guest_lspci_again
   test_static_role_account_acls
   test_11_ss_nfs_version_on_ssvm
   test_nested_virtualization_vmware
   test_3d_gpu_support
   test_deploy_vgpu_enabled_vm
   
   **Passed test suits:**
   test_deploy_vm_with_userdata.py
   test_affinity_groups_projects.py
   test_portable_publicip.py
   test_vm_snapshots.py
   test_over_provisioning.py
   test_global_settings.py
   test_router_dnsservice.py
   test_scale_vm.py
   test_service_offerings.py
   test_routers_iptables_default_policy.py
   test_loadbalance.py
   test_routers.py
   test_reset_vm_on_reboot.py
   test_deploy_vms_with_varied_deploymentplanners.py
   test_network.py
   test_router_dns.py
   test_login.py
   test_deploy_vm_iso.py
   test_public_ip_range.py
   test_multipleips_per_nic.py
   test_metrics_api.py
   test_regions.py
   test_affinity_groups.py
   test_network_acl.py
   test_pvlan.py
   test_nic.py
   test_deploy_vm_root_resize.py
   test_resource_detail.py
   test_secondary_storage.py
   test_vm_life_cycle.py
   test_disk_offerings.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] mrunalinikankariya commented on a change in pull request #2244: CLOUDSTACK-10054:Volume download times out in 3600 seconds

2017-08-29 Thread git
mrunalinikankariya commented on a change in pull request #2244: 
CLOUDSTACK-10054:Volume download times out in 3600 seconds
URL: https://github.com/apache/cloudstack/pull/2244#discussion_r135963512
 
 

 ##
 File path: 
plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/manager/VmwareManager.java
 ##
 @@ -39,6 +39,9 @@
 public static final ConfigKey s_vmwareCleanOldWorderVMs = new 
ConfigKey("Advanced", Boolean.class, "vmware.clean.old.worker.vms", 
"false",
 "If a worker vm is older then twice the 'job.expire.minutes' + 
'job.cancel.threshold.minutes' , remove it.", true, ConfigKey.Scope.Global);
 
+static final ConfigKey s_vmwareOVAPackageTimeout = new 
ConfigKey(Integer.class, "vmware.package.ova.timeout", "Advanced", 
"3600",
 
 Review comment:
   corrected the description
 

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] mrunalinikankariya commented on a change in pull request #1985: CLOUDSTACK-9812:Update "updatePortForwardingRule" api to include additional parameter to update the end port in case of port ra

2017-08-29 Thread git
mrunalinikankariya commented on a change in pull request #1985: 
CLOUDSTACK-9812:Update "updatePortForwardingRule" api to include additional 
parameter to update the end port in case of port range
URL: https://github.com/apache/cloudstack/pull/1985#discussion_r135963268
 
 

 ##
 File path: test/integration/component/test_portforwardingrules.py
 ##
 @@ -0,0 +1,429 @@
+# 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.
+
+# Import Local Modules
+from marvin.cloudstackTestCase import cloudstackTestCase, unittest
+from marvin.lib.base import (PublicIPAddress,
+ NetworkOffering,
+ Autoscale,
+ Network,
+ NetworkServiceProvider,
+ Template,
+ VirtualMachine,
+ VPC,
+ VpcOffering,
+ StaticNATRule,
+ FireWallRule,
+ NATRule,
+ Vpn,
+ VpnUser,
+ LoadBalancerRule,
+ Account,
+ ServiceOffering,
+ PhysicalNetwork,
+ User)
+from marvin.lib.common import (get_domain,
+   get_zone,
+   get_template)
+from marvin.lib.utils import validateList, cleanup_resources
+from marvin.codes import PASS
+from nose.plugins.attrib import attr
+
+class TestPortForwardingRules(cloudstackTestCase):
 
 Review comment:
   done
 

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] nvazquez commented on issue #2158: CLOUDSTACK-9972: Enhance listVolume API to include physical size and ?

2017-08-29 Thread git
nvazquez commented on issue #2158: CLOUDSTACK-9972: Enhance listVolume API to 
include physical size and ?
URL: https://github.com/apache/cloudstack/pull/2158#issuecomment-325850583
 
 
   @rhtyd issues fixed, test failing on Travis seems unrelated: 
`"CloudstackAPIException: Execute cmd: createproject failed, due to: errorCode: 
535, errorText:Maximum number of resources of type 'project' for domain id=5 
has been exceeded.\n"]`
 

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] nvazquez commented on issue #2158: CLOUDSTACK-9972: Enhance listVolume API to include physical size and ?

2017-08-29 Thread git
nvazquez commented on issue #2158: CLOUDSTACK-9972: Enhance listVolume API to 
include physical size and ?
URL: https://github.com/apache/cloudstack/pull/2158#issuecomment-325850583
 
 
   @rhtyd issues fixed, tests failing on Travis seems unrelated: 
`"CloudstackAPIException: Execute cmd: createproject failed, due to: errorCode: 
535, errorText:Maximum number of resources of type 'project' for domain id=5 
has been exceeded.\n"]`
 

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] cloudmonger commented on issue #1960: [4.11/Future] CLOUDSTACK-9782: Host HA and KVM HA provider

2017-08-29 Thread git
cloudmonger commented on issue #1960: [4.11/Future] CLOUDSTACK-9782: Host HA 
and KVM HA provider
URL: https://github.com/apache/cloudstack/pull/1960#issuecomment-325502654
 
 
   ### ACS CI BVT Run
**Sumarry:**
Build Number 1159
Hypervisor xenserver
NetworkType Advanced
Passed=115
Failed=5
Skipped=40
   
   _Link to logs Folder (search by build_no):_ 
https://www.dropbox.com/sh/r2si930m8xxzavs/AAAzNrnoF1fC3auFrvsKo_8-a?dl=0
   
   **Failed tests:**
   * test_volumes.py
   
* test_06_download_detached_volume Failed
   
   * test_routers_network_ops.py
   
* test_01_isolate_network_FW_PF_default_routes_egress_true Failing since 24 
runs
   
* test_02_isolate_network_FW_PF_default_routes_egress_false Failing since 
151 runs
   
* test_01_RVR_Network_FW_PF_SSH_default_routes_egress_true Failing since 
146 runs
   
* test_02_RVR_Network_FW_PF_SSH_default_routes_egress_false Failing since 
146 runs
   
   
   **Skipped tests:**
   test_vm_nic_adapter_vmxnet3
   test_01_verify_libvirt
   test_02_verify_libvirt_after_restart
   test_03_verify_libvirt_attach_disk
   test_04_verify_guest_lspci
   test_05_change_vm_ostype_restart
   test_06_verify_guest_lspci_again
   test_disable_oobm_ha_state_ineligible
   test_ha_kvm_host_degraded
   test_ha_kvm_host_fencing
   test_ha_kvm_host_recovering
   test_hostha_configure_default_driver
   test_hostha_enable_ha_when_host_disabled
   test_hostha_enable_ha_when_host_disconected
   test_hostha_enable_ha_when_host_in_maintenance
   test_remove_ha_provider_not_possible
   test_configure_ha_provider_invalid
   test_configure_ha_provider_valid
   test_ha_disable_feature_invalid
   test_ha_enable_feature_invalid
   test_ha_enabledisable_across_clusterzones
   test_ha_list_providers
   test_ha_multiple_mgmt_server_ownership
   test_ha_verify_fsm_available
   test_ha_verify_fsm_degraded
   test_ha_verify_fsm_fenced
   test_ha_verify_fsm_recovering
   test_hostha_configure_default_driver
   test_hostha_configure_invalid_provider
   test_hostha_disable_feature_valid
   test_hostha_enable_feature_valid
   test_hostha_enable_feature_without_setting_provider
   test_list_ha_for_host
   test_list_ha_for_host_invalid
   test_list_ha_for_host_valid
   test_static_role_account_acls
   test_11_ss_nfs_version_on_ssvm
   test_nested_virtualization_vmware
   test_3d_gpu_support
   test_deploy_vgpu_enabled_vm
   
   **Passed test suits:**
   test_deploy_vm_with_userdata.py
   test_affinity_groups_projects.py
   test_portable_publicip.py
   test_vm_snapshots.py
   test_over_provisioning.py
   test_global_settings.py
   test_router_dnsservice.py
   test_scale_vm.py
   test_service_offerings.py
   test_routers_iptables_default_policy.py
   test_loadbalance.py
   test_routers.py
   test_reset_vm_on_reboot.py
   test_deploy_vms_with_varied_deploymentplanners.py
   test_network.py
   test_router_dns.py
   test_outofbandmanagement_nestedplugin.py
   test_non_contigiousvlan.py
   test_login.py
   test_deploy_vm_iso.py
   test_list_ids_parameter.py
   test_public_ip_range.py
   test_multipleips_per_nic.py
   test_metrics_api.py
   test_regions.py
   test_affinity_groups.py
   test_network_acl.py
   test_pvlan.py
   test_nic.py
   test_deploy_vm_root_resize.py
   test_resource_detail.py
   test_secondary_storage.py
   test_vm_life_cycle.py
   test_disk_offerings.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] cloudmonger commented on issue #1960: [4.11/Future] CLOUDSTACK-9782: Host HA and KVM HA provider

2017-08-29 Thread git
cloudmonger commented on issue #1960: [4.11/Future] CLOUDSTACK-9782: Host HA 
and KVM HA provider
URL: https://github.com/apache/cloudstack/pull/1960#issuecomment-325843732
 
 
   ### ACS CI BVT Run
**Sumarry:**
Build Number 1164
Hypervisor xenserver
NetworkType Advanced
Passed=109
Failed=8
Skipped=40
   
   _Link to logs Folder (search by build_no):_ 
https://www.dropbox.com/sh/r2si930m8xxzavs/AAAzNrnoF1fC3auFrvsKo_8-a?dl=0
   
   **Failed tests:**
   * test_vm_snapshots.py
   
* test_change_service_offering_for_vm_with_snapshots Failed
   
   * test_router_dnsservice.py
   
* test_router_dns_guestipquery Failing since 2 runs
   
   * test_non_contigiousvlan.py
   
* test_extendPhysicalNetworkVlan Failed
   
   * test_volumes.py
   
* test_06_download_detached_volume Failed
   
   * test_routers_network_ops.py
   
* test_01_isolate_network_FW_PF_default_routes_egress_true Failing since 28 
runs
   
* test_02_isolate_network_FW_PF_default_routes_egress_false Failing since 
155 runs
   
* test_01_RVR_Network_FW_PF_SSH_default_routes_egress_true Failing since 
150 runs
   
* test_02_RVR_Network_FW_PF_SSH_default_routes_egress_false Failing since 
150 runs
   
   
   **Skipped tests:**
   test_vm_nic_adapter_vmxnet3
   test_01_verify_libvirt
   test_02_verify_libvirt_after_restart
   test_03_verify_libvirt_attach_disk
   test_04_verify_guest_lspci
   test_05_change_vm_ostype_restart
   test_06_verify_guest_lspci_again
   test_disable_oobm_ha_state_ineligible
   test_ha_kvm_host_degraded
   test_ha_kvm_host_fencing
   test_ha_kvm_host_recovering
   test_hostha_configure_default_driver
   test_hostha_enable_ha_when_host_disabled
   test_hostha_enable_ha_when_host_disconected
   test_hostha_enable_ha_when_host_in_maintenance
   test_remove_ha_provider_not_possible
   test_configure_ha_provider_invalid
   test_configure_ha_provider_valid
   test_ha_disable_feature_invalid
   test_ha_enable_feature_invalid
   test_ha_enabledisable_across_clusterzones
   test_ha_list_providers
   test_ha_multiple_mgmt_server_ownership
   test_ha_verify_fsm_available
   test_ha_verify_fsm_degraded
   test_ha_verify_fsm_fenced
   test_ha_verify_fsm_recovering
   test_hostha_configure_default_driver
   test_hostha_configure_invalid_provider
   test_hostha_disable_feature_valid
   test_hostha_enable_feature_valid
   test_hostha_enable_feature_without_setting_provider
   test_list_ha_for_host
   test_list_ha_for_host_invalid
   test_list_ha_for_host_valid
   test_static_role_account_acls
   test_11_ss_nfs_version_on_ssvm
   test_nested_virtualization_vmware
   test_3d_gpu_support
   test_deploy_vgpu_enabled_vm
   
   **Passed test suits:**
   test_deploy_vm_with_userdata.py
   test_affinity_groups_projects.py
   test_portable_publicip.py
   test_over_provisioning.py
   test_global_settings.py
   test_scale_vm.py
   test_service_offerings.py
   test_routers_iptables_default_policy.py
   test_loadbalance.py
   test_routers.py
   test_reset_vm_on_reboot.py
   test_deploy_vms_with_varied_deploymentplanners.py
   test_network.py
   test_router_dns.py
   test_outofbandmanagement_nestedplugin.py
   test_login.py
   test_deploy_vm_iso.py
   test_list_ids_parameter.py
   test_public_ip_range.py
   test_multipleips_per_nic.py
   test_metrics_api.py
   test_regions.py
   test_affinity_groups.py
   test_network_acl.py
   test_pvlan.py
   test_nic.py
   test_deploy_vm_root_resize.py
   test_resource_detail.py
   test_secondary_storage.py
   test_disk_offerings.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 #2158: CLOUDSTACK-9972: Enhance listVolume API to include physical size and ?

2017-08-29 Thread git
blueorangutan commented on issue #2158: CLOUDSTACK-9972: Enhance listVolume API 
to include physical size and ?
URL: https://github.com/apache/cloudstack/pull/2158#issuecomment-325823241
 
 
   Trillian test result (tid-1452)
   Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
   Total time taken: 33295 seconds
   Marvin logs: 
https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr2158-t1452-kvm-centos7.zip
   Intermitten failure detected: /marvin/tests/smoke/test_iso.py
   Intermitten failure detected: /marvin/tests/smoke/test_privategw_acl.py
   Intermitten failure detected: /marvin/tests/smoke/test_vpc_vpn.py
   Test completed. 55 look OK, 3 have error(s)
   
   
   Test | Result | Time (s) | Test File
   --- | --- | --- | ---
   test_01_vpc_remote_access_vpn | `Failure` | 60.74 | test_vpc_vpn.py
   test_04_rvpc_privategw_static_routes | `Failure` | 349.28 | 
test_privategw_acl.py
   test_05_iso_permissions | `Failure` | 0.04 | test_iso.py
   test_02_edit_iso | `Failure` | 0.03 | test_iso.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.01 | test_ssvm.py
   test_01_scale_vm | Skipped | 0.00 | test_scale_vm.py
   test_01_primary_storage_iscsi | Skipped | 0.02 | 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_deploy_vgpu_enabled_vm | Skipped | 0.02 | test_deploy_vgpu_enabled_vm.py
   test_3d_gpu_support | Skipped | 0.02 | 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-08-29 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-325814286
 
 
   Trillian test result (tid-1451)
   Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
   Total time taken: 31070 seconds
   Marvin logs: 
https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr2246-t1451-kvm-centos7.zip
   Intermitten failure detected: /marvin/tests/smoke/test_iso.py
   Intermitten failure detected: /marvin/tests/smoke/test_privategw_acl.py
   Intermitten failure detected: /marvin/tests/smoke/test_vpc_vpn.py
   Test completed. 55 look OK, 3 have error(s)
   
   
   Test | Result | Time (s) | Test File
   --- | --- | --- | ---
   test_01_vpc_remote_access_vpn | `Failure` | 45.88 | test_vpc_vpn.py
   test_04_rvpc_privategw_static_routes | `Failure` | 360.67 | 
test_privategw_acl.py
   test_05_iso_permissions | `Failure` | 0.06 | test_iso.py
   test_02_edit_iso | `Failure` | 0.05 | test_iso.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_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] cloudmonger commented on issue #2234: CLOUDSTACK-10041: Support to use ikev1 for site-to-site VPN connections

2017-08-29 Thread git
cloudmonger commented on issue #2234: CLOUDSTACK-10041: Support to use ikev1 
for site-to-site VPN connections
URL: https://github.com/apache/cloudstack/pull/2234#issuecomment-325798334
 
 
   ### ACS CI BVT Run
**Sumarry:**
Build Number 1163
Hypervisor xenserver
NetworkType Advanced
Passed=106
Failed=5
Skipped=12
   
   _Link to logs Folder (search by build_no):_ 
https://www.dropbox.com/sh/r2si930m8xxzavs/AAAzNrnoF1fC3auFrvsKo_8-a?dl=0
   
   **Failed tests:**
   * test_router_dns.py
   
* test_router_dns_guestipquery Failing since 1 runs
   
   * test_routers_network_ops.py
   
* test_01_isolate_network_FW_PF_default_routes_egress_true Failing since 27 
runs
   
* test_02_isolate_network_FW_PF_default_routes_egress_false Failing since 
154 runs
   
* test_01_RVR_Network_FW_PF_SSH_default_routes_egress_true Failing since 
149 runs
   
* test_02_RVR_Network_FW_PF_SSH_default_routes_egress_false Failing since 
149 runs
   
   
   **Skipped tests:**
   test_vm_nic_adapter_vmxnet3
   test_01_verify_libvirt
   test_02_verify_libvirt_after_restart
   test_03_verify_libvirt_attach_disk
   test_04_verify_guest_lspci
   test_05_change_vm_ostype_restart
   test_06_verify_guest_lspci_again
   test_static_role_account_acls
   test_11_ss_nfs_version_on_ssvm
   test_nested_virtualization_vmware
   test_3d_gpu_support
   test_deploy_vgpu_enabled_vm
   
   **Passed test suits:**
   test_deploy_vm_with_userdata.py
   test_affinity_groups_projects.py
   test_portable_publicip.py
   test_vm_snapshots.py
   test_over_provisioning.py
   test_global_settings.py
   test_router_dnsservice.py
   test_scale_vm.py
   test_service_offerings.py
   test_routers_iptables_default_policy.py
   test_loadbalance.py
   test_routers.py
   test_reset_vm_on_reboot.py
   test_deploy_vms_with_varied_deploymentplanners.py
   test_network.py
   test_non_contigiousvlan.py
   test_login.py
   test_deploy_vm_iso.py
   test_list_ids_parameter.py
   test_public_ip_range.py
   test_multipleips_per_nic.py
   test_metrics_api.py
   test_regions.py
   test_affinity_groups.py
   test_network_acl.py
   test_pvlan.py
   test_volumes.py
   test_nic.py
   test_deploy_vm_root_resize.py
   test_resource_detail.py
   test_secondary_storage.py
   test_disk_offerings.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 #2109: CLOUDSTACK-9922 Unable to use 8081 port for Load balancing

2017-08-29 Thread git
blueorangutan commented on issue #2109: CLOUDSTACK-9922 Unable to use 8081 port 
for Load balancing
URL: https://github.com/apache/cloudstack/pull/2109#issuecomment-325744432
 
 
   Trillian test result (tid-1448)
   Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
   Total time taken: 33355 seconds
   Marvin logs: 
https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr2109-t1448-kvm-centos7.zip
   Intermitten failure detected: 
/marvin/tests/smoke/test_deploy_virtio_scsi_vm.py
   Intermitten failure detected: /marvin/tests/smoke/test_iso.py
   Intermitten failure detected: /marvin/tests/smoke/test_privategw_acl.py
   Intermitten failure detected: /marvin/tests/smoke/test_vpc_vpn.py
   Test completed. 55 look OK, 3 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` | 340.38 | 
test_privategw_acl.py
   test_05_iso_permissions | `Failure` | 0.04 | test_iso.py
   test_02_edit_iso | `Failure` | 0.04 | test_iso.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_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
   
 

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] cloudmonger commented on issue #2148: CLOUDSTACK-9963 Root Disk controller value is changing while migrating VM

2017-08-29 Thread git
cloudmonger commented on issue #2148: CLOUDSTACK-9963 Root Disk controller 
value is changing while migrating VM
URL: https://github.com/apache/cloudstack/pull/2148#issuecomment-325729042
 
 
   ### ACS CI BVT Run
**Sumarry:**
Build Number 1162
Hypervisor xenserver
NetworkType Advanced
Passed=108
Failed=6
Skipped=12
   
   _Link to logs Folder (search by build_no):_ 
https://www.dropbox.com/sh/r2si930m8xxzavs/AAAzNrnoF1fC3auFrvsKo_8-a?dl=0
   
   **Failed tests:**
   * test_router_dns.py
   
* test_router_dns_guestipquery Failed
   
   * test_routers_network_ops.py
   
* test_01_isolate_network_FW_PF_default_routes_egress_true Failing since 26 
runs
   
* test_02_isolate_network_FW_PF_default_routes_egress_false Failing since 
153 runs
   
* test_01_RVR_Network_FW_PF_SSH_default_routes_egress_true Failing since 
148 runs
   
* test_02_RVR_Network_FW_PF_SSH_default_routes_egress_false Failing since 
148 runs
   
* test_03_RVR_Network_check_router_state Failed
   
   
   **Skipped tests:**
   test_vm_nic_adapter_vmxnet3
   test_01_verify_libvirt
   test_02_verify_libvirt_after_restart
   test_03_verify_libvirt_attach_disk
   test_04_verify_guest_lspci
   test_05_change_vm_ostype_restart
   test_06_verify_guest_lspci_again
   test_static_role_account_acls
   test_11_ss_nfs_version_on_ssvm
   test_nested_virtualization_vmware
   test_3d_gpu_support
   test_deploy_vgpu_enabled_vm
   
   **Passed test suits:**
   test_deploy_vm_with_userdata.py
   test_affinity_groups_projects.py
   test_portable_publicip.py
   test_vm_snapshots.py
   test_over_provisioning.py
   test_global_settings.py
   test_router_dnsservice.py
   test_scale_vm.py
   test_service_offerings.py
   test_routers_iptables_default_policy.py
   test_loadbalance.py
   test_routers.py
   test_reset_vm_on_reboot.py
   test_deploy_vms_with_varied_deploymentplanners.py
   test_network.py
   test_non_contigiousvlan.py
   test_login.py
   test_deploy_vm_iso.py
   test_list_ids_parameter.py
   test_public_ip_range.py
   test_multipleips_per_nic.py
   test_metrics_api.py
   test_regions.py
   test_affinity_groups.py
   test_network_acl.py
   test_pvlan.py
   test_volumes.py
   test_nic.py
   test_deploy_vm_root_resize.py
   test_resource_detail.py
   test_secondary_storage.py
   test_vm_life_cycle.py
   test_disk_offerings.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] sgoeminn opened a new pull request #2250: CLOUDSTACK-10057: listNetworkOfferings now returns the correct number of offerings.

2017-08-29 Thread git
sgoeminn opened a new pull request #2250: CLOUDSTACK-10057: 
listNetworkOfferings now returns the correct number  of offerings.
URL: https://github.com/apache/cloudstack/pull/2250
 
 
   https://issues.apache.org/jira/browse/CLOUDSTACK-10057
 

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 #2214: Speed-up VR initialisation/configuration

2017-08-29 Thread git
blueorangutan commented on issue #2214: Speed-up VR initialisation/configuration
URL: https://github.com/apache/cloudstack/pull/2214#issuecomment-325664974
 
 
   Packaging result: ?centos6 ?centos7 ?debian. JID-1043
 

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 #2158: CLOUDSTACK-9972: Enhance listVolume API to include physical size and ?

2017-08-29 Thread git
blueorangutan commented on issue #2158: CLOUDSTACK-9972: Enhance listVolume API 
to include physical size and ?
URL: https://github.com/apache/cloudstack/pull/2158#issuecomment-325659844
 
 
   @nvazquez 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] nvazquez commented on issue #2158: CLOUDSTACK-9972: Enhance listVolume API to include physical size and ?

2017-08-29 Thread git
nvazquez commented on issue #2158: CLOUDSTACK-9972: Enhance listVolume API to 
include physical size and ?
URL: https://github.com/apache/cloudstack/pull/2158#issuecomment-325659765
 
 
   @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 #2158: CLOUDSTACK-9972: Enhance listVolume API to include physical size and ?

2017-08-29 Thread git
blueorangutan commented on issue #2158: CLOUDSTACK-9972: Enhance listVolume API 
to include physical size and ?
URL: https://github.com/apache/cloudstack/pull/2158#issuecomment-325659279
 
 
   Packaging result: ?centos6 ?centos7 ?debian. JID-1042
 

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-08-29 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-325658509
 
 
   @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-08-29 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-325658307
 
 
   @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 #2246: CLOUDSTACK-10046 checksum validation for any java supported Digests-type

2017-08-29 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-325657709
 
 
   Packaging result: ?centos6 ?centos7 ?debian. JID-1041
 

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 #2214: Speed-up VR initialisation/configuration

2017-08-29 Thread git
blueorangutan commented on issue #2214: Speed-up VR initialisation/configuration
URL: https://github.com/apache/cloudstack/pull/2214#issuecomment-325657737
 
 
   @DagSonsteboSB 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] DagSonsteboSB commented on issue #2214: Speed-up VR initialisation/configuration

2017-08-29 Thread git
DagSonsteboSB commented on issue #2214: Speed-up VR initialisation/configuration
URL: https://github.com/apache/cloudstack/pull/2214#issuecomment-325657630
 
 
   @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 #2158: CLOUDSTACK-9972: Enhance listVolume API to include physical size and ?

2017-08-29 Thread git
blueorangutan commented on issue #2158: CLOUDSTACK-9972: Enhance listVolume API 
to include physical size and ?
URL: https://github.com/apache/cloudstack/pull/2158#issuecomment-325652483
 
 
   @nvazquez 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] nvazquez commented on issue #2158: CLOUDSTACK-9972: Enhance listVolume API to include physical size and ?

2017-08-29 Thread git
nvazquez commented on issue #2158: CLOUDSTACK-9972: Enhance listVolume API to 
include physical size and ?
URL: https://github.com/apache/cloudstack/pull/2158#issuecomment-325652319
 
 
   @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 #2246: CLOUDSTACK-10046 checksum validation for any java supported Digests-type

2017-08-29 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-325650324
 
 
   @borisstoyanov 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] borisstoyanov commented on issue #2246: CLOUDSTACK-10046 checksum validation for any java supported Digests-type

2017-08-29 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-325650154
 
 
   @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] sgoeminn commented on issue #2004: [4.11/Future] CLOUDSTACK-9832: Do not assign public IP NIC to the VPC VR when the VPC offering does not contain VpcVirtualRouter as a SourceNat provider

2017-08-29 Thread git
sgoeminn commented on issue #2004: [4.11/Future] CLOUDSTACK-9832: Do not assign 
public IP NIC to the VPC VR when the VPC offering does not contain 
VpcVirtualRouter as a SourceNat provider
URL: https://github.com/apache/cloudstack/pull/2004#issuecomment-325650035
 
 
   Currently we will not change the check (It's also not checked in isolated 
network case). We will change it as soon as we add support for site-to-site VPN 
for Nuage where Nuage will be the source NAT provider. 
 

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 issue #2054: CLOUDSTACK-9886 : After restarting cloudstack-management , It takes time to connect hosts

2017-08-29 Thread git
vedulasantosh commented on issue #2054: CLOUDSTACK-9886 : After restarting 
cloudstack-management , It takes time to connect hosts
URL: https://github.com/apache/cloudstack/pull/2054#issuecomment-325647854
 
 
   Test LGTM
   
   Before Fix:
   
   It took about 12 mins to connect ESXi host after MS was restarted.
   
   2017-08-29 15:55:57,441 INFO  [o.a.c.s.l.CloudStackExtendedLifeCycle] 
(main:null) (logid:) Running system integrity checker 
org.apache.cloudstack.utils.identity.ManagementServerNode@56c9aa12
   2017-08-29 15:55:57,441 INFO  [o.a.c.s.l.CloudStackExtendedLifeCycle] 
(main:null) (logid:) Configuring CloudStack Components
   2017-08-29 16:08:30,772 DEBUG [c.c.a.m.AgentManagerImpl] 
(AgentTaskPool-1:ctx-e4b9546b) (logid:7831f281) Simulating start for resource 
10.112.3.36 id 1
   2017-08-29 16:08:30,772 DEBUG [c.c.a.m.AgentManagerImpl] 
(AgentTaskPool-1:ctx-e4b9546b) (logid:7831f281) Creating agent for host 1
   2017-08-29 16:08:30,951 DEBUG [c.c.s.StatsCollector] 
(StatsCollector-4:ctx-200c10b0) (logid:88d478b5) 
HostOutOfBandManagementStatsCollector is running...
   
   
   After Fix:
   
   It took about 38 sec to connect ESXi host after MS was restarted.
   
   2017-08-29 17:42:14,432 INFO  [o.a.c.s.l.CloudStackExtendedLifeCycle] 
(main:null) (logid:) Running system integrity checker 
org.apache.cloudstack.utils.identity.ManagementServerNode@4d1847de
   2017-08-29 17:42:14,433 INFO  [o.a.c.s.l.CloudStackExtendedLifeCycle] 
(main:null) (logid:) Configuring CloudStack Components
   2017-08-29 17:42:52,302 DEBUG [c.c.a.m.AgentManagerImpl] 
(AgentTaskPool-1:ctx-2410f7cb) (logid:b06510de) Simulating start for resource 
10.112.3.36 id 1
   2017-08-29 17:42:52,302 DEBUG [c.c.a.m.AgentManagerImpl] 
(AgentTaskPool-1:ctx-2410f7cb) (logid:b06510de) Creating agent for host 1
   2017-08-29 17:42:53,748 DEBUG [c.c.s.StatsCollector] 
(StatsCollector-1:ctx-5c38226e) (logid:203d8311) HostStatsCollector is 
running...
 

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-08-29 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-325641496
 
 
   Packaging result: ?centos6 ?centos7 ?debian. JID-1040
 

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-08-29 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-325640331
 
 
   @borisstoyanov 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 #2214: Speed-up VR initialisation/configuration

2017-08-29 Thread git
blueorangutan commented on issue #2214: Speed-up VR initialisation/configuration
URL: https://github.com/apache/cloudstack/pull/2214#issuecomment-325640087
 
 
   Packaging result: ?centos6 ?centos7 ?debian. JID-1039
 

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-08-29 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-325640134
 
 
   @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 #1960: [4.11/Future] CLOUDSTACK-9782: Host HA and KVM HA provider

2017-08-29 Thread git
rhtyd commented on issue #1960: [4.11/Future] CLOUDSTACK-9782: Host HA and KVM 
HA provider
URL: https://github.com/apache/cloudstack/pull/1960#issuecomment-325638899
 
 
   Ignoring intermittent test failures and new test failures around `isos`, 
this PR has enough LGTMs and test results now, I'll merge this tomorrow and 
wait for any new comments and objections.
 

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 #1960: [4.11/Future] CLOUDSTACK-9782: Host HA and KVM HA provider

2017-08-29 Thread git
rhtyd commented on issue #1960: [4.11/Future] CLOUDSTACK-9782: Host HA and KVM 
HA provider
URL: https://github.com/apache/cloudstack/pull/1960#issuecomment-325638685
 
 
   @DaanHoogland I've fixed the issue now, it was not used so removed the method
 

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 #1960: [4.11/Future] CLOUDSTACK-9782: Host HA and KVM HA provider

2017-08-29 Thread git
rhtyd commented on a change in pull request #1960: [4.11/Future] 
CLOUDSTACK-9782: Host HA and KVM HA provider
URL: https://github.com/apache/cloudstack/pull/1960#discussion_r135765061
 
 

 ##
 File path: 
api/src/org/apache/cloudstack/api/command/admin/host/PrepareForMaintenanceCmd.java
 ##
 @@ -108,4 +108,8 @@ public void execute() {
 throw new ServerApiException(ApiErrorCode.INTERNAL_ERROR, "Failed 
to prepare host for maintenance");
 }
 }
+
+public void setHostId(final Long hostId) {
+id = hostId;
 
 Review comment:
   Fixed @DaanHoogland please see, turns out we dont' need this method.
 

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 #1960: [4.11/Future] CLOUDSTACK-9782: Host HA and KVM HA provider

2017-08-29 Thread git
rhtyd commented on a change in pull request #1960: [4.11/Future] 
CLOUDSTACK-9782: Host HA and KVM HA provider
URL: https://github.com/apache/cloudstack/pull/1960#discussion_r135765061
 
 

 ##
 File path: 
api/src/org/apache/cloudstack/api/command/admin/host/PrepareForMaintenanceCmd.java
 ##
 @@ -108,4 +108,8 @@ public void execute() {
 throw new ServerApiException(ApiErrorCode.INTERNAL_ERROR, "Failed 
to prepare host for maintenance");
 }
 }
+
+public void setHostId(final Long hostId) {
+id = hostId;
 
 Review comment:
   Can fix that
 

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 #2214: Speed-up VR initialisation/configuration

2017-08-29 Thread git
blueorangutan commented on issue #2214: Speed-up VR initialisation/configuration
URL: https://github.com/apache/cloudstack/pull/2214#issuecomment-325635204
 
 
   @DagSonsteboSB 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] DagSonsteboSB commented on issue #2214: Speed-up VR initialisation/configuration

2017-08-29 Thread git
DagSonsteboSB commented on issue #2214: Speed-up VR initialisation/configuration
URL: https://github.com/apache/cloudstack/pull/2214#issuecomment-325635050
 
 
   @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] DaanHoogland commented on a change in pull request #1960: [4.11/Future] CLOUDSTACK-9782: Host HA and KVM HA provider

2017-08-29 Thread git
DaanHoogland commented on a change in pull request #1960: [4.11/Future] 
CLOUDSTACK-9782: Host HA and KVM HA provider
URL: https://github.com/apache/cloudstack/pull/1960#discussion_r135757989
 
 

 ##
 File path: 
api/src/org/apache/cloudstack/api/command/admin/host/PrepareForMaintenanceCmd.java
 ##
 @@ -108,4 +108,8 @@ public void execute() {
 throw new ServerApiException(ApiErrorCode.INTERNAL_ERROR, "Failed 
to prepare host for maintenance");
 }
 }
+
+public void setHostId(final Long hostId) {
+id = hostId;
 
 Review comment:
   this setter is still to be called setId(..) instead of setHostId(..), or 
should the field be renamed?
 

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


[cloudstack-docs-rn] branch master updated (bfee260 -> f36f2f4)

2017-08-29 Thread bhaisaab
This is an automated email from the ASF dual-hosted git repository.

bhaisaab pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/cloudstack-docs-rn.git.


from bfee260  about: update what's new in 4.9.2.0
 add 9b7c995  upgrade: 4.10 JRE1.8 and systemvmtemplate url updates
 add 34a7972  upgrade: fix section usage for Java8 installation on Ubuntu
 add 8581314  upgrade: commit java8 and upgrade49 documents
 add 065f7f4  4.10 api changes and fixed issues copied from list generated 
by @swill
 add 1a75888  corrected PR #2071
 add 62505e7  Merge pull request #30 from 
karuturi/4.10-api-changes-issues-fixed
 add 102f554  Whats new in 4.10.0.0
 add 92c9836  corrected indentation
 add b836233  Changed thejasypt jar file version from 1.9.0 to 1.9.2
 add c287855  upgrade: fix jasypt-1.9.2 version/jar usage
 add f36f2f4  Merge branch '4.9' into 4.10

No new revisions were added by this update.

Summary of changes:
 source/about.rst   |   85 +-
 source/api-changes.rst |  653 ++---
 source/compat.rst  |2 +-
 source/conf.py |4 +-
 source/fixed_issues.rst| 2546 +++-
 source/global.rst  |   10 +-
 source/index.rst   |1 +
 ...{_no-sysvm_templates.rst => _java_8_ubuntu.rst} |   16 +-
 source/upgrade/_no-sysvm_templates.rst |2 +-
 source/upgrade/_sysvm_templates.rst|   16 +-
 source/upgrade/upgrade-4.2.rst |5 +-
 source/upgrade/upgrade-4.3.rst |4 +-
 source/upgrade/upgrade-4.4.rst |4 +-
 source/upgrade/upgrade-4.5.rst |4 +-
 source/upgrade/upgrade-4.6.rst |5 +-
 source/upgrade/upgrade-4.7.rst |6 +-
 source/upgrade/upgrade-4.8.rst |5 +-
 .../upgrade/{upgrade-4.8.rst => upgrade-4.9.rst}   |   29 +-
 source/upgrade/upgrade_notes.rst   |7 +-
 source/upgrade/valid_source.rst|   14 +-
 20 files changed, 1726 insertions(+), 1692 deletions(-)
 copy source/upgrade/{_no-sysvm_templates.rst => _java_8_ubuntu.rst} (59%)
 copy source/upgrade/{upgrade-4.8.rst => upgrade-4.9.rst} (97%)

-- 
To stop receiving notification emails like this one, please contact
['"commits@cloudstack.apache.org" '].


[cloudstack-docs-rn] branch 4.9 updated: upgrade: fix jasypt-1.9.2 version/jar usage

2017-08-29 Thread bhaisaab
This is an automated email from the ASF dual-hosted git repository.

bhaisaab pushed a commit to branch 4.9
in repository https://gitbox.apache.org/repos/asf/cloudstack-docs-rn.git


The following commit(s) were added to refs/heads/4.9 by this push:
 new c287855  upgrade: fix jasypt-1.9.2 version/jar usage
c287855 is described below

commit c2878550d60e0f96c605208261818ac99a68daa1
Author: Rohit Yadav 
AuthorDate: Tue Aug 29 13:05:12 2017 +0200

upgrade: fix jasypt-1.9.2 version/jar usage

Signed-off-by: Rohit Yadav 
---
 source/upgrade/upgrade-4.2.rst | 2 +-
 source/upgrade/upgrade-4.3.rst | 2 +-
 source/upgrade/upgrade-4.4.rst | 2 +-
 source/upgrade/upgrade-4.5.rst | 2 +-
 source/upgrade/upgrade-4.6.rst | 2 +-
 source/upgrade/upgrade-4.7.rst | 2 +-
 source/upgrade/upgrade-4.8.rst | 2 +-
 7 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/source/upgrade/upgrade-4.2.rst b/source/upgrade/upgrade-4.2.rst
index d0c8017..714b9ed 100644
--- a/source/upgrade/upgrade-4.2.rst
+++ b/source/upgrade/upgrade-4.2.rst
@@ -280,7 +280,7 @@ are required only for clouds using VMware clusters:
 
.. sourcecode:: bash
 
-  $ java -classpath /usr/share/cloudstack-common/lib/jasypt-1.9.0.jar 
org.jasypt.intf.cli.JasyptPBEStringEncryptionCLI encrypt.sh 
input="_your_vCenter_password_" password="`cat /etc/cloudstack/management/key`" 
verbose=false
+  $ java -classpath /usr/share/cloudstack-common/lib/jasypt-1.9.2.jar 
org.jasypt.intf.cli.JasyptPBEStringEncryptionCLI encrypt.sh 
input="_your_vCenter_password_" password="`cat /etc/cloudstack/management/key`" 
verbose=false
 
Store the output from this step, we need to add this in
cluster\_details table and vmware\_data\_center tables in place of
diff --git a/source/upgrade/upgrade-4.3.rst b/source/upgrade/upgrade-4.3.rst
index 05b7cc5..8b7bfd2 100644
--- a/source/upgrade/upgrade-4.3.rst
+++ b/source/upgrade/upgrade-4.3.rst
@@ -283,7 +283,7 @@ are required only for clouds using VMware clusters:
 
.. sourcecode:: bash
 
-  $ java -classpath /usr/share/cloudstack-common/lib/jasypt-1.9.0.jar 
org.jasypt.intf.cli.JasyptPBEStringEncryptionCLI encrypt.sh 
input="_your_vCenter_password_" password="`cat /etc/cloudstack/management/key`" 
verbose=false
+  $ java -classpath /usr/share/cloudstack-common/lib/jasypt-1.9.2.jar 
org.jasypt.intf.cli.JasyptPBEStringEncryptionCLI encrypt.sh 
input="_your_vCenter_password_" password="`cat /etc/cloudstack/management/key`" 
verbose=false
 
Store the output from this step, we need to add this in
cluster\_details table and vmware\_data\_center tables in place of
diff --git a/source/upgrade/upgrade-4.4.rst b/source/upgrade/upgrade-4.4.rst
index 9e8e2ac..1e5ab92 100644
--- a/source/upgrade/upgrade-4.4.rst
+++ b/source/upgrade/upgrade-4.4.rst
@@ -285,7 +285,7 @@ are required only for clouds using VMware clusters:
 
.. sourcecode:: bash
 
-  $ java -classpath /usr/share/cloudstack-common/lib/jasypt-1.9.0.jar 
org.jasypt.intf.cli.JasyptPBEStringEncryptionCLI encrypt.sh 
input="_your_vCenter_password_" password="`cat /etc/cloudstack/management/key`" 
verbose=false
+  $ java -classpath /usr/share/cloudstack-common/lib/jasypt-1.9.2.jar 
org.jasypt.intf.cli.JasyptPBEStringEncryptionCLI encrypt.sh 
input="_your_vCenter_password_" password="`cat /etc/cloudstack/management/key`" 
verbose=false
 
Store the output from this step, we need to add this in
cluster\_details table and vmware\_data\_center tables in place of
diff --git a/source/upgrade/upgrade-4.5.rst b/source/upgrade/upgrade-4.5.rst
index e6403e8..e2ffc74 100644
--- a/source/upgrade/upgrade-4.5.rst
+++ b/source/upgrade/upgrade-4.5.rst
@@ -283,7 +283,7 @@ are required only for clouds using VMware clusters:
 
.. sourcecode:: bash
 
-  $ java -classpath /usr/share/cloudstack-common/lib/jasypt-1.9.0.jar 
org.jasypt.intf.cli.JasyptPBEStringEncryptionCLI encrypt.sh 
input="_your_vCenter_password_" password="`cat /etc/cloudstack/management/key`" 
verbose=false
+  $ java -classpath /usr/share/cloudstack-common/lib/jasypt-1.9.2.jar 
org.jasypt.intf.cli.JasyptPBEStringEncryptionCLI encrypt.sh 
input="_your_vCenter_password_" password="`cat /etc/cloudstack/management/key`" 
verbose=false
 
 Store the output from this step, we need to add this in
 cluster\_details table and vmware\_data\_center tables in place of
diff --git a/source/upgrade/upgrade-4.6.rst b/source/upgrade/upgrade-4.6.rst
index 5954609..fc0651f 100644
--- a/source/upgrade/upgrade-4.6.rst
+++ b/source/upgrade/upgrade-4.6.rst
@@ -280,7 +280,7 @@ are required only for clouds using VMware clusters:
 
.. sourcecode:: bash
 
-  $ java -classpath /usr/share/cloudstack-common/lib/jasypt-1.9.0.jar 
org.jasypt.intf.cli.JasyptPBEStringEncryptionCLI encrypt.sh 
input="_your_vCenter_password_" password="`cat /etc/cloudstack/management/key`" 
verbose=false
+  $ java -classpath 

[cloudstack-docs-rn] 01/01: Merge branch '4.9' into 4.10

2017-08-29 Thread bhaisaab
This is an automated email from the ASF dual-hosted git repository.

bhaisaab pushed a commit to branch 4.10
in repository https://gitbox.apache.org/repos/asf/cloudstack-docs-rn.git

commit f36f2f458ae7de0f7d38d3c06cb4c0bc7200ceee
Merge: b836233 c287855
Author: Rohit Yadav 
AuthorDate: Tue Aug 29 13:06:14 2017 +0200

Merge branch '4.9' into 4.10

 source/upgrade/upgrade-4.2.rst | 2 +-
 source/upgrade/upgrade-4.3.rst | 2 +-
 source/upgrade/upgrade-4.4.rst | 2 +-
 source/upgrade/upgrade-4.5.rst | 2 +-
 source/upgrade/upgrade-4.6.rst | 2 +-
 source/upgrade/upgrade-4.7.rst | 2 +-
 source/upgrade/upgrade-4.8.rst | 2 +-
 7 files changed, 7 insertions(+), 7 deletions(-)


-- 
To stop receiving notification emails like this one, please contact
"commits@cloudstack.apache.org" .


[cloudstack-docs-rn] branch 4.10 updated (b836233 -> f36f2f4)

2017-08-29 Thread bhaisaab
This is an automated email from the ASF dual-hosted git repository.

bhaisaab pushed a change to branch 4.10
in repository https://gitbox.apache.org/repos/asf/cloudstack-docs-rn.git.


from b836233  Changed thejasypt jar file version from 1.9.0 to 1.9.2
 add c287855  upgrade: fix jasypt-1.9.2 version/jar usage
 new f36f2f4  Merge branch '4.9' into 4.10

The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.


Summary of changes:
 source/upgrade/upgrade-4.2.rst | 2 +-
 source/upgrade/upgrade-4.3.rst | 2 +-
 source/upgrade/upgrade-4.4.rst | 2 +-
 source/upgrade/upgrade-4.5.rst | 2 +-
 source/upgrade/upgrade-4.6.rst | 2 +-
 source/upgrade/upgrade-4.7.rst | 2 +-
 source/upgrade/upgrade-4.8.rst | 2 +-
 7 files changed, 7 insertions(+), 7 deletions(-)

-- 
To stop receiving notification emails like this one, please contact
['"commits@cloudstack.apache.org" '].


[cloudstack-docs-rn] branch 4.10 updated: Changed thejasypt jar file version from 1.9.0 to 1.9.2

2017-08-29 Thread bhaisaab
This is an automated email from the ASF dual-hosted git repository.

bhaisaab pushed a commit to branch 4.10
in repository https://gitbox.apache.org/repos/asf/cloudstack-docs-rn.git


The following commit(s) were added to refs/heads/4.10 by this push:
 new b836233  Changed thejasypt jar file version from 1.9.0 to 1.9.2
b836233 is described below

commit b836233028d3c2bddb6b39668cc8d75902c67e27
Author: Glenn Wagner 
AuthorDate: Tue Aug 29 12:56:54 2017 +0200

Changed thejasypt jar file version from 1.9.0 to 1.9.2
---
 source/upgrade/upgrade-4.9.rst | 24 
 1 file changed, 12 insertions(+), 12 deletions(-)

diff --git a/source/upgrade/upgrade-4.9.rst b/source/upgrade/upgrade-4.9.rst
index eae26ea..7f6af71 100644
--- a/source/upgrade/upgrade-4.9.rst
+++ b/source/upgrade/upgrade-4.9.rst
@@ -19,7 +19,7 @@
 Upgrade Instruction from |version_to_upgrade|
 =
 
-This section will guide you from CloudStack |version_to_upgrade| to CloudStack 
+This section will guide you from CloudStack |version_to_upgrade| to CloudStack
 |version|.
 
 Any steps that are hypervisor-specific will be called out with a note.
@@ -28,7 +28,7 @@ We recommend reading through this section once or twice 
before beginning
 your upgrade procedure, and working through it on a test system before
 working on a production system.
 
-.. note:: 
+.. note::
 The following upgrade instructions should be performed regardless of
 hypervisor type.
 
@@ -53,9 +53,9 @@ Create RPM or Debian packages (as appropriate) and a 
repository from
 the |version| source, or check the Apache CloudStack downloads page at
 http://cloudstack.apache.org/downloads.html
 for package repositories supplied by community members. You will need
-them for :ref:`ubuntu48` or :ref:`rhel48` and :ref:`kvm48` hosts upgrade. 
+them for :ref:`ubuntu48` or :ref:`rhel48` and :ref:`kvm48` hosts upgrade.
 
-Instructions for creating packages from the CloudStack source are in the 
+Instructions for creating packages from the CloudStack source are in the
 `CloudStack Installation Guide`_.
 
 
@@ -109,10 +109,10 @@ Backup current database
 Management Server on Ubuntu
 ---
 
-If you are using Ubuntu, follow this procedure to upgrade your packages. If 
+If you are using Ubuntu, follow this procedure to upgrade your packages. If
 not, skip to step :ref:`rhel48`.
 
-.. note:: 
+.. note::
**Community Packages:** This section assumes you're using the community
supplied packages for CloudStack. If you've created your own packages and
APT repository, substitute your own URL for the ones used in these examples.
@@ -179,10 +179,10 @@ read as appropriate for your |version| repository.
 Management Server on CentOS/RHEL
 
 
-If you are using CentOS or RHEL, follow this procedure to upgrade your 
+If you are using CentOS or RHEL, follow this procedure to upgrade your
 packages. If not, skip to hypervisors section, then :ref:`upg-sysvm48`.
 
-.. note:: 
+.. note::
**Community Packages:** This section assumes you're using the community
supplied packages for CloudStack. If you've created your own packages and
yum repository, substitute your own URL for the ones used in these examples.
@@ -230,7 +230,7 @@ Setup the GPG public key if you wish to enable 
``gpgcheck=1``:
 If you're using your own package repository, change this line to
 read as appropriate for your |version| repository.
 
-#. Now that you have the repository configured, it's time to upgrade the 
+#. Now that you have the repository configured, it's time to upgrade the
``cloudstack-management``.
 
.. sourcecode:: bash
@@ -279,7 +279,7 @@ are required only for clouds using VMware clusters:
 
.. sourcecode:: bash
 
-  $ java -classpath /usr/share/cloudstack-common/lib/jasypt-1.9.0.jar 
org.jasypt.intf.cli.JasyptPBEStringEncryptionCLI encrypt.sh 
input="_your_vCenter_password_" password="`cat /etc/cloudstack/management/key`" 
verbose=false
+  $ java -classpath /usr/share/cloudstack-common/lib/jasypt-1.9.2.jar 
org.jasypt.intf.cli.JasyptPBEStringEncryptionCLI encrypt.sh 
input="_your_vCenter_password_" password="`cat /etc/cloudstack/management/key`" 
verbose=false
 
 Store the output from this step, we need to add this in
 cluster\_details table and vmware\_data\_center tables in place of
@@ -357,7 +357,7 @@ hosts.
 
   $ sudo apt-get upgrade cloudstack-agent
 
-#. Verify that the file ``/etc/cloudstack/agent/environment.properties`` has a 
+#. Verify that the file ``/etc/cloudstack/agent/environment.properties`` has a
 line that reads:
 
.. sourcecode:: bash
@@ -383,7 +383,7 @@ For KVM hosts, upgrade the ``cloudstack-agent`` package
 
   $ sudo yum upgrade cloudstack-agent
 
-#. Verify that the file ``/etc/cloudstack/agent/environment.properties`` has a 
+#. Verify that the file ``/etc/cloudstack/agent/environment.properties`` 

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

2017-08-29 Thread git
DaanHoogland commented on issue #2246: CLOUDSTACK-10046 checksum validation for 
any java supported Digests-type
URL: https://github.com/apache/cloudstack/pull/2246#issuecomment-325611580
 
 
   tnx @borisstoyanov will do
 

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-08-29 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-325609930
 
 
   @DaanHoogland I've added tests, can you please resolve the merge conflicts 
so we can kick tests across hypervisors? 
 

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-08-29 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-325609930
 
 
   @DaanHoogland I've added tests, can you please resolve the merge conflicts? 
 

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


[cloudstack-docs-admin] branch master updated: conf: update version info

2017-08-29 Thread bhaisaab
This is an automated email from the ASF dual-hosted git repository.

bhaisaab pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/cloudstack-docs-admin.git


The following commit(s) were added to refs/heads/master by this push:
 new 3f3430b  conf: update version info
3f3430b is described below

commit 3f3430b4f9379ca4604bf66308d1994f7f8ff20c
Author: Rohit Yadav 
AuthorDate: Tue Aug 29 10:52:41 2017 +0200

conf: update version info

Signed-off-by: Rohit Yadav 
---
 source/conf.py | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/source/conf.py b/source/conf.py
index 368a6b5..9a3173d 100644
--- a/source/conf.py
+++ b/source/conf.py
@@ -60,9 +60,9 @@ copyright = u'2016, Apache Software Foundation'
 # built documents.
 #
 # The short X.Y version.
-version = '4.8'
+version = '4.11'
 # The full version, including alpha/beta/rc tags.
-release = '4.8.0'
+release = '4.11.0.0'
 
 # The language for content autogenerated by Sphinx. Refer to documentation
 # for a list of supported languages.

-- 
To stop receiving notification emails like this one, please contact
['"commits@cloudstack.apache.org" '].


[cloudstack-docs-admin] branch master updated: hosts: update hosts spec, remove deprecated global setting

2017-08-29 Thread bhaisaab
This is an automated email from the ASF dual-hosted git repository.

bhaisaab pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/cloudstack-docs-admin.git


The following commit(s) were added to refs/heads/master by this push:
 new a9d00e9  hosts: update hosts spec, remove deprecated global setting
a9d00e9 is described below

commit a9d00e921950c7c027e2ca18a51a68bdb9b99dd4
Author: Rohit Yadav 
AuthorDate: Tue Aug 29 10:48:52 2017 +0200

hosts: update hosts spec, remove deprecated global setting

Signed-off-by: Rohit Yadav 
---
 source/hosts.rst | 13 ++---
 1 file changed, 6 insertions(+), 7 deletions(-)

diff --git a/source/hosts.rst b/source/hosts.rst
index d9970da..68f2e94 100644
--- a/source/hosts.rst
+++ b/source/hosts.rst
@@ -588,7 +588,7 @@ To enable you to assign VLANs to Isolated networks,
 
 
 Out-of-band Management
-~~
+--
 
 CloudStack provides Root admins the ability to configure and use supported
 out-of-band management interface (e.g. IPMI, iLO, DRAC, etc.) on a physical
@@ -608,15 +608,14 @@ outofbandmanagement.action.timeout60  
The ou
 outofbandmanagement.ipmitool.interfacelanplus The 
out of band management IpmiTool driver interface to use. Valid values are: lan, 
lanplus etc
 outofbandmanagement.ipmitool.path /usr/bin/ipmitool   The 
out of band management ipmitool path used by the IpmiTool driver
 outofbandmanagement.ipmitool.retries  1   The 
out of band management IpmiTool driver retries option -R
-outofbandmanagement.sync.interval 300 The 
out of band management background sync thread interval in seconds
 outofbandmanagement.sync.poolsize 50  The 
out of band management background sync thread pool size 50
 ===   =   

 
-A change in `outofbandmanagement.sync.interval` or 
`outofbandmanagement.sync.poolsize`
-settings requires restarting of management server(s) as the thread pool and a
-background (power state) sync thread are configured during load time when
-CloudStack management server starts. Rest of the global settings can be changed
-without requiring restarting of management server(s).
+A change in `outofbandmanagement.sync.poolsize` settings requires restarting of
+management server(s) as the thread pool and a background (power state) sync
+thread are configured during load time when CloudStack management server 
starts.
+Rest of the global settings can be changed without requiring restarting of
+management server(s).
 
 The `outofbandmanagement.sync.poolsize` is the maximum number of ipmitool
 background power state scanners that can run at a time. Based on the maximum

-- 
To stop receiving notification emails like this one, please contact
['"commits@cloudstack.apache.org" '].


[GitHub] blueorangutan commented on issue #2046: CLOUDSTACK-7958: Add configuration for limit to CIDRs for Admin API calls

2017-08-29 Thread git
blueorangutan commented on issue #2046: CLOUDSTACK-7958: Add configuration for 
limit to CIDRs for Admin API calls
URL: https://github.com/apache/cloudstack/pull/2046#issuecomment-325596074
 
 
   @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] blueorangutan commented on issue #2109: CLOUDSTACK-9922 Unable to use 8081 port for Load balancing

2017-08-29 Thread git
blueorangutan commented on issue #2109: CLOUDSTACK-9922 Unable to use 8081 port 
for Load balancing
URL: https://github.com/apache/cloudstack/pull/2109#issuecomment-325596065
 
 
   @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 #2109: CLOUDSTACK-9922 Unable to use 8081 port for Load balancing

2017-08-29 Thread git
rhtyd commented on issue #2109: CLOUDSTACK-9922 Unable to use 8081 port for 
Load balancing
URL: https://github.com/apache/cloudstack/pull/2109#issuecomment-325596019
 
 
   @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 #2046: CLOUDSTACK-7958: Add configuration for limit to CIDRs for Admin API calls

2017-08-29 Thread git
rhtyd commented on issue #2046: CLOUDSTACK-7958: Add configuration for limit to 
CIDRs for Admin API calls
URL: https://github.com/apache/cloudstack/pull/2046#issuecomment-325595908
 
 
   @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 #2109: CLOUDSTACK-9922 Unable to use 8081 port for Load balancing

2017-08-29 Thread git
blueorangutan commented on issue #2109: CLOUDSTACK-9922 Unable to use 8081 port 
for Load balancing
URL: https://github.com/apache/cloudstack/pull/2109#issuecomment-325594899
 
 
   Packaging result: ?centos6 ?centos7 ?debian. JID-1037
 

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 #2046: CLOUDSTACK-7958: Add configuration for limit to CIDRs for Admin API calls

2017-08-29 Thread git
blueorangutan commented on issue #2046: CLOUDSTACK-7958: Add configuration for 
limit to CIDRs for Admin API calls
URL: https://github.com/apache/cloudstack/pull/2046#issuecomment-325594914
 
 
   Packaging result: ?centos6 ?centos7 ?debian. JID-1038
 

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] DagSonsteboSB commented on issue #2214: Speed-up VR initialisation/configuration

2017-08-29 Thread git
DagSonsteboSB commented on issue #2214: Speed-up VR initialisation/configuration
URL: https://github.com/apache/cloudstack/pull/2214#issuecomment-325587510
 
 
   Testing has identified issue with service startup order which needs resolved 
- cloud-early-config starts before cloud-early-init is finished, working on fix.
 

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 #2046: CLOUDSTACK-7958: Add configuration for limit to CIDRs for Admin API calls

2017-08-29 Thread git
blueorangutan commented on issue #2046: CLOUDSTACK-7958: Add configuration for 
limit to CIDRs for Admin API calls
URL: https://github.com/apache/cloudstack/pull/2046#issuecomment-325587132
 
 
   @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 #2109: CLOUDSTACK-9922 Unable to use 8081 port for Load balancing

2017-08-29 Thread git
blueorangutan commented on issue #2109: CLOUDSTACK-9922 Unable to use 8081 port 
for Load balancing
URL: https://github.com/apache/cloudstack/pull/2109#issuecomment-325587118
 
 
   @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 #2109: CLOUDSTACK-9922 Unable to use 8081 port for Load balancing

2017-08-29 Thread git
rhtyd commented on issue #2109: CLOUDSTACK-9922 Unable to use 8081 port for 
Load balancing
URL: https://github.com/apache/cloudstack/pull/2109#issuecomment-325587080
 
 
   @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 #2046: CLOUDSTACK-7958: Add configuration for limit to CIDRs for Admin API calls

2017-08-29 Thread git
rhtyd commented on issue #2046: CLOUDSTACK-7958: Add configuration for limit to 
CIDRs for Admin API calls
URL: https://github.com/apache/cloudstack/pull/2046#issuecomment-325587051
 
 
   @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] sachinnitw1317 commented on a change in pull request #2204: [CLOUDSTACK-10025] Adding Support for NoVNC Console for KVM and XENSERVER

2017-08-29 Thread git
sachinnitw1317 commented on a change in pull request #2204: [CLOUDSTACK-10025] 
Adding Support for NoVNC Console for KVM and XENSERVER
URL: https://github.com/apache/cloudstack/pull/2204#discussion_r135715681
 
 

 ##
 File path: systemvm/noVNC/dashboard.css
 ##
 @@ -0,0 +1,20 @@
+.vnc-controls {
 
 Review comment:
   added apache license to file I have created. rest files belong to noVNC
 

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 #2204: [CLOUDSTACK-10025] Adding Support for NoVNC Console for KVM and XENSERVER

2017-08-29 Thread git
rhtyd commented on issue #2204: [CLOUDSTACK-10025] Adding Support for NoVNC 
Console for KVM and XENSERVER
URL: https://github.com/apache/cloudstack/pull/2204#issuecomment-325569080
 
 
   @sachinnitw1317 good work overall, when you're done with additional changes 
and refactorings, let me know I would help kick some tests with 
Trillian/blueorangutan
 

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] sachinnitw1317 commented on a change in pull request #2204: [CLOUDSTACK-10025] Adding Support for NoVNC Console for KVM and XENSERVER

2017-08-29 Thread git
sachinnitw1317 commented on a change in pull request #2204: [CLOUDSTACK-10025] 
Adding Support for NoVNC Console for KVM and XENSERVER
URL: https://github.com/apache/cloudstack/pull/2204#discussion_r135704375
 
 

 ##
 File path: 
services/console-proxy/server/src/com/cloud/consoleproxy/ConsoleProxySecureServerFactoryImpl.java
 ##
 @@ -112,8 +117,7 @@ public SSLServerSocket createSSLServerSocket(int port) 
throws IOException {
 SSLServerSocket srvSock = null;
 SSLServerSocketFactory ssf = sslContext.getServerSocketFactory();
 srvSock = (SSLServerSocket)ssf.createServerSocket(port);
-srvSock.setEnabledProtocols(SSLUtils.getRecommendedProtocols());
-srvSock.setEnabledCipherSuites(SSLUtils.getRecommendedCiphers());
 
 Review comment:
   done
 

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