[GitHub] cloudmonger commented on issue #2190: CLOUDSTACK-10010: Fixed the negotiation of S2S VPN connections

[git]

cloudmonger commented on issue #2190: CLOUDSTACK-10010: Fixed the negotiation 
of S2S VPN connections
URL: https://github.com/apache/cloudstack/pull/2190#issuecomment-317631490
 
 
   ### ACS CI BVT Run
**Sumarry:**
Build Number 1007
Hypervisor xenserver
NetworkType Advanced
Passed=102
Failed=11
Skipped=12
   
   _Link to logs Folder (search by build_no):_ 
https://www.dropbox.com/sh/r2si930m8xxzavs/AAAzNrnoF1fC3auFrvsKo_8-a?dl=0
   
   **Failed tests:**
   * test_loadbalance.py
   
* test_01_create_lb_rule_src_nat Failing since 2 runs
   
* test_02_create_lb_rule_non_nat Failing since 2 runs
   
* test_assign_and_removal_lb Failed
   
   * test_non_contigiousvlan.py
   
* test_extendPhysicalNetworkVlan Failing since 2 runs
   
   * test_deploy_vm_iso.py
   
* test_deploy_vm_from_iso Failing since 64 runs
   
   * test_volumes.py
   
* test_06_download_detached_volume Failing since 4 runs
   
   * test_vm_life_cycle.py
   
* test_10_attachAndDetach_iso Failing since 64 runs
   
   * test_routers_network_ops.py
   
* test_01_isolate_network_FW_PF_default_routes_egress_true Failing since 97 
runs
   
* test_02_isolate_network_FW_PF_default_routes_egress_false Failing since 
97 runs
   
* test_01_RVR_Network_FW_PF_SSH_default_routes_egress_true Failing since 95 
runs
   
* test_02_RVR_Network_FW_PF_SSH_default_routes_egress_false Failing since 
95 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_scale_vm.py
   test_service_offerings.py
   test_routers_iptables_default_policy.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_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] cloudmonger commented on issue #2190: CLOUDSTACK-10010: Fixed the negotiation of S2S VPN connections

[git]

cloudmonger commented on issue #2190: CLOUDSTACK-10010: Fixed the negotiation 
of S2S VPN connections
URL: https://github.com/apache/cloudstack/pull/2190#issuecomment-318260244
 
 
   ### ACS CI BVT Run
**Sumarry:**
Build Number 1015
Hypervisor xenserver
NetworkType Advanced
Passed=101
Failed=10
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_non_contigiousvlan.py
   
* test_extendPhysicalNetworkVlan Failed
   
   * test_deploy_vm_iso.py
   
* test_deploy_vm_from_iso Failing since 70 runs
   
   * test_list_ids_parameter.py
   
* ContextSuite context=TestListIdsParams>:setup Failing since 46 runs
   
   * test_volumes.py
   
* test_06_download_detached_volume Failing since 2 runs
   
   * test_vm_life_cycle.py
   
* test_10_attachAndDetach_iso Failing since 70 runs
   
   * test_routers_network_ops.py
   
* test_01_isolate_network_FW_PF_default_routes_egress_true Failing since 
103 runs
   
* test_02_isolate_network_FW_PF_default_routes_egress_false Failing since 
103 runs
   
* test_01_RVR_Network_FW_PF_SSH_default_routes_egress_true Failing since 
101 runs
   
* test_02_RVR_Network_FW_PF_SSH_default_routes_egress_false Failing since 
101 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_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_login.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 #2177: CLOUDSTACK-9977: Enhance SSL protocol used by Console Proxy

[git]

blueorangutan commented on issue #2177: CLOUDSTACK-9977: Enhance SSL protocol 
used by Console Proxy
URL: https://github.com/apache/cloudstack/pull/2177#issuecomment-318220045
 
 
   Trillian test result (tid-1253)
   Environment: vmware-55u3 (x2), Advanced Networking with Mgmt server 7
   Total time taken: 50326 seconds
   Marvin logs: 
https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr2177-t1253-vmware-55u3.zip
   Intermitten failure detected: /marvin/tests/smoke/test_privategw_acl.py
   Intermitten failure detected: /marvin/tests/smoke/test_routers_network_ops.py
   Intermitten failure detected: /marvin/tests/smoke/test_vm_life_cycle.py
   Intermitten failure detected: /marvin/tests/smoke/test_volumes.py
   Intermitten failure detected: /marvin/tests/smoke/test_vpc_redundant.py
   Test completed. 48 look ok, 4 have error(s)
   
   
   Test | Result | Time (s) | Test File
   --- | --- | --- | ---
   test_02_redundant_VPC_default_routes | `Failure` | 616.16 | 
test_vpc_redundant.py
   test_01_create_volume | `Failure` | 193.76 | test_volumes.py
   test_04_rvpc_privategw_static_routes | `Failure` | 1012.18 | 
test_privategw_acl.py
   test_06_download_detached_volume | `Error` | 100.91 | test_volumes.py
   test_01_vpc_site2site_vpn | Success | 387.00 | test_vpc_vpn.py
   test_01_vpc_remote_access_vpn | Success | 268.11 | test_vpc_vpn.py
   test_01_redundant_vpc_site2site_vpn | Success | 790.00 | test_vpc_vpn.py
   test_02_VPC_default_routes | Success | 381.16 | test_vpc_router_nics.py
   test_01_VPC_nics_after_destroy | Success | 749.61 | test_vpc_router_nics.py
   test_05_rvpc_multi_tiers | Success | 542.75 | test_vpc_redundant.py
   test_04_rvpc_network_garbage_collector_nics | Success | 1588.62 | 
test_vpc_redundant.py
   test_03_create_redundant_VPC_1tier_2VMs_2IPs_2PF_ACL_reboot_routers | 
Success | 831.19 | test_vpc_redundant.py
   test_01_create_redundant_VPC_2tiers_4VMs_4IPs_4PF_ACL | Success | 1204.55 | 
test_vpc_redundant.py
   test_09_delete_detached_volume | Success | 26.04 | test_volumes.py
   test_05_detach_volume | Success | 110.36 | test_volumes.py
   test_04_delete_attached_volume | Success | 20.27 | test_volumes.py
   test_03_download_attached_volume | Success | 25.35 | test_volumes.py
   test_02_attach_volume | Success | 58.86 | test_volumes.py
   test_change_service_offering_for_vm_with_snapshots | Success | 483.15 | 
test_vm_snapshots.py
   test_03_delete_vm_snapshots | Success | 280.26 | test_vm_snapshots.py
   test_02_revert_vm_snapshots | Success | 234.51 | test_vm_snapshots.py
   test_01_create_vm_snapshots | Success | 162.06 | test_vm_snapshots.py
   test_deploy_vm_multiple | Success | 232.84 | test_vm_life_cycle.py
   test_deploy_vm | Success | 0.04 | test_vm_life_cycle.py
   test_advZoneVirtualRouter | Success | 0.03 | test_vm_life_cycle.py
   test_10_attachAndDetach_iso | Success | 27.00 | test_vm_life_cycle.py
   test_09_expunge_vm | Success | 125.28 | test_vm_life_cycle.py
   test_08_migrate_vm | Success | 66.27 | test_vm_life_cycle.py
   test_07_restore_vm | Success | 0.10 | test_vm_life_cycle.py
   test_06_destroy_vm | Success | 10.16 | test_vm_life_cycle.py
   test_03_reboot_vm | Success | 5.16 | test_vm_life_cycle.py
   test_02_start_vm | Success | 20.26 | test_vm_life_cycle.py
   test_01_stop_vm_forced | Success | 5.18 | test_vm_life_cycle.py
   test_01_stop_vm | Success | 10.17 | test_vm_life_cycle.py
   test_CreateTemplateWithDuplicateName | Success | 398.24 | test_templates.py
   test_08_list_system_templates | Success | 0.03 | test_templates.py
   test_07_list_public_templates | Success | 0.04 | test_templates.py
   test_05_template_permissions | Success | 0.06 | test_templates.py
   test_04_extract_template | Success | 15.29 | test_templates.py
   test_03_delete_template | Success | 5.12 | test_templates.py
   test_02_edit_template | Success | 90.18 | test_templates.py
   test_01_create_template | Success | 271.93 | test_templates.py
   test_10_destroy_cpvm | Success | 292.14 | test_ssvm.py
   test_09_destroy_ssvm | Success | 274.15 | test_ssvm.py
   test_08_reboot_cpvm | Success | 156.64 | test_ssvm.py
   test_07_reboot_ssvm | Success | 218.87 | test_ssvm.py
   test_06_stop_cpvm | Success | 202.14 | test_ssvm.py
   test_05_stop_ssvm | Success | 234.34 | test_ssvm.py
   test_04_cpvm_internals | Success | 1.33 | test_ssvm.py
   test_03_ssvm_internals | Success | 4.24 | test_ssvm.py
   test_02_list_cpvm_vm | Success | 0.14 | test_ssvm.py
   test_01_list_sec_storage_vm | Success | 0.14 | test_ssvm.py
   test_02_list_snapshots_with_removed_data_store | Success | 233.36 | 
test_snapshots.py
   test_01_snapshot_root_disk | Success | 71.62 | test_snapshots.py
   test_04_change_offering_small | Success | 98.09 | test_service_offerings.py
   test_03_delete_service_offering | Success | 0.04 | test_service_offerings.py
   test_02_edit_service_offering | Success | 0.10 | test_service_offerings.py
   test_01_create_service_offering | Success | 0.10 | 

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

[git]

cloudmonger 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-318210744
 
 
   ### ACS CI BVT Run
**Sumarry:**
Build Number 1014
Hypervisor xenserver
NetworkType Advanced
Passed=102
Failed=9
Skipped=12
   
   _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_deploy_vm_iso.py
   
* test_deploy_vm_from_iso Failing since 69 runs
   
   * test_list_ids_parameter.py
   
* ContextSuite context=TestListIdsParams>:setup Failing since 45 runs
   
   * test_volumes.py
   
* test_06_download_detached_volume Failed
   
   * test_vm_life_cycle.py
   
* test_10_attachAndDetach_iso Failing since 69 runs
   
   * test_routers_network_ops.py
   
* test_01_isolate_network_FW_PF_default_routes_egress_true Failing since 
102 runs
   
* test_02_isolate_network_FW_PF_default_routes_egress_false Failing since 
102 runs
   
* test_01_RVR_Network_FW_PF_SSH_default_routes_egress_true Failing since 
100 runs
   
* test_02_RVR_Network_FW_PF_SSH_default_routes_egress_false Failing since 
100 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_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_non_contigiousvlan.py
   test_login.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] cloudmonger commented on issue #2046: CLOUDSTACK-7958: Add configuration for limit to CIDRs for Admin API calls

[git]

cloudmonger 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-317584046
 
 
   ### ACS CI BVT Run
**Sumarry:**
Build Number 1006
Hypervisor xenserver
NetworkType Advanced
Passed=102
Failed=11
Skipped=12
   
   _Link to logs Folder (search by build_no):_ 
https://www.dropbox.com/sh/r2si930m8xxzavs/AAAzNrnoF1fC3auFrvsKo_8-a?dl=0
   
   **Failed tests:**
   * test_scale_vm.py
   
* ContextSuite context=TestScaleVm>:setup Failing since 33 runs
   
   * test_loadbalance.py
   
* test_01_create_lb_rule_src_nat Failed
   
* test_02_create_lb_rule_non_nat Failed
   
   * test_non_contigiousvlan.py
   
