<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Apr 29, 2013 at 10:29 AM, Mark McLoughlin <span dir="ltr"><<a href="mailto:markmc@redhat.com" target="_blank">markmc@redhat.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On Mon, 2013-04-29 at 09:45 -0400, Eric Windisch wrote:<br>
> ><br>
> > It was, originally, but create_worker() uses the RPC dispatcher so we had to change. Now that we have join_consumer_pool() in the public API, we could change back.<br>
> ><br>
><br>
><br>
> Is create_worker not used in Ceilometer anymore? This was added for<br>
> Ceilometer's benefit. If it isn't used, we might want to deprecate it.<br>
<br>
</div>It is used to listen on the 'metering' topic ... but I'm not entirely<br>
sure why.<br>
<br>
        # Set ourselves up as a separate worker for the metering data,<br>
        # since the default for service is to use create_consumer().<br>
        self.conn.create_worker(<br>
            cfg.CONF.metering_topic,<br>
            rpc_dispatcher.RpcDispatcher([self]),<br>
            'ceilometer.collector.' + cfg.CONF.metering_topic,<br>
        )<br>
<br>
The best guess I can come up for this is that we want to use a<br>
non-default worker pool so that third party listeners can use the<br>
default worker pool.<br></blockquote><div><br></div><div style>If I had written join_consumer_pool() before create_worker() I would never have needed create_worker(). I'd like to wait for the new rpc API to come out before changing ceilometer, but I think we can assume create_worker() could go away.</div>
<div style><br></div><div style>Doug</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Cheers,<br>
Mark.<br>
<div class="HOEnZb"><div class="h5"><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>
</div></div></blockquote></div><br></div></div>