Skip to content
New issue

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

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

Already on GitHub? # to your account

feat: add support for sr-iov network interface #2059

Merged

Conversation

vasilsatanasov
Copy link
Contributor

@vasilsatanasov vasilsatanasov commented Nov 10, 2023

Description

  • Ported the code introduced with feat: Add support for SR-IOV Network Adapters #1417

  • Verified that virtual machines with SR-IOV network adapters can be created

  • Added basic e2e to verify that SR-IOV is supporte4d with ported code

  • Verified that there are no regressions introduced ran TestAccResourceVSphereVirtualMachine_staticMAC
    TestAccResourceVSphereVirtualMachine_TestAccResourceVSphereVirtualMachine_hardwareVersionClone TestAccResourceVSphereVirtualMachine_disksKeepOnRemove TestAccResourceVSphereVirtualMachine_addDevices
    TestAccResourceVSphereVirtualMachine_highDiskUnitInsufficientBus TestAccResourceVSphereVirtualMachine_highDiskUnitNumbers TestAccResourceVSphereVirtualMachine_removeMiddleDevicesChangeDiskUnit TestAccResourceVSphereVirtualMachine_vAppIsoNoCdrom TestAccResourceVSphereVirtualMachine_vAppIsoBasic
    TestAccResourceVSphereVirtualMachine_vAppIsoChangeCdromBacking TestAccResourceVSphereVirtualMachine_cloneCustomizeWithNewResourcePool TestAccResourceVSphereVirtualMachine_cloneBlockESXi TestAccResourceVSphereVirtualMachine_clonePoweredOn TestAccResourceVSphereVirtualMachine_cloneWithBadSizeWithLinkedClone TestAccResourceVSphereVirtualMachine_cloneWithDifferentHostname TestAccResourceVSphereVirtualMachine_cpuHotAdd
    TestAccResourceVSphereVirtualMachine_dualStackIPv4AndIPv6 TestAccResourceVSphereVirtualMachine_hostVMotion

on the branch with the changes + same tests on the main branch with identical results

Acceptance tests

  • Have you added an acceptance test for the functionality being added?
  • Have you run the acceptance tests on this branch?

Output from acceptance testing:

$ make testacc TESTARGS='-run=TestAccResourceVSphereVirtualMachine_SRIOV'
==> Checking that code complies with gofmt requirements...
TF_ACC=1 go test $(go list ./... |grep -v 'vendor') -v -run=TestAccResourceVSphereVirtualMachine_SRIOV -timeout 360m
?       github.com/hashicorp/terraform-provider-vsphere [no test files]
?       github.com/hashicorp/terraform-provider-vsphere/vsphere/internal/administrationroles    [no test files]
?       github.com/hashicorp/terraform-provider-vsphere/vsphere/internal/helper/computeresource [no test files]
?       github.com/hashicorp/terraform-provider-vsphere/vsphere/internal/helper/contentlibrary  [no test files]
?       github.com/hashicorp/terraform-provider-vsphere/vsphere/internal/helper/customattribute [no test files]
?       github.com/hashicorp/terraform-provider-vsphere/vsphere/internal/helper/clustercomputeresource  [no test files]
?       github.com/hashicorp/terraform-provider-vsphere/vsphere/internal/helper/datastore       [no test files]
?       github.com/hashicorp/terraform-provider-vsphere/vsphere/internal/helper/dvportgroup     [no test files]
?       github.com/hashicorp/terraform-provider-vsphere/vsphere/internal/helper/envbrowse       [no test files]
?       github.com/hashicorp/terraform-provider-vsphere/vsphere/internal/helper/datacenter      [no test files]
?       github.com/hashicorp/terraform-provider-vsphere/vsphere/internal/helper/folder  [no test files]
?       github.com/hashicorp/terraform-provider-vsphere/vsphere/internal/helper/resourcepool    [no test files]
?       github.com/hashicorp/terraform-provider-vsphere/vsphere/internal/helper/hostsystem      [no test files]
?       github.com/hashicorp/terraform-provider-vsphere/vsphere/internal/helper/spbm    [no test files]
?       github.com/hashicorp/terraform-provider-vsphere/vsphere/internal/helper/ovfdeploy       [no test files]
?       github.com/hashicorp/terraform-provider-vsphere/vsphere/internal/helper/nsx     [no test files]
?       github.com/hashicorp/terraform-provider-vsphere/vsphere/internal/helper/storagepod      [no test files]
?       github.com/hashicorp/terraform-provider-vsphere/vsphere/internal/helper/network [no test files]
?       github.com/hashicorp/terraform-provider-vsphere/vsphere/internal/helper/structure       [no test files]
?       github.com/hashicorp/terraform-provider-vsphere/vsphere/internal/helper/testhelper      [no test files]
?       github.com/hashicorp/terraform-provider-vsphere/vsphere/internal/helper/utils   [no test files]
?       github.com/hashicorp/terraform-provider-vsphere/vsphere/internal/helper/vappcontainer   [no test files]
?       github.com/hashicorp/terraform-provider-vsphere/vsphere/internal/helper/provider        [no test files]
?       github.com/hashicorp/terraform-provider-vsphere/vsphere/internal/helper/virtualmachine  [no test files]
?       github.com/hashicorp/terraform-provider-vsphere/vsphere/internal/helper/vsanclient      [no test files]
?       github.com/hashicorp/terraform-provider-vsphere/vsphere/internal/helper/vsansystem      [no test files]
?       github.com/hashicorp/terraform-provider-vsphere/vsphere/internal/vmworkflow     [no test files]
=== RUN   TestAccResourceVSphereVirtualMachine_SRIOV
--- PASS: TestAccResourceVSphereVirtualMachine_SRIOV (125.39s)
PASS
ok      github.com/hashicorp/terraform-provider-vsphere/vsphere 126.512s
testing: warning: no tests to run
PASS
ok      github.com/hashicorp/terraform-provider-vsphere/vsphere/internal/helper/viapi   0.865s [no tests to run]
testing: warning: no tests to run
PASS
ok      github.com/hashicorp/terraform-provider-vsphere/vsphere/internal/helper/virtualdisk     0.931s [no tests to run]
testing: warning: no tests to run
PASS
ok      github.com/hashicorp/terraform-provider-vsphere/vsphere/internal/virtualdevice  1.091s [no tests to run]

