| thumper | axw: thanks for that controller login branch, I'm sure it'll help some 2.2.4 controllers going to 2.3 | 00:19 |
|---|---|---|
| axw | thumper: np | 00:53 |
| axw | thumper: I think it's a little tidier now anyway, but I'm biased | 00:53 |
| * babbageclunk steps out to look at a house | 01:08 | |
| axw | jam: I think I'm going to have to back out the change to stop running a pinger for other controllers | 01:20 |
| axw | jam: it seems there's no guarantee that the controller workers will connect to their own apiserver. that seems wrong, but I don't know that I can fix it in time for 2.3 | 01:20 |
| axw | jam: post 2.3, each controller machine can just run a pinger for itself in its dependency engine | 01:21 |
| axw | thumper: can you please review https://github.com/juju/juju/pull/8122? | 01:40 |
| thumper | axw: sure | 01:43 |
| thumper | axw: we should talk about this change, because I don't think it is right | 01:48 |
| axw | thumper: it's reverting a change to how it was, but ok | 01:48 |
| thumper | oh... | 01:48 |
| thumper | right you are | 01:49 |
| thumper | this is fine | 01:49 |
| axw | thumper: you wanna HO? | 01:49 |
| thumper | although, just 5m to confirm | 01:49 |
| thumper | lets 1:1 HO | 01:49 |
| axw | sure | 01:49 |
| babbageclunk | thumper: are we going to have something that's going to collect and amalgamate audit logs from different controller machines? | 02:41 |
| thumper | babbageclunk: um... | 04:19 |
| babbageclunk | ? | 04:19 |
| thumper | we aren't looking at writing a tool at this stage | 04:19 |
| thumper | they are aware that an amalgamated view would be needed | 04:19 |
| babbageclunk | thumper: just occurred to me as I was ripping out the DB storage pieces. | 04:20 |
| * thumper nods | 04:20 | |
| babbageclunk | thumper: since this is a stream of either calls or facade methods, it should probably include a type indicator on each record. Maybe just have a top-level item struct with an omitempty field for each of the two records? | 04:55 |
| === frankban|afk is now known as frankban | ||
| === mup_ is now known as mup | ||
| === frankban is now known as frankban|afk | ||
| babbageclunk | thumper: I remember you saying something about introspecting the types of API results to find errors - what was that about? | 20:20 |
| thumper | babbageclunk: otp right now | 21:32 |
| thumper | with you after | 21:32 |
| babbageclunk | tsk you're always on the phone | 21:39 |
| thumper | that's what managers do | 21:40 |
| babbageclunk | :( | 21:41 |
| axw | veebers: can https://github.com/juju/juju/pull/7715 be closed? | 23:41 |
| axw | veebers: and is https://github.com/juju/juju/pull/7858 still something you intend to land? | 23:45 |
| * veebers looks | 23:49 | |
| veebers | axw: ah right, ci-run is work in progress I need to get back to it and land it, same with the 2nd, I can bring down the PR for that one if it's distracting as there is a little more work needed there (was put up as a PR for the discussion) | 23:50 |
| axw | veebers: no worries, just checking | 23:50 |
| axw | sometimes PRs just hang around forever | 23:50 |
| veebers | axw: axk, thanks for the heads up; I do need to move them forward | 23:51 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!