* test_extendPhysicalNetworkVlan Failed
   
   * test_deploy_vm_iso.py
   
* test_deploy_vm_from_iso Failing since 63 runs
   
   * test_volumes.py
   
* test_06_download_detached_volume Failing since 3 runs
   
   * test_vm_life_cycle.py
   
* test_10_attachAndDetach_iso Failing since 63 runs
   
   * test_routers_network_ops.py
   
* test_01_isolate_network_FW_PF_default_routes_egress_true Failing since 96 
runs
   
* test_02_isolate_network_FW_PF_default_routes_egress_false Failing since 
96 runs
   
* test_01_RVR_Network_FW_PF_SSH_default_routes_egress_true Failing since 94 
runs
   
* test_02_RVR_Network_FW_PF_SSH_default_routes_egress_false Failing since 
94 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_service_offerings.py
   test_routers_iptables_default_policy.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_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 #2190: CLOUDSTACK-10010: Fixed the negotiation of S2S VPN connections

[git]

blueorangutan commented on issue #2190: CLOUDSTACK-10010: Fixed the negotiation 
of S2S VPN connections
URL: https://github.com/apache/cloudstack/pull/2190#issuecomment-318196029
 
 
   @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 #2190: CLOUDSTACK-10010: Fixed the negotiation of S2S VPN connections

[git]

rhtyd commented on issue #2190: CLOUDSTACK-10010: Fixed the negotiation of S2S 
VPN connections
URL: https://github.com/apache/cloudstack/pull/2190#issuecomment-318195815
 
 
   @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 #2190: CLOUDSTACK-10010: Fixed the negotiation of S2S VPN connections

[git]

blueorangutan commented on issue #2190: CLOUDSTACK-10010: Fixed the negotiation 
of S2S VPN connections
URL: https://github.com/apache/cloudstack/pull/2190#issuecomment-318185096
 
 
   Packaging result: ?centos6 ?centos7 ?debian. JID-865
 

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 #2190: CLOUDSTACK-10010: Fixed the negotiation of S2S VPN connections

[git]

blueorangutan commented on issue #2190: CLOUDSTACK-10010: Fixed the negotiation 
of S2S VPN connections
URL: https://github.com/apache/cloudstack/pull/2190#issuecomment-318179849
 
 
   @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 #2190: CLOUDSTACK-10010: Fixed the negotiation of S2S VPN connections

[git]

rhtyd commented on issue #2190: CLOUDSTACK-10010: Fixed the negotiation of S2S 
VPN connections
URL: https://github.com/apache/cloudstack/pull/2190#issuecomment-318179778
 
 
   @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] swill commented on issue #2190: CLOUDSTACK-10010: Fixed the negotiation of S2S VPN connections

[git]

swill commented on issue #2190: CLOUDSTACK-10010: Fixed the negotiation of S2S 
VPN connections
URL: https://github.com/apache/cloudstack/pull/2190#issuecomment-318179550
 
 
   @borisstoyanov or @rhtyd please kick off the tests again.  I forgot that we 
have not pushed the marvin update to properly test projects yet.  That PR is 
coming, but for now I have change the test to use the standard `account` and 
`domain` style testing.  @syed will likely push the addition to properly test 
projects through marvin as a different PR at some point soon.
   
   Please kick off the tests again.  Thanks...
 

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


With regards,
Apache Git Services

[GitHub] blueorangutan commented on issue #2177: CLOUDSTACK-9977: Enhance SSL protocol used by Console Proxy

[git]

blueorangutan commented on issue #2177: CLOUDSTACK-9977: Enhance SSL protocol 
used by Console Proxy
URL: https://github.com/apache/cloudstack/pull/2177#issuecomment-318171273
 
 
   Trillian test result (tid-1252)
   Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
   Total time taken: 36665 seconds
   Marvin logs: 
https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr2177-t1252-kvm-centos7.zip
   Intermitten failure detected: 
/marvin/tests/smoke/test_deploy_virtio_scsi_vm.py
   Intermitten failure detected: /marvin/tests/smoke/test_privategw_acl.py
   Intermitten failure detected: /marvin/tests/smoke/test_vpc_redundant.py
   Test completed. 50 look ok, 1 have error(s)
   
   
   Test | Result | Time (s) | Test File
   --- | --- | --- | ---
   test_04_rvpc_privategw_static_routes | `Failure` | 349.06 | 
test_privategw_acl.py
   test_01_vpc_site2site_vpn | Success | 163.97 | test_vpc_vpn.py
   test_01_vpc_remote_access_vpn | Success | 65.75 | test_vpc_vpn.py
   test_01_redundant_vpc_site2site_vpn | Success | 234.25 | test_vpc_vpn.py
   test_02_VPC_default_routes | Success | 257.63 | test_vpc_router_nics.py
   test_01_VPC_nics_after_destroy | Success | 503.17 | test_vpc_router_nics.py
   test_05_rvpc_multi_tiers | Success | 497.26 | test_vpc_redundant.py
   test_04_rvpc_network_garbage_collector_nics | Success | 1396.08 | 
test_vpc_redundant.py
   test_03_create_redundant_VPC_1tier_2VMs_2IPs_2PF_ACL_reboot_routers | 
Success | 540.25 | test_vpc_redundant.py
   test_02_redundant_VPC_default_routes | Success | 740.29 | 
test_vpc_redundant.py
   test_01_create_redundant_VPC_2tiers_4VMs_4IPs_4PF_ACL | Success | 1309.39 | 
test_vpc_redundant.py
   test_09_delete_detached_volume | Success | 151.03 | test_volumes.py
   test_08_resize_volume | Success | 155.98 | test_volumes.py
   test_07_resize_fail | Success | 160.97 | test_volumes.py
   test_06_download_detached_volume | Success | 155.89 | test_volumes.py
   test_05_detach_volume | Success | 155.59 | test_volumes.py
   test_04_delete_attached_volume | Success | 150.80 | test_volumes.py
   test_03_download_attached_volume | Success | 155.92 | test_volumes.py
   test_02_attach_volume | Success | 90.90 | test_volumes.py
   test_01_create_volume | Success | 711.61 | test_volumes.py
   test_03_delete_vm_snapshots | Success | 275.13 | test_vm_snapshots.py
   test_02_revert_vm_snapshots | Success | 100.70 | test_vm_snapshots.py
   test_01_create_vm_snapshots | Success | 163.91 | test_vm_snapshots.py
   test_deploy_vm_multiple | Success | 271.98 | test_vm_life_cycle.py
   test_deploy_vm | Success | 0.02 | test_vm_life_cycle.py
   test_advZoneVirtualRouter | Success | 0.01 | test_vm_life_cycle.py
   test_10_attachAndDetach_iso | Success | 26.74 | test_vm_life_cycle.py
   test_09_expunge_vm | Success | 125.14 | test_vm_life_cycle.py
   test_08_migrate_vm | Success | 30.54 | test_vm_life_cycle.py
   test_07_restore_vm | Success | 0.08 | test_vm_life_cycle.py
   test_06_destroy_vm | Success | 125.62 | test_vm_life_cycle.py
   test_03_reboot_vm | Success | 130.62 | test_vm_life_cycle.py
   test_02_start_vm | Success | 10.11 | test_vm_life_cycle.py
   test_01_stop_vm_forced | Success | 5.09 | test_vm_life_cycle.py
   test_01_stop_vm | Success | 35.22 | test_vm_life_cycle.py
   test_CreateTemplateWithDuplicateName | Success | 100.59 | test_templates.py
   test_08_list_system_templates | Success | 0.03 | test_templates.py
   test_07_list_public_templates | Success | 0.03 | test_templates.py
   test_05_template_permissions | Success | 0.05 | test_templates.py
   test_04_extract_template | Success | 5.15 | test_templates.py
   test_03_delete_template | Success | 5.11 | test_templates.py
   test_02_edit_template | Success | 90.15 | test_templates.py
   test_01_create_template | Success | 130.69 | test_templates.py
   test_10_destroy_cpvm | Success | 191.38 | test_ssvm.py
   test_09_destroy_ssvm | Success | 163.14 | test_ssvm.py
   test_08_reboot_cpvm | Success | 131.62 | test_ssvm.py
   test_07_reboot_ssvm | Success | 138.53 | test_ssvm.py
   test_06_stop_cpvm | Success | 161.93 | test_ssvm.py
   test_05_stop_ssvm | Success | 164.71 | test_ssvm.py
   test_04_cpvm_internals | Success | 1.26 | test_ssvm.py
   test_03_ssvm_internals | Success | 3.79 | test_ssvm.py
   test_02_list_cpvm_vm | Success | 0.08 | test_ssvm.py
   test_01_list_sec_storage_vm | Success | 0.09 | test_ssvm.py
   test_02_list_snapshots_with_removed_data_store | Success | 81.51 | 
test_snapshots.py
   test_01_snapshot_root_disk | Success | 16.00 | test_snapshots.py
   test_04_change_offering_small | Success | 204.25 | test_service_offerings.py
   test_03_delete_service_offering | Success | 0.03 | test_service_offerings.py
   test_02_edit_service_offering | Success | 0.05 | test_service_offerings.py
   test_01_create_service_offering | Success | 0.08 | test_service_offerings.py
   test_02_sys_template_ready | Success | 0.07 | test_secondary_storage.py
   test_01_sys_vm_start 

[GitHub] blueorangutan commented on issue #2177: CLOUDSTACK-9977: Enhance SSL protocol used by Console Proxy

[git]

blueorangutan commented on issue #2177: CLOUDSTACK-9977: Enhance SSL protocol 
used by Console Proxy
URL: https://github.com/apache/cloudstack/pull/2177#issuecomment-318171223
 
 
   Trillian test result (tid-1251)
   Environment: xenserver-65sp1 (x2), Advanced Networking with Mgmt server 6
   Total time taken: 36981 seconds
   Marvin logs: 
https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr2177-t1251-xenserver-65sp1.zip
   Intermitten failure detected: /marvin/tests/smoke/test_privategw_acl.py
   Intermitten failure detected: /marvin/tests/smoke/test_routers_network_ops.py
   Intermitten failure detected: /marvin/tests/smoke/test_vpc_redundant.py
   Test completed. 49 look ok, 2 have error(s)
   
   
   Test | Result | Time (s) | Test File
   --- | --- | --- | ---
   test_05_rvpc_multi_tiers | `Failure` | 361.71 | test_vpc_redundant.py
   test_04_rvpc_network_garbage_collector_nics | `Failure` | 1274.56 | 
test_vpc_redundant.py
   test_01_create_redundant_VPC_2tiers_4VMs_4IPs_4PF_ACL | `Failure` | 368.72 | 
test_vpc_redundant.py
   test_04_rvpc_privategw_static_routes | `Failure` | 546.07 | 
test_privategw_acl.py
   test_01_vpc_site2site_vpn | Success | 244.93 | test_vpc_vpn.py
   test_01_vpc_remote_access_vpn | Success | 121.19 | test_vpc_vpn.py
   test_01_redundant_vpc_site2site_vpn | Success | 405.77 | test_vpc_vpn.py
   test_02_VPC_default_routes | Success | 248.42 | test_vpc_router_nics.py
   test_01_VPC_nics_after_destroy | Success | 555.20 | test_vpc_router_nics.py
   test_03_create_redundant_VPC_1tier_2VMs_2IPs_2PF_ACL_reboot_routers | 
Success | 691.83 | test_vpc_redundant.py
   test_02_redundant_VPC_default_routes | Success | 907.29 | 
