Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support autoinstall for ubuntu 23.04 desktop #457

Merged
merged 33 commits into from
Apr 19, 2023

Conversation

ZouYuhua
Copy link
Collaborator

Task:
Support autoinstall for ubuntu desktop 23.04

Test results:

  1. Pass for ubuntu 23.04 desktop with bios/efi:

image

image

  1. Pass for ubuntu 23.04 live server

image

  1. Pass for ubuntu 22.04 desktop

image

  1. Pass for ubuntu 22.04 live server

image

Copy link
Contributor

@keirazhang keirazhang left a comment

Choose a reason for hiding this comment

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

linux/deploy_vm/eject_del_unattend_install_iso.yml and linux/deploy_vm/collect_vm_logs.yml shall also be updated.

autoinstall/README.md Outdated Show resolved Hide resolved
autoinstall/Ubuntu/Flutter/user-data.j2 Outdated Show resolved Hide resolved
autoinstall/Ubuntu/Flutter/user-data.j2 Outdated Show resolved Hide resolved
linux/deploy_vm/rebuild_unattend_install_iso.yml Outdated Show resolved Hide resolved
linux/deploy_vm/ubuntu/ubuntu_install_os.yml Outdated Show resolved Hide resolved
linux/deploy_vm/create_unattend_install_iso.yml Outdated Show resolved Hide resolved
@ZouYuhua ZouYuhua force-pushed the zyh_ubuntu23.04_desktop branch from c91c9f1 to 37626d2 Compare April 13, 2023 03:13
@ZouYuhua
Copy link
Collaborator Author

linux/deploy_vm/eject_del_unattend_install_iso.yml and linux/deploy_vm/collect_vm_logs.yml shall also be updated.

Thanks. Change it.

@ZouYuhua
Copy link
Collaborator Author

ZouYuhua commented Apr 13, 2023

Hit issue for the full testing after rebase from upstream:

2023-04-13 06:03:56,013 | TASK [1_deploy_vm][Check vm_primary_nic_mac is set for VM 'test_vm_1'] 
task path: /home/worker/workspace/Ansible_Ubuntu_23.04_Desktop_ISO_80GA_NVME_E1000E_EFI/ansible-vsphere-gos-validation/common/vm_get_ip.yml:10
fatal: [localhost]: FAILED! => {
    "assertion": "vm_primary_nic_mac",
    "changed": false,
    "evaluated_to": false,
    "msg": "Parameter 'vm_primary_nic_mac' must be set for getting VM's guest IPv4 address."
}
error message:
Parameter 'vm_primary_nic_mac' must be set for getting VM's guest IPv4 address.

I am working on the fix. Thanks

@ZouYuhua
Copy link
Collaborator Author

ZouYuhua commented Apr 15, 2023

Hit issue for the full testing after rebase from upstream:

2023-04-13 06:03:56,013 | TASK [1_deploy_vm][Check vm_primary_nic_mac is set for VM 'test_vm_1'] 
task path: /home/worker/workspace/Ansible_Ubuntu_23.04_Desktop_ISO_80GA_NVME_E1000E_EFI/ansible-vsphere-gos-validation/common/vm_get_ip.yml:10
fatal: [localhost]: FAILED! => {
    "assertion": "vm_primary_nic_mac",
    "changed": false,
    "evaluated_to": false,
    "msg": "Parameter 'vm_primary_nic_mac' must be set for getting VM's guest IPv4 address."
}
error message:
Parameter 'vm_primary_nic_mac' must be set for getting VM's guest IPv4 address.

The module community.vmware.vmware_guest_network doesn't get the mac address:

TASK [Set fact of VM network adapters info] ************************************
task path: /home/worker/workspace/Ansible_Ubuntu_23.04_Desktop_ISO_MAIN_SATA_VMXNET3_EFI/ansible-vsphere-gos-validation/common/vm_get_network_facts.yml:21
ok: [localhost] => {
    "ansible_facts": {
        "vm_network_adapters": {
            "0": {
                "allow_guest_ctl": true,
                "connected": false,
                "device_type": "vmxnet3",
                "label": "Network adapter 1",
                "mac_addr": null,
                "mac_address": null,
                "name": "VM Network",
                "network_name": "VM Network",
                "start_connected": true,
                "switch": "vSwitch0",
                "unit_number": 7,
                "vlan_id": 0,
                "wake_onlan": true
            }
        }
    },
    "changed": false
}

