feature: add field 'parent_id' to Location Resource & Data Source

[GennadySpb]

Hello there!

We actively use Location nesting. And I really miss the ability to read and write the “parent_id” value in a DCIM model resource Location.

This PR is about the same as in #508.

PTAL.

[fbreckle]

Can you add a test that UNSETs the parent_id? I assume it will fail because of reasons similar to the linked issue. I will then fix it in fbreckle/go-netbox, if necessary.

[GennadySpb]

Can you add a test that UNSETs the parent_id? I assume it will fail because of reasons similar to the linked issue. I will then fix it in fbreckle/go-netbox, if necessary.

You're right, the update doesn't work.
I'm trying to update go-netbox locally for testing (use the x-omitempty: false option), but it failed on generate goal when running:

make: *** [generate] Error 137

What could be wrong?

p.s. Docker at macOS on ARM CPU.

[fbreckle]

Googling the error gives me mostly "out of memory" results. I will generate a branch with the change for you.

[fbreckle]

Change is at https://github.com/fbreckle/go-netbox/tree/feature/parent_id

You should be able to use go get github.com/fbreckle/go-netbox@3b369d7c46cfb41981a33275e1a1ae841248d68d now

[GennadySpb]

Did a quick check. Looks like it works :)

[fbreckle]

Can you add the test that unsets a parent id as well? Else LGTM

Just add one more step to one of the tests where the parent id is unset and then check if its actually unset or 0 or whatever.

[GennadySpb]

Can you add the test that unsets a parent id as well? Else LGTM
Check Acceptance test TestAccNetboxLocationsDataSource_sublocations, it contains two configs:

1. create hierarchy of Locations
2. remove parent_id