test_vpc_redundant.py
   test_09_delete_detached_volume | Success | 15.53 | test_volumes.py
   test_08_resize_volume | Success | 75.69 | test_volumes.py
   test_07_resize_fail | Success | 90.76 | test_volumes.py
   test_06_download_detached_volume | Success | 20.32 | test_volumes.py
   test_05_detach_volume | Success | 100.27 | test_volumes.py
   test_04_delete_attached_volume | Success | 10.20 | test_volumes.py
   test_03_download_attached_volume | Success | 10.24 | test_volumes.py
   test_02_attach_volume | Success | 43.77 | test_volumes.py
   test_01_create_volume | Success | 392.27 | test_volumes.py
   test_change_service_offering_for_vm_with_snapshots | Success | 356.73 | 
test_vm_snapshots.py
   test_03_delete_vm_snapshots | Success | 280.24 | test_vm_snapshots.py
   test_02_revert_vm_snapshots | Success | 209.16 | test_vm_snapshots.py
   test_01_create_vm_snapshots | Success | 133.75 | test_vm_snapshots.py
   test_deploy_vm_multiple | Success | 176.83 | test_vm_life_cycle.py
   test_deploy_vm | Success | 0.02 | test_vm_life_cycle.py
   test_advZoneVirtualRouter | Success | 0.02 | test_vm_life_cycle.py
   test_10_attachAndDetach_iso | Success | 26.83 | test_vm_life_cycle.py
   test_09_expunge_vm | Success | 125.29 | test_vm_life_cycle.py
   test_08_migrate_vm | Success | 80.98 | test_vm_life_cycle.py
   test_07_restore_vm | Success | 0.08 | test_vm_life_cycle.py
   test_06_destroy_vm | Success | 15.19 | test_vm_life_cycle.py
   test_03_reboot_vm | Success | 15.18 | test_vm_life_cycle.py
   test_02_start_vm | Success | 15.20 | test_vm_life_cycle.py
   test_01_stop_vm_forced | Success | 5.13 | test_vm_life_cycle.py
   test_01_stop_vm | Success | 30.28 | test_vm_life_cycle.py
   test_CreateTemplateWithDuplicateName | Success | 80.67 | test_templates.py
   test_08_list_system_templates | Success | 0.03 | test_templates.py
   test_07_list_public_templates | Success | 0.03 | test_templates.py
   test_05_template_permissions | Success | 0.05 | test_templates.py
   test_04_extract_template | Success | 5.13 | test_templates.py
   test_03_delete_template | Success | 5.08 | test_templates.py
   test_02_edit_template | Success | 90.06 | test_templates.py
   test_01_create_template | Success | 40.39 | test_templates.py
   test_10_destroy_cpvm | Success | 191.60 | test_ssvm.py
   test_09_destroy_ssvm | Success | 168.75 | test_ssvm.py
   test_08_reboot_cpvm | Success | 136.59 | test_ssvm.py
   test_07_reboot_ssvm | Success | 138.59 | test_ssvm.py
   test_06_stop_cpvm | Success | 161.63 | test_ssvm.py
   test_05_stop_ssvm | Success | 133.57 | test_ssvm.py
   test_04_cpvm_internals | Success | 1.15 | test_ssvm.py
   test_03_ssvm_internals | Success | 3.51 | test_ssvm.py
   test_02_list_cpvm_vm | Success | 0.10 | test_ssvm.py
   test_01_list_sec_storage_vm | Success | 0.11 | test_ssvm.py
   test_02_list_snapshots_with_removed_data_store | Success | 109.18 | 
test_snapshots.py
   test_01_snapshot_root_disk | Success | 21.27 | test_snapshots.py
   test_04_change_offering_small | Success | 58.91 | test_service_offerings.py
   test_03_delete_service_offering | Success | 0.03 | test_service_offerings.py
   test_02_edit_service_offering | Success | 0.04 | test_service_offerings.py
   test_01_create_service_offering | Success | 0.07 | test_service_offerings.py
  

[GitHub] rhtyd commented on issue #2190: CLOUDSTACK-10010: Fixed the negotiation of S2S VPN connections

[git]

rhtyd commented on issue #2190: CLOUDSTACK-10010: Fixed the negotiation of S2S 
VPN connections
URL: https://github.com/apache/cloudstack/pull/2190#issuecomment-318169197
 
 
   @swill `test_01_vpc_site2site_vpn_multiple_options` has failed, please see
 

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 #2193: CLOUDSTACK-10007 Isolation methods

[git]

blueorangutan commented on issue #2193: CLOUDSTACK-10007 Isolation methods
URL: https://github.com/apache/cloudstack/pull/2193#issuecomment-318165296
 
 
   Trillian test result (tid-1250)
   Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
   Total time taken: 43383 seconds
   Marvin logs: 
https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr2193-t1250-kvm-centos7.zip
   Intermitten failure detected: /marvin/tests/smoke/test_privategw_acl.py
   Intermitten failure detected: /marvin/tests/smoke/test_routers_network_ops.py
   Intermitten failure detected: /marvin/tests/smoke/test_vpc_redundant.py
   Test completed. 50 look ok, 1 have error(s)
   
   
   Test | Result | Time (s) | Test File
   --- | --- | --- | ---
   test_04_rvpc_privategw_static_routes | `Failure` | 523.07 | 
test_privategw_acl.py
   test_01_vpc_site2site_vpn | Success | 205.41 | test_vpc_vpn.py
   test_01_vpc_remote_access_vpn | Success | 91.39 | test_vpc_vpn.py
   test_01_redundant_vpc_site2site_vpn | Success | 351.55 | test_vpc_vpn.py
   test_02_VPC_default_routes | Success | 286.99 | test_vpc_router_nics.py
   test_01_VPC_nics_after_destroy | Success | 663.72 | test_vpc_router_nics.py
   test_05_rvpc_multi_tiers | Success | 552.01 | test_vpc_redundant.py
   test_04_rvpc_network_garbage_collector_nics | Success | 1371.07 | 
test_vpc_redundant.py
   test_03_create_redundant_VPC_1tier_2VMs_2IPs_2PF_ACL_reboot_routers | 
Success | 653.93 | test_vpc_redundant.py
   test_02_redundant_VPC_default_routes | Success | 799.96 | 
test_vpc_redundant.py
   test_01_create_redundant_VPC_2tiers_4VMs_4IPs_4PF_ACL | Success | 1376.63 | 
test_vpc_redundant.py
   test_09_delete_detached_volume | Success | 156.77 | test_volumes.py
   test_08_resize_volume | Success | 156.59 | test_volumes.py
   test_07_resize_fail | Success | 161.78 | test_volumes.py
   test_06_download_detached_volume | Success | 156.47 | test_volumes.py
   test_05_detach_volume | Success | 150.87 | test_volumes.py
   test_04_delete_attached_volume | Success | 151.37 | test_volumes.py
   test_03_download_attached_volume | Success | 156.55 | test_volumes.py
   test_02_attach_volume | Success | 124.70 | test_volumes.py
   test_01_create_volume | Success | 719.25 | test_volumes.py
   test_03_delete_vm_snapshots | Success | 275.24 | test_vm_snapshots.py
   test_02_revert_vm_snapshots | Success | 100.99 | test_vm_snapshots.py
   test_01_create_vm_snapshots | Success | 174.52 | test_vm_snapshots.py
   test_deploy_vm_multiple | Success | 328.24 | test_vm_life_cycle.py
   test_deploy_vm | Success | 0.04 | test_vm_life_cycle.py
   test_advZoneVirtualRouter | Success | 0.03 | test_vm_life_cycle.py
   test_10_attachAndDetach_iso | Success | 28.74 | test_vm_life_cycle.py
   test_09_expunge_vm | Success | 125.26 | test_vm_life_cycle.py
   test_08_migrate_vm | Success | 30.97 | test_vm_life_cycle.py
   test_07_restore_vm | Success | 0.13 | test_vm_life_cycle.py
   test_06_destroy_vm | Success | 130.91 | test_vm_life_cycle.py
   test_03_reboot_vm | Success | 125.92 | test_vm_life_cycle.py
   test_02_start_vm | Success | 10.19 | test_vm_life_cycle.py
   test_01_stop_vm_forced | Success | 5.19 | test_vm_life_cycle.py
   test_01_stop_vm | Success | 40.36 | test_vm_life_cycle.py
   test_CreateTemplateWithDuplicateName | Success | 161.42 | test_templates.py
   test_08_list_system_templates | Success | 0.06 | test_templates.py
   test_07_list_public_templates | Success | 0.04 | test_templates.py
   test_05_template_permissions | Success | 0.08 | test_templates.py
   test_04_extract_template | Success | 5.30 | test_templates.py
   test_03_delete_template | Success | 5.12 | test_templates.py
   test_02_edit_template | Success | 90.18 | test_templates.py
   test_01_create_template | Success | 115.95 | test_templates.py
   test_10_destroy_cpvm | Success | 166.90 | test_ssvm.py
   test_09_destroy_ssvm | Success | 194.22 | test_ssvm.py
   test_08_reboot_cpvm | Success | 131.80 | test_ssvm.py
   test_07_reboot_ssvm | Success | 134.75 | test_ssvm.py
   test_06_stop_cpvm | Success | 137.02 | test_ssvm.py
   test_05_stop_ssvm | Success | 164.18 | test_ssvm.py
   test_04_cpvm_internals | Success | 1.45 | test_ssvm.py
   test_03_ssvm_internals | Success | 4.08 | test_ssvm.py
   test_02_list_cpvm_vm | Success | 0.13 | test_ssvm.py
   test_01_list_sec_storage_vm | Success | 0.17 | test_ssvm.py
   test_02_list_snapshots_with_removed_data_store | Success | 87.40 | 
test_snapshots.py
   test_01_snapshot_root_disk | Success | 16.37 | test_snapshots.py
   test_04_change_offering_small | Success | 240.17 | test_service_offerings.py
   test_03_delete_service_offering | Success | 0.05 | test_service_offerings.py
   test_02_edit_service_offering | Success | 0.06 | test_service_offerings.py
   test_01_create_service_offering | Success | 0.15 | test_service_offerings.py
   test_02_sys_template_ready | Success | 0.16 | test_secondary_storage.py
   test_01_sys_vm_start | Success | 0.24 | 

[GitHub] wido commented on issue #2028: CLOUDSTACK-9853: Add support for Secondary IPv6 Addresses and Subnets

[git]

wido commented on issue #2028: CLOUDSTACK-9853: Add support for Secondary IPv6 
Addresses and Subnets
URL: https://github.com/apache/cloudstack/pull/2028#issuecomment-318157801
 
 
   Test is fixed! Odd that it came through. I swear I fixed this earlier. 
Anyway, it's fixed now.
 

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 #2190: CLOUDSTACK-10010: Fixed the negotiation of S2S VPN connections

[git]

blueorangutan commented on issue #2190: CLOUDSTACK-10010: Fixed the negotiation 
of S2S VPN connections
URL: https://github.com/apache/cloudstack/pull/2190#issuecomment-318156501
 
 
   Trillian test result (tid-1255)
   Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
   Total time taken: 30689 seconds
   Marvin logs: 
https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr2190-t1255-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. 49 look ok, 2 have error(s)
   
   
   Test | Result | Time (s) | Test File
   --- | --- | --- | ---
   test_01_vpc_site2site_vpn_multiple_options | `Failure` | 91.00 | 
test_vpc_vpn.py
   test_04_rvpc_privategw_static_routes | `Failure` | 304.69 | 
