<div dir="ltr">Any reason why destroy-controller and kill-controller would not also remove the local reference (purge-controller)?</div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Apr 6, 2016 at 1:54 PM, Tim Penhey <span dir="ltr"><<a href="mailto:tim.penhey@canonical.com" target="_blank">tim.penhey@canonical.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 06/04/16 23:13, Nick Veitch wrote:<br>
> Sure, I am just concerned about a proliferation of commands to do the<br>
> same (ultimately) task<br>
><br>
> destroy-controller<br>
<br>
</span>The most correct way to take down a controller.<br>
<br>
> kill-controller<br>
<br>
The OMG it is broken, please do as much as you can and I know I'm going<br>
to have to manually check any resources left around that it couldn't<br>
clean up.<br>
<br>
> forget/purge-controller<br>
<br>
Remove local references to the controller.<br>
<br>
<br>
Not really the same things at all.<br>
<span class="HOEnZb"><font color="#888888"><br>
Tim<br>
</font></span><span class="im HOEnZb"><br>
<br>
><br>
><br>
><br>
> On 6 April 2016 at 11:59, Horacio Duran <<a href="mailto:horacio.duran@canonical.com">horacio.duran@canonical.com</a><br>
</span><span class="im HOEnZb">> <mailto:<a href="mailto:horacio.duran@canonical.com">horacio.duran@canonical.com</a>>> wrote:<br>
><br>
>     The issue I see with that approach is that in that case<br>
>     kill-controller might be doing less than you expect instead of more,<br>
>     suppose the controller is having transient issues and kill<br>
>     controller cannot reach the cloud for deletion, this would forget<br>
>     the controller and leave it in the cloud, forget-controller instead<br>
>     tells us very clearly what is going to happen, the change is going<br>
>     to be local and not affect the controller.<br>
>     My 2c<br>
><br>
><br>
>     On Wednesday, 6 April 2016, Nick Veitch <<a href="mailto:nick.veitch@canonical.com">nick.veitch@canonical.com</a><br>
</span><div class="HOEnZb"><div class="h5">>     <mailto:<a href="mailto:nick.veitch@canonical.com">nick.veitch@canonical.com</a>>> wrote:<br>
><br>
>         just my tuppence<br>
><br>
>         instead of having another command, can't we just add this as an<br>
>         option to kill-controller?<br>
><br>
>         juju kill-controller --cleanup <controller><br>
><br>
><br>
><br>
>         On 6 April 2016 at 11:05, Horacio Duran<br>
>         <<a href="mailto:horacio.duran@canonical.com">horacio.duran@canonical.com</a>> wrote:<br>
><br>
><br>
>             I might be biased by years of apt-get but purge makes me<br>
>             think that you are going to do what kill is supposed to do,<br>
>             forget sound more aligned whit what you are really aiming to.<br>
><br>
>             On Wednesday, 6 April 2016, Andrew Wilkins<br>
>             <<a href="mailto:andrew.wilkins@canonical.com">andrew.wilkins@canonical.com</a>> wrote:<br>
><br>
>                 On Tue, Apr 5, 2016 at 2:29 AM Cheryl Jennings<br>
>                 <<a href="mailto:cheryl.jennings@canonical.com">cheryl.jennings@canonical.com</a>> wrote:<br>
><br>
>                     Relevant bug:<br>
>                      <a href="https://bugs.launchpad.net/juju-core/+bug/1553059" rel="noreferrer" target="_blank">https://bugs.launchpad.net/juju-core/+bug/1553059</a><br>
><br>
>                     We should provide a way to clean up controllers<br>
>                     without making the user manually edit juju's files.<br>
><br>
><br>
>                 Unless anyone objects, or has a better spelling, I will<br>
>                 be adding a command to do this:<br>
><br>
>                     juju purge-controller <controller-name><br>
><br>
>                 The command will require a "-y" or prompt for<br>
>                 confirmation, like kill-controller. It will not attempt<br>
>                 to destroy the controller, it will just remove the<br>
>                 details of it from the client.<br>
><br>
>                 (Alternative suggestion for spelling: "juju<br>
>                 forget-controller". Purge-controller may suggest that<br>
>                 we're purging a controller of its contents, rather than<br>
>                 purging the controller from the client?)<br>
><br>
>                 Cheers,<br>
>                 Andrew<br>
><br>
>                     On Mon, Apr 4, 2016 at 7:05 AM, Nate Finch<br>
>                     <<a href="mailto:nate.finch@canonical.com">nate.finch@canonical.com</a>> wrote:<br>
><br>
>                         This just happened to me, too.  Kill-controller<br>
>                         needs to work if at all possible.  That's the<br>
>                         whole point.  And yes, users may not hit<br>
>                         specific problems, but devs do, and that wastes<br>
>                         our time trying to figure out how to manually<br>
>                         clean up the garbage.<br>
><br>
>                         On Mon, Apr 4, 2016 at 8:33 AM Rick Harding<br>
>                         <<a href="mailto:rick.harding@canonical.com">rick.harding@canonical.com</a>> wrote:<br>
><br>
>                             On Sun, Apr 3, 2016 at 6:56 PM Andrew<br>
>                             Wilkins <<a href="mailto:andrew.wilkins@canonical.com">andrew.wilkins@canonical.com</a>> wrote:<br>
><br>
>                                 In a non-beta release we would make sure<br>
>                                 that the config changes aren't backwards<br>
>                                 incompatible.<br>
><br>
><br>
>                             I think this is the key thing. I think that<br>
>                             kill-controller is an exception to this<br>
>                             rule. I think we should always at least give<br>
>                             the user the ability to remove their stuff<br>
>                             and start over with the new alpha/beta/rc<br>
>                             release. I'd like to ask us to explore<br>
>                             making kill-controller an exception to this<br>
>                             policy and that if tests prove we can't<br>
>                             bootstrap on one beta and kill with trunk<br>
>                             that it's a blocking bug for us.<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:<br>
>                             <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>
><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:<br>
>                         <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>
><br>
><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:<br>
>             <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>
><br>
><br>
><br>
>         --<br>
>         Nick Veitch,<br>
>         CDO Documentation<br>
>         Canonical<br>
><br>
><br>
><br>
><br>
> --<br>
> Nick Veitch,<br>
> CDO Documentation<br>
> Canonical<br>
><br>
><br>
<br>
<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>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr">Alexis Bruemmer<div>Juju Core Manager, Canonical Ltd.</div><div>(503) 686-5018</div><div><a href="mailto:alexis.bruemmer@canonical.com" target="_blank">alexis.bruemmer@canonical.com</a></div></div></div>
</div>