Skip to content

Conversation

@weizhouapache
Copy link
Member

@weizhouapache weizhouapache commented Mar 29, 2023

Description

This PR adds

  • The support (hypervisor capabilities) for VMware 8.0b (8.0.0.2) and 8.0c (8.0.0.3)

Refer to
https://docs.vmware.com/en/VMware-vSphere/8.0/rn/vsphere-esxi-80b-release-notes/index.html
https://docs.vmware.com/en/VMware-vSphere/8.0/rn/vsphere-esxi-80c-release-notes/index.html

  • The new guest os mappings introduced in VMware 8.0 (also supported by 8.0a and 8.0b/8.0c)

Refer to https://vdc-repo.vmware.com/vmwb-repository/dcr-public/c476b64b-c93c-4b21-9d76-be14da0148f9/04ca12ad-59b9-4e1c-8232-fd3d4276e52c/SDK/vsphere-ws/docs/ReferenceGuide/vim.vm.GuestOsDescriptor.GuestOsIdentifier.html

  • An extra step to determine the guest os mappings for a hypervisor version

Since VMware 8, each minor update has a newer patch version. For example ESXi host of VMware 8.0b has hypervisor version 8.0.0.2. It is different from previous versions, for example, ESXi host of VMware 7.0 update 3a/3b/3c have the same hypervisor version 7.0.3.0 (not 7.0.3.1/7.0.3.2/7.0.3.3).

This PR adds an extra step to determine the guest os mappings for a hypervisor version: If guest os mapping does not exist, then search for the guest os mapping of parent version. For example
X if guest os mapping of 8.0.0.2 does not exist, then search for guest os mapping for 8.0
X if guest os mapping of 8.0.3.2 does not exist, then search for guest os mapping for 8.0.3

Based on this, we do not need to add guest os mappings for 8.0b (8.0.0.2) and 8.0c (8.0.0.3), the guest os mappings for VMware 8.0 will be used.

Pleae note:
The hypervisor capabilitiy for VMware 8.0U1 (8.0.1.0) has been added in this PR, but there are some critical failures in the testing on VMware 8.0U1, thus it is not supported. (see more information at #7572 )

Types of changes

  • Breaking change (fix or feature that would cause existing functionality to change)
  • 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)

Feature/Enhancement Scale or Bug Severity

Feature/Enhancement Scale

  • Major
  • Minor

Bug Severity

  • BLOCKER
  • Critical
  • Major
  • Minor
  • Trivial

Screenshots (if appropriate):

How Has This Been Tested?

@weizhouapache
Copy link
Member Author

@blueorangutan package

@blueorangutan
Copy link

@weizhouapache a Jenkins job has been kicked to build packages. It will be bundled with KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

@blueorangutan
Copy link

Packaging result: ✔️ el7 ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 5805

@weizhouapache
Copy link
Member Author

@blueorangutan package

@blueorangutan
Copy link

@weizhouapache a Jenkins job has been kicked to build packages. It will be bundled with KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

@blueorangutan
Copy link

Packaging result: ✔️ el7 ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 5825

@codecov
Copy link

codecov bot commented Mar 31, 2023

Codecov Report

Merging #7380 (fc07405) into 4.18 (feb9509) will increase coverage by 0.01%.
The diff coverage is 20.27%.

@@             Coverage Diff              @@
##               4.18    #7380      +/-   ##
============================================
+ Coverage     13.02%   13.03%   +0.01%     
- Complexity     9032     9060      +28     
============================================
  Files          2720     2720              
  Lines        257080   257220     +140     
  Branches      40088    40098      +10     
============================================
+ Hits          33476    33539      +63     
- Misses       219400   219471      +71     
- Partials       4204     4210       +6     
Files Changed Coverage Δ
.../hypervisor/dao/HypervisorCapabilitiesDaoImpl.java 0.00% <0.00%> (ø)
...ava/com/cloud/upgrade/dao/Upgrade41800to41810.java 2.36% <0.00%> (-4.31%) ⬇️
...oud/hypervisor/vmware/resource/VmwareResource.java 4.49% <0.00%> (-0.02%) ⬇️
...om/cloud/storage/dao/GuestOSHypervisorDaoImpl.java 33.66% <50.00%> (+1.40%) ⬆️
...src/main/java/com/cloud/upgrade/GuestOsMapper.java 24.19% <62.96%> (+19.09%) ⬆️
...com/cloud/hypervisor/guru/VmwareVmImplementer.java 14.36% <88.88%> (+14.36%) ⬆️

... and 1 file with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@github-advanced-security
Copy link

You have successfully added a new SonarCloud configuration ``. As part of the setup process, we have scanned this repository and found no existing alerts. In the future, you will see all code scanning alerts on the repository Security tab.

