| thumper | anyone around who can review? | 22:20 |
|---|---|---|
| thumper | https://github.com/juju/juju/pull/6851 | 22:20 |
| redir | looking | 22:22 |
| redir | LGTM | 22:23 |
| thumper | redir: ta | 22:32 |
| redir | good find | 22:32 |
| redir | np | 22:32 |
| thumper | well, looking through the uploaded info for the bug | 22:33 |
| thumper | seemed obvious that we were leaking, and just went through the codebase looking for all cases | 22:33 |
| * redir wonders if there's a static analysis tool for that | 22:39 | |
| thumper | wallyworld: ping | 23:33 |
| wallyworld | yo | 23:33 |
| thumper | wallyworld: can you please enable the pre-push hook? | 23:34 |
| thumper | wallyworld: you are committing code that doesn't pass go vet | 23:34 |
| thumper | apiserver/remoterelations/remoterelations.go:344: and 357 | 23:35 |
| wallyworld | ok. but our landing bot hsould pick that up | 23:35 |
| thumper | yes, but it doesn't | 23:35 |
| wallyworld | sigh | 23:35 |
| nurfet | guys, when juju ssh into a machine which user does it use by default? | 23:45 |
| axw | nurfet: the "ubuntu" user | 23:55 |
| nurfet | axw: thanks. how can I change the user during bootstraping? | 23:57 |
| axw | nurfet: are you using the manual provider? | 23:57 |
| nurfet | no, I am developing a new public cloud provider | 23:58 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!