| wallyworld | menn0: you have a sec? https://hangouts.google.com/hangouts/_/canonical.com/ian-horatio | 00:00 |
|---|---|---|
| menn0 | axw or redir or wallyworld: http://reviews.vapour.ws/r/5408/ pls | 00:30 |
| wallyworld | menn0: otp, will look soon | 00:30 |
| redir | menn0: looking | 00:34 |
| menn0 | redir: thanks | 00:39 |
| axw | redir: are you a graduated reviewer yet? | 00:54 |
| redir | axw: nope | 00:54 |
| redir | IANAGR | 00:54 |
| anastasiamac | axw: do we have the idea of graduating reviewers, now that we have a checklist? | 00:54 |
| axw | anastasiamac: dunno | 00:54 |
| anastasiamac | axw: for graduating idea to work, each un-graduate reviewer should have a graduated mentor. I don;t think that was followed thru... | 00:55 |
| perrito666 | anastasiamac: we used to have achecklist in a spreadsheet, I presume is no longer there | 00:59 |
| anastasiamac | perrito666: m talking about code review checklist on wiki | 01:00 |
| anastasiamac | perrito666: gimme a sec to pull up the link.. | 01:00 |
| perrito666 | ahh I mixed what you where saying | 01:00 |
| anastasiamac | perrito666: https://github.com/juju/juju/wiki/Code-Review-Checklists | 01:00 |
| perrito666 | anastasiamac: yes, that one is always open | 01:01 |
| axw | anastasiamac: do you have time for a review? http://reviews.vapour.ws/r/5407/ | 01:17 |
| axw | fairly straight forward | 01:17 |
| anastasiamac | axw: for u- always :D | 01:17 |
| axw | anastasiamac redir: thanks both for the reviews | 01:30 |
| anastasiamac | axw: it's an awesome addition! thanks for doing it :) | 01:30 |
| redir | axw: np | 01:51 |
| redir | menn0: I haven't been able to make it through the QA steps as expected. I have to go now | 02:11 |
| redir | but I'll look when I get home and fill in details | 02:11 |
| redir | no QUIES... i n logs | 02:11 |
| redir | bbl | 02:11 |
| anastasiamac | axw: wallyworld: scream-related http://reviews.vapour.ws/r/5411/ :) | 02:13 |
| wallyworld | joy | 02:13 |
| anastasiamac | wallyworld: this one is the easiest by far :D | 02:14 |
| axw | anastasiamac: did you run the QA steps on one of those at least? | 02:15 |
| axw | "desired provider" is a bit vague :) | 02:15 |
| anastasiamac | yes. i'll change to "from the list above" or could just list them in [,,,] | 02:16 |
| axw | anastasiamac: as long as a live test has been done, I'm happy | 02:16 |
| anastasiamac | axw: \o/ i'll double-;triple-check tonight :) don't need it in beta15 anyway \o/ but it will make the diff with anything else I propose so much more manageable :) | 02:19 |
| wallyworld | so long as supportedArchitecures() only hits mongo | 02:19 |
| wallyworld | before we stored metadata in state, looking up image metadata was expensive | 02:19 |
| wallyworld | hence the caching | 02:19 |
| anastasiamac | wallyworld: ? unrelated... this change only removes cached supported arch collection from provider environ | 02:20 |
| wallyworld | exactly | 02:20 |
| wallyworld | and hence the caching is now gone | 02:20 |
| wallyworld | so if suportedArchitecures() only hits mongo, that's fine | 02:20 |
| anastasiamac | wallyworld: the problem was that we'd never pick up new arch even if images are updated | 02:20 |
| anastasiamac | wallyworld: caching was undesired from the start i believe.. it was an oversigh | 02:20 |
| wallyworld | nope | 02:20 |
| wallyworld | was deliberate | 02:20 |
| wallyworld | the expectation was that the providers would have all images defined | 02:21 |
| anastasiamac | wallyworld: if it was deliberate, it was shooting juju in the leg/foot.. | 02:21 |
| wallyworld | and generally they do | 02:21 |
| wallyworld | aws, google, all public clouds have images defined | 02:21 |
| wallyworld | and the expectation with openstack clouds also is that they are set up with all images in glance | 02:22 |
| axw | anastasiamac wallyworld: once we stop query simplestreams, this is all moot. the caching will then be done by the image metadata update worker only | 02:23 |
| wallyworld | exactly, that's what i was asking | 02:23 |
| axw | stop querying simplestreams from the providers* | 02:23 |
| wallyworld | s/asking/confirming | 02:23 |
| wallyworld | i just wanted to make sure that the query to mongo was confirmed before we removed the caching | 02:23 |
| axw | wallyworld anastasiamac: we're still doing it. I assumed that the "precursor to..." statement in the description referred to that being done as a follow-u | 02:23 |
| axw | p | 02:23 |
| anastasiamac | axw: exactly, so this work is a precursor to stop provider looking ins simple streams.. m expecting the whole lot will end up in one beta without biting anyone... but wanted to break it into separate PRs for ease of review :) | 02:24 |
| wallyworld | if we can do it all in one beta cycle that would be desireable | 02:24 |
| wallyworld | yay, ty | 02:24 |
| anastasiamac | :D | 02:24 |
| wallyworld | just wanted to be sure that not having the cache escapes | 02:25 |
| wallyworld | otherwise performance will regress badly | 02:25 |
| wallyworld | s/will/probably | 02:25 |
| axw | anastasiamac: maybe you could line up the next one and get a LGTM and land at the same time then? | 02:26 |
| axw | anastasiamac: no value in landing this one by itself I think | 02:26 |
| menn0 | redir: really? did you use the debug-log command in the QA steps? | 02:28 |
| mup | Bug #1612026 opened: depoloyer/jujuclient broken: show-model flag provided but not defined: -m <ci> <deployer> <regression> <juju-core:Incomplete> <juju-deployer:Confirmed> <https://launchpad.net/bugs/1612026> | 02:33 |
| anastasiamac | axw: agreed | 02:35 |
| mup | Bug #1612026 changed: depoloyer/jujuclient broken: show-model flag provided but not defined: -m <ci> <deployer> <regression> <juju-core:Incomplete> <juju-deployer:Confirmed> <https://launchpad.net/bugs/1612026> | 02:39 |
| mup | Bug #1612026 opened: depoloyer/jujuclient broken: show-model flag provided but not defined: -m <ci> <deployer> <regression> <juju-ci-tools:Triaged> <juju-core:Incomplete> <juju-deployer:Confirmed> <https://launchpad.net/bugs/1612026> | 02:46 |
| menn0 | redir: I'm going to merge as I'm pretty sure it's right and I have to positive reviews | 03:54 |
| menn0 | redir: let me know if you really can't repro the QA steps | 03:54 |
| redir | menn0: OK, I am back and running through QA one more time cut and paste style | 03:54 |
| menn0 | redir: ok great. | 03:55 |
| redir | just waiting for apt-get up* | 03:55 |
| redir | seems slow right now | 03:55 |
| menn0 | redir: make sure you're waiting long enough for the migration to finish (check the top of "juju status -m A:foo --format yaml") | 03:55 |
| menn0 | redir: apt-get update always seems a bit slow | 03:56 |
| redir | menn0: yeah was doing watch juju status | 03:56 |
| redir | but re running to make sure I am on correct branch | 03:56 |
| menn0 | redir: you should set up squid-deb-proxy if you haven't already | 03:56 |
| redir | that I did go install ... and not juju install ....(which is in my history so I've definitely done that at some point) | 03:57 |
| redir | I should set that up, it sounds like it would be awesome. | 03:57 |
| * redir googles | 03:57 | |
| menn0 | redir: also for this test using --config enable-os-refresh-upgrade=false and --config enable-os-upgrade=false is totally fine | 03:57 |
| menn0 | redir: I usually use this config when testing stuff locally | 03:58 |
| menn0 | default-series: xenial | 03:58 |
| menn0 | logging-config: "<root>=DEBUG" | 03:58 |
| menn0 | apt-http-proxy: http://10.0.8.1:8888 | 03:58 |
| menn0 | enable-os-refresh-update: true | 03:58 |
| menn0 | enable-os-upgrade: false | 03:58 |
| redir | I thought i had that set as defaults in my yaml config | 03:58 |
| menn0 | where the service on 8888 is squid-deb-proxy | 03:58 |
| redir | is that on the wiki? | 03:58 |
| redir | :) | 03:58 |
| menn0 | probably not | 03:58 |
| * menn0 adds that to his todo list | 03:59 | |
| redir | meh, I'll bug you for details tomorrow afternoon and put it up while I do it. | 04:00 |
| redir | menn0: ^^ | 04:00 |
| redir | my afternoon | 04:00 |
| menn0 | redir: ok | 04:00 |
| mup | Bug #1587993 changed: lxc creation fails, invalid include <canonical-bootstack> <intermittent> <juju-core:Invalid> <juju-core 1.25:Incomplete> <https://launchpad.net/bugs/1587993> | 04:01 |
| menn0 | redir: how's the bootstrap going? | 04:03 |
| redir | menn0: on the second one | 04:03 |
| menn0 | redir: another thing you can do is bootstrap multiple controllers in parallel | 04:04 |
| menn0 | redir: that makes a big difference | 04:04 |
| redir | with above config... so hopefully much faster | 04:04 |
| redir | I'm in cut an paste mode:) | 04:04 |
| menn0 | redir: I didn't put it in the instructions because it complicates things | 04:04 |
| redir | no worries | 04:05 |
| redir | I just figure I fat fingered something earlier | 04:06 |
| redir | ok | 04:10 |
| redir | I see no quies | 04:11 |
| redir | menn0: ^ | 04:11 |
| menn0 | redir: hmmm ok | 04:11 |
| redir | I can try migrating it back to A | 04:12 |
| redir | :) | 04:12 |
| natefinch | axw: you around? | 04:12 |
| menn0 | redir: did the migration complete? i.e. juju status -m A:foo now fails, and juju status -m B:foo now works? | 04:12 |
| redir | yeah it disappeared from status | 04:12 |
| mup | Bug #1469077 changed: Leadership claims, document larger than capped size <landscape> <leadership> <juju-core:Invalid> <juju-core 1.24:Won't Fix by fwereade> <https://launchpad.net/bugs/1469077> | 04:13 |
| mup | Bug #1612043 opened: improvement to juju register text <juju-core:New> <https://launchpad.net/bugs/1612043> | 04:13 |
| veebers | axw, wallyworld: I have a query re: model config. If I have a clouds.yaml file as shown in this paste and run the bootstrap shown in the paste, I should expect to see ftp-proxy set to the controller value when using 'juju model-config' right? (Or have I misconfigured something): http://pastebin.ubuntu.com/22991841/ | 04:13 |
| menn0 | redir: what does "juju debug-log -m A:controller --replay -T | grep migrationmaster" show? | 04:14 |
| wallyworld | veebers: no, you are overriding it | 04:14 |
| wallyworld | --config ftp-proxy=ftp://fake.com | 04:15 |
| redir | menn0: nada | 04:15 |
| axw | natefinch: I am around, what's up? | 04:15 |
| menn0 | redir: nothing at all? | 04:15 |
| redir | correct | 04:15 |
| wallyworld | veebers: if you juju unset ftp-proxy, it wil revert to the controller value | 04:16 |
| natefinch | axw: saw your comments on that windows bug after I pinged you. Was going to ask about the ResizeDiskError | 04:16 |
| menn0 | redir: ok, what about: juju list-controllers? | 04:16 |
| redir | menn0: http://paste.ubuntu.com/22992085/ | 04:16 |
| axw | natefinch: I have a patch ready. just need to change the azure provider to set a minimum for the root-disk constraint to 127G | 04:16 |
| veebers | wallyworld: ah right that's the one. I had in my mind that model overwrites default and controller overwrites model | 04:16 |
| axw | natefinch: it seemed that there were other errors after that though | 04:17 |
| wallyworld | ah, i see | 04:17 |
| redir | menn0: wanna live look for a minute? | 04:17 |
| menn0 | redir: that's probably easier | 04:17 |
| natefinch | axw: I didn't look much further than that... I poked around, but I don't know the azure interface very well | 04:18 |
| natefinch | axw: were there other logs that indicated an error? | 04:18 |
| veebers | wallyworld: So it's actually, model overwrites controller and controller overwrites default? | 04:18 |
| wallyworld | yes, and soon model > region > controller > default | 04:18 |
| axw | natefinch: in the portal, if you go to the VM and click on VM Extensions, you can see the output of the script we run on machine startup that effectively does the same thing as cloud-init | 04:18 |
| redir | menn0: pm'ed link | 04:19 |
| axw | natefinch: there were some errors in there to do with contacting the controller for tools | 04:19 |
| natefinch | axw: ok... yeah, that's the bug I was trying to look into :) | 04:19 |
| axw | natefinch: so if you want to repro and get past the root-disk resizing thing, pass "--constraints root-disk=127G" | 04:20 |
| natefinch | axw: I'm surprised it got that far... seemed like the VM failed to even start, but maybe I was misinterpreting the UI | 04:20 |
| natefinch | axw: nice | 04:20 |
| axw | natefinch: yes it did fail to start, because of the root disk thing | 04:20 |
| axw | natefinch: I mean after you get past that issue, you'll see the other one :) | 04:20 |
| natefinch | oh I see. Ok :) | 04:20 |
| veebers | wallyworld: cool thanks. Oh and you said the other day that if model and controller values are the same it should say controller in the output? | 04:24 |
| wallyworld | yes | 04:24 |
| wallyworld | and that is calculated dynamically | 04:24 |
| mup | Bug #1612043 changed: improvement to juju register text <juju-core:New> <https://launchpad.net/bugs/1612043> | 04:25 |
| mup | Bug #1469077 opened: Leadership claims, document larger than capped size <landscape> <leadership> <juju-core:Invalid> <juju-core 1.24:Won't Fix by fwereade> <https://launchpad.net/bugs/1469077> | 04:25 |
| veebers | ah cool, I'll ensure that's tested too. | 04:26 |
| mup | Bug #1469077 changed: Leadership claims, document larger than capped size <landscape> <leadership> <juju-core:Invalid> <juju-core 1.24:Won't Fix by fwereade> <https://launchpad.net/bugs/1469077> | 04:31 |
| mup | Bug #1612043 opened: improvement to juju register text <juju-core:New> <https://launchpad.net/bugs/1612043> | 04:31 |
| mup | Bug #1612046 opened: inconsistent error text when attempting unauthorised command <juju-core:New> <https://launchpad.net/bugs/1612046> | 04:43 |
| axw | wallyworld: I don't think we should be testing the provisioningState != Succeeded in the provider. That's handled by the Azure SDK. | 04:47 |
| wallyworld | axw: yeah, that is true and i agree. i just wanted to be 100% sure by asking | 04:48 |
| axw | wallyworld: okey dokey. just bringing it up here in case you wanted to debate :) | 04:49 |
| wallyworld | that we didn't need to have a unit test for a failure case where provider didn't succeed | 04:49 |
| wallyworld | you can drop, feel free | 04:49 |
| veebers | wallyworld, axw: When you have a moment could you take a quick shufti at my revised testplan for the config tree and confirm if it's sensible please? https://pastebin.canonical.com/162911/ | 04:50 |
| axw | anastasiamac natefinch menn0: http://reviews.vapour.ws/r/5412/ is just over 500 lines, do any of you have a little time to spare for a second review? | 04:50 |
| axw | a what now :) | 04:51 |
| wallyworld | veebers: sure, give me 5 | 04:51 |
| axw | veebers: sounds fine to me | 04:52 |
| veebers | cool thanks. | 04:53 |
| veebers | oh axw, can I set a global setting in clouds.yaml, or does it need to be under a clouds -> cloudname -> config stanza? | 04:54 |
| axw | veebers: the latter | 04:54 |
| veebers | axw cool thanks | 04:54 |
| natefinch | axw: I'm a bit knackered, otherwise I'd take a shufti at your review | 04:59 |
| axw | natefinch: heh ;p | 04:59 |
| axw | natefinch: no worries | 04:59 |
| natefinch | axw: btw, I'm thinking of putting the json tags back on the schema object, if for informational purposes only | 05:02 |
| natefinch | axw: I would imagine that someone wanting to write a schema would go look at this package... even though jsonschema is standard, it's nice to give the standard names here where they'd already be looking | 05:03 |
| axw | natefinch: if they're not being used, seems more like misinformation to me. if I come across a type with json tags, I expect that they're used for json un/marshaling | 05:03 |
| * axw shrugs | 05:03 | |
| axw | not terribly fussed | 05:04 |
| natefinch | axw: well... they do indicate the correct json tags for marshaling/unmarshaling. The fact that we ignore them is an implementation detail ;) | 05:04 |
| axw | natefinch: I guess it doesn't matter since they can't reasonably be changed anyway | 05:05 |
| natefinch | yep | 05:05 |
| mup | Bug #1612048 opened: juju needs a "list-shares" command for a controller <juju-core:New> <https://launchpad.net/bugs/1612048> | 05:13 |
| === mpontillo_ is now known as mpontillo | ||
| anastasiamac | axw: sorry was afk - school pickup. did u get a 2nd review on 5412? | 05:53 |
| axw | anastasiamac: no, if you have time I would greatly appreciate it | 05:54 |
| anastasiamac | axw: looking now \o/ | 05:54 |
| natefinch | wallyworld: what does logout actually do? | 06:00 |
| wallyworld | natefinch: logs out of the specified controller, meaning yuo need to login again to do stuff | 06:00 |
| natefinch | wallyworld: so like uncaches whatever credentials? | 06:01 |
| wallyworld | it is reflected in list-controllers | 06:01 |
| wallyworld | yeah | 06:01 |
| natefinch | ok | 06:01 |
| === sinzui_ is now known as sinzui | ||
| natefinch | axw: I ended up tweaking the schema struct a bit more to make it more friendly to struct literals, but it's working for marshal/unmarshal now: https://godoc.org/github.com/natefinch/cloudschema | 06:07 |
| natefinch | axw: this let me drop all visible references to go-jsschema, so consumers don't even need to know about it (presuming we add a validation function to this package) | 06:08 |
| axw | natefinch: yeah, I was thinking about that. sounds good | 06:08 |
| natefinch | cool. Bedtime for me. | 06:08 |
| axw | natefinch: good night | 06:09 |
| anastasiamac | axw: LGTM and saw that u have already confirmed that u've live-tested :) | 06:12 |
| axw | anastasiamac: thank you | 06:29 |
| anastasiamac | axw: \o/ | 06:29 |
| axw | anastasiamac: next time in the code I'll make that number a var. it's a bit magic either way - I'll see if I can think of a better way to do it | 06:30 |
| anastasiamac | axw: nps - just saw that it was shuffled in couple of places from previous codebase :) was just a note \o/ | 06:31 |
| veebers | axw: Another clouds.yaml file, if I have a bootstrap arg like in this paste, I should need option B (in the paste) in clouds.yaml right? http://pastebin.ubuntu.com/23001462/ | 06:59 |
| axw | veebers: correct | 07:00 |
| veebers | axw: awesome, thanks :-) | 07:00 |
| axw | veebers: assuming you meant shouldn't :) | 07:01 |
| veebers | axw: oh hah, um. Better asked, which of A or B should work in this example? | 07:02 |
| axw | veebers: you don't need either of them | 07:03 |
| axw | veebers: (unless you want to use config of course; I think you should just use B) | 07:03 |
| axw | veebers: --of course. I don't think we're yet merging config with existing cloud definitions. so if you want to define cloud config, you need an entry in clouds.yaml | 07:03 |
| axw | veebers: gtg pick up my daughter, be back in a little while | 07:04 |
| veebers | axw: To clarify I'm wanting to set a controller config setting (which I do via clouds.yaml as my understanding), using --config for bootstrap will set a model config option right? | 07:05 |
| === frankban|afk is now known as frankban | ||
| axw | veebers: correct. so use B from the example, and add a "cloud:" section as a sibling of "regions:" | 07:24 |
| veebers | axw: should 'cloud:' contain anything/ | 07:28 |
| axw | veebers: gah, I meant "config:" not "cloud:" | 07:29 |
| axw | veebers: i.e. with the controller-level attributes in it | 07:29 |
| veebers | axw: ah right, that makes sense :-) Thanks | 07:30 |
| mup | Bug #1612099 opened: mark currently logged in user in "list-users" <juju-core:Triaged> <https://launchpad.net/bugs/1612099> | 07:40 |
| mup | Bug #1612105 opened: juju help usage output <usability> <juju-core:Triaged> <https://launchpad.net/bugs/1612105> | 07:49 |
| mup | Bug #1612110 opened: update README.md to include running godeps <adoption> <juju-core:Triaged> <https://launchpad.net/bugs/1612110> | 08:13 |
| mup | Bug #1612112 opened: upgrade-charm should warn when the same version is detected <juju-core:Triaged> <https://launchpad.net/bugs/1612112> | 08:13 |
| babbageclunk | fwereade, dimitern: I've just realised I also need to include subnet info in the NetworkConfig I return from the API (if I want it to be complete), which means my mocks and shims need to cover State -> Machine -> LinkLayerDevice -> Address -> Subnet. | 08:52 |
| babbageclunk | fwereade, dimitern: I feel like this is increasingly impractical - is there a better way of doing it? | 08:53 |
| dimitern | babbageclunk: I don't think that's necessary | 08:53 |
| babbageclunk | dimitern: Good! :) What should I do instead? | 08:54 |
| dimitern | babbageclunk: having the IP and MAC addresses, along with the provider IDs of the interface and the address will be sufficient I think | 08:54 |
| dimitern | babbageclunk: in fact, I'm OK with changing the definition of ReleaseContainerAddresses not to take []params.NetworkConfig, but a slice of a new, simpler type | 08:55 |
| babbageclunk | dimitern: Yeah, I was reaching towards something like that too. | 08:55 |
| dimitern | babbageclunk: e.g. params.InterfaceAssignments { InterfaceName, InterfaceProviderID, MACAddress, IPAddress, IPAddressProviderID } | 08:57 |
| dimitern | (not necessarily with these names | 08:57 |
| dimitern | ) | 08:57 |
| babbageclunk | dimitern: Then I think why include the ip address details? We don't need them at the moment. | 08:58 |
| * dimitern *thinks* | 08:59 | |
| fwereade | babbageclunk, dimitern: I would say that the db-backed-object style of state is the root problem here | 08:59 |
| dimitern | babbageclunk: we do, if we're going to do IPAM for MAAS as well | 08:59 |
| dimitern | babbageclunk: and it shouldn't be a huge deal to add it | 08:59 |
| babbageclunk | fwereade: Yeah, I think so too, but I'm worried about trying to change that as part of that work. | 09:00 |
| babbageclunk | fwereade: s/that/this | 09:00 |
| fwereade | babbageclunk, dimitern: I don't think we need to *change* it so much as just route around it | 09:00 |
| dimitern | fwereade: I tried to make the model inside network/ and extract the bits to store in the db and pass over the api from there | 09:00 |
| * dimitern needs to step out to the ironmonger quickly.. | 09:01 | |
| babbageclunk | fwereade: So I could make other methods in state that return LinkLayerDeviceArgs instead of LinkLayerDevices? | 09:02 |
| fwereade | babbageclunk, dimitern: `func (*State) ContainerAddressStuff() *ContainerAddressStuff` with the methods you need, calling down to implementations shared with machine et al, means you need to write and test that... but it also means you can make the apiserver tests much simpler | 09:02 |
| fwereade | babbageclunk, I'm -1 on exposing params stuff from state vs core types | 09:03 |
| fwereade | babbageclunk, I think it's the apiserver's job to convert dependency-free core types into wire representations | 09:03 |
| fwereade | babbageclunk, similarly to how it's state's job to (internally!) convert core types back and forth into db representations | 09:04 |
| fwereade | babbageclunk, making any sense? | 09:04 |
| babbageclunk | fwereade: So what about moving the {LinkLayerDevice,Address}Args types from state into core and making methods to return them? | 09:05 |
| babbageclunk | fwereade: And SubnetInfo | 09:06 |
| babbageclunk | fwereade: Sorry, got the name of the address one wrong - it's LinkLayerDeviceAddress. | 09:09 |
| babbageclunk | fwereade: Then I think that matches what you're saying - they're just plain old data that the tests can construct, no nasty *States or private ctors. | 09:10 |
| babbageclunk | fwereade: I think this is what katco was suggesting in a review a while ago but I didn't really understand because I hadn't experienced the full horror of all of these nested shims and mocks. | 09:15 |
| * babbageclunk thinks maybe fwereade's popped out to the apothecary? | 09:16 | |
| * babbageclunk should really visit the barber-surgeon sometime. | 09:16 | |
| babbageclunk | fwereade: Oh, hang on - I'd forgotten that those types already exist in core/description. Can I use those? | 09:24 |
| babbageclunk | fwereade: Annoyingly they also have private constructors, so what I'd end up returning from the methods in state would be core.description.{LinkLayerDevice,IPAddress,Subnet}Args. | 09:27 |
| fwereade | babbageclunk, sorry -- but, yes, I think what you're saying makes sense | 09:30 |
| fwereade | babbageclunk, I'm not 100% sure whether they're description types, or whether description should be pulling from core/network or something, but it seems like a good start | 09:31 |
| fwereade | babbageclunk, the more things we can represent and communicate as POD the better, I think | 09:31 |
| babbageclunk | fwereade: Ok - I'm going to start adding the methods to state to return core.description.*Args first - if we want to move those types elsewhere we can do that. Should I go as far as State.Machine(id) -> State.MachineArgs(id) | 09:34 |
| babbageclunk | ? | 09:35 |
| babbageclunk | fwereade: There must already be something that converts from eg a state.LinkLayerDevice to a core.description.LinkLayerDevice, right? | 09:35 |
| fwereade | babbageclunk, apart from suspicion re "Args" being a suboptimal name vs "Spec" or "Info" or something, I think that's the right structure | 09:36 |
| fwereade | babbageclunk, I'm sure there is, but I suspect it's hidden away in migration_export | 09:36 |
| babbageclunk | fwereade: Yeah, the name was putting me off too. | 09:37 |
| babbageclunk | fwereade: Gah! BDSM languages with their access rules and their enforced interfaces! :) | 09:37 |
| fwereade | babbageclunk, oddly enough I was just fulminating privately about the CharmInfo type we use to add charms but don't ever return from state | 09:37 |
| fwereade | hahaha | 09:37 |
| dimitern | babbageclunk: I was trying to follow along, but getting distracted - please ping me when you have those changes to have a look though | 09:42 |
| mup | Bug #1612163 opened: state refcounts should be consolidated <tech-debt> <juju-core:Triaged> <https://launchpad.net/bugs/1612163> | 09:58 |
| babbageclunk | fwereade, dimitern: I feel like this might be a boil the ocean solution. | 10:01 |
| fwereade | babbageclunk, if it's too much, you're closer than me and your judgment is probably clearer | 10:02 |
| fwereade | babbageclunk, I'd been hoping we could add a relatively clean/narrow interface to the functionality we need | 10:02 |
| fwereade | babbageclunk, i.e. we don't need to rewrite what exists, just hide it behind something more fit for purpose | 10:03 |
| voidspace | fwereade: team meeting? | 10:04 |
| fwereade | voidspace, thanks | 10:05 |
| voidspace | dimitern: do you have an example /e/n/i for a maas rack controller that defines vlans? I just have one subnet (and one nic) on the rack controller - but can reconfigure if needed (all KVM). | 10:10 |
| dimitern | voidspace: yep, just a sec | 10:12 |
| voidspace | dimitern: sure | 10:13 |
| dimitern | voidspace: http://paste.ubuntu.com/23010779/ here's my 2.0 vmaas | 10:13 |
| voidspace | dimitern: thanks! | 10:14 |
| rick_h_ | morning | 11:18 |
| anastasiamac | rick_h_: \o/ | 11:20 |
| rick_h_ | katco: dimitern ping for standup | 14:01 |
| rick_h_ | katco: can you take a peek at https://github.com/juju/juju/pull/5978/files please for ui folks? | 14:20 |
| katco | rick_h_: sure | 14:20 |
| rick_h_ | katco: ty | 14:20 |
| mup | Bug #1520292 changed: Upgrade from 1.21.3 -> 1.22.8 -> 1.23.3 fails with 'ERROR a hosted environment cannot have a higher version than the server environment: 1.23.3.5 > 1.22.8.1' <bug-squad> <upgrade-juju> <juju-core:Invalid> <https://launchpad.net/bugs/1520292> | 14:23 |
| mup | Bug #1520292 opened: Upgrade from 1.21.3 -> 1.22.8 -> 1.23.3 fails with 'ERROR a hosted environment cannot have a higher version than the server environment: 1.23.3.5 > 1.22.8.1' <bug-squad> <upgrade-juju> <juju-core:Invalid> <https://launchpad.net/bugs/1520292> | 14:26 |
| katco | frankban: hey, left a new comment on http://reviews.vapour.ws/r/5419/ | 14:26 |
| frankban | katco: sure, I like this new rules actually, good to know | 14:27 |
| mup | Bug #1427257 changed: Juju backup doesn't contain .juju files <backup-restore> <juju-core:Invalid> <https://launchpad.net/bugs/1427257> | 14:35 |
| mup | Bug #1520292 changed: Upgrade from 1.21.3 -> 1.22.8 -> 1.23.3 fails with 'ERROR a hosted environment cannot have a higher version than the server environment: 1.23.3.5 > 1.22.8.1' <bug-squad> <upgrade-juju> <juju-core:Invalid> <https://launchpad.net/bugs/1520292> | 14:35 |
| mup | Bug #1545686 changed: Multi-series not backwards compatible <openstack> <uosci> <juju-core:Triaged by uros-jovanovic> <https://launchpad.net/bugs/1545686> | 14:35 |
| frankban | katco: updated instructions at http://reviews.vapour.ws/r/5419/ | 14:36 |
| katco | frankban: ta, tal | 14:37 |
| frankban | katco or axw: could you please take a look? | 14:37 |
| frankban | we'd like to include that in next beta | 14:37 |
| katco | frankban: did you publish your changes? | 14:37 |
| frankban | katco: I just did, sorry | 14:38 |
| katco | frankban: np, ty | 14:38 |
| katco | frankban: this is great, ty. please also amend your commit message with this information (you can elide the qa portion if you like) | 14:46 |
| frankban | katco: done | 14:48 |
| frankban | katco: thanks for the review, shipping it | 14:49 |
| katco | frankban: thanks for the change! <3 | 14:49 |
| babbageclunk | dimitern: Could you take a look at http://reviews.vapour.ws/r/5420/? Should be a quick one! | 15:02 |
| === benji___ is now known as benji | ||
| dimitern | babbageclunk: looking | 15:17 |
| dimitern | babbageclunk: LGTM | 15:20 |
| babbageclunk | dimitern: Yay, thanks! | 15:20 |
| babbageclunk | dimitern: This is the beginning of the end for this bug, I can feel it in me waters. | 15:21 |
| dimitern | :D indded | 15:23 |
| redir | morning | 15:29 |
| mup | Bug #1466152 changed: [RFE] log locks and other relevant information when jujud receives SIGUSR1 <feature> <logging> <juju-core:Invalid> <https://launchpad.net/bugs/1466152> | 15:29 |
| balloons | mgz, so I saw you got a little further on my PR woes | 16:13 |
| mgz | balloons: yeah, I hoped that would be enough to get you unstuck | 16:13 |
| balloons | mgz, so do you have any thoughts then on getting it landed? I broke it up a little, but my attempt failed (beyond landing just the snapcraft.yaml) | 16:14 |
| mgz | I'm not totally clear from william what his preferered solution is, but we need to change the utils/exec code at least partly back to how it was before | 16:14 |
| balloons | is fwereade still about? | 16:15 |
| balloons | mgz, if not, I'd appreciate your thoughts on what we can try | 16:22 |
| balloons | shall I land something in juju/utils to undo the updates to exec? | 16:23 |
| mgz | that's certainly an option | 16:23 |
| balloons | mgz, seems simple enough honestly I suppose | 16:27 |
| balloons | bogdanteleaga are you about? | 16:27 |
| balloons | mgz, thank you for figuring out the bad commit btw :-) | 16:27 |
| mup | Bug #1612335 opened: Azure rate limit leads to catastrophic failure for subscription <azure-provider> <bootstrap> <ci> <destroy-controller> <kill-controller> <regression> <juju-core:Triaged> <https://launchpad.net/bugs/1612335> | 16:29 |
| balloons | mgz, I like that option. See https://github.com/juju/utils/pull/232 | 16:38 |
| balloons | I'll notify bogdanteleaga he'll need to redo his PR | 16:38 |
| mgz | balloons: okay, poke ocr for a review (I probably shouldn't as I was also involved in the change) | 16:39 |
| balloons | mgz, ack | 16:40 |
| balloons | mgz, I believe that's frobware though and he's EOD | 16:41 |
| balloons | redir, perhaps you can review for me? https://github.com/juju/utils/pull/232 | 16:42 |
| mgz | pretty sure he's on holiday so it must be someone else? | 16:42 |
| balloons | calendar says Andy; which I suppose is ambigous :p | 16:42 |
| balloons | I'm going to pretend it's Wednesday and ask Reed | 16:43 |
| === frankban is now known as frankban|afk | ||
| dimitern | if anyone has half a minute, I'd appreciate a review on this: http://reviews.vapour.ws/r/5421/ | 17:54 |
| dimitern | it's quite tiny | 17:54 |
| dimitern | rick_h_: here's the extra logging PR ^^ | 17:54 |
| redir | balloons: looking, lost in apt-get failures:/ | 17:56 |
| redir | balloons: so reverting a revert? | 18:01 |
| balloons | redir, reverting https://github.com/juju/utils/commit/8a9dea08709bd9f65bbf7ed2c9bcfa0656eb8a7b, which causes a unit test failure | 18:01 |
| balloons | and I'd like to land my otherwise clean PR without being hit by hit (because I need a new version of juju/utils than juju/juju has) | 18:02 |
| redir | changes made to 10adcbf. | 18:05 |
| natefinch | man I hate middle click paste | 18:06 |
| redir | natefinch: weechat multiline paste to msg command | 18:09 |
| natefinch | I can't tell you how many times I have random bits of text appear in my code because I pressed too hard while scrolling with scroll wheel | 18:09 |
| * rick_h_ can't live w/o middle click paste | 18:10 | |
| redir | I don't have a scroll wheel:) | 18:10 |
| natefinch | if I had a button separate from the scroll wheel it might be ok | 18:10 |
| natefinch | redir: OMG, how can you live like that? | 18:10 |
| redir | natefinch: can you give a graduated review to balloons PR above? | 18:10 |
| balloons | natefinch, touchpad scrolling doesn't work for you? 2 fingers! | 18:11 |
| redir | natefinch: with a trackball when I really have to | 18:11 |
| balloons | natefinch, i have itchy fingers, but if it's not up to snuff, I'll can the merge | 18:11 |
| redir | balloons: touchpad constantly reads my palms | 18:12 |
| natefinch | balloons: sure, when i'm forced to use a touchpad like a sucker. When I'm at a desk, I always have a real mouse. | 18:12 |
| balloons | redir, ahh.. yea, the laptop matters SO much. The one I have behaves really nicely. No palm reading! | 18:12 |
| natefinch | yeah, what redir said... linux "disable touchpad while typing" 100% does not work on my laptop (not sure about others) | 18:12 |
| redir | anyone else having 404 issues wiht archive.ubuntu.com? | 18:13 |
| redir | I cannot bootstrap a controller:/ | 18:13 |
| natefinch | balloons: ship it | 18:13 |
| rick_h_ | redir: just bootstrapped here in the last 3min | 18:13 |
| balloons | :-) | 18:13 |
| rick_h_ | redir: pipes your way to your mirror cranky? | 18:14 |
| natefinch | redir: I can get to it via my browser | 18:14 |
| natefinch | balloons: oops... I reviewed dimtern's change, not yours | 18:14 |
| redir | natefinch: http://archive.ubuntu.com/ubuntu/pool/main/q/qemu/qemu-utils_2.5+dfsg-5ubuntu10.2_amd64.deb | 18:15 |
| balloons | natefinch, :p. I trust you'll come to the same conclusion | 18:15 |
| rick_h_ | and dimiter goes "winning!" | 18:15 |
| redir | rick_h_: ^ | 18:15 |
| * balloons waves hand in front of natefinch's eyes | 18:15 | |
| natefinch | redir: that's a 404 year | 18:15 |
| natefinch | yeah | 18:15 |
| redir | whelp seems bootstrap in LXD hands on a handfull of those for me | 18:16 |
| natefinch | balloons: link me? | 18:16 |
| redir | tried deleting the lxc images for fun, cleaned out dns, etc | 18:16 |
| * redir scratches head | 18:16 | |
| natefinch | I think the people who wrote Ubuntu SSO forgot what that first S stands for | 18:16 |
| rick_h_ | redir: apt-get update fail? there's a 10.3 in the archive not a .2 | 18:17 |
| redir | Serial Sign On | 18:17 |
| rick_h_ | redir: looks like it's just out of date index atm | 18:17 |
| redir | rick_h_: k | 18:17 |
| redir | I'll just do something else for a while | 18:18 |
| * redir looks at email! | 18:18 | |
| rick_h_ | redir: so http://archive.ubuntu.com/ubuntu/pool/main/q/qemu/qemu-utils_2.5+dfsg-5ubuntu10.3_amd64.deb is there | 18:18 |
| mup | Bug #1583771 changed: RunnerSuite.TestOneWorkerStartWhenStopping timed out <ci> <intermittent-failure> <race-condition> <regression> <unit-tests> <juju-core:Fix Released by fwereade> <https://launchpad.net/bugs/1583771> | 18:29 |
| mup | Bug #1596462 changed: Deployment failed because state DB is locked <ci> <deploy> <reliability> <juju-core:Fix Released by cox-katherine-e> <https://launchpad.net/bugs/1596462> | 18:29 |
| mup | Bug #1600301 changed: cmd/jujud/agent MachineSuite.TestHostedModelWorkers fails because compute-provisioner never really starts <ci> <intermittent-failure> <regression> <unit-tests> <juju-core:Fix Released by menno.smits> <https://launchpad.net/bugs/1600301> | 18:29 |
| redir | rick_h_: grumble unfixing enable-os-refresh-update for speed fixes my problem. but also unfixes speedup. | 18:32 |
| redir | I guess speedup doesn't matter if it doesn't work | 18:32 |
| rick_h_ | redir: if you want it to go fast...add caching. If you want it to work... | 18:32 |
| * rick_h_ thinks that's famous ancient chinese proverb | 18:33 | |
| redir | I want both | 18:33 |
| * rick_h_ goes to find some sort of ancient proverb about greed | 18:34 | |
| redir | all three really | 18:34 |
| redir | rick_h_: yu gung can move mountains | 18:35 |
| natefinch | I think a meeting with 97 people is gonna be a little crowded | 18:42 |
| perrito666 | natefinch: ? | 18:43 |
| natefinch | perrito666: just got an invite to the juju cross team meeting... which evidently includes everyone except you | 18:44 |
| natefinch | perrito666: nope, I'm wrong, you're on there | 18:44 |
| perrito666 | natefinch: it includes me | 18:44 |
| perrito666 | i just ignore it | 18:44 |
| perrito666 | I just went when I was replacing you | 18:45 |
| natefinch | I thought it might be new, I didn't remember it being on my calendar | 18:47 |
| alexisb | perrito666, ping | 18:57 |
| perrito666 | alexisb: pong | 18:57 |
| alexisb | sounds like some brokeness with your permissions commit | 18:58 |
| alexisb | can you please reach out to sinzui and team | 18:58 |
| perrito666 | alexisb: uff, life cant be easy on me apparently | 18:58 |
| perrito666 | alexisb: sure | 18:58 |
| alexisb | thanks | 18:58 |
| mup | Bug # changed: 1558061, 1566011, 1570657, 1577776, 1587653, 1588390, 1594720, 1597170, 1597601, 1597720, 1599972, 1604482, 1605096, 1607457, 1607620, 1608421, 1608818, 1608821, 1609343, 1610012 | 19:17 |
| mup | Bug #1612395 opened: TaaS RestAPI fails due to "Unknown column 'tap_services.project_id' <tap-as-a-service> <juju-core:New> <https://launchpad.net/bugs/1612395> | 19:47 |
| mup | Bug #1612397 opened: juju 2.0 does not have an unset-config command anymore <juju-core:New> <https://launchpad.net/bugs/1612397> | 19:47 |
| balloons | morning wallyworld | 20:01 |
| wallyworld | hey | 20:01 |
| katco | wallyworld: you're up early | 20:01 |
| katco | or maybe just on early | 20:01 |
| wallyworld | goota drive the kid to school in a sec, early start today | 20:02 |
| katco | cool | 20:02 |
| balloons | wallyworld, anyways as you guessed, we're releasing beta15 atm, but yea, I would like it to land right after | 20:02 |
| katco | that will be me in a week | 20:02 |
| wallyworld | :-) | 20:02 |
| wallyworld | balloons: no worries, willdo asap | 20:02 |
| wallyworld | after release | 20:02 |
| wallyworld | been wanting to land | 20:03 |
| wallyworld | balloons: we still got to sort out how to deal with settings, as per email conversations | 20:03 |
| balloons | wallyworld, brillant. I'd like to also talk with you about how you want to snap the agents as well | 20:03 |
| balloons | wallyworld, right. We need to sync up a bit on the next steps | 20:03 |
| wallyworld | plus i wish the godeps plugin could be pointed to a source dir | 20:04 |
| balloons | it's just simple python. I'd like it fixed too, and I can't believe it's a difficult thing to do | 20:04 |
| balloons | but for my needs, re-pulling is fine, heh | 20:04 |
| wallyworld | yeah agreed, patches accepted i'm srue | 20:04 |
| wallyworld | it works, but is sloooooow | 20:05 |
| wallyworld | and doesn't really fit dev workflow very well | 20:05 |
| balloons | well, it's horrible for developer workflow | 20:05 |
| balloons | but for releasing, it's just fine. I have to pull anyway | 20:05 |
| wallyworld | true for release | 20:05 |
| wallyworld | right, bbiab, need coffee and breakfast after school drop off | 20:06 |
| natefinch | gah, why does yaml.Unmarshal spit out map[interface{}]interface{} instead of map[string]interface{}? | 20:10 |
| katco | gah... my computer's been freezing all day. finally figured out gocode was eating 8GB of memory o.0 | 20:42 |
| katco | ...and wants to continue doing so? wtf | 20:49 |
| redir | katco: I blame emacs | 20:58 |
| redir | egacs | 20:58 |
| redir | eight gigs and continuous swapping | 20:59 |
| redir | jk | 20:59 |
| katco | redir: emacs irc module, erc, has this great "fool" feature which demphasizes/ignores fools who don't know what they're talking about. ;p | 21:01 |
| === icey is now known as icey|vacation | ||
| rick_h_ | natefinch: what's up with this one? https://bugs.launchpad.net/juju-core/+bug/1552274 is that something still looking at? | 21:08 |
| mup | Bug #1552274: juju list-credentials inconsistencies between format output <2.0-count> <bitesize> <conjure> <juju-release-support> <rc1> <usability> <juju-core:In Progress by natefinch> <https://launchpad.net/bugs/1552274> | 21:08 |
| mup | Bug #1612417 opened: revoke failed: User checked status without read permission <ci> <regression> <revoke> <juju-ci-tools:In Progress by sinzui> <juju-core:In Progress by hduran-8> <https://launchpad.net/bugs/1612417> | 21:18 |
| mup | Bug #1484606 changed: bootstrap fails if control bucket not specified and exists <config> <openstack-provider> <juju-core:Invalid> <juju-core 1.25:Won't Fix> <https://launchpad.net/bugs/1484606> | 21:36 |
| mup | Bug #1612395 changed: TaaS RestAPI fails due to "Unknown column 'tap_services.project_id' <tap-as-a-service> <juju-core:Invalid> <https://launchpad.net/bugs/1612395> | 21:36 |
| redir | katco: and psychoanalyze-pinhead | 21:48 |
| katco | redir: lol | 21:48 |
| redir | wallyworld: you back? | 21:53 |
| wallyworld | in a meeting, but yeah | 21:53 |
| redir | k | 21:54 |
| niedbalski | rick_h_, Hey rick ? how's going? I saw that you assigned 1611981 to yourself :), let me know if you have any question. | 21:54 |
| redir | holler if you get free wallyworld | 21:54 |
| wallyworld | redir: will do, got another meeting straight after this one, will ping as soon as i can | 21:56 |
| redir | wallyworld: no sweat | 22:01 |
| rick_h_ | wallyworld: want to have our chat? | 22:06 |
| wallyworld | rick_h_: ok, if you are free, can you give me 5? i thought i saw a decline so i am having another meetig, so give me a sec | 22:06 |
| rick_h_ | niedbalski: cool, that's my way of saying my team will look at it in the near future. will bug if we have questions | 22:07 |
| wallyworld | rick_h_: i'm free now if you are | 22:22 |
| mup | Bug #1605986 opened: Confusing error: Creating container: can't get info for image 'ubuntu-trusty' <oil> <oil-2.0> <juju-core:New> <https://launchpad.net/bugs/1605986> | 22:39 |
| mup | Bug #1612397 changed: juju 2.0 does not have an unset-config command anymore <juju-core:Invalid> <https://launchpad.net/bugs/1612397> | 22:48 |
| wallyworld | redir: finally finished other meetings if you wanted to talk. oh how i love meetings | 22:49 |
| perrito666 | I definitely need an apt proxy or lxd provider is sloooow | 23:00 |
| alexisb | perrito666, it has gotten a lot slower recentl | 23:00 |
| alexisb | I meant to fire up beta8 and see if it was faster | 23:00 |
| perrito666 | alexisb: this seems to be just cloudinit downloading ~50M from a repo that might not be too close | 23:01 |
| perrito666 | and then juju most likely downloads another 50 | 23:01 |
| alexisb | yes but th eimage I have should be cached at this point | 23:02 |
| perrito666 | alexisb: the image is, notice how it goes fast until you reach cloudinit | 23:03 |
| perrito666 | then aptget update/upgrade | 23:03 |
| === akhavr1 is now known as akhavr | ||
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!