=== tvansteenburgh1 is now known as tvansteenburg
=== tvansteenburg is now known as tvansteenburgh
[01:09] <babbageclunk> axw: reviewed #7599
[01:09] <axw> babbageclunk: thanks
[01:10] <babbageclunk> axw: Ooh, I hadn't seen how big #7602 was!
[01:11] <axw> babbageclunk: sorry :( ignore the first commit. and I split it into multiple commits to help. I can split into multiple PRs if you like
[01:11] <babbageclunk> axw: Thanks - I'll go by commit, that'll definitely help!
[01:14] <babbageclunk> axw: yay, first two commits done already! :)
[01:14] <axw> babbageclunk: :p
[01:15] <babbageclunk> axw: (hmm, rereading, those exclamation points sound sarcastic but weren't meant to - the separate commits look much more manageable for reals.)
[01:16] <axw> babbageclunk: no worries, I didn't read any sarcasm
[01:16] <babbageclunk> good, just checking
[01:29]  * babbageclunk goes for a wee run
[02:44] <axw> babbageclunk: so it's https://bugs.launchpad.net/juju/+bug/1696509 that I'm picking up? what's the current status?
[02:44] <mup> Bug #1696509: status-history-pruner fails under load <adrastea> <performance> <pruning> <statuseshistory> <juju:In Progress by 2-xtian> <https://launchpad.net/bugs/1696509>
[02:47] <axw> babbageclunk: oh I see you have a PR up, so I just need to test it?
[02:49] <babbageclunk> axw: that's right - basically what I was planning to do was... actually, shall we do a hangout?
[02:49] <axw> babbageclunk: okey dokey, see you in standuo
[02:50] <axw> p
[02:50] <axw> babbageclunk: a team
[03:04] <blahdeblah> axw: Are the commits you recently made on https://bugs.launchpad.net/juju/+bug/1700451 a full fix or WIP?
[03:04] <mup> Bug #1700451: Upgrade from 2.1.x to 2.2.1 blocked by missing Azure resources <canonical-is> <upgrade-juju> <juju:In Progress by axwalk> <https://launchpad.net/bugs/1700451>
[03:05] <axw> blahdeblah: they're the full fix for 2.2.2, only part of it has landed though. the main commits are still being reviewed
[03:05] <blahdeblah> cool - thanks
[03:06] <blahdeblah> axw: But the plan is that they'll be included in 2.2.2?
[03:06] <axw> blahdeblah: yes, definitely
[03:06] <blahdeblah> awesome
[04:15] <babbageclunk> axw: reviewed!
[04:30] <axw> babbageclunk: tyvm
[04:55] <axw> babbageclunk: I've got 10 agents pounding the controller with status updates, and it's not really making a dent in the status history. I think it'll be much quicker, and just as illustrative, to write a standalone benchmark program. sound reasonable to you?
[04:56] <babbageclunk> axw: Do you mean something that just inserts lots of records into the DB directly?
[04:56] <axw> babbageclunk: yep, then runs PruneStatusHistory
[04:58] <babbageclunk> axw: I've done a bit of that already by increasing the number of rows the tests insert
[04:58] <babbageclunk> axw: I think one of the main things jam was interested in was that the pruner is running in a realistic-ish context.
[04:59] <babbageclunk> axw: What about something that just primes the status history with a lot of records, and then lets pruning run in the controller (with inserts happening around it)?
[05:01] <axw> babbageclunk: can do.
[05:01] <babbageclunk> axw: thanks man
[05:02] <thumper> axw: do you remember how the juju log forwarding works?
[05:02] <axw> thumper: from agents to controller, or from controller to syslog?
[05:03] <thumper> the latter
[05:03] <axw> thumper: only roughly, why?
[05:03] <thumper> bug 1657724
[05:03] <mup> Bug #1657724: Juju2 should have the option to feed logs into rsyslog <canonical-bootstack> <juju:New> <https://launchpad.net/bugs/1657724>
[05:04] <thumper> I'm trying to see if it is documented in our docs anywhere
[05:04] <axw> thumper: I'm hesitant to encourage its use, but I'll comment on the bug with the config needed
[05:04] <thumper> yeah, I get that
[05:05] <thumper> axw: perhaps say it is a relatively new feature with not a lot of robust testing
[05:05] <thumper> and if they would like to help test, that'd be awesome :)
[05:06] <axw> thumper: sure
[05:55] <babbageclunk> menn0: still around? last minute quick review? https://github.com/juju/juju/pull/7604
[06:00] <babbageclunk> axw: hmm, he might not be - could you take a look if you get a moment? ^
[06:00] <axw> babbageclunk: yup
[06:01] <babbageclunk> axw: ta
[06:04] <axw> babbageclunk: LGTM. I have a branch which does a few more (namely DestroyModel), but it's in a messy state. will probably wait till after 2.2.2 now
[07:47] <jamespage> hmm - anyone know when we might expect artful tools to appear in the streams for juju agent binaries?
=== rogpeppe2 is now known as rogpeppe
=== frankban|afk is now known as frankban
[10:18] <rogpeppe> jam: you might want to take a look at https://github.com/go-goose/goose/pull/53 which makes it more efficient for us to layer zip file (or other seek-requiring functionality) on top of swift: https://github.com/go-goose/goose/pull/53
[10:18] <jam> rogpeppe: weekend for me today
[10:18] <rogpeppe> jam: ah, ok, np
[11:16] <wpk> $ juju add-subnet  --help
[11:16] <wpk> Usage: juju add-subnet [options] <CIDR>|<provider-id> <space> [<zone1> <zone2> ...]
[11:16] <wpk> (...)
[11:16] <wpk> Details:
[11:16] <wpk> (...) Unlike "juju add-subnet", this command does not create a new subnet, so it
[11:16] <wpk> is supported on a wider variety of clouds (where SDN features are not
[11:16] <wpk> available, e.g. MAAS).
[11:16] <wpk> that doesn't sound right
=== frankban is now known as frankban|afk