The idea is that we want to be able to support multiple instances of the network barclamp at some point in the future.  :id is the id of the Proposal object that is the barclamp instance, and network_id is the id of the Network associated with the Proposal.  Both are required because of the convention that we adopted where either the id of an object can be passed or the name.  A client could call in to this API method with the name of the network, in which case a handle to the Proposal is required in order to be able to distinguish between identically named networks that are associated with different network barclamp instances.

For this port, I am trying to keep the interface the same as much as possible as it was in crowbar 1.0.  That interface required a barclamp instance id, a network name, and a node name.  It didn't pass the allocated IP address, since it looked that up from the supplied arguments.

I looked for the usage of deallocate IP in our existing code to determine if changing the interface to this call would make sense, and it appears that we don't use this call at all!