=== tvansteenburgh1 is now known as tvansteenburg | ||
=== tvansteenburg is now known as tvansteenburgh | ||
babbageclunk | axw: reviewed #7599 | 01:09 |
---|---|---|
axw | babbageclunk: thanks | 01:09 |
babbageclunk | axw: Ooh, I hadn't seen how big #7602 was! | 01:10 |
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:11 |
babbageclunk | axw: yay, first two commits done already! :) | 01:14 |
axw | babbageclunk: :p | 01:14 |
babbageclunk | axw: (hmm, rereading, those exclamation points sound sarcastic but weren't meant to - the separate commits look much more manageable for reals.) | 01:15 |
axw | babbageclunk: no worries, I didn't read any sarcasm | 01:16 |
babbageclunk | good, just checking | 01:16 |
* babbageclunk goes for a wee run | 01:29 | |
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:44 |
axw | babbageclunk: oh I see you have a PR up, so I just need to test it? | 02:47 |
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:49 |
axw | p | 02:50 |
axw | babbageclunk: a team | 02:50 |
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:04 |
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:05 |
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 | 03:06 |
babbageclunk | axw: reviewed! | 04:15 |
axw | babbageclunk: tyvm | 04:30 |
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:55 |
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:56 |
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:58 |
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)? | 04:59 |
axw | babbageclunk: can do. | 05:01 |
babbageclunk | axw: thanks man | 05:01 |
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:02 |
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:03 |
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:04 |
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:05 |
axw | thumper: sure | 05:06 |
babbageclunk | menn0: still around? last minute quick review? https://github.com/juju/juju/pull/7604 | 05:55 |
babbageclunk | axw: hmm, he might not be - could you take a look if you get a moment? ^ | 06:00 |
axw | babbageclunk: yup | 06:00 |
babbageclunk | axw: ta | 06:01 |
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 | 06:04 |
jamespage | hmm - anyone know when we might expect artful tools to appear in the streams for juju agent binaries? | 07:47 |
=== rogpeppe2 is now known as rogpeppe | ||
=== frankban|afk is now known as frankban | ||
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 | 10:18 |
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 | 11:16 |
=== frankban is now known as frankban|afk |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!