<div dir="ltr">Hi Matt!<div><br></div><div>I've looked into this a bit now, too, and don't have a conclusive answer as to how (or even whether) it's working today. </div><div><br></div><div>Instead, I'd like to point out that Ironic is aiming to deprecate nova.virt.baremetal and nova.scheduler.baremetal_host_manager, in favor of nova.virt.ironic and nova.scheduler.ironic_host_manager, both of which currently live in Ironic's git tree. These were initially based on the baremetal code that you're looking at, and so probably inherited this problem (if it is a problem). Let's fix it in Ironic.</div>

<div><br></div><div>Thanks!</div><div>-Devananda</div><div><br></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Apr 4, 2014 at 7:46 AM, Matthew Booth <span dir="ltr"><<a href="mailto:mbooth@redhat.com" target="_blank">mbooth@redhat.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Whilst looking at something unrelated in HostManager, I noticed that<br>
HostManager.service_states appears to be unused, and decided to remove<br>
it. This seems to have a number of implications:<br>
<br>
1. capabilities in HostManager.get_all_host_states will always be None.<br>
2. capabilities passed to host_state_cls() will always be None<br>
(host_state_cls doesn't appear to be used anywhere else)<br>
3. baremetal_host_manager.new_host_state() capabilities will always be None.<br>
4. cap will always be {}, so will never contain 'baremetal_driver'<br>
5. BaremetalNodeState will never be instantiated<br>
6. BaremetalHostManager is a no-op<br>
<br>
possibly resulting in<br>
<br>
7. The filter scheduler could try to put multiple instances on a single<br>
bare metal host<br>
<br>
This was going to be a 3 line cleanup, but it looks like a can of worms<br>
so I'm going to drop it. It's entirely possible that I've missed another<br>
entry point in to this code, but it might be worth a quick look.<br>
Incidentally, the tests seem to populate service_states in fake, so the<br>
behaviour of the automated tests probably isn't reliable.<br>
<br>
Matt<br>
<span class="HOEnZb"><font color="#888888">--<br>
Matthew Booth, RHCA, RHCSS<br>
Red Hat Engineering, Virtualisation Team<br>
<br>
GPG ID:  D33C3490<br>
GPG FPR: 3733 612D 2D05 5458 8A8A 1600 3441 EA19 D33C 3490<br>
<br>
_______________________________________________<br>
OpenStack-dev mailing list<br>
<a href="mailto:OpenStack-dev@lists.openstack.org">OpenStack-dev@lists.openstack.org</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
</font></span></blockquote></div><br></div>