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

New resource: vsphere_distributed_virtual_switch (refactor) #188

Merged
merged 21 commits into from
Oct 10, 2017

Conversation

vancluever
Copy link
Contributor

This commit is a major refactor of the DVS resource provided in PR #135.

More options have been added, the DVS has been fixed so that it uses the
VMware DVS object instead of the generic DVS object, which is more or
less just used for 3rd party DVS switches. This meant that the DVS was
missing a number of features that more than likely would have been
expected in the DVS.

The breadth of the features added can be seen in the documentation. It
should be noted that this does not include the ability to set private
VLANs, filtering policies, or port mirroring sessions, which will be
added at a later date and pending business case.

This work also contains a bunch of up front work that will be necessary
for the vsphere_distributed_port_group resource, which will be following
not too long after this resource.

jorgenunez and others added 21 commits October 2, 2017 15:17
This commit is a major refactor of the DVS resource provided in PR #135.

More options have been added, the DVS has been fixed so that it uses the
VMware DVS object instead of the generic DVS object, which is more or
less just used for 3rd party DVS switches. This meant that the DVS was
missing a number of features that more than likely would have been
expected in the DVS.

The breadth of the features added can be seen in the documentation. It
should be noted that this does not include the ability to set private
VLANs, filtering policies, or port mirroring sessions, which will be
added at a later date and pending business case.

This work also contains a bunch of up front work that will be necessary
for the vsphere_distributed_port_group resource, which will be following
not too long after this resource.
@vancluever vancluever added enhancement Type: Enhancement new-resource Feature: New Resource labels Oct 8, 2017
vancluever added a commit that referenced this pull request Oct 9, 2017
This commit adds the vsphere_distributed_port_group resource, allowing
port groups to be created on DVS switches, introducing the ability to
manage real networks with Terraform that can be used to set up virtual
machines on.

This update also includes various fixes to #188 to fix spurious diffs
and expected behaviour when both resources are used in tandem.
Copy link
Member

@mbfrahry mbfrahry left a comment

Choose a reason for hiding this comment

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

Looks really good!

@vancluever
Copy link
Contributor Author

Thanks @mbfrahry!

@vancluever vancluever merged commit f54ff44 into master Oct 10, 2017
vancluever added a commit that referenced this pull request Oct 10, 2017
This commit adds the vsphere_distributed_port_group resource, allowing
port groups to be created on DVS switches, introducing the ability to
manage real networks with Terraform that can be used to set up virtual
machines on.

This update also includes various fixes to #188 to fix spurious diffs
and expected behaviour when both resources are used in tandem.
@vancluever vancluever deleted the f-dvs-resource branch October 12, 2017 00:30
@ghost ghost locked and limited conversation to collaborators Apr 19, 2020
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
enhancement Type: Enhancement new-resource Feature: New Resource
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants