I have to disagree slightly, currently (and thankfully?) the current way most charm-helpers aim to be included in charms is by embedding a working snapshot of that version of charm-helpers. So merging this wouldn't affect a huge number of charms, instead maybe the 5 or 6 at most that download from pypi and of those none may actual be broken. As we rethink the distribution model for charm-helpers (making it a debian package, having actual releases, etc) this would certainly become a larger concern. The changes look good at my cursory check, and I hope to review in more detail later this week.<br><div><br></div><div>That said I just wanted to point out that I don't think a change, even as "drastic" as this would have a huge effect on the ecosystem. As charm authors sync charm-helpers they'll end up in our testing environment and should be caught then.<br></div><div><br></div><div>Marco</div><br><div class="gmail_quote">On Thu Nov 20 2014 at 12:33:55 PM Charles Butler <<a href="mailto:charles.butler@canonical.com">charles.butler@canonical.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Jay,<div><br></div><div>I think you're touching on some important topics. Tuning and optimizing charm installation without fetching tons of extra packages around an install is certainly something I'm very open to experimentation with.</div><div><br></div><div>This particular merge makes me suggest we run this through the gauntlet of testing scenarios - as if we merge this as is it effects the default expected behavior of the apt_install, with > 200 charms in the store - and im' going to venture to guess ~ 40% of those are python and charm-helpers based charms. This means a very expensive test would follow to ensure we don't break whats in the wild. If you were to refactor this to make that a parameter on apt_install, and leave the default behavior as is - I would be more confident in making the merge happen.</div><div><br></div><div><br></div><div>All the best, </div></div><div dir="ltr"><div><br></div><div>Charles</div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Nov 19, 2014 at 6:27 PM, Jay Wren <span dir="ltr"><<a href="mailto:jay.wren@canonical.com" target="_blank">jay.wren@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"><div><font face="arial, helvetica, sans-serif">Greetings,</font></div><div><font face="arial, helvetica, sans-serif"><br></font></div><div><font face="arial, helvetica, sans-serif">I wanted to bring attention to a pending MR <a href="https://code.launchpad.net/~evarlast/charm-helpers/use-no-install-recommends/+merge/242270" target="_blank">https://code.launchpad.net/~evarlast/charm-helpers/use-no-install-recommends/+merge/242270</a> which I feel can help charm-helpers.</font></div><div><font face="arial, helvetica, sans-serif"><br></font></div><div><font face="arial, helvetica, sans-serif"><span style="color:rgb(51,51,51);line-height:18px">By not installing recommended packages, charms will install packages faster, with less unneeded packages.</span><br></font></div><div><span style="color:rgb(51,51,51);line-height:18px"><font face="arial, helvetica, sans-serif"><br></font></span></div><div><span style="color:rgb(51,51,51);line-height:18px"><font face="arial, helvetica, sans-serif">This leads to faster deploys and faster experience for new users and an overall better feeling about juju.</font></span></div><div><span style="color:rgb(51,51,51);line-height:18px"><font face="arial, helvetica, sans-serif"><br></font></span></div><div><font color="#333333" face="arial, helvetica, sans-serif"><span style="line-height:18px">On a similar subject, I am asking for feedback from charm-helpers users regarding apt related activity. I feel it might make sense use apt_pkg for as many operations as possible. Would a merge request of this nature be considered or is fork and exec to the more familiar system commands preferred? </span></font></div><div><font color="#333333" face="arial, helvetica, sans-serif"><span style="line-height:18px"><br></span></font></div><div><font color="#333333" face="arial, helvetica, sans-serif"><span style="line-height:18px">Thanks,</span></font></div><div><font color="#333333" face="arial, helvetica, sans-serif"><span style="line-height:18px">--</span></font></div><div><font color="#333333" face="arial, helvetica, sans-serif"><span style="line-height:18px">Jay</span></font></div></div>
<br>--<br>
Juju mailing list<br>
<a href="mailto:Juju@lists.ubuntu.com" target="_blank">Juju@lists.ubuntu.com</a><br>
Modify settings or unsubscribe at: <a href="https://lists.ubuntu.com/mailman/listinfo/juju" target="_blank">https://lists.ubuntu.com/mailman/listinfo/juju</a><br>
<br></blockquote></div><br></div>
--<br>
Juju mailing list<br>
<a href="mailto:Juju@lists.ubuntu.com" target="_blank">Juju@lists.ubuntu.com</a><br>
Modify settings or unsubscribe at: <a href="https://lists.ubuntu.com/mailman/listinfo/juju" target="_blank">https://lists.ubuntu.com/<u></u>mailman/listinfo/juju</a><br>
</blockquote></div>