-
-
Notifications
You must be signed in to change notification settings - Fork 112
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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(vm): add VLAN
trunk support
#1086
Conversation
Signed-off-by: Jack Hodgkiss <identity@jackhodgkiss.uk>
proxmoxtf/resource/vm/vm.go
Outdated
if err != nil { | ||
// Error parsing VLAN trunks. | ||
os.Exit(1) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would appreciate help on how best to handle errors.
I am new to golang.
Thanks so much for contribution @jackhodgkiss! I'll take a look today / tomorrow. |
Signed-off-by: Pavel Boldyrev <627562+bpg@users.noreply.github.com>
Signed-off-by: Pavel Boldyrev <627562+bpg@users.noreply.github.com>
Signed-off-by: Pavel Boldyrev <627562+bpg@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work, thanks @jackhodgkiss!
I've update the error handling code and also add trunks to the automated acceptance test. The acceptance tests are still somewhat experimental, but i'm planning to use them more, esp. for new functionality.
LGTM! 馃殌
@all-contributors please add @jackhodgkiss for code |
I've put up a pull request to add @jackhodgkiss! 馃帀 |
Contributor's Note
I have been using the provider to create a series of Proxmox VMs that require multiple VLANs on a single interface. The approach I have taken is to use trunks to pass through multiple tagged VLANs for which interfaces can be created post deployment such as
eth0.10
.Trunks whilst missing from the UI within Proxmox are supported and documented.
I have been using this small change without issue for a couple months.
/docs
for any user-facing features or additions./example
for any new or updated resources / data sources.make example
to verify that the change works as expected.Proof of Work
The screenshot below shows a network interface with the trunks property containing
10;20;30
as trunked VLANs which can the be utilised within the VM after any OS specific network configuration is applied.Community Note
Closes #0000 | Relates #0000