@blueorangutan
Copy link

Trillian test result (tid-6381)
Environment: vmware-80 (x2), Advanced Networking with Mgmt server r8
Total time taken: 42108 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr7380-t6381-vmware-80.zip
Smoke tests completed. 108 look OK, 0 have errors, 0 did not run
Only failed and skipped tests results shown below:

Test Result Time (s) Test File

@DaanHoogland
Copy link
Contributor

@blueorangutan package

@blueorangutan
Copy link

@DaanHoogland a Jenkins job has been kicked to build packages. It will be bundled with KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

@weizhouapache
Copy link
Member Author

@blueorangutan package

@blueorangutan
Copy link

@weizhouapache a Jenkins job has been kicked to build packages. It will be bundled with KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

@blueorangutan
Copy link

Packaging result: ✔️ el7 ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 5876

Copy link
Contributor

@harikrishna-patnala harikrishna-patnala left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code LGTM.

@rohityadavcloud rohityadavcloud added this to the 4.18.1.0 milestone Apr 13, 2023
@weizhouapache weizhouapache changed the title VMware: add support for 8.0b (8.0.0.2) VMware: add support for 8.0b (8.0.0.2) and 8.0c (8.0.0.3) Apr 17, 2023
@blueorangutan
Copy link

Trillian test result (tid-6414)
Environment: vmware-80 (x2), Advanced Networking with Mgmt server r8
Total time taken: 43090 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr7380-t6414-vmware-80.zip
Smoke tests completed. 108 look OK, 0 have errors, 0 did not run
Only failed and skipped tests results shown below:

Test Result Time (s) Test File

@blueorangutan
Copy link

Trillian test result (tid-6415)
Environment: vmware-80 (x2), Advanced Networking with Mgmt server r8
Total time taken: 41770 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr7380-t6415-vmware-80.zip
Smoke tests completed. 108 look OK, 0 have errors, 0 did not run
Only failed and skipped tests results shown below:

Test Result Time (s) Test File

@blueorangutan
Copy link

Packaging result [SF]: ✔️ el7 ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 6771

@andrijapanicsb
Copy link
Contributor

@blueorangutan test rocky8 vmware-80

@blueorangutan
Copy link

@andrijapanicsb a [SF] Trillian-Jenkins test job (rocky8 mgmt + vmware-80) has been kicked to run smoke tests

@andrijapanicsb
Copy link
Contributor

@blueorangutan test ubuntu22 vmware-80

@blueorangutan
Copy link

@andrijapanicsb a [SF] Trillian-Jenkins test job (ubuntu22 mgmt + vmware-80) has been kicked to run smoke tests

@blueorangutan
Copy link

[LL]Trillian test result (tid-6693)
Environment: vmware-80 (x2), Advanced Networking with Mgmt server u22
Total time taken: 45726 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr7380-t6693-vmware-80.zip
Smoke tests completed. 106 look OK, 2 have errors, 0 did not run
Only failed and skipped tests results shown below:

Test Result Time (s) Test File
test_04_rvpc_internallb_haproxy_stats_on_all_interfaces Error 193.87 test_internal_lb.py
test_01_RVR_Network_FW_PF_SSH_default_routes_egress_true Error 131.06 test_routers_network_ops.py
test_02_RVR_Network_FW_PF_SSH_default_routes_egress_false Error 160.30 test_routers_network_ops.py

@blueorangutan
Copy link

[SF] Trillian test result (tid-7407)
Environment: vmware-80 (x2), Advanced Networking with Mgmt server r8
Total time taken: 46606 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr7380-t7407-vmware-80.zip
Smoke tests completed. 107 look OK, 1 have errors, 0 did not run
Only failed and skipped tests results shown below:

Test Result Time (s) Test File
test_create_pvlan_network Error 0.09 test_pvlan.py

@blueorangutan
Copy link

[LL]Trillian test result (tid-6694)
Environment: vmware-80 (x2), Advanced Networking with Mgmt server r8
Total time taken: 50983 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr7380-t6694-vmware-80.zip
Smoke tests completed. 106 look OK, 2 have errors, 0 did not run
Only failed and skipped tests results shown below:

Test Result Time (s) Test File
test_02_port_fwd_on_non_src_nat Error 13.52 test_network.py
test_01_deploy_vm_on_specific_host Error 18.57 test_vm_deployment_planner.py
test_02_deploy_vm_on_specific_cluster Error 3602.66 test_vm_deployment_planner.py
test_03_deploy_vm_on_specific_pod Error 1.23 test_vm_deployment_planner.py
test_04_deploy_vm_on_host_override_pod_and_cluster Error 2.23 test_vm_deployment_planner.py
test_05_deploy_vm_on_cluster_override_pod Error 2.24 test_vm_deployment_planner.py