test_privategw_acl.py
   test_01_vpc_site2site_vpn | Success | 159.16 | test_vpc_vpn.py
   test_01_vpc_remote_access_vpn | Success | 60.84 | test_vpc_vpn.py
   test_01_redundant_vpc_site2site_vpn | Success | 234.44 | test_vpc_vpn.py
   test_02_VPC_default_routes | Success | 243.14 | test_vpc_router_nics.py
   test_01_VPC_nics_after_destroy | Success | 557.00 | test_vpc_router_nics.py
   test_05_rvpc_multi_tiers | Success | 507.89 | test_vpc_redundant.py
   test_04_rvpc_network_garbage_collector_nics | Success | 1319.45 | 
test_vpc_redundant.py
   test_03_create_redundant_VPC_1tier_2VMs_2IPs_2PF_ACL_reboot_routers | 
Success | 540.52 | test_vpc_redundant.py
   test_02_redundant_VPC_default_routes | Success | 745.99 | 
test_vpc_redundant.py
   test_01_create_redundant_VPC_2tiers_4VMs_4IPs_4PF_ACL | Success | 1280.19 | 
test_vpc_redundant.py
   test_09_delete_detached_volume | Success | 156.22 | test_volumes.py
   test_08_resize_volume | Success | 156.06 | test_volumes.py
   test_07_resize_fail | Success | 156.09 | test_volumes.py
   test_06_download_detached_volume | Success | 156.00 | test_volumes.py
   test_05_detach_volume | Success | 145.57 | test_volumes.py
   test_04_delete_attached_volume | Success | 150.96 | test_volumes.py
   test_03_download_attached_volume | Success | 151.02 | test_volumes.py
   test_02_attach_volume | Success | 92.06 | test_volumes.py
   test_01_create_volume | Success | 621.35 | test_volumes.py
   test_03_delete_vm_snapshots | Success | 275.16 | test_vm_snapshots.py
   test_02_revert_vm_snapshots | Success | 95.78 | test_vm_snapshots.py
   test_01_create_vm_snapshots | Success | 133.81 | test_vm_snapshots.py
   test_deploy_vm_multiple | Success | 256.99 | test_vm_life_cycle.py
   test_deploy_vm | Success | 0.02 | test_vm_life_cycle.py
   test_advZoneVirtualRouter | Success | 0.02 | test_vm_life_cycle.py
   test_10_attachAndDetach_iso | Success | 26.60 | test_vm_life_cycle.py
   test_09_expunge_vm | Success | 125.24 | test_vm_life_cycle.py
   test_08_migrate_vm | Success | 30.73 | test_vm_life_cycle.py
   test_07_restore_vm | Success | 0.09 | test_vm_life_cycle.py
   test_06_destroy_vm | Success | 125.69 | test_vm_life_cycle.py
   test_03_reboot_vm | Success | 125.68 | test_vm_life_cycle.py
   test_02_start_vm | Success | 10.13 | test_vm_life_cycle.py
   test_01_stop_vm_forced | Success | 5.11 | test_vm_life_cycle.py
   test_01_stop_vm | Success | 35.26 | test_vm_life_cycle.py
   test_CreateTemplateWithDuplicateName | Success | 75.70 | test_templates.py
   test_08_list_system_templates | Success | 0.02 | test_templates.py
   test_07_list_public_templates | Success | 0.03 | test_templates.py
   test_05_template_permissions | Success | 0.05 | test_templates.py
   test_04_extract_template | Success | 5.11 | test_templates.py
   test_03_delete_template | Success | 5.10 | test_templates.py
   test_02_edit_template | Success | 90.15 | test_templates.py
   test_01_create_template | Success | 50.39 | test_templates.py
   test_10_destroy_cpvm | Success | 161.55 | test_ssvm.py
   test_09_destroy_ssvm | Success | 163.11 | test_ssvm.py
   test_08_reboot_cpvm | Success | 101.28 | test_ssvm.py
   test_07_reboot_ssvm | Success | 103.02 | test_ssvm.py
   test_06_stop_cpvm | Success | 101.52 | test_ssvm.py
   test_05_stop_ssvm | Success | 133.16 | test_ssvm.py
   test_04_cpvm_internals | Success | 0.97 | test_ssvm.py
   test_03_ssvm_internals | Success | 2.94 | test_ssvm.py
   test_02_list_cpvm_vm | Success | 0.13 | test_ssvm.py
   test_01_list_sec_storage_vm | Success | 0.11 | test_ssvm.py
   test_02_list_snapshots_with_removed_data_store | Success | 86.58 | 
test_snapshots.py
   test_01_snapshot_root_disk | Success | 10.99 | test_snapshots.py
   test_04_change_offering_small | Success | 209.37 | test_service_offerings.py
   test_03_delete_service_offering | Success | 0.03 | test_service_offerings.py
   test_02_edit_service_offering | Success | 0.04 | test_service_offerings.py
   test_01_create_service_offering | Success | 0.07 | test_service_offerings.py
   test_02_sys_template_ready | Success | 0.09 | test_secondary_storage.py
   test_01_sys_vm_start | 

[GitHub] blueorangutan commented on issue #2201: CLOUDSTACK-10016: VPC VR doesn't respond to DNS requests from remote access vpn clients

[git]

blueorangutan commented on issue #2201: CLOUDSTACK-10016: VPC VR doesn't 
respond to DNS requests from remote access vpn clients
URL: https://github.com/apache/cloudstack/pull/2201#issuecomment-318151900
 
 
   Trillian test result (tid-1254)
   Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
   Total time taken: 31394 seconds
   Marvin logs: 
https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr2201-t1254-kvm-centos7.zip
   Intermitten failure detected: /marvin/tests/smoke/test_privategw_acl.py
   Intermitten failure detected: /marvin/tests/smoke/test_snapshots.py
   Intermitten failure detected: /marvin/tests/smoke/test_templates.py
   Test completed. 46 look ok, 3 have error(s)
   
   
   Test | Result | Time (s) | Test File
   --- | --- | --- | ---
   test_04_rvpc_privategw_static_routes | `Failure` | 456.76 | 
test_privategw_acl.py
   test_04_extract_template | `Error` | 5.11 | test_templates.py
   test_03_delete_template | `Error` | 5.12 | test_templates.py
   test_02_list_snapshots_with_removed_data_store | `Error` | 0.04 | 
test_snapshots.py
   test_01_vpc_site2site_vpn | Success | 195.07 | test_vpc_vpn.py
   test_01_vpc_remote_access_vpn | Success | 81.31 | test_vpc_vpn.py
   test_01_redundant_vpc_site2site_vpn | Success | 285.80 | test_vpc_vpn.py
   test_02_VPC_default_routes | Success | 320.14 | test_vpc_router_nics.py
   test_01_VPC_nics_after_destroy | Success | 579.24 | test_vpc_router_nics.py
   test_05_rvpc_multi_tiers | Success | 540.68 | test_vpc_redundant.py
   test_04_rvpc_network_garbage_collector_nics | Success | 1345.63 | 
test_vpc_redundant.py
   test_03_create_redundant_VPC_1tier_2VMs_2IPs_2PF_ACL_reboot_routers | 
Success | 621.39 | test_vpc_redundant.py
   test_02_redundant_VPC_default_routes | Success | 783.96 | 
test_vpc_redundant.py
   test_01_create_redundant_VPC_2tiers_4VMs_4IPs_4PF_ACL | Success | 1320.02 | 
test_vpc_redundant.py
   test_09_delete_detached_volume | Success | 156.57 | test_volumes.py
   test_08_resize_volume | Success | 156.50 | test_volumes.py
   test_07_resize_fail | Success | 161.53 | test_volumes.py
   test_06_download_detached_volume | Success | 156.46 | test_volumes.py
   test_05_detach_volume | Success | 150.81 | test_volumes.py
   test_04_delete_attached_volume | Success | 151.23 | test_volumes.py
   test_03_download_attached_volume | Success | 156.36 | test_volumes.py
   test_02_attach_volume | Success | 89.39 | test_volumes.py
   test_01_create_volume | Success | 712.57 | test_volumes.py
   test_deploy_vm_multiple | Success | 343.37 | test_vm_life_cycle.py
   test_deploy_vm | Success | 0.03 | test_vm_life_cycle.py
   test_advZoneVirtualRouter | Success | 0.02 | test_vm_life_cycle.py
   test_10_attachAndDetach_iso | Success | 37.08 | test_vm_life_cycle.py
   test_09_expunge_vm | Success | 125.33 | test_vm_life_cycle.py
   test_08_migrate_vm | Success | 91.37 | test_vm_life_cycle.py
   test_07_restore_vm | Success | 0.13 | test_vm_life_cycle.py
   test_06_destroy_vm | Success | 130.90 | test_vm_life_cycle.py
   test_03_reboot_vm | Success | 125.90 | test_vm_life_cycle.py
   test_02_start_vm | Success | 10.17 | test_vm_life_cycle.py
   test_01_stop_vm_forced | Success | 5.14 | test_vm_life_cycle.py
   test_01_stop_vm | Success | 35.33 | test_vm_life_cycle.py
   test_CreateTemplateWithDuplicateName | Success | 146.13 | test_templates.py
   test_08_list_system_templates | Success | 0.03 | test_templates.py
   test_07_list_public_templates | Success | 0.04 | test_templates.py
   test_05_template_permissions | Success | 0.06 | test_templates.py
   test_02_edit_template | Success | 90.11 | test_templates.py
   test_01_create_template | Success | 50.52 | test_templates.py
   test_10_destroy_cpvm | Success | 191.73 | test_ssvm.py
   test_09_destroy_ssvm | Success | 164.79 | test_ssvm.py
   test_08_reboot_cpvm | Success | 131.61 | test_ssvm.py
   test_07_reboot_ssvm | Success | 134.11 | test_ssvm.py
   test_06_stop_cpvm | Success | 161.87 | test_ssvm.py
   test_05_stop_ssvm | Success | 134.01 | test_ssvm.py
   test_04_cpvm_internals | Success | 1.24 | test_ssvm.py
   test_03_ssvm_internals | Success | 3.58 | test_ssvm.py
   test_02_list_cpvm_vm | Success | 0.14 | test_ssvm.py
   test_01_list_sec_storage_vm | Success | 0.16 | test_ssvm.py
   test_01_snapshot_root_disk | Success | 16.20 | test_snapshots.py
   test_04_change_offering_small | Success | 239.87 | test_service_offerings.py
   test_03_delete_service_offering | Success | 0.04 | test_service_offerings.py
   test_02_edit_service_offering | Success | 0.07 | test_service_offerings.py
   test_01_create_service_offering | Success | 0.11 | test_service_offerings.py
   test_02_sys_template_ready | Success | 0.17 | test_secondary_storage.py
   test_01_sys_vm_start | Success | 0.22 | test_secondary_storage.py
   test_09_reboot_router | Success | 40.34 | test_routers.py
   test_08_start_router | Success | 30.29 | test_routers.py
   test_07_stop_router | Success | 10.16 | 

[GitHub] cloudmonger commented on issue #2193: CLOUDSTACK-10007 Isolation methods

[git]

cloudmonger commented on issue #2193: CLOUDSTACK-10007 Isolation methods
URL: https://github.com/apache/cloudstack/pull/2193#issuecomment-317176923
 
 
   ### ACS CI BVT Run
**Sumarry:**
Build Number 996
Hypervisor xenserver
NetworkType Advanced
Passed=105
Failed=8
Skipped=12
   
   _Link to logs Folder (search by build_no):_ 
