<div dir="ltr">I think I did remove some of that code, because it wasn't being used and wasn't clear why it was there. Easy enough to put back in.<div><div> </div><div>Cheers,</div><div>Andrew <div class="gmail_quote"><div dir="ltr">On Mon, Mar 7, 2016 at 8:40 AM Tim Penhey <<a href="mailto:tim.penhey@canonical.com">tim.penhey@canonical.com</a>> wrote: </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Ah yeah... it is mostly coming back to me now. I actually have a branch lying around that does most of this. Perhaps I should look at it again :-) Tim On 04/03/16 20:08, Ian Booth wrote: > Hey Tim > > The new bootstrap UX has not removed any --admin-user flag. > I can see that the server jujud bootstrap command has an --admin-user argument > but it appears this is never set anywhere in the cloud init scripts. Or not that > I can see. I've checked older version of the relevant files and can't see where > we've ever used this. > > So maybe we have a capability to bootstrap the controller agent with a specified > admin-user but have not hooked it up yet? > > On 04/03/16 08:11, Tim Penhey wrote: >> Ah... it used to be there :-) At least it is on my feature branch, but I >> don't think I have merged the most recent master updates that has the >> work to re-work bootstrap for the new cloud credentials stuff. >> >> Tim >> >> On 04/03/16 10:09, Rick Harding wrote: >>> If we do that we need to also make it configurable on bootstrap as an >>> option. >>> >>> +1 overall >>> >>> >>> On Thu, Mar 3, 2016, 4:07 PM Tim Penhey <<a href="mailto:tim.penhey@canonical.com" target="_blank">tim.penhey@canonical.com</a> >>> <mailto:<a href="mailto:tim.penhey@canonical.com" target="_blank">tim.penhey@canonical.com</a>>> wrote: >>> >>> Hi folks, >>> >>> I was thinking that with the upcoming big changes with 2.0, we should >>> tackle a long held issue where we have the initial user called "admin". >>> >>> There was a request some time back that we should use the current user's >>> name. The reason it wasn't implemented at that time was due to logging >>> into the GUI issues. These have been resolved some time back with the >>> multiple user support that was added. >>> >>> All the server side code handles the ability to define the initial user >>> for the controller model, and we do this in all the tests, so the >>> default test user is actually called "test-admin". >>> >>> I *think* that all we need to do is change the default value we use in >>> the bootstrap command for the AdminUserName (--admin-user flag) from >>> "admin" to something we derive from the current user. >>> >>> Probably worth doing now. >>> >>> Thoughts? >>> >>> Tim >>> >>> -- >>> Juju-dev mailing list >>> <a href="mailto:Juju-dev@lists.ubuntu.com" target="_blank">Juju-dev@lists.ubuntu.com</a> <mailto:<a href="mailto:Juju-dev@lists.ubuntu.com" target="_blank">Juju-dev@lists.ubuntu.com</a>> >>> 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> >>> >> >> -- Juju-dev mailing list <a href="mailto:Juju-dev@lists.ubuntu.com" target="_blank">Juju-dev@lists.ubuntu.com</a> 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> </blockquote></div></div></div></div>