Hi,<div><br></div><div>I'm trying to extend the os-hosts extensions by adding some attributes, but it seems that this is not possible after a lot of debugging.</div><div><br></div><div>This is my current approach:</div>
<div><a href="http://pastebin.com/D6mFTE0E">http://pastebin.com/D6mFTE0E</a> </div><div><br></div><div>After debugging it seems that it never registers the show function extension here: <a href="https://github.com/openstack/nova/blob/master/nova/api/openstack/wsgi.py#L714">https://github.com/openstack/nova/blob/master/nova/api/openstack/wsgi.py#L714</a> </div>
<div>Should os-hosts first be changed to core feature before it can be extended? Would it be worth to directly patch contrib/hosts extensions instead of trying to extend an extension?</div><div><br></div><div>Some background:</div>
<div>The company I started working for did many small niche things and hacks directly to the nova base code, but I recently saw the nova options for easily extending nearly anything in nova.</div><div>Such as osapi_compute_extensions, compute_manager_class, compute_api_class, compute_driver etc.</div>
<div><br></div><div>So I started working on migrating our changes to extensions where we define custom compute_api_class etc which uses as its parent the original nova.compute.api.API. </div><div>But hit a wall just now with extending the 'os-hosts' part. </div>
<div><br></div><div>Also wondering is this a good approach to do niche changes in this way? It seems that this way we can easily merge the folsom release with our niche changes or at least with less trouble. Because we seperated our niche changes from the openstack / nova code.</div>
<div><br></div><div>I'm also hoping to supply patches, but I may not be good enough yet and most changes we did doesn't seem to be ready yet. A small thing I changed that I think would be usefull for upstream is being able to specicy the compute_hostapi_class, which is a small patch that adds the flag and uses the same way to import the api classes as compute_api_class.</div>
<div>Related patch: <a href="http://pastebin.com/rWQCwDEW">http://pastebin.com/rWQCwDEW</a></div><div><br></div><div>Many thanks in advance! (Feeling a littly shy mailing many so many experts, sorry)</div><div>Sam Stoelinga</div>
<div><br></div><div><br></div><div><br></div><div><br></div>