TASK [Print the VM network adapters info] **************************************
task path: /home/worker/workspace/Ansible_Ubuntu_23.04_Desktop_ISO_MAIN_SATA_VMXNET3_EFI/ansible-vsphere-gos-validation/common/vm_get_network_facts.yml:25
ok: [localhost] => {
    "vm_network_adapters": {
        "0": {
            "allow_guest_ctl": true,
            "connected": false,
            "device_type": "vmxnet3",
            "label": "Network adapter 1",
            "mac_addr": null,
            "mac_address": null,
            "name": "VM Network",
            "network_name": "VM Network",
            "start_connected": true,
            "switch": "vSwitch0",
            "unit_number": 7,
            "vlan_id": 0,
            "wake_onlan": true
        }
    }
}

TASK [Set fact of VM's primary network adapter MAC address] ********************
task path: /home/worker/workspace/Ansible_Ubuntu_23.04_Desktop_ISO_MAIN_SATA_VMXNET3_EFI/ansible-vsphere-gos-validation/common/vm_get_vm_info.yml:30
ok: [localhost] => {
    "ansible_facts": {
        "vm_primary_nic_mac": ""
    },
    "changed": false
}

ZouYuhua added 20 commits April 17, 2023 18:07
Signed-off-by: ZouYuhua <[email protected]>
Signed-off-by: ZouYuhua <[email protected]>
Signed-off-by: ZouYuhua <[email protected]>
Signed-off-by: ZouYuhua <[email protected]>
Signed-off-by: ZouYuhua <[email protected]>
Signed-off-by: ZouYuhua <[email protected]>
Signed-off-by: ZouYuhua <[email protected]>
Signed-off-by: ZouYuhua <[email protected]>
Signed-off-by: ZouYuhua <[email protected]>
Signed-off-by: ZouYuhua <[email protected]>
@ZouYuhua ZouYuhua force-pushed the zyh_ubuntu23.04_desktop branch from 7724019 to 7a450a3 Compare April 17, 2023 10:07
@ZouYuhua
Copy link
Collaborator Author

The module community.vmware.vmware_guest_network doesn't get the mac address

Apply the workaround for the issue:
run vm_get_network_facts.yml to get the mac address before the update_inventory.yml

@ZouYuhua
Copy link
Collaborator Author

Hi Qi,
Is it OK for the workaround to the issue "Parameter 'vm_primary_nic_mac' must be set for getting VM's guest IPv4 address." Thanks

ZouYuhua added 2 commits April 18, 2023 13:51
@ZouYuhua
Copy link
Collaborator Author

The testing is on-going after all the changes. Thanks

common/vm_wait_primary_nic_mac.yml Outdated Show resolved Hide resolved
common/vm_wait_primary_nic_mac.yml Outdated Show resolved Hide resolved
common/vm_wait_primary_nic_mac.yml Show resolved Hide resolved
common/vm_wait_primary_nic_mac.yml Outdated Show resolved Hide resolved
common/vm_get_vm_info.yml Outdated Show resolved Hide resolved
common/vm_wait_primary_nic_mac.yml Outdated Show resolved Hide resolved
common/vm_wait_primary_nic_mac.yml Show resolved Hide resolved
common/vm_wait_primary_nic_mac.yml Outdated Show resolved Hide resolved
common/vm_wait_primary_nic_mac.yml Outdated Show resolved Hide resolved
common/vm_get_vm_info.yml Show resolved Hide resolved
common/vm_wait_primary_nic_mac.yml Outdated Show resolved Hide resolved
common/vm_wait_primary_nic_mac.yml Outdated Show resolved Hide resolved
linux/deploy_vm/deploy_vm_from_iso.yml Show resolved Hide resolved
windows/deploy_vm/deploy_vm_from_iso.yml Show resolved Hide resolved
ZouYuhua added 2 commits April 19, 2023 12:07
Signed-off-by: ZouYuhua <[email protected]>
Signed-off-by: ZouYuhua <[email protected]>
Copy link
Contributor

@keirazhang keirazhang left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks!

@ZouYuhua ZouYuhua merged commit 01379b8 into vmware:main Apr 19, 2023
@keirazhang keirazhang added this to the v2.3 milestone Apr 19, 2023
@ZouYuhua ZouYuhua deleted the zyh_ubuntu23.04_desktop branch April 23, 2023 07:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants