You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cloudstack.apache.org by GitBox <gi...@apache.org> on 2020/10/20 15:31:07 UTC
[GitHub] [cloudstack] nvazquez opened a new pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
nvazquez opened a new pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307
## Description
Combined PR of #4248 and #4250
## Types of changes
- [ ] Breaking change (fix or feature that would cause existing functionality to change)
- [x] New feature (non-breaking change which adds functionality)
- [ ] Bug fix (non-breaking change which fixes an issue)
- [ ] Enhancement (improves an existing feature and functionality)
- [ ] Cleanup (Code refactoring and cleanup, that may add test cases)
## Screenshots (if appropriate):
## How Has This Been Tested?
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-716194287
@borisstoyanov a Trillian-Jenkins test job (centos7 mgmt + vmware-67u3) 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 to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] nvazquez commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
nvazquez commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-714735780
@blueorangutan package
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-703956432
<b>Trillian test result (tid-2892)</b>
Environment: vmware-67u3 (x2), Advanced Networking with Mgmt server 7
Total time taken: 36904 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr4307-t2892-vmware-67u3.zip
Intermittent failure detected: /marvin/tests/smoke/test_vpc_redundant.py
Smoke tests completed. 80 look OK, 1 have error(s)
Only failed tests results shown below:
Test | Result | Time (s) | Test File
--- | --- | --- | ---
test_03_create_redundant_VPC_1tier_2VMs_2IPs_2PF_ACL_reboot_routers | `Failure` | 412.51 | test_vpc_redundant.py
test_04_rvpc_network_garbage_collector_nics | `Error` | 3768.72 | test_vpc_redundant.py
test_04_rvpc_network_garbage_collector_nics | `Error` | 3768.75 | test_vpc_redundant.py
test_05_rvpc_multi_tiers | `Error` | 0.01 | test_vpc_redundant.py
ContextSuite context=TestVPCRedundancy>:teardown | `Error` | 0.01 | test_vpc_redundant.py
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] rhtyd commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
rhtyd commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-698911943
Requesting review cc @sureshanaparti @DaanHoogland, thnx
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] nvazquez commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
nvazquez commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-716542311
@blueorangutan test centos7 vmware-67u3
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] andrijapanicsb commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
andrijapanicsb commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-713706546
LGTM
(after the extensive functional testing side)
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-707065059
Packaging result: ✖centos7 ✖centos8 ✔debian. JID-2159
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-694619118
@rhtyd a Trillian-Jenkins matrix job (centos7 mgmt + xs71, centos7 mgmt + vmware67, 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 to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-689867604
@rhtyd a Trillian-Jenkins test job (centos7 mgmt + vmware-67u3) 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 to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-688960175
<b>Trillian test result (tid-2697)</b>
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 68565 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr4307-t2697-kvm-centos7.zip
Intermittent failure detected: /marvin/tests/smoke/test_kubernetes_clusters.py
Intermittent failure detected: /marvin/tests/smoke/test_kubernetes_supported_versions.py
Intermittent failure detected: /marvin/tests/smoke/test_privategw_acl.py
Intermittent failure detected: /marvin/tests/smoke/test_vpc_redundant.py
Intermittent failure detected: /marvin/tests/smoke/test_host_maintenance.py
Intermittent failure detected: /marvin/tests/smoke/test_hostha_kvm.py
Smoke tests completed. 81 look OK, 4 have error(s)
Only failed tests results shown below:
Test | Result | Time (s) | Test File
--- | --- | --- | ---
test_01_add_delete_kubernetes_supported_version | `Error` | 1807.37 | test_kubernetes_supported_versions.py
test_02_vpc_privategw_static_routes | `Failure` | 282.51 | test_privategw_acl.py
test_03_vpc_privategw_restart_vpc_cleanup | `Failure` | 266.32 | test_privategw_acl.py
test_04_rvpc_privategw_static_routes | `Failure` | 540.15 | test_privategw_acl.py
test_01_create_redundant_VPC_2tiers_4VMs_4IPs_4PF_ACL | `Failure` | 439.35 | test_vpc_redundant.py
test_03_create_redundant_VPC_1tier_2VMs_2IPs_2PF_ACL_reboot_routers | `Failure` | 587.17 | test_vpc_redundant.py
test_05_rvpc_multi_tiers | `Failure` | 517.08 | test_vpc_redundant.py
test_hostha_enable_ha_when_host_disabled | `Error` | 2.62 | test_hostha_kvm.py
test_hostha_enable_ha_when_host_in_maintenance | `Error` | 302.78 | test_hostha_kvm.py
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-692211758
Packaging result: ✔centos7 ✔centos8 ✔debian. JID-1999
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] andrijapanicsb commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
andrijapanicsb commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-713712626
@blueorangutan test centos7 vmware-67u3
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-715980310
Packaging result: ✔centos7 ✔centos8 ✔debian. JID-2271
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] nvazquez commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
nvazquez commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-695594698
@blueorangutan test
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-705709530
@DaanHoogland a Trillian-Jenkins test job (centos7 mgmt + vmware-67u3) 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 to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] nvazquez commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
nvazquez commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-716505340
@blueorangutan package
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] nvazquez removed a comment on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
nvazquez removed a comment on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-698751791
@blueorangutan package
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-707523028
Packaging result: ✔centos7 ✔centos8 ✔debian. JID-2164
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-695566590
Packaging result: ✔centos7 ✔centos8 ✔debian. JID-2049
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] harikrishna-patnala commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
harikrishna-patnala commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-689302695
@blueorangutan package
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] borisstoyanov merged pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
borisstoyanov merged pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan removed a comment on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan removed a comment on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-690944356
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-716172471
<b>Trillian test result (tid-3068)</b>
Environment: xenserver-71 (x2), Advanced Networking with Mgmt server 7
Total time taken: 33437 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr4307-t3068-xenserver-71.zip
Intermittent failure detected: /marvin/tests/smoke/test_privategw_acl.py
Intermittent failure detected: /marvin/tests/smoke/test_scale_vm.py
Intermittent failure detected: /marvin/tests/smoke/test_usage.py
Smoke tests completed. 85 look OK, 1 have error(s)
Only failed tests results shown below:
Test | Result | Time (s) | Test File
--- | --- | --- | ---
test_01_scale_vm | `Failure` | 9.34 | test_scale_vm.py
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-693431479
Packaging result: ✔centos7 ✔centos8 ✔debian. JID-2018
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-713875315
<b>Trillian test result (tid-3039)</b>
Environment: vmware-65u2 (x2), Advanced Networking with Mgmt server 7
Total time taken: 14006 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr4307-t3039-vmware-65u2.zip
Intermittent failure detected: /marvin/tests/smoke/test_accounts.py
Intermittent failure detected: /marvin/tests/smoke/test_affinity_groups_projects.py
Intermittent failure detected: /marvin/tests/smoke/test_async_job.py
Intermittent failure detected: /marvin/tests/smoke/test_deploy_vgpu_enabled_vm.py
Intermittent failure detected: /marvin/tests/smoke/test_deploy_vm_iso.py
Intermittent failure detected: /marvin/tests/smoke/test_deploy_vms_with_varied_deploymentplanners.py
Intermittent failure detected: /marvin/tests/smoke/test_deploy_vm_with_userdata.py
Intermittent failure detected: /marvin/tests/smoke/test_diagnostics.py
Intermittent failure detected: /marvin/tests/smoke/test_domain_service_offerings.py
Intermittent failure detected: /marvin/tests/smoke/test_human_readable_logs.py
Intermittent failure detected: /marvin/tests/smoke/test_internal_lb.py
Intermittent failure detected: /marvin/tests/smoke/test_kubernetes_clusters.py
Intermittent failure detected: /marvin/tests/smoke/test_list_ids_parameter.py
Intermittent failure detected: /marvin/tests/smoke/test_loadbalance.py
Intermittent failure detected: /marvin/tests/smoke/test_metrics_api.py
Intermittent failure detected: /marvin/tests/smoke/test_multipleips_per_nic.py
Intermittent failure detected: /marvin/tests/smoke/test_nested_virtualization.py
Intermittent failure detected: /marvin/tests/smoke/test_network_acl.py
Intermittent failure detected: /marvin/tests/smoke/test_network.py
Intermittent failure detected: /marvin/tests/smoke/test_nic_adapter_type.py
Intermittent failure detected: /marvin/tests/smoke/test_password_server.py
Intermittent failure detected: /marvin/tests/smoke/test_portforwardingrules.py
Intermittent failure detected: /marvin/tests/smoke/test_privategw_acl.py
Intermittent failure detected: /marvin/tests/smoke/test_projects.py
Intermittent failure detected: /marvin/tests/smoke/test_reset_vm_on_reboot.py
Intermittent failure detected: /marvin/tests/smoke/test_resource_accounting.py
Intermittent failure detected: /marvin/tests/smoke/test_router_dhcphosts.py
Intermittent failure detected: /marvin/tests/smoke/test_router_dns.py
Intermittent failure detected: /marvin/tests/smoke/test_router_dnsservice.py
Intermittent failure detected: /marvin/tests/smoke/test_routers_iptables_default_policy.py
Intermittent failure detected: /marvin/tests/smoke/test_routers_network_ops.py
Intermittent failure detected: /marvin/tests/smoke/test_routers.py
Intermittent failure detected: /marvin/tests/smoke/test_service_offerings.py
Intermittent failure detected: /marvin/tests/smoke/test_snapshots.py
Intermittent failure detected: /marvin/tests/smoke/test_storage_policy.py
Intermittent failure detected: /marvin/tests/smoke/test_templates.py
Intermittent failure detected: /marvin/tests/smoke/test_vpc_redundant.py
Intermittent failure detected: /marvin/tests/smoke/test_vpc_router_nics.py
Intermittent failure detected: /marvin/tests/smoke/test_vpc_vpn.py
Smoke tests completed. 48 look OK, 38 have error(s)
Only failed tests results shown below:
Test | Result | Time (s) | Test File
--- | --- | --- | ---
ContextSuite context=TestAccounts>:setup | `Error` | 0.00 | test_accounts.py
ContextSuite context=TestAddVmToSubDomain>:setup | `Error` | 0.00 | test_accounts.py
ContextSuite context=TestDomainForceRemove>:setup | `Error` | 0.00 | test_accounts.py
ContextSuite context=TestRemoveUserFromAccount>:setup | `Error` | 5.83 | test_accounts.py
ContextSuite context=TestTemplateHierarchy>:setup | `Error` | 7.11 | test_accounts.py
ContextSuite context=TestDeployVmWithAffinityGroup>:setup | `Error` | 0.00 | test_affinity_groups_projects.py
ContextSuite context=TestAsyncJob>:setup | `Error` | 0.00 | test_async_job.py
test_3d_gpu_support | `Error` | 1.15 | test_deploy_vgpu_enabled_vm.py
test_deploy_vgpu_enabled_vm | `Error` | 1.15 | test_deploy_vgpu_enabled_vm.py
ContextSuite context=TestDeployVMFromISO>:setup | `Error` | 0.00 | test_deploy_vm_iso.py
ContextSuite context=TestInternalLb>:setup | `Error` | 0.00 | test_internal_lb.py
ContextSuite context=TestRemoteDiagnostics>:setup | `Error` | 0.00 | test_diagnostics.py
ContextSuite context=TestDeployVmWithVariedPlanners>:setup | `Error` | 0.00 | test_deploy_vms_with_varied_deploymentplanners.py
ContextSuite context=TestDeployVmWithUserData>:setup | `Error` | 0.00 | test_deploy_vm_with_userdata.py
ContextSuite context=TestDomainsServiceOfferings>:setup | `Error` | 3.68 | test_domain_service_offerings.py
ContextSuite context=TestKubernetesCluster>:setup | `Error` | 0.00 | test_kubernetes_clusters.py
ContextSuite context=TestListIdsParams>:setup | `Error` | 0.00 | test_list_ids_parameter.py
ContextSuite context=TestLoadBalance>:setup | `Error` | 0.00 | test_loadbalance.py
test_list_clusters_metrics | `Error` | 0.27 | test_metrics_api.py
test_list_hosts_metrics | `Error` | 0.24 | test_metrics_api.py
test_list_infrastructure_metrics | `Error` | 0.24 | test_metrics_api.py
test_list_pstorage_metrics | `Error` | 0.22 | test_metrics_api.py
test_list_vms_metrics | `Error` | 0.21 | test_metrics_api.py
test_list_volumes_metrics | `Error` | 0.29 | test_metrics_api.py
test_list_zones_metrics | `Error` | 0.27 | test_metrics_api.py
test_nic_secondaryip_add_remove | `Error` | 0.09 | test_multipleips_per_nic.py
ContextSuite context=TestNestedVirtualization>:setup | `Error` | 0.00 | test_nested_virtualization.py
ContextSuite context=TestNetworkACL>:setup | `Error` | 0.00 | test_network_acl.py
test_delete_account | `Error` | 0.11 | test_network.py
ContextSuite context=TestL2Networks>:setup | `Error` | 0.87 | test_network.py
ContextSuite context=TestPortForwarding>:setup | `Error` | 0.96 | test_network.py
ContextSuite context=TestPublicIP>:setup | `Error` | 1.93 | test_network.py
test_reboot_router | `Error` | 0.09 | test_network.py
test_releaseIP | `Error` | 0.10 | test_network.py
ContextSuite context=TestRouterRules>:setup | `Error` | 0.21 | test_network.py
ContextSuite context=TestAdapterTypeForNic>:setup | `Error` | 0.00 | test_nic_adapter_type.py
ContextSuite context=TestIsolatedNetworksPasswdServer>:setup | `Error` | 0.00 | test_password_server.py
ContextSuite context=TestPortForwardingRules>:setup | `Error` | 0.00 | test_portforwardingrules.py
ContextSuite context=TestPrivateGwACL>:setup | `Error` | 0.00 | test_privategw_acl.py
ContextSuite context=TestProjectSuspendActivate>:setup | `Error` | 7.48 | test_projects.py
ContextSuite context=TestResetVmOnReboot>:setup | `Error` | 0.00 | test_reset_vm_on_reboot.py
ContextSuite context=TestRAMCPUResourceAccounting>:setup | `Error` | 0.00 | test_resource_accounting.py
ContextSuite context=TestRouterDHCPHosts>:setup | `Error` | 0.00 | test_router_dhcphosts.py
ContextSuite context=TestRouterDHCPOpts>:setup | `Error` | 0.00 | test_router_dhcphosts.py
ContextSuite context=TestRouterDns>:setup | `Error` | 0.00 | test_router_dns.py
ContextSuite context=TestRouterDnsService>:setup | `Error` | 0.00 | test_router_dnsservice.py
ContextSuite context=TestRouterIpTablesPolicies>:setup | `Error` | 0.00 | test_routers_iptables_default_policy.py
ContextSuite context=TestVPCIpTablesPolicies>:setup | `Error` | 0.00 | test_routers_iptables_default_policy.py
ContextSuite context=TestIsolatedNetworks>:setup | `Error` | 0.00 | test_routers_network_ops.py
ContextSuite context=TestRedundantIsolateNetworks>:setup | `Error` | 0.00 | test_routers_network_ops.py
ContextSuite context=TestRouterServices>:setup | `Error` | 0.00 | test_routers.py
ContextSuite context=TestServiceOfferings>:setup | `Error` | 0.19 | test_service_offerings.py
ContextSuite context=TestSnapshotRootDisk>:setup | `Error` | 0.00 | test_snapshots.py
ContextSuite context=TestVMWareStoragePolicies>:setup | `Error` | 0.00 | test_storage_policy.py
test_02_1_create_template_with_checksum_sha1_negative | `Error` | 0.07 | test_templates.py
test_02_create_template_with_checksum_sha1 | `Error` | 0.07 | test_templates.py
test_03_1_create_template_with_checksum_sha256_negative | `Error` | 0.06 | test_templates.py
test_03_create_template_with_checksum_sha256 | `Error` | 0.07 | test_templates.py
test_04_1_create_template_with_checksum_md5_negative | `Error` | 0.06 | test_templates.py
test_04_create_template_with_checksum_md5 | `Error` | 0.06 | test_templates.py
test_05_create_template_with_no_checksum | `Error` | 0.07 | test_templates.py
ContextSuite context=TestVPCRedundancy>:setup | `Error` | 0.00 | test_vpc_redundant.py
ContextSuite context=TestVPCNics>:setup | `Error` | 0.00 | test_vpc_router_nics.py
ContextSuite context=TestRVPCSite2SiteVpn>:setup | `Error` | 0.00 | test_vpc_vpn.py
ContextSuite context=TestVPCSite2SiteVPNMultipleOptions>:setup | `Error` | 0.00 | test_vpc_vpn.py
ContextSuite context=TestVpcRemoteAccessVpn>:setup | `Error` | 0.00 | test_vpc_vpn.py
ContextSuite context=TestVpcSite2SiteVpn>:setup | `Error` | 0.00 | test_vpc_vpn.py
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-713377949
Packaging result: ✖centos7 ✖centos8 ✖debian. JID-2242
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-693097086
<b>Trillian test result (tid-2738)</b>
Environment: xenserver-71 (x2), Advanced Networking with Mgmt server 7
Total time taken: 60199 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr4307-t2738-xenserver-71.zip
Intermittent failure detected: /marvin/tests/smoke/test_accounts.py
Intermittent failure detected: /marvin/tests/smoke/test_internal_lb.py
Intermittent failure detected: /marvin/tests/smoke/test_kubernetes_supported_versions.py
Intermittent failure detected: /marvin/tests/smoke/test_scale_vm.py
Intermittent failure detected: /marvin/tests/smoke/test_ssvm.py
Intermittent failure detected: /marvin/tests/smoke/test_templates.py
Smoke tests completed. 70 look OK, 2 have error(s)
Only failed tests results shown below:
Test | Result | Time (s) | Test File
--- | --- | --- | ---
test_01_add_delete_kubernetes_supported_version | `Error` | 1810.22 | test_kubernetes_supported_versions.py
test_01_scale_vm | `Failure` | 15.89 | test_scale_vm.py
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-711089535
@rhtyd a Trillian-Jenkins matrix job (centos6 mgmt + xs71, centos7 mgmt + vmware65, 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 to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-716188699
@nvazquez a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-698122135
@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 to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] rhtyd commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
rhtyd commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-698122010
@blueorangutan package
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-713655041
Packaging result: ✔centos7 ✔centos8 ✔debian. JID-2253
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-716101499
@nvazquez a Trillian-Jenkins test job (centos7 mgmt + vmware-67u3) 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 to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] nvazquez commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
nvazquez commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-711486202
@blueorangutan package
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-716182505
<b>Trillian test result (tid-3071)</b>
Environment: vmware-67u3 (x2), Advanced Networking with Mgmt server 7
Total time taken: 37791 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr4307-t3071-vmware-67u3.zip
Intermittent failure detected: /marvin/tests/smoke/test_kubernetes_clusters.py
Intermittent failure detected: /marvin/tests/smoke/test_storage_policy.py
Smoke tests completed. 84 look OK, 2 have error(s)
Only failed tests results shown below:
Test | Result | Time (s) | Test File
--- | --- | --- | ---
test_03_deploy_and_upgrade_kubernetes_cluster | `Failure` | 774.30 | test_kubernetes_clusters.py
test_01_import_storage_policies | `Error` | 2.02 | test_storage_policy.py
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-696548965
@harikrishna-patnala 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 to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] rhtyd commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
rhtyd commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-698122010
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] nvazquez commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
nvazquez commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-700383163
@blueorangutan package
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] harikrishna-patnala commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
harikrishna-patnala commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-690943888
@blueorangutan package
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan removed a comment on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan removed a comment on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-698752221
@nvazquez a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] borisstoyanov commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
borisstoyanov commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-692495460
@blueorangutan test matrix
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-692256412
Packaging result: ✔centos7 ✔centos8 ✔debian. JID-2001
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-695931306
<b>Trillian test result (tid-2797)</b>
Environment: vmware-67u3 (x2), Advanced Networking with Mgmt server 7
Total time taken: 45833 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr4307-t2797-vmware-67u3.zip
Intermittent failure detected: /marvin/tests/smoke/test_vpc_redundant.py
Smoke tests completed. 80 look OK, 1 have error(s)
Only failed tests results shown below:
Test | Result | Time (s) | Test File
--- | --- | --- | ---
test_01_create_redundant_VPC_2tiers_4VMs_4IPs_4PF_ACL | `Failure` | 497.47 | test_vpc_redundant.py
test_04_rvpc_network_garbage_collector_nics | `Error` | 3810.98 | test_vpc_redundant.py
test_04_rvpc_network_garbage_collector_nics | `Error` | 3811.01 | test_vpc_redundant.py
test_05_rvpc_multi_tiers | `Error` | 0.01 | test_vpc_redundant.py
ContextSuite context=TestVPCRedundancy>:teardown | `Error` | 0.01 | test_vpc_redundant.py
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-695888136
Packaging result: ✔centos7 ✔centos8 ✔debian. JID-2052
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] DaanHoogland commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
DaanHoogland commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-715986779
> I think test_templates we can ignore, not sure what's wrong with kubernetes_cluster.
> test_storage_policy is introduced in this PR and it looks like the test needs fixing.
the second of the failures is in the cleanup, a network offerning being deleted while a network is still using it. should be easy to fix.
I'll look at the other later
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] borisstoyanov commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
borisstoyanov commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-703583387
@blueorangutan package
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-703152803
@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 to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-705709530
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-705937777
@nvazquez a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-690973819
Packaging result: ✖centos7 ✖centos8 ✖debian. JID-1958
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-714179071
<b>Trillian test result (tid-3038)</b>
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 33007 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr4307-t3038-kvm-centos7.zip
Intermittent failure detected: /marvin/tests/smoke/test_snapshots.py
Intermittent failure detected: /marvin/tests/smoke/test_storage_policy.py
Intermittent failure detected: /marvin/tests/smoke/test_usage.py
Smoke tests completed. 83 look OK, 3 have error(s)
Only failed tests results shown below:
Test | Result | Time (s) | Test File
--- | --- | --- | ---
test_02_list_snapshots_with_removed_data_store | `Error` | 10.90 | test_snapshots.py
ContextSuite context=TestVMWareStoragePolicies>:teardown | `Error` | 1.54 | test_storage_policy.py
test_01_volume_usage | `Error` | 125.06 | test_usage.py
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-693140012
<b>Trillian test result (tid-2746)</b>
Environment: vmware-67u3 (x2), Advanced Networking with Mgmt server 7
Total time taken: 58775 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr4307-t2746-vmware-67u3.zip
Intermittent failure detected: /marvin/tests/smoke/test_deploy_vm_root_resize.py
Intermittent failure detected: /marvin/tests/smoke/test_kubernetes_supported_versions.py
Intermittent failure detected: /marvin/tests/smoke/test_snapshots.py
Intermittent failure detected: /marvin/tests/smoke/test_vpc_vpn.py
Smoke tests completed. 80 look OK, 2 have error(s)
Only failed tests results shown below:
Test | Result | Time (s) | Test File
--- | --- | --- | ---
test_01_add_delete_kubernetes_supported_version | `Error` | 1817.83 | test_kubernetes_supported_versions.py
test_02_list_snapshots_with_removed_data_store | `Error` | 24.04 | test_snapshots.py
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] DaanHoogland commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
DaanHoogland commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-714494799
@blueorangutan test centos6 xenserver-71
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-715211805
@borisstoyanov a Trillian-Jenkins test job (centos7 mgmt + vmware-67u3) 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 to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-716101225
Packaging result: ✔centos7 ✔centos8 ✔debian. JID-2274
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-705902186
Packaging result: ✔centos7 ✔centos8 ✔debian. JID-2151
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] harikrishna-patnala commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
harikrishna-patnala commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-705893170
@blueorangutan package
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] rhtyd commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
rhtyd commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-703179776
@blueorangutan test centos7 vmware-67u3
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] DaanHoogland commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
DaanHoogland commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-714446170
@blueorangutan test centos6 xenserver-71
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-716096319
@nvazquez a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-716542162
@nvazquez a Trillian-Jenkins matrix job (centos7 mgmt + xs71, centos7 mgmt + vmware65, 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 to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-716521280
Packaging result: ✔centos7 ✔centos8 ✔debian. JID-2277
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] rhtyd commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
rhtyd commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-708301374
@nvazquez @harikrishna-patnala please look at Travis tests too after they complete (just rekicked few failed jobs) and ensure they pass, thanks.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] andrijapanicsb removed a comment on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
andrijapanicsb removed a comment on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-713373066
@blueorangutan package
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-689552548
<b>Trillian test result (tid-2699)</b>
Environment: xenserver-71 (x2), Advanced Networking with Mgmt server 7
Total time taken: 101618 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr4307-t2699-xenserver-71.zip
Intermittent failure detected: /marvin/tests/smoke/test_accounts.py
Intermittent failure detected: /marvin/tests/smoke/test_async_job.py
Intermittent failure detected: /marvin/tests/smoke/test_certauthority_root.py
Intermittent failure detected: /marvin/tests/smoke/test_deploy_vm_root_resize.py
Intermittent failure detected: /marvin/tests/smoke/test_domain_vpc_offerings.py
Intermittent failure detected: /marvin/tests/smoke/test_internal_lb.py
Intermittent failure detected: /marvin/tests/smoke/test_kubernetes_supported_versions.py
Intermittent failure detected: /marvin/tests/smoke/test_list_ids_parameter.py
Intermittent failure detected: /marvin/tests/smoke/test_network_acl.py
Intermittent failure detected: /marvin/tests/smoke/test_password_server.py
Intermittent failure detected: /marvin/tests/smoke/test_scale_vm.py
Smoke tests completed. 79 look OK, 6 have error(s)
Only failed tests results shown below:
Test | Result | Time (s) | Test File
--- | --- | --- | ---
ContextSuite context=TestAddVmToSubDomain>:setup | `Error` | 1849.30 | test_accounts.py
ContextSuite context=TestTemplateHierarchy>:setup | `Error` | 1529.07 | test_accounts.py
test_02_internallb_roundrobin_1RVPC_3VM_HTTP_port80 | `Error` | 0.10 | test_internal_lb.py
test_03_vpc_internallb_haproxy_stats_on_all_interfaces | `Failure` | 910.84 | test_internal_lb.py
test_04_rvpc_internallb_haproxy_stats_on_all_interfaces | `Failure` | 1027.43 | test_internal_lb.py
test_03_create_vpc_domain_vpc_offering | `Error` | 917.86 | test_domain_vpc_offerings.py
test_01_add_delete_kubernetes_supported_version | `Error` | 1807.56 | test_kubernetes_supported_versions.py
test_network_acl | `Error` | 912.02 | test_network_acl.py
test_01_scale_vm | `Failure` | 93.40 | test_scale_vm.py
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] harikrishna-patnala commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
harikrishna-patnala commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-690943888
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-693970604
Packaging result: ✖centos7 ✖centos8 ✖debian. JID-2024
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] DaanHoogland removed a comment on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
DaanHoogland removed a comment on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-714494799
@blueorangutan test centos6 xenserver-71
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-716542645
@nvazquez a Trillian-Jenkins test job (centos7 mgmt + vmware-67u3) 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 to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] nvazquez commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
nvazquez commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-713632375
@blueorangutan package
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-713376208
@DaanHoogland a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-707974074
Packaging result: ✔centos7 ✔centos8 ✔debian. JID-2170
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-700419881
@nvazquez a Trillian-Jenkins test job (centos7 mgmt + vmware-67u3) 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 to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-696510052
Packaging result: ✖centos7 ✔centos8 ✔debian. JID-2058
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] nvazquez commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
nvazquez commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-693410185
@blueorangutan package
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-715198287
<b>Trillian test result (tid-3060)</b>
Environment: vmware-67u3 (x2), Advanced Networking with Mgmt server 7
Total time taken: 41208 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr4307-t3060-vmware-67u3.zip
Intermittent failure detected: /marvin/tests/smoke/test_kubernetes_clusters.py
Intermittent failure detected: /marvin/tests/smoke/test_storage_policy.py
Intermittent failure detected: /marvin/tests/smoke/test_templates.py
Intermittent failure detected: /marvin/tests/smoke/test_volumes.py
Smoke tests completed. 82 look OK, 4 have error(s)
Only failed tests results shown below:
Test | Result | Time (s) | Test File
--- | --- | --- | ---
test_03_deploy_and_upgrade_kubernetes_cluster | `Failure` | 769.66 | test_kubernetes_clusters.py
test_01_import_storage_policies | `Error` | 3.44 | test_storage_policy.py
ContextSuite context=TestVMWareStoragePolicies>:teardown | `Error` | 4.57 | test_storage_policy.py
test_02_create_template_with_checksum_sha1 | `Error` | 5.20 | test_templates.py
test_03_create_template_with_checksum_sha256 | `Error` | 5.22 | test_templates.py
test_04_create_template_with_checksum_md5 | `Error` | 5.18 | test_templates.py
test_10_list_volumes | `Failure` | 369.56 | test_volumes.py
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-716523106
@nvazquez a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-694440522
@nvazquez a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-703605476
Packaging result: ✔centos7 ✔centos8 ✔debian. JID-2117
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-698042807
Packaging result: ✔centos7 ✔centos8 ✔debian. JID-2075
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-709443384
@nvazquez a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-698753680
@nvazquez a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-691660190
Packaging result: ✖centos7 ✖centos8 ✖debian. JID-1986
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-694118977
Packaging result: ✔centos7 ✔centos8 ✔debian. JID-2027
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] DaanHoogland commented on a change in pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
DaanHoogland commented on a change in pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#discussion_r496704106
##########
File path: api/src/main/java/org/apache/cloudstack/api/command/user/template/ListTemplatesCmd.java
##########
@@ -82,10 +86,35 @@
@Parameter(name = ApiConstants.PARENT_TEMPLATE_ID, type = CommandType.UUID, entityType = TemplateResponse.class, description = "list datadisk templates by parent template id", since = "4.4")
private Long parentTemplateId;
+ @Parameter(name = ApiConstants.DETAILS,
Review comment:
add a `since = "4.15"`
##########
File path: api/src/main/java/org/apache/cloudstack/api/response/TemplateResponse.java
##########
@@ -194,12 +195,22 @@
@Param(description = "KVM Only: true if template is directly downloaded to Primary Storage bypassing Secondary Storage")
private Boolean directDownload;
+ @SerializedName(ApiConstants.DEPLOY_AS_IS)
+ @Param(description = "VMware only: true if template is deployed without orchestrating disks and networks but \"as-is\" defined in the template.")
+ private Boolean deployAsIs;
+
+ @SerializedName(ApiConstants.DEPLOY_AS_IS_DETAILS)
+ @Param(description = "VMware only: additional key/value details tied with deploy-as-is template")
Review comment:
add `since = "4.15"`
##########
File path: api/src/main/java/com/cloud/agent/api/to/DeployAsIsInfoTO.java
##########
@@ -0,0 +1,58 @@
+// 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.agent.api.to;
Review comment:
new class should be in org.apache.cloudstack
##########
File path: api/src/main/java/com/cloud/agent/api/to/deployasis/OVFNetworkTO.java
##########
@@ -0,0 +1,124 @@
+// 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.agent.api.to.deployasis;
Review comment:
new class should be in org.apache.cloudstack
##########
File path: api/src/main/java/com/cloud/agent/api/to/deployasis/OVFEulaSectionTO.java
##########
@@ -0,0 +1,49 @@
+//
+// 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.agent.api.to.deployasis;
+
+import com.cloud.agent.api.LogLevel;
+
+/**
Review comment:
👍 🎉
##########
File path: engine/api/src/main/java/org/apache/cloudstack/engine/subsystem/api/storage/TemplateService.java
##########
@@ -72,4 +75,6 @@ public TemplateInfo getTemplate() {
void associateCrosszoneTemplatesToZone(long dcId);
AsyncCallFuture<TemplateApiResult> createDatadiskTemplateAsync(TemplateInfo parentTemplate, TemplateInfo dataDiskTemplate, String path, String diskId, long fileSize, boolean bootable);
+
+ List<DatadiskTO> getTemplateDatadisksOnImageStore(TemplateInfo templateInfo, String configurationId);
Review comment:
here the string is `configurationId` instead of `configuration`. would be nice to javadoc the difference
##########
File path: engine/orchestration/src/main/java/org/apache/cloudstack/engine/orchestration/NetworkOrchestrator.java
##########
@@ -1085,7 +1170,7 @@ boolean isNetworkImplemented(final NetworkVO network) {
}
Pair<NetworkGuru, NetworkVO> implementNetwork(final long networkId, final DeployDestination dest, final ReservationContext context, final boolean isRouter) throws ConcurrentOperationException,
- ResourceUnavailableException, InsufficientCapacityException {
+ ResourceUnavailableException, InsufficientCapacityException {
Review comment:
with throws?
##########
File path: engine/orchestration/src/main/java/org/apache/cloudstack/engine/orchestration/NetworkOrchestrator.java
##########
@@ -1661,7 +1746,7 @@ public void doInTransactionWithoutResult(final TransactionStatus status) {
@Override
public void prepare(final VirtualMachineProfile vmProfile, final DeployDestination dest, final ReservationContext context) throws InsufficientCapacityException, ConcurrentOperationException,
- ResourceUnavailableException {
+ ResourceUnavailableException {
Review comment:
with throws?
##########
File path: engine/schema/src/main/java/com/cloud/storage/VMTemplateStoragePoolVO.java
##########
@@ -168,17 +171,18 @@ public Status getDownloadState() {
return downloadState;
}
- public VMTemplateStoragePoolVO(long poolId, long templateId) {
+ public VMTemplateStoragePoolVO(long poolId, long templateId, String configuration) {
super();
this.poolId = poolId;
this.templateId = templateId;
this.downloadState = Status.NOT_DOWNLOADED;
this.state = ObjectInDataStoreStateMachine.State.Allocated;
this.markedForGC = false;
+ this.deploymentOption = configuration;
Review comment:
why this difference in name? what do either mean?
##########
File path: engine/schema/src/main/resources/META-INF/db/schema-41400to41500.sql
##########
@@ -220,3 +516,22 @@ ALTER VIEW `cloud`.`image_store_view` AS
`cloud`.`data_center` ON image_store.data_center_id = data_center.id
left join
`cloud`.`image_store_details` ON image_store_details.store_id = image_store.id;
+
+-- OVF configured OS while registering deploy-as-is templates Linux 3.x Kernel OS
+INSERT IGNORE INTO `cloud`.`guest_os` (id, uuid, category_id, display_name, created) VALUES (305, UUID(), 11, 'OVF Configured OS', now());
+INSERT IGNORE INTO `cloud`.`guest_os` (id, uuid, category_id, display_name, created) VALUES (306, UUID(), 2, 'Linux 3.x Kernel (64 bit)', now());
+INSERT IGNORE INTO `cloud`.`guest_os` (id, uuid, category_id, display_name, created) VALUES (307, UUID(), 2, 'Linux 3.x Kernel (32 bit)', now());
+
+INSERT IGNORE INTO `cloud`.`guest_os_hypervisor` (uuid, hypervisor_type, hypervisor_version, guest_os_name, guest_os_id, created, is_user_defined) VALUES (UUID(), 'VMware', '6.0', 'other3xLinux64Guest', 306, now(), 0);
+INSERT IGNORE INTO `cloud`.`guest_os_hypervisor` (uuid, hypervisor_type, hypervisor_version, guest_os_name, guest_os_id, created, is_user_defined) VALUES (UUID(), 'VMware', '6.5', 'other3xLinux64Guest', 306, now(), 0);
+INSERT IGNORE INTO `cloud`.`guest_os_hypervisor` (uuid, hypervisor_type, hypervisor_version, guest_os_name, guest_os_id, created, is_user_defined) VALUES (UUID(), 'VMware', '6.7', 'other3xLinux64Guest', 306, now(), 0);
+INSERT IGNORE INTO `cloud`.`guest_os_hypervisor` (uuid, hypervisor_type, hypervisor_version, guest_os_name, guest_os_id, created, is_user_defined) VALUES (UUID(), 'VMware', '6.7.1', 'other3xLinux64Guest', 306, now(), 0);
+INSERT IGNORE INTO `cloud`.`guest_os_hypervisor` (uuid, hypervisor_type, hypervisor_version, guest_os_name, guest_os_id, created, is_user_defined) VALUES (UUID(), 'VMware', '6.7.2', 'other3xLinux64Guest', 306, now(), 0);
+INSERT IGNORE INTO `cloud`.`guest_os_hypervisor` (uuid, hypervisor_type, hypervisor_version, guest_os_name, guest_os_id, created, is_user_defined) VALUES (UUID(), 'VMware', '6.7.3', 'other3xLinux64Guest', 306, now(), 0);
+
+INSERT IGNORE INTO `cloud`.`guest_os_hypervisor` (uuid, hypervisor_type, hypervisor_version, guest_os_name, guest_os_id, created, is_user_defined) VALUES (UUID(), 'VMware', '6.0', 'other3xLinuxGuest', 307, now(), 0);
+INSERT IGNORE INTO `cloud`.`guest_os_hypervisor` (uuid, hypervisor_type, hypervisor_version, guest_os_name, guest_os_id, created, is_user_defined) VALUES (UUID(), 'VMware', '6.5', 'other3xLinuxGuest', 307, now(), 0);
+INSERT IGNORE INTO `cloud`.`guest_os_hypervisor` (uuid, hypervisor_type, hypervisor_version, guest_os_name, guest_os_id, created, is_user_defined) VALUES (UUID(), 'VMware', '6.7', 'other3xLinuxGuest', 307, now(), 0);
+INSERT IGNORE INTO `cloud`.`guest_os_hypervisor` (uuid, hypervisor_type, hypervisor_version, guest_os_name, guest_os_id, created, is_user_defined) VALUES (UUID(), 'VMware', '6.7.1', 'other3xLinuxGuest', 307, now(), 0);
+INSERT IGNORE INTO `cloud`.`guest_os_hypervisor` (uuid, hypervisor_type, hypervisor_version, guest_os_name, guest_os_id, created, is_user_defined) VALUES (UUID(), 'VMware', '6.7.2', 'other3xLinuxGuest', 307, now(), 0);
+INSERT IGNORE INTO `cloud`.`guest_os_hypervisor` (uuid, hypervisor_type, hypervisor_version, guest_os_name, guest_os_id, created, is_user_defined) VALUES (UUID(), 'VMware', '6.7.3', 'other3xLinuxGuest', 307, now(), 0);
Review comment:
why are these part of this PR?
##########
File path: api/src/main/java/com/cloud/agent/api/to/deployasis/OVFEulaSectionTO.java
##########
@@ -0,0 +1,49 @@
+//
+// 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.agent.api.to.deployasis;
Review comment:
new class should be in org.apache.cloudstack
##########
File path: core/src/main/java/com/cloud/agent/api/ValidateVcenterDetailsCommand.java
##########
@@ -0,0 +1,40 @@
+// 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.agent.api;
Review comment:
new class should be in org.apache.cloudstack
##########
File path: api/src/main/java/org/apache/cloudstack/api/response/TemplateResponse.java
##########
@@ -194,12 +195,22 @@
@Param(description = "KVM Only: true if template is directly downloaded to Primary Storage bypassing Secondary Storage")
private Boolean directDownload;
+ @SerializedName(ApiConstants.DEPLOY_AS_IS)
+ @Param(description = "VMware only: true if template is deployed without orchestrating disks and networks but \"as-is\" defined in the template.")
Review comment:
add `since = "4.15"`
##########
File path: vmware-base/src/main/java/com/cloud/hypervisor/vmware/mo/HypervisorHostHelper.java
##########
@@ -1773,22 +1824,55 @@ public static void importVmFromOVF(VmwareHypervisorHost host, String ovfFilePath
}
}
- DatacenterMO dcMo = new DatacenterMO(context, host.getHyperHostDatacenter());
- ManagedObjectReference morLease = context.getService().importVApp(morRp, ovfImportResult.getImportSpec(), dcMo.getVmFolder(), morHost);
+ DatacenterMO dcMo = null;
+ try {
+ dcMo = new DatacenterMO(context, host.getHyperHostDatacenter());
+ } catch (Exception e) {
+ throw new CloudRuntimeException(String.format("no datacenter for host '%s' available in context", context.getServerAddress()), e);
+ }
+ ManagedObjectReference folderMO = null;
+ try {
+ folderMO = dcMo.getVmFolder();
+ } catch (Exception e) {
+ throw new CloudRuntimeException("no management handle for VmFolder", e);
+ }
+ ManagedObjectReference morLease = null;
+ try {
+ morLease = context.getService().importVApp(morRp, ovfImportResult.getImportSpec(), folderMO, morHost);
+ } catch (DuplicateNameFaultMsg
+ | FileFaultFaultMsg
+ | InsufficientResourcesFaultFaultMsg
+ | InvalidDatastoreFaultMsg
+ | InvalidNameFaultMsg
+ | OutOfBoundsFaultMsg
+ | RuntimeFaultFaultMsg
+ | VmConfigFaultFaultMsg fault) {
+ throw new CloudRuntimeException("import vApp failed",fault);
+ }
if (morLease == null) {
String msg = "importVApp() failed. ovfFilePath: " + ovfFilePath + ", vmName: " + vmName + ", diskOption: " + diskOption;
s_logger.error(msg);
- throw new Exception(msg);
+ throw new CloudRuntimeException(msg);
}
boolean importSuccess = true;
final HttpNfcLeaseMO leaseMo = new HttpNfcLeaseMO(context, morLease);
- HttpNfcLeaseState state = leaseMo.waitState(new HttpNfcLeaseState[] {HttpNfcLeaseState.READY, HttpNfcLeaseState.ERROR});
+ HttpNfcLeaseState state = null;
+ try {
+ state = leaseMo.waitState(new HttpNfcLeaseState[] {HttpNfcLeaseState.READY, HttpNfcLeaseState.ERROR});
+ } catch (Exception e) {
+ throw new CloudRuntimeException("exception while waiting for leaseMO", e);
+ }
Review comment:
new method
##########
File path: api/src/main/java/com/cloud/agent/api/to/deployasis/OVFConfigurationTO.java
##########
@@ -0,0 +1,61 @@
+//
+// 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.agent.api.to.deployasis;
Review comment:
new class should be in org.apache.cloudstack
##########
File path: vmware-base/src/main/java/com/cloud/hypervisor/vmware/mo/HypervisorHostHelper.java
##########
@@ -1773,22 +1824,55 @@ public static void importVmFromOVF(VmwareHypervisorHost host, String ovfFilePath
}
}
- DatacenterMO dcMo = new DatacenterMO(context, host.getHyperHostDatacenter());
- ManagedObjectReference morLease = context.getService().importVApp(morRp, ovfImportResult.getImportSpec(), dcMo.getVmFolder(), morHost);
+ DatacenterMO dcMo = null;
+ try {
+ dcMo = new DatacenterMO(context, host.getHyperHostDatacenter());
+ } catch (Exception e) {
+ throw new CloudRuntimeException(String.format("no datacenter for host '%s' available in context", context.getServerAddress()), e);
+ }
+ ManagedObjectReference folderMO = null;
+ try {
+ folderMO = dcMo.getVmFolder();
+ } catch (Exception e) {
+ throw new CloudRuntimeException("no management handle for VmFolder", e);
+ }
+ ManagedObjectReference morLease = null;
+ try {
+ morLease = context.getService().importVApp(morRp, ovfImportResult.getImportSpec(), folderMO, morHost);
+ } catch (DuplicateNameFaultMsg
+ | FileFaultFaultMsg
+ | InsufficientResourcesFaultFaultMsg
+ | InvalidDatastoreFaultMsg
+ | InvalidNameFaultMsg
+ | OutOfBoundsFaultMsg
+ | RuntimeFaultFaultMsg
+ | VmConfigFaultFaultMsg fault) {
+ throw new CloudRuntimeException("import vApp failed",fault);
+ }
if (morLease == null) {
String msg = "importVApp() failed. ovfFilePath: " + ovfFilePath + ", vmName: " + vmName + ", diskOption: " + diskOption;
s_logger.error(msg);
- throw new Exception(msg);
+ throw new CloudRuntimeException(msg);
}
boolean importSuccess = true;
final HttpNfcLeaseMO leaseMo = new HttpNfcLeaseMO(context, morLease);
- HttpNfcLeaseState state = leaseMo.waitState(new HttpNfcLeaseState[] {HttpNfcLeaseState.READY, HttpNfcLeaseState.ERROR});
+ HttpNfcLeaseState state = null;
+ try {
+ state = leaseMo.waitState(new HttpNfcLeaseState[] {HttpNfcLeaseState.READY, HttpNfcLeaseState.ERROR});
+ } catch (Exception e) {
+ throw new CloudRuntimeException("exception while waiting for leaseMO", e);
+ }
try {
if (state == HttpNfcLeaseState.READY) {
final long totalBytes = HttpNfcLeaseMO.calcTotalBytes(ovfImportResult);
File ovfFile = new File(ovfFilePath);
- HttpNfcLeaseInfo httpNfcLeaseInfo = leaseMo.getLeaseInfo();
+ HttpNfcLeaseInfo httpNfcLeaseInfo = null;
+ try {
+ httpNfcLeaseInfo = leaseMo.getLeaseInfo();
+ } catch (Exception e) {
+ throw new CloudRuntimeException("error waiting for lease info", e);
+ }
Review comment:
new method
##########
File path: api/src/main/java/com/cloud/agent/api/to/OVFInformationTO.java
##########
@@ -0,0 +1,95 @@
+//
+// 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.agent.api.to;
Review comment:
new class should be in org.apache.cloudstack
##########
File path: plugins/hypervisors/vmware/src/main/java/com/cloud/hypervisor/vmware/resource/VmwareResource.java
##########
@@ -745,20 +752,38 @@ private Answer execute(ResizeVolumeCommand cmd) {
} else if (newSize == oldSize) {
return new ResizeVolumeAnswer(cmd, true, "success", newSize * ResourceType.bytesToKiB);
}
+ /*
+ // FR41 this is yet to fix
+ ManagedObjectReference morDS1 = HypervisorHostHelper.findDatastoreWithBackwardsCompatibility(hyperHost, cmd.getPoolUuid());
+ DatastoreMO dsMo1 = new DatastoreMO(hyperHost.getContext(), morDS1);
+ vmdkDataStorePath = VmwareStorageLayoutHelper.getLegacyDatastorePathFromVmdkFileName(dsMo1, path + VMDK_EXTENSION);
+ DatastoreFile dsFile1 = new DatastoreFile(vmdkDataStorePath);
+
+ s_logger.debug("vDiskid does not exist for volume " + vmdkDataStorePath + " registering the disk now");
+ VirtualStorageObjectManagerMO vStorageObjectManagerMO = new VirtualStorageObjectManagerMO(getServiceContext());
+ try {
+ VStorageObject vStorageObject = vStorageObjectManagerMO.registerVirtualDisk(dsFile1, null, dsMo1.getOwnerDatacenter().second());
+ VStorageObjectConfigInfo diskConfigInfo = vStorageObject.getConfig();
+ ID vdiskId = diskConfigInfo.getId();
+ } catch (Throwable e) {
+ if (e instanceof AlreadyExistsFaultMsg) {
+
+ }
+ }*/
Review comment:
code in comment please remove
##########
File path: api/src/main/java/com/cloud/agent/api/to/deployasis/OVFConfigurationTO.java
##########
@@ -0,0 +1,61 @@
+//
+// 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.agent.api.to.deployasis;
+
+import java.util.List;
+
+public class OVFConfigurationTO implements TemplateDeployAsIsInformationTO {
Review comment:
small javadoc ?
```suggestion
/**
* contains lists of configurations items, parsed from the OVF
*/
public class OVFConfigurationTO implements TemplateDeployAsIsInformationTO {
```
##########
File path: api/src/main/java/com/cloud/agent/api/to/deployasis/OVFVirtualHardwareItemTO.java
##########
@@ -0,0 +1,365 @@
+// 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.agent.api.to.deployasis;
+
+// From: https://schemas.dmtf.org/wbem/wscim/1/cim-schema/2/CIM_ResourceAllocationSettingData.xsd
Review comment:
please change to javadoc, adding two or three words? nothing special
##########
File path: api/src/main/java/com/cloud/agent/api/to/deployasis/OVFVirtualHardwareItemTO.java
##########
@@ -0,0 +1,365 @@
+// 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.agent.api.to.deployasis;
Review comment:
new class should be in `org.apache.cloudstack`
##########
File path: engine/api/src/main/java/org/apache/cloudstack/engine/orchestration/service/VolumeOrchestrationService.java
##########
@@ -117,8 +120,11 @@ DiskProfile allocateRawVolume(Type type, String name, DiskOffering offering, Lon
boolean canVmRestartOnAnotherServer(long vmId);
- DiskProfile allocateTemplatedVolume(Type type, String name, DiskOffering offering, Long rootDisksize, Long minIops, Long maxIops, VirtualMachineTemplate template, VirtualMachine vm,
- Account owner);
+ /**
Review comment:
👍
##########
File path: plugins/hypervisors/vmware/src/main/java/org/apache/cloudstack/api/command/admin/zone/ListVsphereStoragePoliciesCmd.java
##########
@@ -0,0 +1,109 @@
+// 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 org.apache.cloudstack.api.command.admin.zone;
+
+import com.cloud.dc.DataCenter;
+import com.cloud.dc.VsphereStoragePolicy;
+import com.cloud.exception.ConcurrentOperationException;
+import com.cloud.exception.InsufficientCapacityException;
+import com.cloud.exception.NetworkRuleConflictException;
+import com.cloud.exception.ResourceAllocationException;
+import com.cloud.exception.ResourceUnavailableException;
+import com.cloud.hypervisor.vmware.VmwareDatacenterService;
+import org.apache.cloudstack.acl.RoleType;
+import org.apache.cloudstack.api.APICommand;
+import org.apache.cloudstack.api.ApiConstants;
+import org.apache.cloudstack.api.ApiErrorCode;
+import org.apache.cloudstack.api.BaseCmd;
+import org.apache.cloudstack.api.Parameter;
+import org.apache.cloudstack.api.ServerApiException;
+import org.apache.cloudstack.api.response.ListResponse;
+import org.apache.cloudstack.api.response.VsphereStoragePoliciesResponse;
+import org.apache.cloudstack.api.response.ZoneResponse;
+import org.apache.cloudstack.context.CallContext;
+import org.apache.log4j.Logger;
+
+import javax.inject.Inject;
+import java.util.ArrayList;
+import java.util.List;
+
+@APICommand(name = ListVsphereStoragePoliciesCmd.APINAME, description = "List vSphere storage policies",
+ responseObject = VsphereStoragePoliciesResponse.class,
+ requestHasSensitiveInfo = false, responseHasSensitiveInfo = false,
+ authorized = {RoleType.Admin})
+public class ListVsphereStoragePoliciesCmd extends BaseCmd {
Review comment:
```suggestion
@APICommand(name = ListVsphereStoragePoliciesCmd.APINAME, description = "List vSphere storage policies",
responseObject = VsphereStoragePoliciesResponse.class,
requestHasSensitiveInfo = false, responseHasSensitiveInfo = false,
since = "4.15", authorized = {RoleType.Admin})
public class ListVsphereStoragePoliciesCmd extends BaseCmd {
```
##########
File path: api/src/main/java/com/cloud/agent/api/to/deployasis/TemplateDeployAsIsInformationTO.java
##########
@@ -0,0 +1,24 @@
+//
+// 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.agent.api.to.deployasis;
+
+import java.io.Serializable;
+
+public interface TemplateDeployAsIsInformationTO extends Serializable {
Review comment:
what is this needed for?
##########
File path: api/src/main/java/com/cloud/agent/api/to/OVFInformationTO.java
##########
@@ -0,0 +1,95 @@
+//
+// 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.agent.api.to;
+
+import com.cloud.agent.api.LogLevel;
+import com.cloud.agent.api.to.deployasis.OVFEulaSectionTO;
+import com.cloud.agent.api.to.deployasis.OVFNetworkTO;
+import com.cloud.agent.api.to.deployasis.OVFPropertyTO;
+import com.cloud.agent.api.to.deployasis.OVFVirtualHardwareSectionTO;
+import com.cloud.utils.Pair;
+
+import java.util.List;
+
+public class OVFInformationTO {
Review comment:
small javadoc?
##########
File path: api/src/main/java/com/cloud/agent/api/to/deployasis/OVFVirtualHardwareSectionTO.java
##########
@@ -0,0 +1,50 @@
+//
+// 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.agent.api.to.deployasis;
+
+import java.util.List;
+
+public class OVFVirtualHardwareSectionTO implements TemplateDeployAsIsInformationTO {
Review comment:
good name ==> no javadoc needed
##########
File path: api/src/main/java/com/cloud/agent/api/to/deployasis/OVFVirtualHardwareSectionTO.java
##########
@@ -0,0 +1,50 @@
+//
+// 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.agent.api.to.deployasis;
Review comment:
new class should be in `org.apache.cloudstack`
##########
File path: core/src/main/java/com/cloud/agent/api/ValidateVcenterDetailsCommand.java
##########
@@ -0,0 +1,40 @@
+// 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.agent.api;
+
+public class ValidateVcenterDetailsCommand extends Command {
Review comment:
what is it for?
##########
File path: core/src/main/java/org/apache/cloudstack/storage/command/CheckDataStoreStoragePolicyComplainceCommand.java
##########
@@ -0,0 +1,61 @@
+//
+// 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 org.apache.cloudstack.storage.command;
+
+import com.cloud.agent.api.to.StorageFilerTO;
+
+public class CheckDataStoreStoragePolicyComplainceCommand extends StorageSubSystemCommand {
+
+ String storagePolicyId;
+ private StorageFilerTO storagePool;
+
+ public CheckDataStoreStoragePolicyComplainceCommand(String storagePolicyId, StorageFilerTO storagePool) {
Review comment:
short javadoc with descrition of this command-answer pattern? at least what is returned under what condition? just by the name it might be a `boolean` or a `List<StoragePolicy>` that is expected.
##########
File path: plugins/hypervisors/vmware/src/main/java/com/cloud/storage/resource/VmwareStorageProcessor.java
##########
@@ -1124,6 +1203,12 @@ private void postCreatePrivateTemplate(String installFullPath, long templateId,
throw new Exception(msg);
}
+ DatacenterMO dcMo = new DatacenterMO(context, hyperHost.getHyperHostDatacenter());
+ ManagedObjectReference morPool = hyperHost.getHyperHostOwnerResourcePool();
+ vmMo.createFullCloneWithSpecificDisk(templateUniqueName, dcMo.getVmFolder(), morPool, VmwareHelper.getDiskDeviceDatastore(volumeDeviceInfo.first()), volumeDeviceInfo);
+ clonedVm = dcMo.findVm(templateUniqueName);
+
+ /* FR41 THIS IS OLD way of creating template using snapshot
Review comment:
internal project code in open source version, please remove.
##########
File path: api/src/main/java/com/cloud/agent/api/to/DeployAsIsInfoTO.java
##########
@@ -0,0 +1,58 @@
+// 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.agent.api.to;
+
+import com.cloud.agent.api.LogLevel;
+
+import java.util.HashMap;
+import java.util.Map;
+
+public class DeployAsIsInfoTO {
Review comment:
javadoc, what on earth is an `DeployAsIsInfoTO`?
##########
File path: server/src/main/java/com/cloud/storage/StorageManagerImpl.java
##########
@@ -1498,6 +1610,16 @@ public PrimaryDataStoreInfo cancelPrimaryStorageForMaintenance(CancelPrimaryStor
DataStoreProvider provider = _dataStoreProviderMgr.getDataStoreProvider(primaryStorage.getStorageProviderName());
DataStoreLifeCycle lifeCycle = provider.getDataStoreLifeCycle();
DataStore store = _dataStoreMgr.getDataStore(primaryStorage.getId(), DataStoreRole.Primary);
+ if (primaryStorage.getPoolType() == StoragePoolType.DatastoreCluster) {
+ primaryStorage.setStatus(StoragePoolStatus.Up);
+ _storagePoolDao.update(primaryStorage.getId(), primaryStorage);
+ //FR41 need to handle when one of the primary stores is unable to cancel the maintenance mode
+ List<StoragePoolVO> childDatastores = _storagePoolDao.listChildStoragePoolsInDatastoreCluster(primaryStorageId);
+ for (StoragePoolVO childDatastore : childDatastores) {
+ DataStore childStore = _dataStoreMgr.getDataStore(childDatastore.getId(), DataStoreRole.Primary);
+ lifeCycle.cancelMaintain(childStore);
+ }
+ }
Review comment:
separate method
##########
File path: engine/orchestration/src/main/java/org/apache/cloudstack/engine/orchestration/NetworkOrchestrator.java
##########
@@ -747,87 +751,168 @@ public void doInTransactionWithoutResult(final TransactionStatus status) {
@Override
@DB
public void allocate(final VirtualMachineProfile vm, final LinkedHashMap<? extends Network, List<? extends NicProfile>> networks, final Map<String, Map<Integer, String>> extraDhcpOptions) throws InsufficientCapacityException,
Review comment:
fromat request
```suggestion
public void allocate(final VirtualMachineProfile vm, final LinkedHashMap<? extends Network, List<? extends NicProfile>> networks, final Map<String, Map<Integer, String>> extraDhcpOptions)
throws InsufficientCapacityException, ConcurrentOperationException {
```
but of course delete the next line (github sugestions are almost smart enough for this.
##########
File path: core/src/test/java/com/cloud/agent/api/storage/DownloadAnswerTest.java
##########
@@ -0,0 +1,62 @@
+// 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.agent.api.storage;
Review comment:
new class should be in `org.apache.cloudstack`
##########
File path: engine/orchestration/src/main/java/org/apache/cloudstack/engine/orchestration/NetworkOrchestrator.java
##########
@@ -747,87 +751,168 @@ public void doInTransactionWithoutResult(final TransactionStatus status) {
@Override
@DB
public void allocate(final VirtualMachineProfile vm, final LinkedHashMap<? extends Network, List<? extends NicProfile>> networks, final Map<String, Map<Integer, String>> extraDhcpOptions) throws InsufficientCapacityException,
- ConcurrentOperationException {
+ ConcurrentOperationException {
Review comment:
```suggestion
```
but only in combination with above
##########
File path: engine/api/src/main/java/org/apache/cloudstack/engine/subsystem/api/storage/TemplateDataFactory.java
##########
@@ -27,7 +27,7 @@
TemplateInfo getReadyTemplateOnImageStore(long templateId, Long zoneId);
- TemplateInfo getTemplate(DataObject obj, DataStore store);
+ TemplateInfo getTemplate(DataObject obj, DataStore store, String configuration);
Review comment:
i would like a javadoc about what this string entails
##########
File path: engine/orchestration/src/main/java/com/cloud/vm/VirtualMachineManagerImpl.java
##########
@@ -592,6 +597,13 @@ protected void advanceExpunge(VMInstanceVO vm) throws ResourceUnavailableExcepti
//remove the overcommit details from the uservm details
userVmDetailsDao.removeDetails(vm.getId());
+ // Remove details if VM deploy as-is
Review comment:
this comment could be the method-name of a member being called here.
`long templateId` should be scope protected to avoid confusion in this 114! line method.
##########
File path: api/src/main/java/com/cloud/agent/api/to/deployasis/TemplateDeployAsIsInformationTO.java
##########
@@ -0,0 +1,24 @@
+//
+// 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.agent.api.to.deployasis;
Review comment:
new class should be in `org.apache.cloudstack`
##########
File path: engine/orchestration/src/main/java/org/apache/cloudstack/engine/orchestration/NetworkOrchestrator.java
##########
@@ -1105,7 +1190,7 @@ boolean isNetworkImplemented(final NetworkVO network) {
@Override
@DB
public Pair<NetworkGuru, NetworkVO> implementNetwork(final long networkId, final DeployDestination dest, final ReservationContext context) throws ConcurrentOperationException,
- ResourceUnavailableException, InsufficientCapacityException {
+ ResourceUnavailableException, InsufficientCapacityException {
Review comment:
with throws?
##########
File path: engine/orchestration/src/main/java/org/apache/cloudstack/engine/orchestration/NetworkOrchestrator.java
##########
@@ -3056,7 +3141,7 @@ public boolean startNetwork(final long networkId, final DeployDestination dest,
@Override
public boolean restartNetwork(final Long networkId, final Account callerAccount, final User callerUser, final boolean cleanup) throws ConcurrentOperationException, ResourceUnavailableException,
- InsufficientCapacityException {
+ InsufficientCapacityException {
Review comment:
and this one (with `throws`)
##########
File path: engine/orchestration/src/main/java/org/apache/cloudstack/engine/orchestration/VolumeOrchestrator.java
##########
@@ -296,6 +315,34 @@ public StoragePool findStoragePool(DiskProfile dskCh, DataCenter dc, Pod pod, Lo
return null;
}
+ @Override
+ public StoragePool findChildDataStoreInDataStoreCluster(DataCenter dc, Pod pod, Long clusterId, Long hostId, VirtualMachine vm, Long datastoreClusterId) {
+ Long podId = null;
+ if (pod != null) {
+ podId = pod.getId();
+ } else if (clusterId != null) {
+ Cluster cluster = _entityMgr.findById(Cluster.class, clusterId);
+ if (cluster != null) {
+ podId = cluster.getPodId();
+ }
+ }
+ List<StoragePoolVO> childDatastores = _storagePoolDao.listChildStoragePoolsInDatastoreCluster(datastoreClusterId);
+ List<StoragePool> suitablePools = new ArrayList<StoragePool>();
+
+ for (StoragePoolVO childDatastore: childDatastores)
+ suitablePools.add((StoragePool)dataStoreMgr.getDataStore(childDatastore.getId(), DataStoreRole.Primary));
+
+ VirtualMachineProfile profile = new VirtualMachineProfileImpl(vm);
+ for (StoragePoolAllocator allocator : _storagePoolAllocators) {
+ DataCenterDeployment plan = new DataCenterDeployment(dc.getId(), podId, clusterId, hostId, null, null);
+ final List<StoragePool> poolList = allocator.reorderPools(suitablePools, profile, plan);
+
+ if (poolList != null && !poolList.isEmpty()) {
+ return (StoragePool)dataStoreMgr.getDataStore(poolList.get(0).getId(), DataStoreRole.Primary);
+ }
+ }
+ return null;
+ }
Review comment:
newline:
```suggestion
}
```
##########
File path: engine/orchestration/src/main/java/org/apache/cloudstack/engine/orchestration/NetworkOrchestrator.java
##########
@@ -747,87 +751,168 @@ public void doInTransactionWithoutResult(final TransactionStatus status) {
@Override
@DB
public void allocate(final VirtualMachineProfile vm, final LinkedHashMap<? extends Network, List<? extends NicProfile>> networks, final Map<String, Map<Integer, String>> extraDhcpOptions) throws InsufficientCapacityException,
- ConcurrentOperationException {
+ ConcurrentOperationException {
Transaction.execute(new TransactionCallbackWithExceptionNoReturn<InsufficientCapacityException>() {
@Override
public void doInTransactionWithoutResult(final TransactionStatus status) throws InsufficientCapacityException {
- int deviceId = 0;
- int size = 0;
- for (final Network ntwk : networks.keySet()) {
- final List<? extends NicProfile> profiles = networks.get(ntwk);
- if (profiles != null && !profiles.isEmpty()) {
- size = size + profiles.size();
- } else {
- size = size + 1;
- }
+ if (s_logger.isTraceEnabled()) {
+ s_logger.trace(String.format("allocating networks for %s(template %s); %d networks",vm.getInstanceName(), vm.getTemplate().getUuid(), networks.size()));
}
+ int deviceId = 0;
+ int size;
+ size = determineNumberOfNicsRequired();
final boolean[] deviceIds = new boolean[size];
Arrays.fill(deviceIds, false);
+ List<Pair<Network, NicProfile>> profilesList = getOrderedNetworkNicProfileMapping(networks);
final List<NicProfile> nics = new ArrayList<NicProfile>(size);
NicProfile defaultNic = null;
+ Network nextNetwork = null;
+ for (Pair <Network, NicProfile> networkNicPair : profilesList) {
+ nextNetwork = networkNicPair.first();
+ Pair<NicProfile, Integer> newDeviceInfo = addRequestedNicToNicListWithDeviceNumberAndRetrieveDefaultDevice(networkNicPair.second(), deviceIds, deviceId, nextNetwork, nics, defaultNic);
+ defaultNic = newDeviceInfo.first();
+ deviceId = newDeviceInfo.second();
+ }
+ createExtraNics(size, nics, nextNetwork);
+
+ if (nics.size() == 1) {
+ nics.get(0).setDefaultNic(true);
+ }
+ }
+
+ /**
+ * private transaction method to check and add devices to the nic list and update the info
+ */
+ Pair<NicProfile,Integer> addRequestedNicToNicListWithDeviceNumberAndRetrieveDefaultDevice(NicProfile requested, boolean[] deviceIds, int deviceId, Network nextNetwork, List<NicProfile> nics, NicProfile defaultNic)
+ throws InsufficientAddressCapacityException, InsufficientVirtualNetworkCapacityException {
Review comment:
👍 this is the kind of formatting i'd like to see for all methods (i.e. throws on a new line if the signature is (too) long
##########
File path: engine/schema/src/main/java/com/cloud/dc/dao/VsphereStoragePolicyDao.java
##########
@@ -14,18 +14,17 @@
// KIND, either express or implied. See the License for the
// specific language governing permissions and limitations
// under the License.
+package com.cloud.dc.dao;
Review comment:
new classes should be in org.apache.cloudstack
##########
File path: engine/orchestration/src/main/java/org/apache/cloudstack/engine/orchestration/NetworkOrchestrator.java
##########
@@ -747,87 +751,168 @@ public void doInTransactionWithoutResult(final TransactionStatus status) {
@Override
@DB
public void allocate(final VirtualMachineProfile vm, final LinkedHashMap<? extends Network, List<? extends NicProfile>> networks, final Map<String, Map<Integer, String>> extraDhcpOptions) throws InsufficientCapacityException,
- ConcurrentOperationException {
+ ConcurrentOperationException {
Transaction.execute(new TransactionCallbackWithExceptionNoReturn<InsufficientCapacityException>() {
@Override
public void doInTransactionWithoutResult(final TransactionStatus status) throws InsufficientCapacityException {
- int deviceId = 0;
- int size = 0;
- for (final Network ntwk : networks.keySet()) {
- final List<? extends NicProfile> profiles = networks.get(ntwk);
- if (profiles != null && !profiles.isEmpty()) {
- size = size + profiles.size();
- } else {
- size = size + 1;
- }
+ if (s_logger.isTraceEnabled()) {
+ s_logger.trace(String.format("allocating networks for %s(template %s); %d networks",vm.getInstanceName(), vm.getTemplate().getUuid(), networks.size()));
}
+ int deviceId = 0;
+ int size;
+ size = determineNumberOfNicsRequired();
final boolean[] deviceIds = new boolean[size];
Arrays.fill(deviceIds, false);
+ List<Pair<Network, NicProfile>> profilesList = getOrderedNetworkNicProfileMapping(networks);
final List<NicProfile> nics = new ArrayList<NicProfile>(size);
NicProfile defaultNic = null;
+ Network nextNetwork = null;
+ for (Pair <Network, NicProfile> networkNicPair : profilesList) {
+ nextNetwork = networkNicPair.first();
+ Pair<NicProfile, Integer> newDeviceInfo = addRequestedNicToNicListWithDeviceNumberAndRetrieveDefaultDevice(networkNicPair.second(), deviceIds, deviceId, nextNetwork, nics, defaultNic);
+ defaultNic = newDeviceInfo.first();
+ deviceId = newDeviceInfo.second();
+ }
+ createExtraNics(size, nics, nextNetwork);
+
+ if (nics.size() == 1) {
+ nics.get(0).setDefaultNic(true);
+ }
+ }
+
+ /**
+ * private transaction method to check and add devices to the nic list and update the info
+ */
+ Pair<NicProfile,Integer> addRequestedNicToNicListWithDeviceNumberAndRetrieveDefaultDevice(NicProfile requested, boolean[] deviceIds, int deviceId, Network nextNetwork, List<NicProfile> nics, NicProfile defaultNic)
+ throws InsufficientAddressCapacityException, InsufficientVirtualNetworkCapacityException {
+ Pair<NicProfile, Integer> rc = new Pair<>(null,null);
+ Boolean isDefaultNic = false;
+ if (vm != null && requested != null && requested.isDefaultNic()) {
+ isDefaultNic = true;
+ }
+
+ while (deviceIds[deviceId] && deviceId < deviceIds.length) {
+ deviceId++;
+ }
+
+ final Pair<NicProfile, Integer> vmNicPair = allocateNic(requested, nextNetwork, isDefaultNic, deviceId, vm);
+ NicProfile vmNic = null;
+ if (vmNicPair != null) {
+ vmNic = vmNicPair.first();
+ if (vmNic == null) {
+ return rc;
+ }
+ deviceId = vmNicPair.second();
+ }
+
+ final int devId = vmNic.getDeviceId();
+ if (devId >= deviceIds.length) {
+ throw new IllegalArgumentException("Device id for nic is too large: " + vmNic);
+ }
+ if (deviceIds[devId]) {
+ throw new IllegalArgumentException("Conflicting device id for two different nics: " + vmNic);
+ }
+
+ deviceIds[devId] = true;
+ if (vmNic.isDefaultNic()) {
+ if (defaultNic != null) {
+ throw new IllegalArgumentException("You cannot specify two nics as default nics: nic 1 = " + defaultNic + "; nic 2 = " + vmNic);
+ }
+ defaultNic = vmNic;
+ }
+
+ nics.add(vmNic);
+ vm.addNic(vmNic);
+ saveExtraDhcpOptions(nextNetwork.getUuid(), vmNic.getId(), extraDhcpOptions);
+ rc.first(defaultNic);
+ rc.second(deviceId);
+ return rc;
+ }
+
+ /**
+ * private transaction method to get oredered list of Network and NicProfile pair
+ * @return ordered list of Network and NicProfile pair
+ * @param networks the map od networks to nic profiles list
+ */
+ private List<Pair<Network, NicProfile>> getOrderedNetworkNicProfileMapping(final LinkedHashMap<? extends Network, List<? extends NicProfile>> networks) {
+ List<Pair<Network, NicProfile>> profilesList = new ArrayList<>();
for (final Map.Entry<? extends Network, List<? extends NicProfile>> network : networks.entrySet()) {
- final Network config = network.getKey();
List<? extends NicProfile> requestedProfiles = network.getValue();
if (requestedProfiles == null) {
requestedProfiles = new ArrayList<NicProfile>();
}
if (requestedProfiles.isEmpty()) {
requestedProfiles.add(null);
}
-
for (final NicProfile requested : requestedProfiles) {
- Boolean isDefaultNic = false;
- if (vm != null && requested != null && requested.isDefaultNic()) {
- isDefaultNic = true;
- }
-
- while (deviceIds[deviceId] && deviceId < deviceIds.length) {
- deviceId++;
- }
-
- final Pair<NicProfile, Integer> vmNicPair = allocateNic(requested, config, isDefaultNic, deviceId, vm);
- NicProfile vmNic = null;
- if (vmNicPair != null) {
- vmNic = vmNicPair.first();
- if (vmNic == null) {
- continue;
- }
- deviceId = vmNicPair.second();
- }
-
- final int devId = vmNic.getDeviceId();
- if (devId >= deviceIds.length) {
- throw new IllegalArgumentException("Device id for nic is too large: " + vmNic);
- }
- if (deviceIds[devId]) {
- throw new IllegalArgumentException("Conflicting device id for two different nics: " + vmNic);
+ profilesList.add(new Pair<Network, NicProfile>(network.getKey(), requested));
+ }
+ }
+ profilesList.sort(new Comparator<Pair<Network, NicProfile>>() {
+ @Override
+ public int compare(Pair<Network, NicProfile> pair1, Pair<Network, NicProfile> pair2) {
+ int profile1Order = Integer.MAX_VALUE;
+ int profile2Order = Integer.MAX_VALUE;
+ if (pair1 != null && pair1.second() != null && pair1.second().getOrderIndex() != null) {
+ profile1Order = pair1.second().getOrderIndex();
}
-
- deviceIds[devId] = true;
-
- if (vmNic.isDefaultNic()) {
- if (defaultNic != null) {
- throw new IllegalArgumentException("You cannot specify two nics as default nics: nic 1 = " + defaultNic + "; nic 2 = " + vmNic);
- }
- defaultNic = vmNic;
+ if (pair2 != null && pair2.second() != null && pair2.second().getOrderIndex() != null) {
+ profile2Order = pair2.second().getOrderIndex();
}
+ return profile1Order - profile2Order;
+ }
+ });
+ return profilesList;
+ }
- nics.add(vmNic);
- vm.addNic(vmNic);
- saveExtraDhcpOptions(config.getUuid(), vmNic.getId(), extraDhcpOptions);
+ /**
+ * private transaction method to run over the objects and determine nic requirements
+ * @return the total numer of nics required
+ */
+ private int determineNumberOfNicsRequired() {
+ int size = 0;
+ for (final Network ntwk : networks.keySet()) {
+ final List<? extends NicProfile> profiles = networks.get(ntwk);
+ if (profiles != null && !profiles.isEmpty()) {
+ size = size + profiles.size();
+ } else {
+ size = size + 1;
}
}
+
+ List<OVFNetworkTO> netprereqs = templateDeployAsIsDetailsDao.listNetworkRequirementsByTemplateId(vm.getTemplate().getId());
+ if (size < netprereqs.size()) {
+ size = netprereqs.size();
+ }
+ return size;
+ }
+
+ /**
+ * private transaction method to add nics as required
+ * @param size the number needed
+ * @param nics the list of nics present
+ * @param finalNetwork the network to add the nics to
+ * @throws InsufficientVirtualNetworkCapacityException great
+ * @throws InsufficientAddressCapacityException also magnificent, as the name sugests
+ */
+ private void createExtraNics(int size, List<NicProfile> nics, Network finalNetwork) throws InsufficientVirtualNetworkCapacityException, InsufficientAddressCapacityException {
if (nics.size() != size) {
s_logger.warn("Number of nics " + nics.size() + " doesn't match number of requested nics " + size);
- throw new CloudRuntimeException("Number of nics " + nics.size() + " doesn't match number of requested networks " + size);
- }
-
- if (nics.size() == 1) {
- nics.get(0).setDefaultNic(true);
+ if (nics.size() > size) {
+ throw new CloudRuntimeException("Number of nics " + nics.size() + " doesn't match number of requested networks " + size);
+ } else {
+ if (finalNetwork == null) {
+ throw new CloudRuntimeException(String.format("can not assign network to %d remaining required NICs", size - nics.size()));
+ }
+ // create extra
Review comment:
method name?
##########
File path: engine/orchestration/src/main/java/org/apache/cloudstack/engine/orchestration/NetworkOrchestrator.java
##########
@@ -3033,7 +3118,7 @@ public void reallyRun() {
@Override
public boolean startNetwork(final long networkId, final DeployDestination dest, final ReservationContext context) throws ConcurrentOperationException, ResourceUnavailableException,
- InsufficientCapacityException {
+ InsufficientCapacityException {
Review comment:
`throws`
##########
File path: engine/orchestration/src/main/java/org/apache/cloudstack/engine/orchestration/VolumeOrchestrator.java
##########
@@ -1704,4 +1801,4 @@ public void doInTransactionWithoutResult(TransactionStatus status) {
}
});
}
-}
+}
Review comment:
```suggestion
}
```
##########
File path: engine/schema/src/main/java/com/cloud/dc/dao/VsphereStoragePolicyDaoImpl.java
##########
@@ -0,0 +1,64 @@
+// 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.dc.dao;
Review comment:
new classes should be in `org.apache.cloudstack`
##########
File path: engine/schema/src/main/java/com/cloud/dc/VsphereStoragePolicyVO.java
##########
@@ -0,0 +1,126 @@
+// 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.dc;
Review comment:
new classes should be in `org.apache.cloudstack`
##########
File path: engine/schema/src/main/java/com/cloud/deployasis/dao/TemplateDeployAsIsDetailsDao.java
##########
@@ -0,0 +1,32 @@
+// 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.deployasis.dao;
Review comment:
new classes should be in `org.apache.cloudstack`
##########
File path: engine/schema/src/main/java/com/cloud/deployasis/dao/TemplateDeployAsIsDetailsDaoImpl.java
##########
@@ -0,0 +1,83 @@
+// 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.deployasis.dao;
Review comment:
new classes should be in `org.apache.cloudstack`
##########
File path: engine/schema/src/main/java/com/cloud/storage/dao/VMTemplateDetailsDaoImpl.java
##########
@@ -30,4 +30,4 @@
public void addDetail(long resourceId, String key, String value, boolean display) {
super.addDetail(new VMTemplateDetailVO(resourceId, key, value, display));
}
-}
+}
Review comment:
```suggestion
}
```
##########
File path: engine/schema/src/main/java/com/cloud/deployasis/UserVmDeployAsIsDetailVO.java
##########
@@ -0,0 +1,94 @@
+// 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.deployasis;
Review comment:
new classes should be in `org.apache.cloudstack`
##########
File path: engine/schema/src/main/java/com/cloud/deployasis/TemplateDeployAsIsDetailVO.java
##########
@@ -0,0 +1,94 @@
+// 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.deployasis;
Review comment:
new classes should be in `org.apache.cloudstack`
##########
File path: engine/storage/image/src/main/java/org/apache/cloudstack/storage/image/TemplateServiceImpl.java
##########
@@ -1280,4 +1290,4 @@ protected Void createDatadiskTemplateCallback(AsyncCallbackDispatcher<TemplateSe
future.complete(dataDiskTemplateResult);
return null;
}
-}
+}
Review comment:
```suggestion
}
```
##########
File path: engine/schema/src/main/java/com/cloud/deployasis/dao/UserVmDeployAsIsDetailsDao.java
##########
@@ -0,0 +1,24 @@
+// 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.deployasis.dao;
Review comment:
new classes should be in `org.apache.cloudstack`
##########
File path: engine/schema/src/main/java/com/cloud/deployasis/dao/UserVmDeployAsIsDetailsDaoImpl.java
##########
@@ -0,0 +1,30 @@
+// 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.deployasis.dao;
Review comment:
new classes should be in `org.apache.cloudstack`
##########
File path: engine/storage/src/main/java/org/apache/cloudstack/storage/volume/datastore/PrimaryDataStoreHelper.java
##########
@@ -178,6 +185,13 @@ public DataStore attachZone(DataStore store, HypervisorType hypervisor) {
pool.setHypervisor(hypervisor);
pool.setStatus(StoragePoolStatus.Up);
this.dataStoreDao.update(pool.getId(), pool);
+ if(pool.getPoolType() == StoragePoolType.DatastoreCluster && pool.getParent() == 0) {
+ List<StoragePoolVO> childDatastores = dataStoreDao.listChildStoragePoolsInDatastoreCluster(pool.getId());
+ for (StoragePoolVO child : childDatastores) {
+ child.setScope(ScopeType.ZONE);
+ this.dataStoreDao.update(child.getId(), child);
+ }
+ }
Review comment:
call of `doSomeChildClusterStuff(ScopeType.ZONE, ...)`
##########
File path: engine/storage/volume/src/main/java/org/apache/cloudstack/storage/volume/VolumeServiceImpl.java
##########
@@ -2125,4 +2132,4 @@ public void unmanageVolume(long volumeId) {
volDao.remove(vol.getId());
}
}
-}
+}
Review comment:
```suggestion
}
```
##########
File path: engine/storage/src/main/java/org/apache/cloudstack/storage/image/deployasis/DeployAsIsHelper.java
##########
@@ -0,0 +1,34 @@
+// 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 org.apache.cloudstack.storage.image.deployasis;
+
+import com.cloud.agent.api.storage.DownloadAnswer;
+import com.cloud.agent.api.to.NicTO;
+import com.cloud.vm.VirtualMachineProfile;
+
+import java.util.Map;
+
+public interface DeployAsIsHelper {
+
+ void persistTemplateDeployAsIsDetails(long templateId, DownloadAnswer answer);
+ Map<String, String> getVirtualMachineDeployAsIsProperties(VirtualMachineProfile vmId);
+
+ String getAllocatedVirtualMachineTemplatePath(VirtualMachineProfile vm, String configuration, String destStoragePool);
+ String getAllocatedVirtualMachineDestinationStoragePool(VirtualMachineProfile vm);
+
+ Map<Integer, String> getAllocatedVirtualMachineNicsAdapterMapping(VirtualMachineProfile vm, NicTO[] nics);
+}
Review comment:
can we have some javadoc here? these are helper functions for use in... (storage drivers/network orchestrators/Virtualmachien orchestrators) ?
##########
File path: vmware-base/src/main/java/com/cloud/hypervisor/vmware/mo/VirtualStorageObjectManagerMO.java
##########
@@ -0,0 +1,94 @@
+// 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.hypervisor.vmware.mo;
Review comment:
new classes should go in `org.apache.cloudstack`
##########
File path: vmware-base/src/main/java/com/cloud/hypervisor/vmware/util/VcenterSessionHandler.java
##########
@@ -0,0 +1,88 @@
+// 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.hypervisor.vmware.util;
Review comment:
new classes should go in `org.apache.cloudstack`
##########
File path: engine/storage/src/main/java/org/apache/cloudstack/storage/image/BaseImageStoreDriverImpl.java
##########
@@ -413,25 +383,16 @@ private Answer sendToLeastBusyEndpoint(List<EndPoint> eps, CopyCommand cmd) {
return answer;
} catch (AgentUnavailableException e) {
errMsg = e.toString();
- s_logger.debug("Failed to send command, due to Agent:" + endPoint.getId() + ", " + e.toString());
+ LOGGER.debug("Failed to send command, due to Agent:" + endPoint.getId() + ", " + e.toString());
} catch (OperationTimedoutException e) {
errMsg = e.toString();
- s_logger.debug("Failed to send command, due to Agent:" + endPoint.getId() + ", " + e.toString());
+ LOGGER.debug("Failed to send command, due to Agent:" + endPoint.getId() + ", " + e.toString());
}
throw new CloudRuntimeException("Failed to send command, due to Agent:" + endPoint.getId() + ", " + errMsg);
}
@Override
public boolean canCopy(DataObject srcData, DataObject destData) {
- DataStore srcStore = srcData.getDataStore();
- DataStore destStore = destData.getDataStore();
- if ((srcData.getDataStore().getTO() instanceof NfsTO && destData.getDataStore().getTO() instanceof NfsTO) &&
- (srcStore.getRole() == DataStoreRole.Image && destStore.getRole() == DataStoreRole.Image) &&
- ((srcData.getType() == DataObjectType.TEMPLATE && destData.getType() == DataObjectType.TEMPLATE) ||
- (srcData.getType() == DataObjectType.SNAPSHOT && destData.getType() == DataObjectType.SNAPSHOT) ||
- (srcData.getType() == DataObjectType.VOLUME && destData.getType() == DataObjectType.VOLUME))) {
- return true;
- }
Review comment:
kind of surprising that the BaseImageStoreDriverImpl would be able to copy anything anymore in a PR that is Vmware specific... Why is this?
##########
File path: vmware-base/src/main/java/com/cloud/hypervisor/vmware/mo/HypervisorHostHelper.java
##########
@@ -1738,33 +1775,47 @@ public static String removeOVFNetwork(final String ovfString) {
return ovfString;
}
+ /**
+ * deploys a new VM from a ovf spec. It ignores network, defaults locale to 'US'
+ * @throws Exception shoud be a VmwareResourceException
Review comment:
```suggestion
* @throws CloudRuntimeException should be a VmwareResourceException
* @throws IOException if ovfFilePath not valid?
```
##########
File path: vmware-base/src/main/java/com/cloud/hypervisor/vmware/mo/HypervisorHostHelper.java
##########
@@ -1738,33 +1775,47 @@ public static String removeOVFNetwork(final String ovfString) {
return ovfString;
}
+ /**
Review comment:
👍
##########
File path: engine/storage/src/main/java/org/apache/cloudstack/storage/volume/datastore/PrimaryDataStoreHelper.java
##########
@@ -161,6 +161,13 @@ public DataStore attachCluster(DataStore store) {
pool.setScope(ScopeType.CLUSTER);
pool.setStatus(StoragePoolStatus.Up);
this.dataStoreDao.update(pool.getId(), pool);
+ if(pool.getPoolType() == StoragePoolType.DatastoreCluster && pool.getParent() == 0) {
+ List<StoragePoolVO> childDatastores = dataStoreDao.listChildStoragePoolsInDatastoreCluster(pool.getId());
+ for (StoragePoolVO child : childDatastores) {
+ child.setScope(ScopeType.CLUSTER);
+ this.dataStoreDao.update(child.getId(), child);
+ }
+ }
Review comment:
new method `doSomeChildClusterStuff(ScopeType scopeType, ...)`
##########
File path: vmware-base/src/main/java/com/cloud/hypervisor/vmware/mo/PbmProfileManagerMO.java
##########
@@ -0,0 +1,94 @@
+// 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.hypervisor.vmware.mo;
Review comment:
new classes should go in `org.apache.cloudstack`
##########
File path: vmware-base/src/main/java/com/cloud/hypervisor/vmware/mo/StoragepodMO.java
##########
@@ -0,0 +1,48 @@
+// 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.hypervisor.vmware.mo;
Review comment:
new classes should go in `org.apache.cloudstack`
##########
File path: vmware-base/src/main/java/com/cloud/hypervisor/vmware/mo/HypervisorHostHelper.java
##########
@@ -1773,22 +1824,55 @@ public static void importVmFromOVF(VmwareHypervisorHost host, String ovfFilePath
}
}
- DatacenterMO dcMo = new DatacenterMO(context, host.getHyperHostDatacenter());
- ManagedObjectReference morLease = context.getService().importVApp(morRp, ovfImportResult.getImportSpec(), dcMo.getVmFolder(), morHost);
+ DatacenterMO dcMo = null;
+ try {
+ dcMo = new DatacenterMO(context, host.getHyperHostDatacenter());
+ } catch (Exception e) {
+ throw new CloudRuntimeException(String.format("no datacenter for host '%s' available in context", context.getServerAddress()), e);
+ }
Review comment:
factor in new method
##########
File path: vmware-base/src/main/java/com/cloud/hypervisor/vmware/mo/HypervisorHostHelper.java
##########
@@ -1738,33 +1775,47 @@ public static String removeOVFNetwork(final String ovfString) {
return ovfString;
}
+ /**
+ * deploys a new VM from a ovf spec. It ignores network, defaults locale to 'US'
+ * @throws Exception shoud be a VmwareResourceException
+ */
public static void importVmFromOVF(VmwareHypervisorHost host, String ovfFilePath, String vmName, DatastoreMO dsMo, String diskOption, ManagedObjectReference morRp,
- ManagedObjectReference morHost) throws Exception {
+ ManagedObjectReference morHost, String configurationId) throws CloudRuntimeException, IOException {
assert (morRp != null);
OvfCreateImportSpecParams importSpecParams = new OvfCreateImportSpecParams();
importSpecParams.setHostSystem(morHost);
importSpecParams.setLocale("US");
importSpecParams.setEntityName(vmName);
- importSpecParams.setDeploymentOption("");
+ String deploymentOption = StringUtils.isNotBlank(configurationId) ? configurationId : "";
+ importSpecParams.setDeploymentOption(deploymentOption);
importSpecParams.setDiskProvisioning(diskOption); // diskOption: thin, thick, etc
String ovfDescriptor = removeOVFNetwork(HttpNfcLeaseMO.readOvfContent(ovfFilePath));
VmwareContext context = host.getContext();
- OvfCreateImportSpecResult ovfImportResult =
- context.getService().createImportSpec(context.getServiceContent().getOvfManager(), ovfDescriptor, morRp, dsMo.getMor(), importSpecParams);
-
+ OvfCreateImportSpecResult ovfImportResult = null;
+ try {
+ ovfImportResult = context.getService().createImportSpec(context.getServiceContent().getOvfManager(), ovfDescriptor, morRp, dsMo.getMor(), importSpecParams);
+ } catch (ConcurrentAccessFaultMsg
+ | FileFaultFaultMsg
+ | InvalidDatastoreFaultMsg
+ | InvalidStateFaultMsg
+ | RuntimeFaultFaultMsg
+ | TaskInProgressFaultMsg
+ | VmConfigFaultFaultMsg error) {
+ throw new CloudRuntimeException("ImportSpec creation failed", error);
+ }
Review comment:
factorred out in new method?
##########
File path: vmware-base/src/main/java/com/cloud/hypervisor/vmware/mo/HypervisorHostHelper.java
##########
@@ -1773,22 +1824,55 @@ public static void importVmFromOVF(VmwareHypervisorHost host, String ovfFilePath
}
}
- DatacenterMO dcMo = new DatacenterMO(context, host.getHyperHostDatacenter());
- ManagedObjectReference morLease = context.getService().importVApp(morRp, ovfImportResult.getImportSpec(), dcMo.getVmFolder(), morHost);
+ DatacenterMO dcMo = null;
+ try {
+ dcMo = new DatacenterMO(context, host.getHyperHostDatacenter());
+ } catch (Exception e) {
+ throw new CloudRuntimeException(String.format("no datacenter for host '%s' available in context", context.getServerAddress()), e);
+ }
+ ManagedObjectReference folderMO = null;
+ try {
+ folderMO = dcMo.getVmFolder();
+ } catch (Exception e) {
+ throw new CloudRuntimeException("no management handle for VmFolder", e);
+ }
+ ManagedObjectReference morLease = null;
+ try {
+ morLease = context.getService().importVApp(morRp, ovfImportResult.getImportSpec(), folderMO, morHost);
+ } catch (DuplicateNameFaultMsg
+ | FileFaultFaultMsg
+ | InsufficientResourcesFaultFaultMsg
+ | InvalidDatastoreFaultMsg
+ | InvalidNameFaultMsg
+ | OutOfBoundsFaultMsg
+ | RuntimeFaultFaultMsg
+ | VmConfigFaultFaultMsg fault) {
+ throw new CloudRuntimeException("import vApp failed",fault);
+ }
Review comment:
new method
##########
File path: vmware-base/src/main/java/com/cloud/hypervisor/vmware/mo/PbmPlacementSolverMO.java
##########
@@ -0,0 +1,68 @@
+// 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.hypervisor.vmware.mo;
Review comment:
new classes should go in `org.apache.cloudstack`
##########
File path: vmware-base/src/main/java/com/cloud/hypervisor/vmware/mo/HypervisorHostHelper.java
##########
@@ -1704,6 +1736,11 @@ public static String resolveHostNameInUrl(DatacenterMO dcMo, String url) {
return url;
}
+ /**
Review comment:
👍
##########
File path: plugins/hypervisors/vmware/src/main/java/com/cloud/storage/resource/VmwareStorageProcessor.java
##########
@@ -1720,13 +1801,19 @@ public Answer createTemplateFromSnapshot(CopyCommand cmd) {
String disks[] = vmMo.getCurrentSnapshotDiskChainDatastorePaths(diskDevice);
if (clonedWorkerVMNeeded) {
// 4 MB is the minimum requirement for VM memory in VMware
Review comment:
is this comment still valid? not by hte looks of the code below.
##########
File path: vmware-base/src/main/java/com/cloud/hypervisor/vmware/mo/HypervisorHostHelper.java
##########
@@ -1818,31 +1903,44 @@ public void action(Long param) {
String erroMsg = "File upload task failed to complete due to: " + e.getMessage();
s_logger.error(erroMsg);
importSuccess = false; // Set flag to cleanup the stale template left due to failed import operation, if any
- throw new Exception(erroMsg, e);
+ throw new CloudRuntimeException(erroMsg, e);
} catch (Throwable th) {
String errorMsg = "throwable caught during file upload task: " + th.getMessage();
s_logger.error(errorMsg);
importSuccess = false; // Set flag to cleanup the stale template left due to failed import operation, if any
- throw new Exception(errorMsg, th);
+ throw new CloudRuntimeException(errorMsg, th);
} finally {
progressReporter.close();
}
if (bytesAlreadyWritten == totalBytes) {
- leaseMo.updateLeaseProgress(100);
+ try {
+ leaseMo.updateLeaseProgress(100);
+ } catch (Exception e) {
+ throw new CloudRuntimeException("error while waiting for lease update", e);
+ }
}
} else if (state == HttpNfcLeaseState.ERROR) {
- LocalizedMethodFault error = leaseMo.getLeaseError();
+ LocalizedMethodFault error = null;
+ try {
+ error = leaseMo.getLeaseError();
+ } catch (Exception e) {
+ throw new CloudRuntimeException("error getting lease error", e);
+ }
MethodFault fault = error.getFault();
String erroMsg = "Object creation on vCenter failed due to: Exception: " + fault.getClass().getName() + ", message: " + error.getLocalizedMessage();
s_logger.error(erroMsg);
- throw new Exception(erroMsg);
+ throw new CloudRuntimeException(erroMsg);
}
} finally {
- if (!importSuccess) {
- s_logger.error("Aborting the lease on " + vmName + " after import operation failed.");
- leaseMo.abortLease();
- } else {
- leaseMo.completeLease();
+ try {
+ if (!importSuccess) {
+ s_logger.error("Aborting the lease on " + vmName + " after import operation failed.");
+ leaseMo.abortLease();
+ } else {
+ leaseMo.completeLease();
+ }
Review comment:
cleanup/finalize method
##########
File path: plugins/hypervisors/vmware/src/main/java/com/cloud/storage/resource/VmwareStorageProcessor.java
##########
@@ -891,6 +886,92 @@ public Answer cloneVolumeFromBaseTemplate(CopyCommand cmd) {
}
}
+ private String cloneVMwithVMname(VmwareContext context, VmwareHypervisorHost hyperHost, TemplateObjectTO template,
+ VirtualMachineMO vmTemplate, VolumeObjectTO volume, DatacenterMO dcMo, DatastoreMO dsMo) throws Exception {
+ ManagedObjectReference morDatastore = dsMo.getMor();
+ ManagedObjectReference morPool = hyperHost.getHyperHostOwnerResourcePool();
+ ManagedObjectReference morCluster = hyperHost.getHyperHostCluster();
+ if (template.getSize() != null) {
+ _fullCloneFlag = volume.getSize() > template.getSize() ? true : _fullCloneFlag;
+ }
+ if (!_fullCloneFlag) {
+ createVMLinkedClone(vmTemplate, dcMo, volume.getVmName(), morDatastore, morPool);
+ } else {
+ createVMFullClone(vmTemplate, dcMo, dsMo, volume.getVmName(), morDatastore, morPool);
+ }
+
+ VirtualMachineMO vmMo = new ClusterMO(context, morCluster).findVmOnHyperHost(volume.getVmName());
+ assert (vmMo != null);
+
+ return vmMo.getVmdkFileBaseNames().get(0);
+ }
+
+ private String createVMFolderWithVMName(VmwareContext context, VmwareHypervisorHost hyperHost, TemplateObjectTO template,
Review comment:
long method, can you split it?
##########
File path: vmware-base/src/main/java/com/cloud/hypervisor/vmware/mo/HypervisorHostHelper.java
##########
@@ -1773,22 +1824,55 @@ public static void importVmFromOVF(VmwareHypervisorHost host, String ovfFilePath
}
}
- DatacenterMO dcMo = new DatacenterMO(context, host.getHyperHostDatacenter());
- ManagedObjectReference morLease = context.getService().importVApp(morRp, ovfImportResult.getImportSpec(), dcMo.getVmFolder(), morHost);
+ DatacenterMO dcMo = null;
+ try {
+ dcMo = new DatacenterMO(context, host.getHyperHostDatacenter());
+ } catch (Exception e) {
+ throw new CloudRuntimeException(String.format("no datacenter for host '%s' available in context", context.getServerAddress()), e);
+ }
+ ManagedObjectReference folderMO = null;
+ try {
+ folderMO = dcMo.getVmFolder();
+ } catch (Exception e) {
+ throw new CloudRuntimeException("no management handle for VmFolder", e);
+ }
Review comment:
factor in new method
##########
File path: plugins/hypervisors/vmware/src/main/java/com/cloud/hypervisor/vmware/resource/VmwareResource.java
##########
@@ -745,20 +752,38 @@ private Answer execute(ResizeVolumeCommand cmd) {
} else if (newSize == oldSize) {
return new ResizeVolumeAnswer(cmd, true, "success", newSize * ResourceType.bytesToKiB);
}
+ /*
+ // FR41 this is yet to fix
Review comment:
internal project code in open source version. please remove/reformulate this comment.
##########
File path: api/src/main/java/com/cloud/agent/api/to/deployasis/OVFVirtualHardwareSectionTO.java
##########
@@ -0,0 +1,50 @@
+//
+// 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.agent.api.to.deployasis;
+
+import java.util.List;
+
+public class OVFVirtualHardwareSectionTO implements TemplateDeployAsIsInformationTO {
Review comment:
though it has contents that might need explanation to the new...
##########
File path: plugins/hypervisors/vmware/src/main/java/com/cloud/storage/resource/VmwareStorageProcessor.java
##########
@@ -1134,7 +1219,7 @@ private void postCreatePrivateTemplate(String installFullPath, long templateId,
Pair<VirtualMachineMO, String[]> cloneResult =
vmMo.cloneFromCurrentSnapshot(workerVmName, 0, 4, volumeDeviceInfo.second(), VmwareHelper.getDiskDeviceDatastore(volumeDeviceInfo.first()));
clonedVm = cloneResult.first();
-
+ * */
Review comment:
code in comment, please remove.
##########
File path: plugins/hypervisors/vmware/src/main/java/org/apache/cloudstack/api/command/admin/zone/ImportVsphereStoragePoliciesCmd.java
##########
@@ -0,0 +1,111 @@
+// 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 org.apache.cloudstack.api.command.admin.zone;
+
+import com.cloud.dc.DataCenter;
+import com.cloud.dc.VsphereStoragePolicy;
+import com.cloud.exception.ConcurrentOperationException;
+import com.cloud.exception.InsufficientCapacityException;
+import com.cloud.exception.NetworkRuleConflictException;
+import com.cloud.exception.ResourceAllocationException;
+import com.cloud.exception.ResourceUnavailableException;
+import com.cloud.hypervisor.vmware.VmwareDatacenterService;
+
+import org.apache.cloudstack.acl.RoleType;
+import org.apache.cloudstack.api.APICommand;
+import org.apache.cloudstack.api.ApiConstants;
+import org.apache.cloudstack.api.ApiErrorCode;
+import org.apache.cloudstack.api.BaseCmd;
+import org.apache.cloudstack.api.Parameter;
+import org.apache.cloudstack.api.ServerApiException;
+import org.apache.cloudstack.api.response.VsphereStoragePoliciesResponse;
+import org.apache.cloudstack.api.response.ListResponse;
+import org.apache.cloudstack.api.response.ZoneResponse;
+import org.apache.cloudstack.context.CallContext;
+import org.apache.log4j.Logger;
+
+import javax.inject.Inject;
+import java.util.ArrayList;
+import java.util.List;
+
+@APICommand(name = ImportVsphereStoragePoliciesCmd.APINAME, description = "Import vSphere storage policies",
+ responseObject = VsphereStoragePoliciesResponse.class,
+ requestHasSensitiveInfo = false, responseHasSensitiveInfo = false,
+ authorized = {RoleType.Admin})
Review comment:
```suggestion
@APICommand(name = ImportVsphereStoragePoliciesCmd.APINAME, description = "Import vSphere storage policies",
responseObject = VsphereStoragePoliciesResponse.class,
requestHasSensitiveInfo = false, responseHasSensitiveInfo = false,
since = "4.15", authorized = {RoleType.Admin})
```
##########
File path: utils/src/test/java/com/cloud/utils/compression/CompressionUtilTest.java
##########
@@ -0,0 +1,128 @@
+//
+// 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.utils.compression;
Review comment:
new classes should go in `org.apache.cloudstack`
##########
File path: utils/src/main/java/com/cloud/utils/compression/CompressionUtil.java
##########
@@ -0,0 +1,58 @@
+//
+// 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.utils.compression;
Review comment:
new classes should go in `org.apache.cloudstack`
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-704730518
@harikrishna-patnala 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 to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-698843786
@borisstoyanov a Trillian-Jenkins test job (centos7 mgmt + vmware-65u2) 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 to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-712040097
Packaging result: ✔centos7 ✔centos8 ✔debian. JID-2206
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-709732065
@nvazquez a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] nvazquez commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
nvazquez commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-716541344
@blueorangutan test matrix
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan removed a comment on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan removed a comment on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-716096319
@nvazquez a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-715590825
<b>Trillian test result (tid-3064)</b>
Environment: vmware-67u3 (x2), Advanced Networking with Mgmt server 7
Total time taken: 41239 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr4307-t3064-vmware-67u3.zip
Intermittent failure detected: /marvin/tests/smoke/test_kubernetes_clusters.py
Intermittent failure detected: /marvin/tests/smoke/test_storage_policy.py
Intermittent failure detected: /marvin/tests/smoke/test_templates.py
Intermittent failure detected: /marvin/tests/smoke/test_vpc_vpn.py
Smoke tests completed. 83 look OK, 3 have error(s)
Only failed tests results shown below:
Test | Result | Time (s) | Test File
--- | --- | --- | ---
test_03_deploy_and_upgrade_kubernetes_cluster | `Failure` | 793.48 | test_kubernetes_clusters.py
test_01_import_storage_policies | `Error` | 3.49 | test_storage_policy.py
ContextSuite context=TestVMWareStoragePolicies>:teardown | `Error` | 4.64 | test_storage_policy.py
test_02_create_template_with_checksum_sha1 | `Error` | 5.20 | test_templates.py
test_03_create_template_with_checksum_sha256 | `Error` | 5.21 | test_templates.py
test_04_create_template_with_checksum_md5 | `Error` | 5.21 | test_templates.py
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-716184355
<b>Trillian test result (tid-3070)</b>
Environment: vmware-65u2 (x2), Advanced Networking with Mgmt server 7
Total time taken: 38687 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr4307-t3070-vmware-65u2.zip
Intermittent failure detected: /marvin/tests/smoke/test_kubernetes_clusters.py
Intermittent failure detected: /marvin/tests/smoke/test_storage_policy.py
Smoke tests completed. 84 look OK, 2 have error(s)
Only failed tests results shown below:
Test | Result | Time (s) | Test File
--- | --- | --- | ---
test_03_deploy_and_upgrade_kubernetes_cluster | `Failure` | 1455.72 | test_kubernetes_clusters.py
test_01_import_storage_policies | `Error` | 2.32 | test_storage_policy.py
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan removed a comment on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan removed a comment on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-692164357
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-705942785
<b>Trillian test result (tid-2915)</b>
Environment: vmware-67u3 (x2), Advanced Networking with Mgmt server 7
Total time taken: 34044 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr4307-t2915-vmware-67u3.zip
Intermittent failure detected: /marvin/tests/smoke/test_vpc_vpn.py
Smoke tests completed. 82 look OK, 0 have error(s)
Only failed tests results shown below:
Test | Result | Time (s) | Test File
--- | --- | --- | ---
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] harikrishna-patnala removed a comment on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
harikrishna-patnala removed a comment on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-690943888
@blueorangutan package
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-691731088
@rhtyd a Trillian-Jenkins matrix job (centos7 mgmt + xs71, centos7 mgmt + vmware67, 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 to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan removed a comment on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan removed a comment on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-698752221
@nvazquez a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] nvazquez commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
nvazquez commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-688459825
@blueorangutan package
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan removed a comment on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan removed a comment on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-696510052
Packaging result: ✖centos7 ✔centos8 ✔debian. JID-2058
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] rhtyd removed a comment on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
rhtyd removed a comment on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-711078483
@blueorangutan package
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-716100992
<b>Trillian test result (tid-3067)</b>
Environment: vmware-67u3 (x2), Advanced Networking with Mgmt server 7
Total time taken: 47924 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr4307-t3067-vmware-67u3.zip
Intermittent failure detected: /marvin/tests/smoke/test_kubernetes_clusters.py
Intermittent failure detected: /marvin/tests/smoke/test_storage_policy.py
Intermittent failure detected: /marvin/tests/smoke/test_volumes.py
Smoke tests completed. 83 look OK, 2 have error(s)
Only failed tests results shown below:
Test | Result | Time (s) | Test File
--- | --- | --- | ---
test_03_deploy_and_upgrade_kubernetes_cluster | `Failure` | 809.16 | test_kubernetes_clusters.py
test_01_import_storage_policies | `Error` | 2.90 | test_storage_policy.py
ContextSuite context=TestVMWareStoragePolicies>:teardown | `Error` | 3.96 | test_storage_policy.py
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-705246414
<b>Trillian test result (tid-2907)</b>
Environment: vmware-67u3 (x2), Advanced Networking with Mgmt server 7
Total time taken: 38793 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr4307-t2907-vmware-67u3.zip
Intermittent failure detected: /marvin/tests/smoke/test_vpc_redundant.py
Smoke tests completed. 84 look OK, 1 have error(s)
Only failed tests results shown below:
Test | Result | Time (s) | Test File
--- | --- | --- | ---
test_01_create_redundant_VPC_2tiers_4VMs_4IPs_4PF_ACL | `Failure` | 402.70 | test_vpc_redundant.py
test_03_create_redundant_VPC_1tier_2VMs_2IPs_2PF_ACL_reboot_routers | `Failure` | 401.02 | test_vpc_redundant.py
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] nvazquez commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
nvazquez commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-692216930
@blueorangutan package
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] rhtyd commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
rhtyd commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-696499355
@nvazquez @harikrishna-patnala can you resolve the schema conflict?
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] borisstoyanov commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
borisstoyanov commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-716515467
Kubernetes cluster test seems to be present in other PRs as well, I've run this on the test env we have with this PR and got
```
=== TestName: test_01_deploy_kubernetes_cluster | Status : SUCCESS ===
=== TestName: test_02_invalid_upgrade_kubernetes_cluster | Status : SUCCESS ===
=== TestName: test_03_deploy_and_upgrade_kubernetes_cluster | Status : FAILED ===
=== TestName: test_04_deploy_and_scale_kubernetes_cluster | Status : SUCCESS ===
=== TestName: test_05_delete_kubernetes_cluster | Status : SUCCESS ===
=== TestName: test_06_deploy_invalid_kubernetes_ha_cluster | Status : SUCCESS ===
=== TestName: test_07_deploy_kubernetes_ha_cluster | Status : SUCCESS ===
=== TestName: test_08_deploy_and_upgrade_kubernetes_ha_cluster | Status : SUCCESS ===
=== TestName: test_09_delete_kubernetes_ha_cluster | Status : SUCCESS ===
```
Then the same error with test03 has been seen here as well:
https://github.com/apache/cloudstack/pull/4382#issuecomment-706347417
The storage policy failure is related to setup and using an incompatible policy with our infra, we'll fix the test now to evade this action and we'll look to fix this one later as it's not breaking anything.
I think we're in a good position to merge this!
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-716505957
@nvazquez a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] sureshanaparti commented on a change in pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
sureshanaparti commented on a change in pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#discussion_r529209046
##########
File path: server/src/main/java/com/cloud/vm/UserVmManagerImpl.java
##########
@@ -2899,14 +2899,20 @@ public UserVm destroyVm(DestroyVMCmd cmd) throws ResourceUnavailableException, C
}
s_logger.debug("Found no ongoing snapshots on volume of type ROOT, for the vm with id " + vmId);
- List<VolumeVO> volumes = getVolumesFromIds(cmd);
+ List<VolumeVO> volumesToBeDeleted = getVolumesFromIds(cmd);
- checkForUnattachedVolumes(vmId, volumes);
- validateVolumes(volumes);
+ checkForUnattachedVolumes(vmId, volumesToBeDeleted);
+ validateVolumes(volumesToBeDeleted);
stopVirtualMachine(vmId, VmDestroyForcestop.value());
- detachVolumesFromVm(volumes);
+ if (vm.getHypervisorType() == HypervisorType.VMware) {
+ List<VolumeVO> allVolumes = _volsDao.findByInstance(vm.getId());
+ allVolumes.removeIf(vol -> vol.getVolumeType() == Volume.Type.ROOT);
+ detachVolumesFromVm(allVolumes);
+ } else {
+ detachVolumesFromVm(volumesToBeDeleted);
+ }
Review comment:
these changes are reverted in the PR https://github.com/apache/cloudstack/pull/4493 , as the data disks (which are not to be deleted) are being detached on destroy VM, which is breaking the existing behavior.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-712615216
@nvazquez a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-690155795
@harikrishna-patnala 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 to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] rhtyd commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
rhtyd commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-711078774
@blueorangutan package
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] nvazquez commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
nvazquez commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-712614507
@blueorangutan package
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] nvazquez commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
nvazquez commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-697048189
@blueorangutan package
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-704052728
@harikrishna-patnala 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 to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] alexandremattioli commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
alexandremattioli commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-713707828
LGTM
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] nvazquez removed a comment on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
nvazquez removed a comment on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-716096196
@blueorangutan package
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-700767733
<b>Trillian test result (tid-2870)</b>
Environment: vmware-67u3 (x2), Advanced Networking with Mgmt server 7
Total time taken: 36216 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr4307-t2870-vmware-67u3.zip
Smoke tests completed. 85 look OK, 0 have error(s)
Only failed tests results shown below:
Test | Result | Time (s) | Test File
--- | --- | --- | ---
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-711083849
Packaging result: ✔centos7 ✔centos8 ✔debian. JID-2202
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] andrijapanicsb removed a comment on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
andrijapanicsb removed a comment on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-713708538
@blueorangutan test centos7 vmware-67u3
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-713705765
@andrijapanicsb a Trillian-Jenkins matrix job (centos6 mgmt + xs71, centos7 mgmt + vmware65, 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 to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] nvazquez commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
nvazquez commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-696497201
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] borisstoyanov commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
borisstoyanov commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-707509018
@blueorangutan package
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] harikrishna-patnala commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
harikrishna-patnala commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-699753623
@blueorangutan package
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-715908816
<b>Trillian test result (tid-3066)</b>
Environment: vmware-67u3 (x2), Advanced Networking with Mgmt server 7
Total time taken: 78093 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr4307-t3066-vmware-67u3.zip
Intermittent failure detected: /marvin/tests/smoke/test_kubernetes_clusters.py
Intermittent failure detected: /marvin/tests/smoke/test_storage_policy.py
Intermittent failure detected: /marvin/tests/smoke/test_templates.py
Smoke tests completed. 82 look OK, 3 have error(s)
Only failed tests results shown below:
Test | Result | Time (s) | Test File
--- | --- | --- | ---
test_03_deploy_and_upgrade_kubernetes_cluster | `Failure` | 787.74 | test_kubernetes_clusters.py
test_01_import_storage_policies | `Error` | 3.48 | test_storage_policy.py
ContextSuite context=TestVMWareStoragePolicies>:teardown | `Error` | 4.62 | test_storage_policy.py
test_02_create_template_with_checksum_sha1 | `Error` | 5.20 | test_templates.py
test_03_create_template_with_checksum_sha256 | `Error` | 5.19 | test_templates.py
test_04_create_template_with_checksum_md5 | `Error` | 5.19 | test_templates.py
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-708945535
Packaging result: ✔centos7 ✔centos8 ✔debian. JID-2183
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-688475225
Packaging result: ✔centos7 ✔centos8 ✔debian. JID-1911
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-714736228
@nvazquez a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-705308542
@harikrishna-patnala 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 to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan removed a comment on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan removed a comment on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-716098637
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] rhtyd commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
rhtyd commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-688492162
@blueorangutan test matrix
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-716539931
Packaging result: ✔centos7 ✔centos8 ✔debian. JID-2278
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] harikrishna-patnala commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
harikrishna-patnala commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-693217928
@blueorangutan package
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-700417021
Packaging result: ✔centos7 ✔centos8 ✔debian. JID-2096
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-713713478
@andrijapanicsb a Trillian-Jenkins test job (centos7 mgmt + vmware-67u3) 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 to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-714227108
Packaging result: ✔centos7 ✔centos8 ✔debian. JID-2257
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-716101364
@nvazquez a Trillian-Jenkins matrix job (centos7 mgmt + xs71, centos7 mgmt + vmware65, 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 to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-690944356
@harikrishna-patnala 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 to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] nvazquez commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
nvazquez commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-716098907
@blueorangutan package
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-706873950
@harikrishna-patnala 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 to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] rhtyd commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
rhtyd commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-689867233
@blueorangutan test centos7 vmware-67u3
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] rhtyd commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
rhtyd commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-693837790
@harikrishna-patnala can you merge latest master and fix conflicts, then run pkging and smoketests?
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-707509594
@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 to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] nvazquez removed a comment on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
nvazquez removed a comment on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-716098579
@blueorangutan package
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] nvazquez removed a comment on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
nvazquez removed a comment on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-692216930
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-698033925
@harikrishna-patnala 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 to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] harikrishna-patnala removed a comment on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
harikrishna-patnala removed a comment on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-690943888
@blueorangutan package
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-716098637
@nvazquez a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-714754645
@nvazquez a Trillian-Jenkins test job (centos7 mgmt + vmware-67u3) 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 to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-694099769
@harikrishna-patnala 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 to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-693148959
<b>Trillian test result (tid-2739)</b>
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 73046 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr4307-t2739-kvm-centos7.zip
Intermittent failure detected: /marvin/tests/smoke/test_diagnostics.py
Intermittent failure detected: /marvin/tests/smoke/test_internal_lb.py
Intermittent failure detected: /marvin/tests/smoke/test_iso.py
Intermittent failure detected: /marvin/tests/smoke/test_kubernetes_supported_versions.py
Intermittent failure detected: /marvin/tests/smoke/test_outofbandmanagement.py
Intermittent failure detected: /marvin/tests/smoke/test_privategw_acl.py
Intermittent failure detected: /marvin/tests/smoke/test_router_dhcphosts.py
Intermittent failure detected: /marvin/tests/smoke/test_templates.py
Intermittent failure detected: /marvin/tests/smoke/test_host_maintenance.py
Intermittent failure detected: /marvin/tests/smoke/test_hostha_kvm.py
Smoke tests completed. 78 look OK, 7 have error(s)
Only failed tests results shown below:
Test | Result | Time (s) | Test File
--- | --- | --- | ---
test_03_vpc_internallb_haproxy_stats_on_all_interfaces | `Error` | 422.31 | test_internal_lb.py
test_04_extract_Iso | `Failure` | 128.09 | test_iso.py
test_01_add_delete_kubernetes_supported_version | `Error` | 1807.99 | test_kubernetes_supported_versions.py
test_02_vpc_privategw_static_routes | `Failure` | 534.25 | test_privategw_acl.py
test_03_vpc_privategw_restart_vpc_cleanup | `Failure` | 658.05 | test_privategw_acl.py
test_router_dhcphosts | `Failure` | 177.32 | test_router_dhcphosts.py
ContextSuite context=TestRouterDHCPHosts>:teardown | `Error` | 201.16 | test_router_dhcphosts.py
ContextSuite context=TestTemplates>:setup | `Error` | 1333.20 | test_templates.py
test_hostha_enable_ha_when_host_disabled | `Error` | 1.86 | test_hostha_kvm.py
test_hostha_enable_ha_when_host_in_maintenance | `Error` | 301.05 | test_hostha_kvm.py
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] rhtyd commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
rhtyd commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-711089475
@blueorangutan test matrix
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] nvazquez commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
nvazquez commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-706231802
@blueorangutan test centos7 vmware-67u3
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-692144707
@nvazquez a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-713378669
@DaanHoogland a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-714718352
@DaanHoogland a Trillian-Jenkins test job (centos7 mgmt + xenserver-71) 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 to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-715958266
@nvazquez a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-714216118
@nvazquez a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] nvazquez commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
nvazquez commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-692193401
@blueorangutan package
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-689041717
<b>Trillian test result (tid-2698)</b>
Environment: vmware-67u3 (x2), Advanced Networking with Mgmt server 7
Total time taken: 75260 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr4307-t2698-vmware-67u3.zip
Intermittent failure detected: /marvin/tests/smoke/test_iso.py
Intermittent failure detected: /marvin/tests/smoke/test_kubernetes_supported_versions.py
Intermittent failure detected: /marvin/tests/smoke/test_snapshots.py
Intermittent failure detected: /marvin/tests/smoke/test_ssvm.py
Smoke tests completed. 81 look OK, 4 have error(s)
Only failed tests results shown below:
Test | Result | Time (s) | Test File
--- | --- | --- | ---
test_03_create_iso_with_checksum_md5 | `Error` | 72.52 | test_iso.py
test_03_create_iso_with_checksum_md5 | `Error` | 73.56 | test_iso.py
test_04_create_iso_with_no_checksum | `Error` | 73.64 | test_iso.py
test_01_add_delete_kubernetes_supported_version | `Error` | 1813.90 | test_kubernetes_supported_versions.py
test_02_list_snapshots_with_removed_data_store | `Error` | 179.25 | test_snapshots.py
test_07_reboot_ssvm | `Failure` | 57.99 | test_ssvm.py
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] nvazquez commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
nvazquez commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-716188592
@blueorangutan package
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] nvazquez commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
nvazquez commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-715982462
@blueorangutan test centos7 vmware-67u3
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-716901178
<b>Trillian test result (tid-3078)</b>
Environment: vmware-65u2 (x2), Advanced Networking with Mgmt server 7
Total time taken: 38296 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr4307-t3078-vmware-65u2.zip
Intermittent failure detected: /marvin/tests/smoke/test_kubernetes_clusters.py
Smoke tests completed. 85 look OK, 1 have error(s)
Only failed tests results shown below:
Test | Result | Time (s) | Test File
--- | --- | --- | ---
test_03_deploy_and_upgrade_kubernetes_cluster | `Failure` | 782.42 | test_kubernetes_clusters.py
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] nvazquez commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
nvazquez commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-692144201
@blueorangutan package
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] weizhouapache commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
weizhouapache commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-716649162
@borisstoyanov @nvazquez
I think we normally merge a pr with only 1 commit.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan removed a comment on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan removed a comment on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-692234413
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-700409095
@nvazquez a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] nvazquez commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
nvazquez commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-694440179
@blueorangutan package
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-695810848
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan removed a comment on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan removed a comment on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-714446959
@DaanHoogland a Trillian-Jenkins test job (centos6 mgmt + xenserver-71) 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 to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] rhtyd commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
rhtyd commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-703152643
@blueorangutan package
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-698122135
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-692217289
@nvazquez a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] nvazquez commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
nvazquez commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-700437781
@blueorangutan package
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-716166827
<b>Trillian test result (tid-3069)</b>
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 30614 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr4307-t3069-kvm-centos7.zip
Smoke tests completed. 86 look OK, 0 have error(s)
Only failed tests results shown below:
Test | Result | Time (s) | Test File
--- | --- | --- | ---
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-695596443
@nvazquez a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-711494911
Packaging result: ✔centos7 ✔centos8 ✔debian. JID-2203
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] rhtyd commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
rhtyd commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-706842572
@harikrishna-patnala @nvazquez can you fix conflicts and kick test matrix? Thnx
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] borisstoyanov commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
borisstoyanov commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-706906624
@blueorangutan test centos7 vmware-67u3
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] harikrishna-patnala commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
harikrishna-patnala commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-698263502
@blueorangutan package
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-700185228
<b>Trillian test result (tid-2867)</b>
Environment: vmware-67u3 (x2), Advanced Networking with Mgmt server 7
Total time taken: 45529 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr4307-t2867-vmware-67u3.zip
Intermittent failure detected: /marvin/tests/smoke/test_diagnostics.py
Intermittent failure detected: /marvin/tests/smoke/test_internal_lb.py
Intermittent failure detected: /marvin/tests/smoke/test_iso.py
Intermittent failure detected: /marvin/tests/smoke/test_kubernetes_supported_versions.py
Intermittent failure detected: /marvin/tests/smoke/test_loadbalance.py
Intermittent failure detected: /marvin/tests/smoke/test_vpc_redundant.py
Smoke tests completed. 76 look OK, 5 have error(s)
Only failed tests results shown below:
Test | Result | Time (s) | Test File
--- | --- | --- | ---
test_01_internallb_roundrobin_1VPC_3VM_HTTP_port80 | `Failure` | 387.26 | test_internal_lb.py
test_02_internallb_roundrobin_1RVPC_3VM_HTTP_port80 | `Failure` | 548.32 | test_internal_lb.py
test_03_vpc_internallb_haproxy_stats_on_all_interfaces | `Error` | 220.60 | test_internal_lb.py
test_04_rvpc_internallb_haproxy_stats_on_all_interfaces | `Error` | 301.29 | test_internal_lb.py
test_01_ping_in_vr_success | `Failure` | 15.35 | test_diagnostics.py
test_03_ping_in_ssvm_success | `Failure` | 15.34 | test_diagnostics.py
test_05_ping_in_cpvm_success | `Failure` | 15.32 | test_diagnostics.py
test_07_arping_in_vr | `Failure` | 5.15 | test_diagnostics.py
test_08_arping_in_ssvm | `Failure` | 5.16 | test_diagnostics.py
test_09_arping_in_cpvm | `Failure` | 5.16 | test_diagnostics.py
test_13_retrieve_vr_default_files | `Error` | 6.55 | test_diagnostics.py
test_14_retrieve_vr_one_file | `Error` | 13.50 | test_diagnostics.py
test_15_retrieve_ssvm_default_files | `Error` | 13.50 | test_diagnostics.py
test_16_retrieve_ssvm_single_file | `Error` | 13.50 | test_diagnostics.py
test_17_retrieve_cpvm_default_files | `Error` | 13.60 | test_diagnostics.py
test_18_retrieve_cpvm_single_file | `Error` | 13.50 | test_diagnostics.py
test_04_extract_Iso | `Failure` | 6.52 | test_iso.py
test_01_add_delete_kubernetes_supported_version | `Error` | 1807.48 | test_kubernetes_supported_versions.py
test_01_create_redundant_VPC_2tiers_4VMs_4IPs_4PF_ACL | `Failure` | 469.08 | test_vpc_redundant.py
test_05_rvpc_multi_tiers | `Failure` | 225.59 | test_vpc_redundant.py
test_05_rvpc_multi_tiers | `Error` | 225.61 | test_vpc_redundant.py
ContextSuite context=TestVPCRedundancy>:teardown | `Error` | 225.62 | test_vpc_redundant.py
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-691730872
Packaging result: ✔centos7 ✔centos8 ✔debian. JID-1989
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-698843342
@borisstoyanov a Trillian-Jenkins test job (centos7 mgmt + vmware-67u3) 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 to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-698009668
<b>Trillian test result (tid-2839)</b>
Environment: vmware-67u3 (x2), Advanced Networking with Mgmt server 7
Total time taken: 59874 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr4307-t2839-vmware-67u3.zip
Intermittent failure detected: /marvin/tests/smoke/test_deploy_vm_root_resize.py
Intermittent failure detected: /marvin/tests/smoke/test_kubernetes_clusters.py
Intermittent failure detected: /marvin/tests/smoke/test_kubernetes_supported_versions.py
Intermittent failure detected: /marvin/tests/smoke/test_vpc_redundant.py
Smoke tests completed. 85 look OK, 0 have error(s)
Only failed tests results shown below:
Test | Result | Time (s) | Test File
--- | --- | --- | ---
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-699766358
@rhtyd a Trillian-Jenkins test job (centos7 mgmt + vmware-67u3) 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 to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] andrijapanicsb commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
andrijapanicsb commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-713705228
@blueorangutan test matrix
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-716098518
Packaging result: ✖centos7 ✖centos8 ✖debian. JID-2272
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-696497308
@nvazquez a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] borisstoyanov commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
borisstoyanov commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-698842821
@blueorangutan test centos7 vmware-67u3
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-706473183
<b>Trillian test result (tid-2921)</b>
Environment: vmware-67u3 (x2), Advanced Networking with Mgmt server 7
Total time taken: 40704 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr4307-t2921-vmware-67u3.zip
Intermittent failure detected: /marvin/tests/smoke/test_kubernetes_clusters.py
Intermittent failure detected: /marvin/tests/smoke/test_vpc_redundant.py
Smoke tests completed. 79 look OK, 2 have error(s)
Only failed tests results shown below:
Test | Result | Time (s) | Test File
--- | --- | --- | ---
test_03_deploy_and_upgrade_kubernetes_cluster | `Failure` | 790.10 | test_kubernetes_clusters.py
test_03_create_redundant_VPC_1tier_2VMs_2IPs_2PF_ACL_reboot_routers | `Failure` | 410.03 | test_vpc_redundant.py
test_05_rvpc_multi_tiers | `Error` | 323.67 | test_vpc_redundant.py
test_05_rvpc_multi_tiers | `Error` | 323.70 | test_vpc_redundant.py
ContextSuite context=TestVPCRedundancy>:teardown | `Error` | 323.73 | test_vpc_redundant.py
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-716098977
@nvazquez a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] borisstoyanov commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
borisstoyanov commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-708926952
@blueorangutan package
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] harikrishna-patnala commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
harikrishna-patnala commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-690943888
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-697048258
@nvazquez a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] nvazquez commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
nvazquez commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-716098579
@blueorangutan package
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] nvazquez commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
nvazquez commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-714215896
@blueorangutan package
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-714230179
@nvazquez a Trillian-Jenkins test job (centos7 mgmt + vmware-67u3) 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 to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] nvazquez commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
nvazquez commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-709730756
@blueorangutan package
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-716193092
Packaging result: ✔centos7 ✔centos8 ✔debian. JID-2275
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] rhtyd closed pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
rhtyd closed pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-705893382
@harikrishna-patnala 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 to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] borisstoyanov commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
borisstoyanov commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-716532217
cat /marvin//MarvinLogs/test_storage_policy_DFIU0C/results.txt
Test VMWare storage policies ... === TestName: test_01_import_storage_policies | Status : SUCCESS ===
ok
----------------------------------------------------------------------
Ran 1 test in 70.694s
OK
LGTM, locally tested
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-700471666
Packaging result: ✔centos7 ✔centos8 ✔debian. JID-2097
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] harikrishna-patnala commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
harikrishna-patnala commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-704052367
@blueorangutan package
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-689316454
Packaging result: ✔centos7 ✔centos8 ✔debian. JID-1924
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] nvazquez removed a comment on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
nvazquez removed a comment on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-698751791
@blueorangutan package
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] nvazquez commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
nvazquez commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-716101259
@blueorangutan test matrix
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-714495720
@DaanHoogland a Trillian-Jenkins test job (centos6 mgmt + xenserver-71) 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 to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] rhtyd commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
rhtyd commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-711078483
@blueorangutan package
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-709462471
Packaging result: ✔centos7 ✔centos8 ✔debian. JID-2188
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan removed a comment on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan removed a comment on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-690944356
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-703637328
@borisstoyanov a Trillian-Jenkins test job (centos7 mgmt + vmware-67u3) 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 to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] nvazquez commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
nvazquez commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-700408841
@blueorangutan package
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] harikrishna-patnala removed a comment on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
harikrishna-patnala removed a comment on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-690943888
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-715982803
@nvazquez a Trillian-Jenkins test job (centos7 mgmt + vmware-67u3) 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 to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-704744647
Packaging result: ✔centos7 ✔centos8 ✔debian. JID-2142
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-698139088
Packaging result: ✔centos7 ✔centos8 ✔debian. JID-2076
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] nvazquez commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
nvazquez commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-698751791
@blueorangutan package
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] andrijapanicsb commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
andrijapanicsb commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-713373066
@blueorangutan package
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] DaanHoogland commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
DaanHoogland commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-705708999
@blueorangutan test centos7 vmware-67u3
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-701286268
@harikrishna-patnala 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 to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] nvazquez commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
nvazquez commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-714229955
@blueorangutan test centos7 vmware-67u3
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-714446959
@DaanHoogland a Trillian-Jenkins test job (centos6 mgmt + xenserver-71) 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 to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-689302998
@harikrishna-patnala 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 to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan removed a comment on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan removed a comment on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-690944356
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-713879746
<b>Trillian test result (tid-3040)</b>
Environment: vmware-67u3 (x2), Advanced Networking with Mgmt server 7
Total time taken: 13762 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr4307-t3040-vmware-67u3.zip
Intermittent failure detected: /marvin/tests/smoke/test_accounts.py
Intermittent failure detected: /marvin/tests/smoke/test_affinity_groups_projects.py
Intermittent failure detected: /marvin/tests/smoke/test_async_job.py
Intermittent failure detected: /marvin/tests/smoke/test_deploy_vgpu_enabled_vm.py
Intermittent failure detected: /marvin/tests/smoke/test_deploy_vm_iso.py
Intermittent failure detected: /marvin/tests/smoke/test_deploy_vms_with_varied_deploymentplanners.py
Intermittent failure detected: /marvin/tests/smoke/test_deploy_vm_with_userdata.py
Intermittent failure detected: /marvin/tests/smoke/test_diagnostics.py
Intermittent failure detected: /marvin/tests/smoke/test_domain_service_offerings.py
Intermittent failure detected: /marvin/tests/smoke/test_internal_lb.py
Intermittent failure detected: /marvin/tests/smoke/test_kubernetes_clusters.py
Intermittent failure detected: /marvin/tests/smoke/test_list_ids_parameter.py
Intermittent failure detected: /marvin/tests/smoke/test_loadbalance.py
Intermittent failure detected: /marvin/tests/smoke/test_metrics_api.py
Intermittent failure detected: /marvin/tests/smoke/test_multipleips_per_nic.py
Intermittent failure detected: /marvin/tests/smoke/test_nested_virtualization.py
Intermittent failure detected: /marvin/tests/smoke/test_network_acl.py
Intermittent failure detected: /marvin/tests/smoke/test_network.py
Intermittent failure detected: /marvin/tests/smoke/test_nic_adapter_type.py
Intermittent failure detected: /marvin/tests/smoke/test_password_server.py
Intermittent failure detected: /marvin/tests/smoke/test_portforwardingrules.py
Intermittent failure detected: /marvin/tests/smoke/test_privategw_acl.py
Intermittent failure detected: /marvin/tests/smoke/test_projects.py
Intermittent failure detected: /marvin/tests/smoke/test_reset_vm_on_reboot.py
Intermittent failure detected: /marvin/tests/smoke/test_resource_accounting.py
Intermittent failure detected: /marvin/tests/smoke/test_router_dhcphosts.py
Intermittent failure detected: /marvin/tests/smoke/test_router_dns.py
Intermittent failure detected: /marvin/tests/smoke/test_router_dnsservice.py
Intermittent failure detected: /marvin/tests/smoke/test_routers_iptables_default_policy.py
Intermittent failure detected: /marvin/tests/smoke/test_routers_network_ops.py
Intermittent failure detected: /marvin/tests/smoke/test_routers.py
Intermittent failure detected: /marvin/tests/smoke/test_service_offerings.py
Intermittent failure detected: /marvin/tests/smoke/test_snapshots.py
Intermittent failure detected: /marvin/tests/smoke/test_storage_policy.py
Intermittent failure detected: /marvin/tests/smoke/test_templates.py
Intermittent failure detected: /marvin/tests/smoke/test_vpc_redundant.py
Intermittent failure detected: /marvin/tests/smoke/test_vpc_router_nics.py
Intermittent failure detected: /marvin/tests/smoke/test_vpc_vpn.py
Smoke tests completed. 48 look OK, 38 have error(s)
Only failed tests results shown below:
Test | Result | Time (s) | Test File
--- | --- | --- | ---
ContextSuite context=TestAccounts>:setup | `Error` | 0.00 | test_accounts.py
ContextSuite context=TestAddVmToSubDomain>:setup | `Error` | 0.00 | test_accounts.py
ContextSuite context=TestDomainForceRemove>:setup | `Error` | 0.00 | test_accounts.py
ContextSuite context=TestRemoveUserFromAccount>:setup | `Error` | 5.84 | test_accounts.py
ContextSuite context=TestTemplateHierarchy>:setup | `Error` | 7.12 | test_accounts.py
ContextSuite context=TestDeployVmWithAffinityGroup>:setup | `Error` | 0.00 | test_affinity_groups_projects.py
ContextSuite context=TestAsyncJob>:setup | `Error` | 0.00 | test_async_job.py
test_3d_gpu_support | `Error` | 1.10 | test_deploy_vgpu_enabled_vm.py
test_deploy_vgpu_enabled_vm | `Error` | 1.13 | test_deploy_vgpu_enabled_vm.py
ContextSuite context=TestDeployVMFromISO>:setup | `Error` | 0.00 | test_deploy_vm_iso.py
ContextSuite context=TestInternalLb>:setup | `Error` | 0.00 | test_internal_lb.py
ContextSuite context=TestRemoteDiagnostics>:setup | `Error` | 0.00 | test_diagnostics.py
ContextSuite context=TestDeployVmWithVariedPlanners>:setup | `Error` | 0.00 | test_deploy_vms_with_varied_deploymentplanners.py
ContextSuite context=TestDeployVmWithUserData>:setup | `Error` | 0.00 | test_deploy_vm_with_userdata.py
ContextSuite context=TestDomainsServiceOfferings>:setup | `Error` | 3.70 | test_domain_service_offerings.py
ContextSuite context=TestKubernetesCluster>:setup | `Error` | 0.00 | test_kubernetes_clusters.py
ContextSuite context=TestListIdsParams>:setup | `Error` | 0.00 | test_list_ids_parameter.py
ContextSuite context=TestLoadBalance>:setup | `Error` | 0.00 | test_loadbalance.py
test_list_clusters_metrics | `Error` | 0.23 | test_metrics_api.py
test_list_hosts_metrics | `Error` | 0.23 | test_metrics_api.py
test_list_infrastructure_metrics | `Error` | 0.22 | test_metrics_api.py
test_list_pstorage_metrics | `Error` | 0.22 | test_metrics_api.py
test_list_vms_metrics | `Error` | 0.22 | test_metrics_api.py
test_list_volumes_metrics | `Error` | 0.24 | test_metrics_api.py
test_list_zones_metrics | `Error` | 0.23 | test_metrics_api.py
test_nic_secondaryip_add_remove | `Error` | 0.10 | test_multipleips_per_nic.py
ContextSuite context=TestNestedVirtualization>:setup | `Error` | 0.00 | test_nested_virtualization.py
ContextSuite context=TestNetworkACL>:setup | `Error` | 0.00 | test_network_acl.py
test_delete_account | `Error` | 0.10 | test_network.py
ContextSuite context=TestL2Networks>:setup | `Error` | 0.94 | test_network.py
ContextSuite context=TestPortForwarding>:setup | `Error` | 1.04 | test_network.py
ContextSuite context=TestPublicIP>:setup | `Error` | 1.89 | test_network.py
test_reboot_router | `Error` | 0.09 | test_network.py
test_releaseIP | `Error` | 0.09 | test_network.py
ContextSuite context=TestRouterRules>:setup | `Error` | 0.17 | test_network.py
ContextSuite context=TestAdapterTypeForNic>:setup | `Error` | 0.00 | test_nic_adapter_type.py
ContextSuite context=TestIsolatedNetworksPasswdServer>:setup | `Error` | 0.00 | test_password_server.py
ContextSuite context=TestPortForwardingRules>:setup | `Error` | 0.00 | test_portforwardingrules.py
ContextSuite context=TestPrivateGwACL>:setup | `Error` | 0.00 | test_privategw_acl.py
ContextSuite context=TestProjectSuspendActivate>:setup | `Error` | 7.51 | test_projects.py
ContextSuite context=TestResetVmOnReboot>:setup | `Error` | 0.00 | test_reset_vm_on_reboot.py
ContextSuite context=TestRAMCPUResourceAccounting>:setup | `Error` | 0.00 | test_resource_accounting.py
ContextSuite context=TestRouterDHCPHosts>:setup | `Error` | 0.00 | test_router_dhcphosts.py
ContextSuite context=TestRouterDHCPOpts>:setup | `Error` | 0.00 | test_router_dhcphosts.py
ContextSuite context=TestRouterDns>:setup | `Error` | 0.00 | test_router_dns.py
ContextSuite context=TestRouterDnsService>:setup | `Error` | 0.00 | test_router_dnsservice.py
ContextSuite context=TestRouterIpTablesPolicies>:setup | `Error` | 0.00 | test_routers_iptables_default_policy.py
ContextSuite context=TestVPCIpTablesPolicies>:setup | `Error` | 0.00 | test_routers_iptables_default_policy.py
ContextSuite context=TestIsolatedNetworks>:setup | `Error` | 0.00 | test_routers_network_ops.py
ContextSuite context=TestRedundantIsolateNetworks>:setup | `Error` | 0.00 | test_routers_network_ops.py
ContextSuite context=TestRouterServices>:setup | `Error` | 0.00 | test_routers.py
ContextSuite context=TestServiceOfferings>:setup | `Error` | 0.18 | test_service_offerings.py
ContextSuite context=TestSnapshotRootDisk>:setup | `Error` | 0.00 | test_snapshots.py
ContextSuite context=TestVMWareStoragePolicies>:setup | `Error` | 0.00 | test_storage_policy.py
test_02_1_create_template_with_checksum_sha1_negative | `Error` | 0.08 | test_templates.py
test_02_create_template_with_checksum_sha1 | `Error` | 0.06 | test_templates.py
test_03_1_create_template_with_checksum_sha256_negative | `Error` | 0.07 | test_templates.py
test_03_create_template_with_checksum_sha256 | `Error` | 0.07 | test_templates.py
test_04_1_create_template_with_checksum_md5_negative | `Error` | 0.06 | test_templates.py
test_04_create_template_with_checksum_md5 | `Error` | 0.07 | test_templates.py
test_05_create_template_with_no_checksum | `Error` | 0.06 | test_templates.py
ContextSuite context=TestVPCRedundancy>:setup | `Error` | 0.00 | test_vpc_redundant.py
ContextSuite context=TestVPCNics>:setup | `Error` | 0.00 | test_vpc_router_nics.py
ContextSuite context=TestRVPCSite2SiteVpn>:setup | `Error` | 0.00 | test_vpc_vpn.py
ContextSuite context=TestVPCSite2SiteVPNMultipleOptions>:setup | `Error` | 0.00 | test_vpc_vpn.py
ContextSuite context=TestVpcRemoteAccessVpn>:setup | `Error` | 0.00 | test_vpc_vpn.py
ContextSuite context=TestVpcSite2SiteVpn>:setup | `Error` | 0.00 | test_vpc_vpn.py
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] harikrishna-patnala closed pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
harikrishna-patnala closed pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] nvazquez removed a comment on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
nvazquez removed a comment on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-700398258
@blueorangutan package
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] rhtyd commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
rhtyd commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-691705991
@blueorangutan package
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] nvazquez commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
nvazquez commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-715957695
@blueorangutan package
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] rhtyd commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
rhtyd commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-691655444
@blueorangutan package
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] nvazquez commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
nvazquez commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-714753945
@blueorangutan test centos7 vmware-67u3
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] rhtyd commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
rhtyd commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-699766020
@blueorangutan test centos7 vmware-67u3
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-700617403
@harikrishna-patnala 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 to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] vladimirpetrov closed pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
vladimirpetrov closed pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-711990371
@harikrishna-patnala 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 to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] andrijapanicsb commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
andrijapanicsb commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-714999310
@blueorangutan test centos7 xenserver-71
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-709616430
@nvazquez a Trillian-Jenkins test job (centos7 mgmt + vmware-67u3) 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 to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] borisstoyanov commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
borisstoyanov commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-704896445
@blueorangutan test centos7 vmware-67u3
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-692064600
@nvazquez a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-714716948
<b>Trillian test result (tid-3042)</b>
Environment: vmware-67u3 (x2), Advanced Networking with Mgmt server 7
Total time taken: 50431 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr4307-t3042-vmware-67u3.zip
Intermittent failure detected: /marvin/tests/smoke/test_diagnostics.py
Intermittent failure detected: /marvin/tests/smoke/test_kubernetes_clusters.py
Intermittent failure detected: /marvin/tests/smoke/test_outofbandmanagement.py
Intermittent failure detected: /marvin/tests/smoke/test_storage_policy.py
Intermittent failure detected: /marvin/tests/smoke/test_templates.py
Intermittent failure detected: /marvin/tests/smoke/test_volumes.py
Smoke tests completed. 82 look OK, 3 have error(s)
Only failed tests results shown below:
Test | Result | Time (s) | Test File
--- | --- | --- | ---
ContextSuite context=TestKubernetesCluster>:setup | `Error` | 0.00 | test_kubernetes_clusters.py
test_01_import_storage_policies | `Error` | 3.33 | test_storage_policy.py
ContextSuite context=TestVMWareStoragePolicies>:teardown | `Error` | 4.43 | test_storage_policy.py
test_02_1_create_template_with_checksum_sha1_negative | `Error` | 0.07 | test_templates.py
test_02_create_template_with_checksum_sha1 | `Error` | 0.06 | test_templates.py
test_03_1_create_template_with_checksum_sha256_negative | `Error` | 0.06 | test_templates.py
test_03_create_template_with_checksum_sha256 | `Error` | 0.07 | test_templates.py
test_04_1_create_template_with_checksum_md5_negative | `Error` | 0.06 | test_templates.py
test_04_create_template_with_checksum_md5 | `Error` | 0.06 | test_templates.py
test_05_create_template_with_no_checksum | `Error` | 0.06 | test_templates.py
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] borisstoyanov commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
borisstoyanov commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-698842821
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-699799352
Packaging result: ✔centos7 ✔centos8 ✔debian. JID-2091
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] harikrishna-patnala commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
harikrishna-patnala commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-704730114
@blueorangutan package
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] rhtyd commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
rhtyd commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-703285012
@blueorangutan test
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] borisstoyanov commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
borisstoyanov commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-716725427
Sorry what's the best way to do it? Revert will create yet another pull request..
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] harikrishna-patnala commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
harikrishna-patnala commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-705308320
@blueorangutan package
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] DaanHoogland commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
DaanHoogland commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-705708999
@blueorangutan test centos7 vmware-67u3
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-713373585
@andrijapanicsb 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 to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-688460101
@nvazquez a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support
Posted by GitBox <gi...@apache.org>.
blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-692238371
@nvazquez a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org