https://www.dropbox.com/sh/r2si930m8xxzavs/AAAzNrnoF1fC3auFrvsKo_8-a?dl=0
   
   **Failed tests:**
   * test_deploy_vm_iso.py
   
* test_deploy_vm_from_iso Failing since 54 runs
   
   * test_volumes.py
   
* test_06_download_detached_volume Failed
   
   * test_vm_life_cycle.py
   
* test_10_attachAndDetach_iso Failing since 54 runs
   
   * test_routers_network_ops.py
   
* test_01_isolate_network_FW_PF_default_routes_egress_true Failing since 87 
runs
   
* test_02_isolate_network_FW_PF_default_routes_egress_false Failing since 
87 runs
   
* test_01_RVR_Network_FW_PF_SSH_default_routes_egress_true Failing since 85 
runs
   
* test_02_RVR_Network_FW_PF_SSH_default_routes_egress_false Failing since 
85 runs
   
* test_03_RVR_Network_check_router_state Failing since 2 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_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_non_contigiousvlan.py
   test_login.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] cloudmonger commented on issue #2193: CLOUDSTACK-10007 Isolation methods

[git]

cloudmonger commented on issue #2193: CLOUDSTACK-10007 Isolation methods
URL: https://github.com/apache/cloudstack/pull/2193#issuecomment-318120916
 
 
   ### ACS CI BVT Run
**Sumarry:**
Build Number 1013
Hypervisor xenserver
NetworkType Advanced
Passed=107
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_deploy_vm_iso.py
   
* test_deploy_vm_from_iso Failing since 68 runs
   
   * test_vm_life_cycle.py
   
* test_10_attachAndDetach_iso Failing since 68 runs
   
   * test_routers_network_ops.py
   
* test_01_isolate_network_FW_PF_default_routes_egress_true Failing since 
101 runs
   
* test_02_isolate_network_FW_PF_default_routes_egress_false Failing since 
101 runs
   
* test_01_RVR_Network_FW_PF_SSH_default_routes_egress_true Failing since 99 
runs
   
* test_02_RVR_Network_FW_PF_SSH_default_routes_egress_false Failing since 
99 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_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_non_contigiousvlan.py
   test_login.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] wido commented on issue #2028: CLOUDSTACK-9853: Add support for Secondary IPv6 Addresses and Subnets

[git]

wido commented on issue #2028: CLOUDSTACK-9853: Add support for Secondary IPv6 
Addresses and Subnets
URL: https://github.com/apache/cloudstack/pull/2028#issuecomment-318085384
 
 
   I see, looking at this one:
   
   Tests run: 3, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 0.013 
sec <<< FAILURE! - in com.cloud.agent.api.SecurityGroupRulesCmdTest
   testCompressStringifiedRules(com.cloud.agent.api.SecurityGroupRulesCmdTest)  
Time elapsed: 0.01 sec  <<< FAILURE!
   java.lang.AssertionError: null
at org.junit.Assert.fail(Assert.java:86)
at org.junit.Assert.assertTrue(Assert.java:41)
at org.junit.Assert.assertTrue(Assert.java:52)
at 
com.cloud.agent.api.SecurityGroupRulesCmdTest.testCompressStringifiedRules(SecurityGroupRulesCmdTest.java:91)
 

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 opened a new pull request #2204: Http handler

[git]

sachinnitw1317 opened a new pull request #2204: Http handler
URL: https://github.com/apache/cloudstack/pull/2204
 
 
   Work Done So far : 
   - noVNC integrated and tested for KVM environment.
   - shifted entire consoleProxy server to jetty server on port 80.
   - added SSL support not tested currently.
 

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] branch master updated (dbf11d5 -> be8ae98)

