Missing restart_if_needed parameter

fanatic / go-infoblox

Golang Infoblox WAPI Library (Deprecated)

16 stars 26 forks source link

Missing restart_if_needed parameter #40

Closed dn1s closed 5 years ago

dn1s commented 5 years ago

If configure_for_dns or configure_for_dhcp are set to true for a host object you might need to restart dhcp or dns service.

dn1s commented 5 years ago

Just tried it out in my infoblox instance. restart_if_needed needs a newer api version than 1.4.1. I'm not really sure when this feature was introduced and I don't have an support account for infoblox so I can't look it up. So testing this I just bumped version to 2.1 which seems to be fine. Obviously I didn't test all functionality with this api version. I can open a pull request but somebody has to test if api 2.1 is working with all features.

dn1s commented 5 years ago

Well I just worked around this problem as you can see here: #41 So no need for updating api version or adding restart_if_needed parameter which on a side note did not work properly for me either.