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

Add device_id and device_name and tests for virtual_machine #477

Merged

Conversation

zeddD1abl0
Copy link
Contributor

There is no associated Issue/Feature Request, but I went to use this provider with the ProxMox provider and a few other things, and I realised that I needed the Device Name and/or the Device ID.

This update simply adds the required values/fields to the data source.

NOTE: There are a number of lines that start with "var" that were removed. It appears that go linters prefer lines like
var thing = "Hello"
to instead look similar to
thing := "Hello"

@fbreckle fbreckle merged commit 29f856e into e-breuninger:master Sep 20, 2023
7 checks passed
@zeddD1abl0 zeddD1abl0 deleted the fr-add-device-id-to-vm-datasource branch September 22, 2023 11:51
w87x pushed a commit to w87x/terraform-provider-netbox that referenced this pull request Oct 5, 2023
…nger#477)

* Add device_id and device_name and tests

* Restore removed var
w87x pushed a commit to w87x/terraform-provider-netbox that referenced this pull request Oct 5, 2023
…nger#477)

* Add device_id and device_name and tests

* Restore removed var
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