[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.git.


from dbf11d5  API: display zone capacity data only to admin
 add 03aca88  CLOUDSTACK-8186: make setRemoved to null possible
 add c1f30d9  Merge branch '4.9' into 4.10
 new be8ae98  Merge branch '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:
 engine/schema/src/com/cloud/storage/dao/VMTemplateDaoImpl.java | 2 +-
 framework/db/pom.xml   | 2 +-
 framework/db/src/com/cloud/utils/db/GenericDaoBase.java| 3 ++-
 3 files changed, 4 insertions(+), 3 deletions(-)

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

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

[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.git

commit be8ae9822dcbf55132412191950cd95e07a8eb8d
Merge: dbf11d5 c1f30d9
Author: Rohit Yadav 
AuthorDate: Wed Jul 26 15:19:13 2017 +0200

Merge branch '4.10'

 engine/schema/src/com/cloud/storage/dao/VMTemplateDaoImpl.java | 2 +-
 framework/db/pom.xml   | 2 +-
 framework/db/src/com/cloud/utils/db/GenericDaoBase.java| 3 ++-
 3 files changed, 4 insertions(+), 3 deletions(-)


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

[cloudstack] branch 4.10 updated (83e9de4 -> c1f30d9)

[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.git.


from 83e9de4  Merge branch '4.9' into 4.10
 add 03aca88  CLOUDSTACK-8186: make setRemoved to null possible
 new c1f30d9  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:
 engine/schema/src/com/cloud/storage/dao/VMTemplateDaoImpl.java | 2 +-
 framework/db/pom.xml   | 2 +-
 framework/db/src/com/cloud/utils/db/GenericDaoBase.java| 3 ++-
 3 files changed, 4 insertions(+), 3 deletions(-)

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

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

[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.git

commit c1f30d954be6741862c533c47f2980dce62e4c2a
Merge: 83e9de4 03aca88
Author: Rohit Yadav 
AuthorDate: Wed Jul 26 15:18:42 2017 +0200

Merge branch '4.9' into 4.10

 engine/schema/src/com/cloud/storage/dao/VMTemplateDaoImpl.java | 2 +-
 framework/db/pom.xml   | 2 +-
 framework/db/src/com/cloud/utils/db/GenericDaoBase.java| 3 ++-
 3 files changed, 4 insertions(+), 3 deletions(-)


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

[GitHub] rhtyd closed pull request #2087: [CLOUDSTACK-9910] API: display zone capacity data only to admin

[git]

rhtyd closed pull request #2087: [CLOUDSTACK-9910] API: display zone capacity 
data only to admin
URL: https://github.com/apache/cloudstack/pull/2087
 
 
   
 

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] branch master updated: API: display zone capacity data only to admin

[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.git


The following commit(s) were added to refs/heads/master by this push:
 new dbf11d5  API: display zone capacity data only to admin
dbf11d5 is described below

commit dbf11d5e2d95851d7c581b1a7d3430a344d3d518
Author: Marc-Aurèle Brothier 
AuthorDate: Mon May 8 09:55:03 2017 +0200

API: display zone capacity data only to admin
---
 server/src/com/cloud/api/query/dao/DataCenterJoinDaoImpl.java | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/server/src/com/cloud/api/query/dao/DataCenterJoinDaoImpl.java 
b/server/src/com/cloud/api/query/dao/DataCenterJoinDaoImpl.java
index 97a7497..ffc7530 100644
--- a/server/src/com/cloud/api/query/dao/DataCenterJoinDaoImpl.java
+++ b/server/src/com/cloud/api/query/dao/DataCenterJoinDaoImpl.java
@@ -76,10 +76,10 @@ public class DataCenterJoinDaoImpl extends 
GenericDaoBase'].

[GitHub] rhtyd commented on issue #2039: rbd: Use libvirt to create new volumes and not rados-java

[git]

rhtyd commented on issue #2039: rbd: Use libvirt to create new volumes and not 
rados-java
URL: https://github.com/apache/cloudstack/pull/2039#issuecomment-318049818
 
 
   Alright, with that LGTM. @kiwiflyer do you have any comment/feedback to 
share?
 

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 #2039: rbd: Use libvirt to create new volumes and not rados-java

[git]

wido commented on issue #2039: rbd: Use libvirt to create new volumes and not 
rados-java
URL: https://github.com/apache/cloudstack/pull/2039#issuecomment-318048191
 
 
   CentOS 7 has libvirt 2.0.0 and therefor is no problem.
   
   CentOS 6 has an ancient version, but Ceph doesn't support CentOS 6 anymore.
 

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

[git]

wido 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-318047626
 
 
   Aha @rhtyd!
   
   I've fixed 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] rhtyd commented on issue #2052: [4.9] Backported fixes for 4.9.3.0 release effort

[git]

rhtyd commented on issue #2052: [4.9] Backported fixes for 4.9.3.0 release 
effort
URL: https://github.com/apache/cloudstack/pull/2052#issuecomment-318045595
 
 
   Test results look okay to me, however most errors on XenServer are related 
to r/vpc based tests and may be ignored.
 

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] branch 4.9 updated: CLOUDSTACK-8186: make setRemoved to null possible

[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.git


The following commit(s) were added to refs/heads/4.9 by this push:
 new 03aca88  CLOUDSTACK-8186: make setRemoved to null possible
03aca88 is described below

commit 03aca885ae9ce53232d0ba3759d14737ba40f288
Author: Wei Zhou 
AuthorDate: Fri Apr 14 22:41:17 2017 +0800

CLOUDSTACK-8186: make setRemoved to null possible
---
 engine/schema/src/com/cloud/storage/dao/VMTemplateDaoImpl.java | 2 +-
 framework/db/pom.xml   | 2 +-
 framework/db/src/com/cloud/utils/db/GenericDaoBase.java| 3 ++-
 3 files changed, 4 insertions(+), 3 deletions(-)

diff --git a/engine/schema/src/com/cloud/storage/dao/VMTemplateDaoImpl.java 
b/engine/schema/src/com/cloud/storage/dao/VMTemplateDaoImpl.java
index 58922fe..2037b3d 100644
--- a/engine/schema/src/com/cloud/storage/dao/VMTemplateDaoImpl.java
+++ b/engine/schema/src/com/cloud/storage/dao/VMTemplateDaoImpl.java
@@ -771,7 +771,7 @@ public class VMTemplateDaoImpl extends 
GenericDaoBase implem
 tmpltZoneVO = new VMTemplateZoneVO(zoneId, tmplt.getId(), new 
Date());
 _templateZoneDao.persist(tmpltZoneVO);
 } else {
-tmpltZoneVO.setRemoved(null);
+tmpltZoneVO.setRemoved(GenericDaoBase.DATE_TO_NULL);
 tmpltZoneVO.setLastUpdated(new Date());
 _templateZoneDao.update(tmpltZoneVO.getId(), tmpltZoneVO);
 }
diff --git a/framework/db/pom.xml b/framework/db/pom.xml
index 421ca5f..137e479 100644
--- a/framework/db/pom.xml
+++ b/framework/db/pom.xml
@@ -11,7 +11,7 @@
   xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 
http://maven.apache.org/xsd/maven-4.0.0.xsd;>
   4.0.0
   cloud-framework-db
-  Apache CloudStack Framework - Event Notification
+  Apache CloudStack Framework - Database
   
 org.apache.cloudstack
 cloudstack-framework
diff --git a/framework/db/src/com/cloud/utils/db/GenericDaoBase.java 
b/framework/db/src/com/cloud/utils/db/GenericDaoBase.java
index f1b8d0d..c5a4cd8 100644
--- a/framework/db/src/com/cloud/utils/db/GenericDaoBase.java
+++ b/framework/db/src/com/cloud/utils/db/GenericDaoBase.java
@@ -158,6 +158,7 @@ public abstract class GenericDaoBase extends Compone
 protected static final String FOR_UPDATE_CLAUSE = " FOR UPDATE ";
 protected static final String SHARE_MODE_CLAUSE = " LOCK IN SHARE MODE";
 protected static final String SELECT_LAST_INSERT_ID_SQL = "SELECT 
LAST_INSERT_ID()";
+public static final Date DATE_TO_NULL = new Date(Long.MIN_VALUE);
 
 protected static final SequenceFetcher s_seqFetcher = 
SequenceFetcher.getInstance();
 
@@ -1537,7 +1538,7 @@ public abstract class GenericDaoBase extends Compone
 }
 } else if (attr.field.getType() == Date.class) {
 final Date date = (Date)value;
-if (date == null) {
+if (date == null || date.equals(DATE_TO_NULL)) {
 pstmt.setObject(j, null);
 return;
 }

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

[GitHub] rhtyd closed pull request #2047: CLOUDSTACK-8186: make setRemoved to null possible

[git]

rhtyd closed pull request #2047: CLOUDSTACK-8186: make setRemoved to null 
possible
URL: https://github.com/apache/cloudstack/pull/2047
 
 
   
 

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 #2039: rbd: Use libvirt to create new volumes and not rados-java

[git]

rhtyd commented on issue #2039: rbd: Use libvirt to create new volumes and not 
rados-java
URL: https://github.com/apache/cloudstack/pull/2039#issuecomment-318044628
 
 
   LGTM, I'm not aware of Ceph/RBD + ACS users using CentOS6/7 based hosts so 
this can be accepted. However, for this change an additional review is asked.
 

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] konstantintrushin opened a new pull request #2203: All private, public, and guest interfaceses are marked as untagged ca?

[git]

konstantintrushin opened a new pull request #2203: All private, public, and 
guest interfaceses are marked as untagged ca?
URL: https://github.com/apache/cloudstack/pull/2203
 
 
   ?using constant Vif creation in Virtual Router VM
   
   When adding static nat eth interfaces are being added constantly to Virtual 
Route VM till they reach PCI slot limit. This is caused by public, prrivate and 
guest interfaces being marked as untagged and in HashMap values are overwritten 
and cleanupNetworkElementCommand dosn't know nic number in VR VM and constantly 
adds eth interfaces.
   
   2017-07-25 10:47:21,054 ERROR [kvm.resource.LibvirtComputingResource] 
(agentRequest-Handler-2:null) (logid:57628c13) Exexutiong 
cleanupNetworkElementCommand
   2017-07-25 10:47:21,054 DEBUG [kvm.resource.LibvirtConnection] 
(agentRequest-Handler-2:null) (logid:57628c13) Looking for libvirtd connection 
at: qemu:///system
   2017-07-25 10:47:21,057 ERROR [kvm.resource.LibvirtComputingResource] 
(agentRequest-Handler-2:null) (logid:57628c13) HASH MAP DUMP: 
{vlan://untagged=0}
   2017-07-25 10:47:21,057 ERROR [kvm.resource.LibvirtComputingResource] 
(agentRequest-Handler-2:null) (logid:57628c13) HASH MAP DUMP: {LinkLocal=1, 
vlan://untagged=0}
   2017-07-25 10:47:21,057 ERROR [kvm.resource.LibvirtComputingResource] 
(agentRequest-Handler-2:null) (logid:57628c13) HASH MAP DUMP: {LinkLocal=1, 
vlan://untagged=2}
   2017-07-25 10:47:21,057 ERROR [kvm.resource.LibvirtComputingResource] 
(agentRequest-Handler-2:null) (logid:57628c13) HASH MAP DUMP: {LinkLocal=1, 
vlan://untagged=3}
   2017-07-25 10:47:21,058 ERROR [kvm.resource.LibvirtComputingResource] 
(agentRequest-Handler-2:null) (logid:57628c13) HASH MAP DUMP: {LinkLocal=1, 
vlan://untagged=4}
   2017-07-25 10:47:21,058 ERROR [kvm.resource.LibvirtComputingResource] 
(agentRequest-Handler-2:null) (logid:57628c13) NIC pos 5
   2017-07-25 10:47:21,058 ERROR [kvm.resource.LibvirtComputingResource] 
(agentRequest-Handler-2:null) (logid:57628c13) HASH MAP DUMP: {LinkLocal=1, 
vlan://untagged=4}
   2017-07-25 10:47:21,058 ERROR [kvm.resource.LibvirtComputingResource] 
(agentRequest-Handler-2:null) (logid:57628c13) Lokking for BroadcastUri 
vlan://261
   2017-07-25 10:47:21,058 ERROR [kvm.resource.LibvirtComputingResource] 
(agentRequest-Handler-2:null) (logid:57628c13) Plugging in VIF with MAC 
1e:00:07:00:02:c8
 

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 #1730: VMTemplateZone needs some love

[git]

cloudmonger commented on issue #1730: VMTemplateZone needs some love
URL: https://github.com/apache/cloudstack/pull/1730#issuecomment-318030875
 
 
   ### ACS CI BVT Run
**Sumarry:**
Build Number 1012
Hypervisor xenserver
NetworkType Advanced
Passed=103
Failed=8
Skipped=12
   
   _Link to logs Folder (search by build_no):_ 
https://www.dropbox.com/sh/r2si930m8xxzavs/AAAzNrnoF1fC3auFrvsKo_8-a?dl=0
   
   **Failed tests:**
   * test_deploy_vms_with_varied_deploymentplanners.py
   
* test_deployvm_firstfit Failed
   
   * test_deploy_vm_iso.py
   
* test_deploy_vm_from_iso Failing since 67 runs
   
   * test_list_ids_parameter.py
   
* ContextSuite context=TestListIdsParams>:setup Failing since 44 runs
   
   * test_vm_life_cycle.py
   
* test_10_attachAndDetach_iso Failing since 67 runs
   
   * test_routers_network_ops.py
   
* test_01_isolate_network_FW_PF_default_routes_egress_true Failing since 
100 runs
   
* test_02_isolate_network_FW_PF_default_routes_egress_false Failing since 
100 runs
   
* test_01_RVR_Network_FW_PF_SSH_default_routes_egress_true Failing since 98 
runs
   
* test_02_RVR_Network_FW_PF_SSH_default_routes_egress_false Failing since 
98 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_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_network.py
   test_router_dns.py
   test_non_contigiousvlan.py
   test_login.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] rhtyd commented on issue #1933: CLOUDSTACK-9569: add router.aggregation.command.each.timeout to agent?

[git]

rhtyd commented on issue #1933: CLOUDSTACK-9569: add 
router.aggregation.command.each.timeout to agent?
URL: https://github.com/apache/cloudstack/pull/1933#issuecomment-318028958
 
 
   @borisstoyanov given Wei's fix, and propagation, @ustcweizhou can comment, 
what do you think Wei?
   I had another look LGTM.
 

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 #1933: CLOUDSTACK-9569: add router.aggregation.command.each.timeout to agent?

[git]

rhtyd commented on a change in pull request #1933: CLOUDSTACK-9569: add 
router.aggregation.command.each.timeout to agent?
URL: https://github.com/apache/cloudstack/pull/1933#discussion_r129548829
 
 

 ##
 File path: 
core/src/com/cloud/agent/resource/virtualnetwork/VirtualRoutingResource.java
 ##
 @@ -272,7 +272,10 @@ public boolean configure(final String name, final 
Map params) th
 _port = NumbersUtil.parseInt(value, 3922);
 
 value = (String)params.get("router.aggregation.command.each.timeout");
-_eachTimeout = Duration.standardSeconds(NumbersUtil.parseInt(value, 
10));
+_eachTimeout = Duration.standardSeconds(NumbersUtil.parseInt(value, 
(int)VRScripts.VR_SCRIPT_EXEC_TIMEOUT.getStandardSeconds()));
 
 Review comment:
   @borisstoyanov however this fix makes sense to use the default value when 
not parse-able.
 

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 #1933: CLOUDSTACK-9569: add router.aggregation.command.each.timeout to agent?

[git]

rhtyd commented on issue #1933: CLOUDSTACK-9569: add 
router.aggregation.command.each.timeout to agent?
URL: https://github.com/apache/cloudstack/pull/1933#issuecomment-318028958
 
 
   @borisstoyanov given Wei's fix, and propagation, this may be unnecessary but 
@ustcweizhou can comment, what do you think Wei?
 

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 #1933: CLOUDSTACK-9569: add router.aggregation.command.each.timeout to agent?

[git]

borisstoyanov commented on issue #1933: CLOUDSTACK-9569: add 
router.aggregation.command.each.timeout to agent?
URL: https://github.com/apache/cloudstack/pull/1933#issuecomment-318025089
 
 
   I think they are both addressing the same thing although adding changes in 
different files. Maybe needs to be reviewed by another dev @rhtyd 
 

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


With regards,
Apache Git Services

[GitHub] rhtyd commented on issue #1933: CLOUDSTACK-9569: add router.aggregation.command.each.timeout to agent?

[git]

rhtyd commented on issue #1933: CLOUDSTACK-9569: add 
router.aggregation.command.each.timeout to agent?
URL: https://github.com/apache/cloudstack/pull/1933#issuecomment-318023014
 
 
   @borisstoyanov is this good to merge or does Wei's fix in #1856 good enough?
 

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 #2190: CLOUDSTACK-10010: Fixed the negotiation of S2S VPN connections

[git]

blueorangutan commented on issue #2190: CLOUDSTACK-10010: Fixed the negotiation 
of S2S VPN connections
URL: https://github.com/apache/cloudstack/pull/2190#issuecomment-318019639
 
 
   @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 #2190: CLOUDSTACK-10010: Fixed the negotiation of S2S VPN connections

[git]

borisstoyanov commented on issue #2190: CLOUDSTACK-10010: Fixed the negotiation 
of S2S VPN connections
URL: https://github.com/apache/cloudstack/pull/2190#issuecomment-318019575
 
 
   @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 #2190: CLOUDSTACK-10010: Fixed the negotiation of S2S VPN connections

[git]

blueorangutan commented on issue #2190: CLOUDSTACK-10010: Fixed the negotiation 
of S2S VPN connections
URL: https://github.com/apache/cloudstack/pull/2190#issuecomment-318018357
 
 
   Packaging result: ?centos6 ?centos7 ?debian. JID-864
 

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

[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-318015760
 
 
   @wido make sure you're not breaking noredist builds as well, see Travis and 
try to get it green;
   
   ```
   [INFO] --- maven-compiler-plugin:3.2:compile 
(default-compile) @ cloud-plugin-hypervisor-vmware ---
   [INFO] Changes detected - recompiling the module!
   [INFO] Compiling 51 source files to 
/home/travis/build/apache/cloudstack/plugins/hypervisors/vmware/target/classes
   [INFO] 
-
   [ERROR] COMPILATION ERROR : 
   [INFO] 
-
   [ERROR] 
/home/travis/build/apache/cloudstack/plugins/hypervisors/vmware/src/com/cloud/hypervisor/guru/VMwareGuru.java:[534,35]
 error: cannot find symbol
   [INFO] 1 error
   ```
 

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 #2182: CLOUDSTACK-10000: FIX remote access vpn does not work due to '#' in password

[git]

rhtyd commented on issue #2182: CLOUDSTACK-1: FIX remote access vpn does 
not work due to '#' in password
URL: https://github.com/apache/cloudstack/pull/2182#issuecomment-318015435
 
 
   LGTM, however can you fix test failure (see Travis) in 
`test_01_VPN_user_limit` @ustcweizhou ?
 

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 #1707: CLOUDSTACK-9397: Add Watchdog timer to KVM Instance

[git]

rhtyd commented on a change in pull request #1707: CLOUDSTACK-9397: Add 
Watchdog timer to KVM Instance
URL: https://github.com/apache/cloudstack/pull/1707#discussion_r129536226
 
 

 ##
 File path: 
plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtComputingResource.java
 ##
 @@ -2051,6 +2066,8 @@ So if getMinSpeed() returns null we fall back to 
getSpeed().
 devices.addDevice(new ChannelDef(_qemuGuestAgentSocketName, 
ChannelDef.ChannelType.UNIX,
   new File(_qemuSocketsPath + "/" + vmTO.getName() + 
"." + _qemuGuestAgentSocketName)));
 
+devices.addDevice(new WatchDogDef(_watchDogAction, _watchDogModel));
 
 Review comment:
   @wido the default action `NONE` is acceptable as adding a watchdog device 
that might possible reset the instance is not acceptable for everyone. Also, 
can there be a case where the default watchdog model `I6300ESB` is not 
available, in which case can we initialize this as null and add the device only 
when _watchDogModel is not null and read options from agent.properties file /cc 
@DaanHoogland @nvazquez @ustcweizhou @remibergsma what do you think?
 

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 #2190: CLOUDSTACK-10010: Fixed the negotiation of S2S VPN connections

[git]

blueorangutan commented on issue #2190: CLOUDSTACK-10010: Fixed the negotiation 
of S2S VPN connections
URL: https://github.com/apache/cloudstack/pull/2190#issuecomment-318013832
 
 
   @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 #2190: CLOUDSTACK-10010: Fixed the negotiation of S2S VPN connections

[git]

rhtyd commented on issue #2190: CLOUDSTACK-10010: Fixed the negotiation of S2S 
VPN connections
URL: https://github.com/apache/cloudstack/pull/2190#issuecomment-318013625
 
 
   @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 #2201: CLOUDSTACK-10016: VPC VR doesn't respond to DNS requests from remote access vpn clients

[git]

blueorangutan commented on issue #2201: CLOUDSTACK-10016: VPC VR doesn't 
respond to DNS requests from remote access vpn clients
URL: https://github.com/apache/cloudstack/pull/2201#issuecomment-318013431
 
 
   @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 #2201: CLOUDSTACK-10016: VPC VR doesn't respond to DNS requests from remote access vpn clients

[git]

rhtyd commented on issue #2201: CLOUDSTACK-10016: VPC VR doesn't respond to DNS 
requests from remote access vpn clients
URL: https://github.com/apache/cloudstack/pull/2201#issuecomment-318013329
 
 
   @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 #2028: CLOUDSTACK-9853: Add support for Secondary IPv6 Addresses and Subnets

[git]

rhtyd commented on issue #2028: CLOUDSTACK-9853: Add support for Secondary IPv6 
Addresses and Subnets
URL: https://github.com/apache/cloudstack/pull/2028#issuecomment-318013201
 
 
   @wido this one is facing build failures, please see.
 

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 #2201: CLOUDSTACK-10016: VPC VR doesn't respond to DNS requests from remote access vpn clients

[git]

blueorangutan commented on issue #2201: CLOUDSTACK-10016: VPC VR doesn't 
respond to DNS requests from remote access vpn clients
URL: https://github.com/apache/cloudstack/pull/2201#issuecomment-318012319
 
 
   Packaging result: ?centos6 ?centos7 ?debian. JID-863
 

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 #2177: CLOUDSTACK-9977: Enhance SSL protocol used by Console Proxy

[git]

blueorangutan commented on issue #2177: CLOUDSTACK-9977: Enhance SSL protocol 
used by Console Proxy
URL: https://github.com/apache/cloudstack/pull/2177#issuecomment-318010048
 
 
   @rhtyd a Trillian-Jenkins matrix job (centos6 mgmt + xs65sp1, centos7 mgmt + 
vmware55u3, centos7 mgmt + kvmcentos7) 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 #2177: CLOUDSTACK-9977: Enhance SSL protocol used by Console Proxy

[git]

rhtyd commented on issue #2177: CLOUDSTACK-9977: Enhance SSL protocol used by 
Console Proxy
URL: https://github.com/apache/cloudstack/pull/2177#issuecomment-318009973
 
 
   @blueorangutan test 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] rhtyd commented on issue #2201: CLOUDSTACK-10016: VPC VR doesn't respond to DNS requests from remote access vpn clients

[git]

rhtyd commented on issue #2201: CLOUDSTACK-10016: VPC VR doesn't respond to DNS 
requests from remote access vpn clients
URL: https://github.com/apache/cloudstack/pull/2201#issuecomment-318007178
 
 
   @Slair1 thanks, please put JIRA id in the git commit message
   @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 #2201: CLOUDSTACK-10016: VPC VR doesn't respond to DNS requests from remote access vpn clients

[git]

blueorangutan commented on issue #2201: CLOUDSTACK-10016: VPC VR doesn't 
respond to DNS requests from remote access vpn clients
URL: https://github.com/apache/cloudstack/pull/2201#issuecomment-318007402
 
 
   @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 a change in pull request #2193: CLOUDSTACK-10007 Isolation methods

[git]

rhtyd commented on a change in pull request #2193: CLOUDSTACK-10007 Isolation 
methods
URL: https://github.com/apache/cloudstack/pull/2193#discussion_r129527430
 
 

 ##
 File path: 
plugins/network-elements/bigswitch-vns/src/com/cloud/network/guru/BigSwitchVnsGuestNetworkGuru.java
 ##
 @@ -0,0 +1,238 @@
+/*
+ * 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.
+ */
+
+package com.cloud.network.guru;
+
+import com.cloud.agent.AgentManager;
+import com.cloud.agent.api.CreateVnsNetworkAnswer;
+import com.cloud.agent.api.CreateVnsNetworkCommand;
+import com.cloud.agent.api.DeleteVnsNetworkCommand;
+import com.cloud.dc.DataCenter;
+import com.cloud.dc.DataCenter.NetworkType;
+import com.cloud.dc.dao.DataCenterDao;
+import com.cloud.deploy.DeployDestination;
+import com.cloud.deploy.DeploymentPlan;
+import com.cloud.exception.InsufficientAddressCapacityException;
+import com.cloud.exception.InsufficientVirtualNetworkCapacityException;
+import com.cloud.host.HostVO;
+import com.cloud.host.dao.HostDao;
+import com.cloud.host.dao.HostDetailsDao;
+import com.cloud.network.BigSwitchVnsDeviceVO;
+import com.cloud.network.Network;
+import com.cloud.network.Network.GuestType;
+import com.cloud.network.Network.State;
+import com.cloud.network.NetworkProfile;
+import com.cloud.network.Networks.BroadcastDomainType;
+import com.cloud.network.PhysicalNetwork;
+import com.cloud.network.PhysicalNetwork.IsolationMethod;
+import com.cloud.network.dao.BigSwitchVnsDao;
+import com.cloud.network.dao.NetworkVO;
+import com.cloud.network.dao.PhysicalNetworkDao;
+import com.cloud.network.dao.PhysicalNetworkVO;
+import com.cloud.offering.NetworkOffering;
+import com.cloud.resource.ResourceManager;
+import com.cloud.user.Account;
+import com.cloud.user.dao.AccountDao;
+import com.cloud.vm.NicProfile;
+import com.cloud.vm.ReservationContext;
+import com.cloud.vm.VirtualMachineProfile;
+import org.apache.log4j.Logger;
+
+import javax.ejb.Local;
+import javax.inject.Inject;
+import java.net.URI;
+import java.net.URISyntaxException;
+import java.util.List;
+
+@Local(value = NetworkGuru.class)
+public class BigSwitchVnsGuestNetworkGuru extends GuestNetworkGuru {
 
 Review comment:
   The PR says adding isolation type, why has this added a new network guru? Do 
we have h/w to test this network guru?
 

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 #2193: CLOUDSTACK-10007 Isolation methods

[git]

rhtyd commented on a change in pull request #2193: CLOUDSTACK-10007 Isolation 
methods
URL: https://github.com/apache/cloudstack/pull/2193#discussion_r129527242
 
 

 ##
 File path: api/src/com/cloud/network/PhysicalNetwork.java
 ##
 @@ -32,8 +36,101 @@
 Disabled, Enabled;
 }
 
-public enum IsolationMethod {
-VLAN, L3, GRE, STT, BCF_SEGMENT, MIDO, SSP, VXLAN, ODL, L3VPN, VSP, 
VCS;
+public class IsolationMethod {
+protected static final String UNKNOWN_PROVIDER = "Unknown";
+static Set registeredIsolationMethods = new 
HashSet<>();
+
+String methodPrefix;
+String provider;
+
+public IsolationMethod(String prfx) {
+this(prfx, UNKNOWN_PROVIDER);
+}
+
+public IsolationMethod(String prfx, String prvdr) {
+methodPrefix = prfx;
+provider = StringUtils.isNotBlank(prvdr)? prvdr : UNKNOWN_PROVIDER;
+registeredIsolationMethods.add(this);
+}
+
+/**
+ * gets a IsolationMethod object that defines this prefix and if any 
it returns the first one found that has a known provider. If none has a known 
provider
 
 Review comment:
   Remove comment, comment is explaining the code that a developer can easily 
read and follow.
 

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 #2193: CLOUDSTACK-10007 Isolation methods

[git]

rhtyd commented on a change in pull request #2193: CLOUDSTACK-10007 Isolation 
methods
URL: https://github.com/apache/cloudstack/pull/2193#discussion_r129527115
 
 

 ##
 File path: api/test/com/cloud/network/IsolationMethodTest.java
 ##
 @@ -0,0 +1,92 @@
+// 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.
+package com.cloud.network;
+
+import org.junit.After;
+import org.junit.Test;
+
+import static org.junit.Assert.assertEquals;
+import static org.junit.Assert.assertFalse;
+
+/**
+ * Created by dahn on 18/07/17.
 
 Review comment:
   Remove this 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 #2193: CLOUDSTACK-10007 Isolation methods

[git]

rhtyd commented on issue #2193: CLOUDSTACK-10007 Isolation methods
URL: https://github.com/apache/cloudstack/pull/2193#issuecomment-318004575
 
 
   @DaanHoogland please put a CLOUDSTACK jira ID on it, squash your changes.
 

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 #2198: CLOUDSTACK-10013: Migrate to Debian9 for systemvmtemplate

[git]

wido commented on issue #2198: CLOUDSTACK-10013: Migrate to Debian9 for 
systemvmtemplate
URL: https://github.com/apache/cloudstack/pull/2198#issuecomment-318003305
 
 
   @rhtyd: Ok, that sounds doable! I've done quite some preseed work. Let me 
see if I can fix this.
 

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


With regards,
Apache Git Services

[GitHub] rhtyd commented on issue #2028: CLOUDSTACK-9853: Add support for Secondary IPv6 Addresses and Subnets

[git]

rhtyd commented on issue #2028: CLOUDSTACK-9853: Add support for Secondary IPv6 
Addresses and Subnets
URL: https://github.com/apache/cloudstack/pull/2028#issuecomment-317999745
 
 
   @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 #2177: CLOUDSTACK-9977: Enhance SSL protocol used by Console Proxy

[git]

rhtyd commented on issue #2177: CLOUDSTACK-9977: Enhance SSL protocol used by 
Console Proxy
URL: https://github.com/apache/cloudstack/pull/2177#issuecomment-317998765
 
 
   @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 #2028: CLOUDSTACK-9853: Add support for Secondary IPv6 Addresses and Subnets

[git]

blueorangutan commented on issue #2028: CLOUDSTACK-9853: Add support for 
Secondary IPv6 Addresses and Subnets
URL: https://github.com/apache/cloudstack/pull/2028#issuecomment-317999897
 
 
   @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 #2171: CLOUDSTACK-9990 : Account name is giving null in event tab after successful creation of account

[git]

rhtyd commented on issue #2171: CLOUDSTACK-9990 : Account name is giving null 
in event tab after successful creation of account
URL: https://github.com/apache/cloudstack/pull/2171#issuecomment-317999477
 
 
   @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 #2171: CLOUDSTACK-9990 : Account name is giving null in event tab after successful creation of account

[git]

blueorangutan commented on issue #2171: CLOUDSTACK-9990 : Account name is 
giving null in event tab after successful creation of account
URL: https://github.com/apache/cloudstack/pull/2171#issuecomment-317999626
 
 
   @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

[cloudstack] branch master updated: [CLOUDSTACK-10005] bugfix in test class (name of exception variable)

[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.git


The following commit(s) were added to refs/heads/master by this push:
 new ae46cab  [CLOUDSTACK-10005] bugfix in test class (name of exception 
variable)
ae46cab is described below

commit ae46cab456eabd98946576cc39bd3f8f729f02d9
Author: Maik Giessler 
AuthorDate: Thu Jul 6 12:59:19 2017 +0200

[CLOUDSTACK-10005] bugfix in test class (name of exception variable)
---
 test/src/com/cloud/test/ui/AddAndDeleteATemplate.java | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/test/src/com/cloud/test/ui/AddAndDeleteATemplate.java 
b/test/src/com/cloud/test/ui/AddAndDeleteATemplate.java
index b0b7183..b46f922 100644
--- a/test/src/com/cloud/test/ui/AddAndDeleteATemplate.java
+++ b/test/src/com/cloud/test/ui/AddAndDeleteATemplate.java
@@ -57,7 +57,7 @@ public class AddAndDeleteATemplate extends 
AbstractSeleniumTestCase {
 }
 } catch (Exception ex) {
 s_logger.info("[ignored]"
-+ "error during clicking test on template: " + 
e.getLocalizedMessage());
++ "error during clicking test on template: " + 
ex.getLocalizedMessage());
 }
 
 for (int second = 0;; second++) {

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

[GitHub] rhtyd commented on issue #2169: [CLOUDSTACK-10005] bugfix in test class (name of exception variable)

[git]

rhtyd commented on issue #2169: [CLOUDSTACK-10005] bugfix in test class (name 
of exception variable)
URL: https://github.com/apache/cloudstack/pull/2169#issuecomment-317998650
 
 
   LGTM.
 

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


With regards,
Apache Git Services

[GitHub] rhtyd closed pull request #2169: [CLOUDSTACK-10005] bugfix in test class (name of exception variable)

[git]

rhtyd closed pull request #2169: [CLOUDSTACK-10005] bugfix in test class (name 
of exception variable)
URL: https://github.com/apache/cloudstack/pull/2169
 
 
   
 

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] Maik1986 commented on issue #2169: [CLOUDSTACK-10005] bugfix in test class (name of exception variable)

[git]

Maik1986 commented on issue #2169: [CLOUDSTACK-10005] bugfix in test class 
(name of exception variable)
URL: https://github.com/apache/cloudstack/pull/2169#issuecomment-317998253
 
 
   jira id has been added to the commit now.
 

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 #2198: CLOUDSTACK-10013: Migrate to Debian9 for systemvmtemplate

[git]

rhtyd commented on issue #2198: CLOUDSTACK-10013: Migrate to Debian9 for 
systemvmtemplate
URL: https://github.com/apache/cloudstack/pull/2198#issuecomment-317998362
 
 
   @wido it gets stuck and asks questions about some packages, we'll need to go 
through the questions it is stuck on and configure those boolean/args. See 
jedi4ever/veewee#1106 discussion.
 

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 #2198: CLOUDSTACK-10013: Migrate to Debian9 for systemvmtemplate

[git]

wido commented on issue #2198: CLOUDSTACK-10013: Migrate to Debian9 for 
systemvmtemplate
URL: https://github.com/apache/cloudstack/pull/2198#issuecomment-317997583
 
 
   @rhtyd: The partitioning is a preseed thing, right? Can you tell what you 
are stuck on?
   
   It's not a Veewee thing, correct?
 

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


With regards,
Apache Git Services

[GitHub] rhtyd closed pull request #2199: CLOUDSTACK-10014: Fix test accounts remove secret key check

[git]

rhtyd closed pull request #2199: CLOUDSTACK-10014: Fix test accounts remove 
secret key check
URL: https://github.com/apache/cloudstack/pull/2199
 
 
   
 

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] 02/02: CLOUDSTACK-10014: fix components/test_accounts.py test_user_key_renew_same_account: use new api getUserKeys to get secrett key

[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.git

commit 899c5c784dfdfc7ba7245aa729c644b905e729ba
Author: Boris 
AuthorDate: Mon Jul 24 17:54:41 2017 +0300

CLOUDSTACK-10014: fix components/test_accounts.py 
test_user_key_renew_same_account: use new api getUserKeys to get secrett key
---
 test/integration/component/test_accounts.py | 20 
 1 file changed, 20 insertions(+)

diff --git a/test/integration/component/test_accounts.py 
b/test/integration/component/test_accounts.py
index 714680b..a64d922 100644
--- a/test/integration/component/test_accounts.py
+++ b/test/integration/component/test_accounts.py
@@ -20,6 +20,7 @@
 from marvin.cloudstackTestCase import cloudstackTestCase
 from marvin.lib.utils import (random_gen,
   cleanup_resources)
+from marvin.cloudstackAPI import *
 from marvin.lib.base import (Domain,
  Account,
  ServiceOffering,
@@ -43,6 +44,8 @@ from nose.plugins.attrib import attr
 from marvin.cloudstackException import CloudstackAPIException
 import time
 
+from pyVmomi.VmomiSupport import GetVersionFromVersionUri
+
 
 class Services:
 
@@ -1653,6 +1656,11 @@ class TestUserAPIKeys(cloudstackTestCase):
 user.apikey,
 userkeys.apikey,
 "Check User api key")
+user.secretkey = self.get_secret_key(user.id)
+self.assertEqual(
+user.secretkey,
+userkeys.secretkey,
+"Check User having secret key")
 
 self.debug("Get test client with user keys")
 cs_api = self.testClient.getUserApiClient(
@@ -1664,6 +1672,17 @@ class TestUserAPIKeys(cloudstackTestCase):
 userkeys.apikey,
 new_keys.apikey,
 "Check API key is different")
+new_keys.secretkey = self.get_secret_key(user_1.id)
+self.assertNotEqual(
+userkeys.secretkey,
+new_keys.secretkey,
+"Check secret key is different")
+
+def get_secret_key(self, id):
+cmd = getUserKeys.getUserKeysCmd()
+cmd.id = id
+keypair = self.apiclient.getUserKeys(cmd)
+return keypair.secretkey
 
 @attr(tags=[
 "role",
@@ -2062,3 +2081,4 @@ class TestDomainForceRemove(cloudstackTestCase):
 with self.assertRaises(Exception):
 domain.delete(self.apiclient, cleanup=False)
 return
+

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

[cloudstack] branch master updated (990e4a6 -> 899c5c7)

[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.git.


from 990e4a6  ipv6: Allow all ICMPv6 traffic if -1 is provided as a ICMP 
type
 new 055ae07  CLOUDSTACK-10014: fix components/test_accounts.py 
test_user_key_renew_same_account: remove secret key checks since it was removed 
from the API response
 new 899c5c7  CLOUDSTACK-10014: fix components/test_accounts.py 
test_user_key_renew_same_account: use new api getUserKeys to get secrett key

The 2 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:
 test/integration/component/test_accounts.py | 12 
 1 file changed, 12 insertions(+)

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

[cloudstack] 01/02: CLOUDSTACK-10014: fix components/test_accounts.py test_user_key_renew_same_account: remove secret key checks since it was removed from the API response

[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.git

commit 055ae078ec02b1f1b9afeba5d69494464fce4d32
Author: Boris 
AuthorDate: Mon Jul 24 16:56:07 2017 +0300

CLOUDSTACK-10014: fix components/test_accounts.py 
test_user_key_renew_same_account: remove secret key checks since it was removed 
from the API response
---
 test/integration/component/test_accounts.py | 8 
 1 file changed, 8 deletions(-)

diff --git a/test/integration/component/test_accounts.py 
b/test/integration/component/test_accounts.py
index 56494e2..714680b 100644
--- a/test/integration/component/test_accounts.py
+++ b/test/integration/component/test_accounts.py
@@ -1653,10 +1653,6 @@ class TestUserAPIKeys(cloudstackTestCase):
 user.apikey,
 userkeys.apikey,
 "Check User api key")
-self.assertEqual(
-user.secretkey,
-userkeys.secretkey,
-"Check User having secret key")
 
 self.debug("Get test client with user keys")
 cs_api = self.testClient.getUserApiClient(
@@ -1668,10 +1664,6 @@ class TestUserAPIKeys(cloudstackTestCase):
 userkeys.apikey,
 new_keys.apikey,
 "Check API key is different")
-self.assertNotEqual(
-userkeys.secretkey,
-new_keys.secretkey,
-"Check secret key is different")
 
 @attr(tags=[
 "role",

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

[GitHub] blueorangutan commented on issue #2193: CLOUDSTACK-10007 Isolation methods

[git]

blueorangutan commented on issue #2193: CLOUDSTACK-10007 Isolation methods
URL: https://github.com/apache/cloudstack/pull/2193#issuecomment-317978291
 
 
   @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 #2193: CLOUDSTACK-10007 Isolation methods

[git]

borisstoyanov commented on issue #2193: CLOUDSTACK-10007 Isolation methods
URL: https://github.com/apache/cloudstack/pull/2193#issuecomment-317978272
 
 
   @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 #2193: CLOUDSTACK-10007 Isolation methods

[git]

blueorangutan commented on issue #2193: CLOUDSTACK-10007 Isolation methods
URL: https://github.com/apache/cloudstack/pull/2193#issuecomment-317977577
 
 
   Packaging result: ?centos6 ?centos7 ?debian. JID-859
 

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 #2200: [CLOUDSTACK-10015] Return storage provider with response to list storage pools

[git]

wido commented on issue #2200: [CLOUDSTACK-10015] Return storage provider with 
response to list storage pools
URL: https://github.com/apache/cloudstack/pull/2200#issuecomment-317973347
 
 
   Code-wise it doesn't matter, but you seem to have optimize imports enabled. 
That seems like a lot of lines changed, but it didn't.
   
   Otherwise: LGTM
   
 

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 #2193: CLOUDSTACK-10007 Isolation methods

[git]

blueorangutan commented on issue #2193: CLOUDSTACK-10007 Isolation methods
URL: https://github.com/apache/cloudstack/pull/2193#issuecomment-317972397
 
 
   @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 #2193: CLOUDSTACK-10007 Isolation methods

[git]

borisstoyanov commented on issue #2193: CLOUDSTACK-10007 Isolation methods
URL: https://github.com/apache/cloudstack/pull/2193#issuecomment-317972236
 
 
   @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