Release Note

Release note for CHANGELOG:

Added support for SR-IOV network devices

References

Closes #1136
Closes #1417

@vasilsatanasov vasilsatanasov requested a review from a team as a code owner November 10, 2023 11:40
@github-actions github-actions bot added documentation Type: Documentation provider Type: Provider size/xl Relative Sizing: Extra-Large labels Nov 10, 2023
@tenthirtyam tenthirtyam added this to the v2.7.0 milestone Nov 10, 2023
@tenthirtyam tenthirtyam changed the title feat: Add support for SR-IOV Network Adapters feat: add support for sr-iov network adapters Nov 13, 2023
Copy link
Collaborator

@tenthirtyam tenthirtyam left a comment

Choose a reason for hiding this comment

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

Minor changes requested. Otherwise LGTM.

@vasilsatanasov vasilsatanasov force-pushed the topic/vasilsatanasov/feat-sriov branch from 0c67c4e to 65af228 Compare November 14, 2023 09:57
Copy link
Collaborator

@tenthirtyam tenthirtyam left a comment

Choose a reason for hiding this comment

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

Minor text changes.

website/docs/r/virtual_machine.html.markdown Outdated Show resolved Hide resolved
website/docs/r/virtual_machine.html.markdown Outdated Show resolved Hide resolved
website/docs/r/virtual_machine.html.markdown Outdated Show resolved Hide resolved
website/docs/r/virtual_machine.html.markdown Outdated Show resolved Hide resolved
@vasilsatanasov vasilsatanasov force-pushed the topic/vasilsatanasov/feat-sriov branch from 65af228 to acf16e9 Compare November 14, 2023 13:59
Copy link
Collaborator

@tenthirtyam tenthirtyam left a comment

Choose a reason for hiding this comment

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

LGTM!

@tenthirtyam tenthirtyam removed this from the v2.7.0 milestone Nov 14, 2023
@tenthirtyam tenthirtyam added this to the v2.6.0 milestone Nov 14, 2023
@tenthirtyam tenthirtyam force-pushed the topic/vasilsatanasov/feat-sriov branch 2 times, most recently from af810ea to a9faecc Compare November 14, 2023 15:00
@tenthirtyam tenthirtyam changed the title feat: add support for sr-iov network adapters feat: add support for sr-iov network interface Nov 14, 2023
Adds support for the SR-IOV (`sriov`) network interface adapter type.

Signed-off-by: Vasil Atanasov <vasila@vmware.com>
@tenthirtyam tenthirtyam force-pushed the topic/vasilsatanasov/feat-sriov branch from a9faecc to 79c2121 Compare November 14, 2023 15:08
@tenthirtyam tenthirtyam merged commit 9c25530 into hashicorp:main Nov 14, 2023
Copy link

This functionality has been released in v2.6.0 of the Terraform Provider. Please see the Terraform documentation on provider versioning or reach out if you need any assistance upgrading.

For further feature requests or bug reports with this functionality, please create a new GitHub issue following the template. Thank you!

1 similar comment
Copy link

This functionality has been released in v2.6.0 of the Terraform Provider. Please see the Terraform documentation on provider versioning or reach out if you need any assistance upgrading.

For further feature requests or bug reports with this functionality, please create a new GitHub issue following the template. Thank you!

Copy link

I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Dec 31, 2023
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
documentation Type: Documentation provider Type: Provider size/xl Relative Sizing: Extra-Large
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add support for SR-IOV network adapters
3 participants