Open Stack

On Jun 9, 2011, at 2:47 AM, Chris Behrens wrote:

> Hm, inject_file is supposed to be tied into instance creation in the API.  If it's not, we have some code missing from API.  Ed Leafe did this work IIRC, as we use this here at Rackspace.  Wonder if some code got dropped at some point or if it was just never completely finished.


	The inject_file() method in nova/compute/api.py was created as essentially a placeholder. The Rackspace use case only called for file injection at instance creation time, which is why the create() method of the API accepts injected files as a parameter. I had planned on implementing the direct inject_file() method, but it was deemed out of scope at the time; perhaps a comment to that effect, or a NotImplemented error, would have made that clearer.



-- Ed Leafe