Fail resource on target_alias change
Fail resource on target_alias change

Previously the resource used `err` which will fail the property change,
but not the resource flush. It also did not set the target_alias on
create, so would generate changes every run. This fixes both of those.

ccaum/puppetlabs-netapp,puppetlabs/puppetlabs-netapp,amit0701/DataONTAPDevice,tphoney/puppetlabs-netapp