@blueorangutan
Copy link

[SF] Trillian test result (tid-7408)
Environment: vmware-80 (x2), Advanced Networking with Mgmt server u22
Total time taken: 52410 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr7380-t7408-vmware-80.zip
Smoke tests completed. 107 look OK, 1 have errors, 0 did not run
Only failed and skipped tests results shown below:

Test Result Time (s) Test File
test_router_dhcphosts Failure 146.75 test_router_dhcphosts.py

@weizhouapache
Copy link
Member Author

@blueorangutan test rocky8 vmware-80

@blueorangutan
Copy link

@weizhouapache a [SF] Trillian-Jenkins test job (rocky8 mgmt + vmware-80) has been kicked to run smoke tests

@weizhouapache
Copy link
Member Author

@blueorangutan test rocky8 vmware-67u3

@blueorangutan
Copy link

@weizhouapache a [SF] Trillian-Jenkins test job (rocky8 mgmt + vmware-67u3) has been kicked to run smoke tests

@weizhouapache
Copy link
Member Author

@blueorangutan test ubuntu22 vmware-80

@blueorangutan
Copy link

@weizhouapache a [SF] Trillian-Jenkins test job (ubuntu22 mgmt + vmware-80) has been kicked to run smoke tests

@weizhouapache weizhouapache self-assigned this Aug 15, 2023
@blueorangutan
Copy link

[SF] Trillian test result (tid-7428)
Environment: vmware-80 (x2), Advanced Networking with Mgmt server u22
Total time taken: 51176 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr7380-t7428-vmware-80.zip
Smoke tests completed. 107 look OK, 1 have errors, 0 did not run
Only failed and skipped tests results shown below:

Test Result Time (s) Test File
test_02_list_cpvm_vm Failure 0.06 test_ssvm.py
test_04_cpvm_internals Failure 0.05 test_ssvm.py
test_12_destroy_cpvm Error 6.31 test_ssvm.py

@blueorangutan
Copy link

[SF] Trillian test result (tid-7427)
Environment: vmware-67u3 (x2), Advanced Networking with Mgmt server r8
Total time taken: 52790 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr7380-t7427-vmware-67u3.zip
Smoke tests completed. 107 look OK, 1 have errors, 0 did not run
Only failed and skipped tests results shown below:

Test Result Time (s) Test File
test_01_deploy_vm_on_specific_host Error 26.95 test_vm_deployment_planner.py
test_02_deploy_vm_on_specific_cluster Error 3605.14 test_vm_deployment_planner.py
test_03_deploy_vm_on_specific_pod Error 1.39 test_vm_deployment_planner.py
test_04_deploy_vm_on_host_override_pod_and_cluster Error 4.46 test_vm_deployment_planner.py
test_05_deploy_vm_on_cluster_override_pod Error 1.33 test_vm_deployment_planner.py

@blueorangutan
Copy link

[SF] Trillian test result (tid-7426)
Environment: vmware-80 (x2), Advanced Networking with Mgmt server r8
Total time taken: 53187 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr7380-t7426-vmware-80.zip
Smoke tests completed. 106 look OK, 2 have errors, 0 did not run
Only failed and skipped tests results shown below:

Test Result Time (s) Test File
test_08_upgrade_kubernetes_ha_cluster Failure 378.91 test_kubernetes_clusters.py
test_09_delete_kubernetes_ha_cluster Failure 0.09 test_kubernetes_clusters.py
ContextSuite context=TestKubernetesCluster>:teardown Error 4052.65 test_kubernetes_clusters.py
test_01_create_redundant_VPC_2tiers_4VMs_4IPs_4PF_ACL Failure 486.80 test_vpc_redundant.py

@weizhouapache
Copy link
Member Author

@blueorangutan test ubuntu22 vmware-80

@weizhouapache
Copy link
Member Author

@blueorangutan test rocky8 vmware-67u3

@blueorangutan
Copy link

@weizhouapache a [SF] Trillian-Jenkins test job (rocky8 mgmt + vmware-67u3) has been kicked to run smoke tests

@weizhouapache
Copy link
Member Author

@blueorangutan test ubuntu22 vmware-80

@blueorangutan
Copy link

@weizhouapache a [SF] Trillian-Jenkins test job (ubuntu22 mgmt + vmware-80) has been kicked to run smoke tests

@blueorangutan
Copy link

[SF] Trillian test result (tid-7448)
Environment: vmware-67u3 (x2), Advanced Networking with Mgmt server r8
Total time taken: 42970 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr7380-t7448-vmware-67u3.zip
Smoke tests completed. 108 look OK, 0 have errors, 0 did not run
Only failed and skipped tests results shown below:

Test Result Time (s) Test File

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants