<div dir="ltr"><div><div><div>Hey Will,<br><br></div>FWIW, I'm responsible for 2 of them - dblogpruner and txnpruner. They were created before I'd ever heard anything about workers not using *state.State directly, certainly before the recent push to clean such workers up. They're not all that new and weren't created in violation of explicit instructions (the instructions came later). <br><br>When I created these workers I  looked at the existing code and saw that  workers which only ran within the state servers  used State directly so that's what I did. These workers are very much state server specific so it seemed sensible to take this approach.<br><br></div>I will update these workers to go via the API and cards already exist on our team's board. It's just a matter of finding time amongst everything else.<br><br></div>- Menno<br><div><br><br><div><br><br><br><br></div></div></div><div class="gmail_extra"><br><div class="gmail_quote">On 8 September 2015 at 19:12, William Reade <span dir="ltr"><<a href="mailto:william.reade@canonical.com" target="_blank">william.reade@canonical.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">People keep writing them, in defiance of sane layering and explicit instructions, for the most embarrassingly trivial tasks (statushistorypruner? dblogpruner? txnpruner? *all* of those can and should pass through a simple api facade, not just dance off to play with the direct-db-access fairies.)<div><br></div><div>There is no justification for *any* of those things to see a *state.State, and I'm going to start treating new workers that violate layering this way as deliberate sabotage attempts. Leads who have overseen the introduction of those workers, sort it out.</div></div>
<br>--<br>
Juju-dev mailing list<br>
<a href="mailto:Juju-dev@lists.ubuntu.com">Juju-dev@lists.ubuntu.com</a><br>
Modify settings or unsubscribe at: <a href="https://lists.ubuntu.com/mailman/listinfo/juju-dev" rel="noreferrer" target="_blank">https://lists.ubuntu.com/mailman/listinfo/juju-dev</a><br>
<br></blockquote></div><br></div>