/srv/irclogs.ubuntu.com/2016/04/25/#juju-dev.txt

babbageclunkdimitern: quick review of this? https://github.com/juju/gomaasapi/pull/4608:51
dimiternbabbageclunk: looking08:51
babbageclunkdimitern: it turns out that as long as the methods return nil literally, the interface values compare == to nil08:51
dimiternbabbageclunk: hmm really? Won't the nil be wrapped still in an nil interface value, which itself is not nil?08:52
babbageclunkdimitern: as long as both the type pointer and the value pointer are nil it'll == nil08:53
babbageclunkdimitern: but returning something like i.subnet when it's nil will set the type pointer.08:54
dimiternbabbageclunk: you're correct, I've just tried this: http://play.golang.org/p/dQSGV-JQ8708:55
dimiternbabbageclunk: so the compiler does the right thing as long as you return an untyped nil08:56
babbageclunkdimitern: yup - and that means we can keep the interface nice while getting the right behaviour08:56
dimiternbabbageclunk: sweet!08:56
babbageclunkdimitern: :)08:57
voidspacedimitern: frobware: will be a couple of minutes late to standup08:59
voidspaceurgent coffee emergency08:59
dimiternvoidspace: np08:59
babbageclunksweet, opportunistically grabbing a tea too then09:00
dimiternbabbageclunk: reviewed09:01
jammgz: how's the release shaping up? Is CI happy or do we need some extra pokinG?09:02
frobwarevoidspace: https://bugs.launchpad.net/maas/+bug/157304609:42
mupBug #1573046: 14.04 images not available for commissioning as distrio-info --lts now reports xenial <landscape> <MAAS:Fix Released by andreserl> <https://launchpad.net/bugs/1573046>09:42
frobwarevoidspace: I got kicked out, back into standup HO again?09:52
voidspacefrobware: I'm still there09:52
frobwaredooferlad: the precise /tmp issue - I wonder how it  ever worked then10:05
dooferladfrobware: I don't know, but it definitely happens.10:06
babbageclunkdimitern, frobware, voidspace: review please? https://github.com/juju/juju/pull/527110:07
babbageclunkalso, trying to post a comment on a bug on launchpad is timing out - is that normal?10:07
babbageclunknvm - fine now10:07
dimiternbabbageclunk: LGTM, thanks!10:10
babbageclunkman, juju on maas is really neat when it's all working smoothly.10:10
babbageclunkdimitern: Thanks!10:10
voidspacebabbageclunk: that was a difficult review...10:10
voidspacedimitern: that's what a review should look like - not 17 pages! ;-)10:10
voidspace(I jest - remove as much code as you want...)10:11
babbageclunkvoidspace: sorry man - I tried to keep it to a 1-line change but it wouldn't compile without the other one.10:11
voidspacebabbageclunk: yeah, I have the same change in my branch10:11
voidspacedimitern: I can confirm that lxd doesn't work on maas for neither trusty nor xenial10:12
voidspacedimitern: due to this bug https://bugs.launchpad.net/juju-core/+bug/156889510:13
mupBug #1568895: Cannot add MAAS-based LXD containers in 2.0beta4 on trusty <juju-core:Triaged> <https://launchpad.net/bugs/1568895>10:13
dimiternbabbageclunk: it should be smooth if we (and the maas guys) are doing it properly :)10:13
babbageclunkdimitern: :)10:13
dimiternvoidspace: oh, not good :/ I'll give it a try here with 1.9 and 2.0 on xenial10:13
voidspaceI also can't commission new nodes at the moment10:14
dimiternvoidspace: some reviews are like that :P10:14
voidspacewhich frobware reckons is probably due to the xenial transition10:14
voidspacenot an issue for me right now, but will be soon10:14
voidspaceheh, yea10:14
dimiternbabbageclunk: if you can bootstrap ok now, you can test a few things - e.g. constraints selection for nodes, bindings, deploying mult-nic lxd backed by a device, ideally as far as checking whether network-get works10:20
voidspacedimitern: can't test lxd, multi-nic or otherwise10:22
dimiternvoidspace: 'otherwise' should work (i.e. with lxc)10:22
voidspacedimitern: well, yes "not lxd" works...10:22
dimiternvoidspace: as long as it works for lxc (and kvm) at least we know the multi-nic code path is exercised10:23
dimiternthen it's down to fixing lxd I guess10:23
voidspacedimitern: by the way - how is AllocateContainerAddresses tested in the maas provider10:24
dimitern(again)10:24
babbageclunkdimitern: ok - I'll probably need some hand-holding for those.10:24
voidspacedimitern: not directly as far as I can tell10:24
voidspaceI'm writing tests for the new code path10:24
dimiternbabbageclunk: sure, just ask10:24
dimiternvoidspace: not tested except manually (live) I'm afraid10:24
voidspacenaughty :-/10:24
voidspacewho let that past review!10:25
voidspaceeasier to test with the new gomaasapi though10:25
dimiternvoidspace: yeah, but at least we can test it properly now10:25
dimiternbabbageclunk: for testing constraints, I'd suggest trying to add a machine with --constraints='...', once for each type (arch, mem, cpu, etc.) on its own, then removing the machine with --force before trying the next10:28
babbageclunkok10:30
babbageclunkdimitern, voidspace: man, reading through gomaasapi/testservice.go was pretty alarming! Basically having to re-implement a baby MAAS!10:31
dimiterndifferent combos are also useful to test, and constraints with negatives as well (tags=^mytag1,^othertag, same for spaces=)10:31
dimiternbabbageclunk: yeah, and that's a simpler test double than we use for ec210:32
babbageclunkdimitern: ouch10:32
voidspacebabbageclunk: yep :-)10:32
dooferladfrobware: have replied to that review...10:38
mupBug #1574564 opened: Juju tools 1.25.5 not found in https://streams.canonical.com/juju/tools/releases/ <juju-core:New> <https://launchpad.net/bugs/1574564>10:42
dimiternvoidspace, frobware, babbageclunk: http://reviews.vapour.ws/r/4697/ please, take a look (one page diff this time :)10:44
fwereadebabbageclunk, dimitern, voidspace: FWIW, I feel quite strongly that complex test doubles are very smelly -- for individual interactions, explicit canned request/response is usually tractable, and for broader tests there's no substitute for a real maas10:52
dimiternfwereade: agreed - we're moving the low-level MAAS API interactions and tests inside gomaasapi itself, and using a pre-canned http server for higher-level interactions10:54
fwereadebabbageclunk, dimitern, voidspace: and it seems like a bad deal to devote effort to exactly aping a substrate -- it's very hard to maintain, and it means that the test infrastructure is biased *against* being able to test weird responses10:55
fwereadedimitern, cool10:55
dimiternfrobware, voidspace, babbageclunk: related, but simpler review for the names package: http://reviews.vapour.ws/r/4698/11:15
babbageclunkdimitern: is there a way to ignore whitespace differences in reviewboard or github? The vertical alignment that gofmt does makes it hard to see the real change in lots of places. (The checkboxes in reviewboard don't seem to do anything for the changes I'm looking at.)11:18
dimiternbabbageclunk: I don't think so, but does it look better on github?11:19
babbageclunkdimitern: oh-ho! https://github.com/blog/967-github-secrets11:20
babbageclunkdimitern (and frobware): add ?w=1 on the end of the url11:21
dimiternbabbageclunk: nice! I knew there was something like that, but haven't used it yet11:21
frobwarebabbageclunk: I was looking for something like that about 30 mins ago when purveying dimitern's PR... thx11:21
dimiternfrobware: fwiw I try to make individual commits make sense on their own11:23
frobwaredimitern: ack. I just find it a toss-up which UI is better. GH or RB.11:23
dimiternok, upgraded both maas-es to latest versions (1.9.1+bzr4543-0ubuntu2 (trusty1) and 2.0.0 (beta4+bzr4944), respectively) now bootstrapping to see if lxd will work11:33
dimiternfwereade: hey, I know we don't usually do our 1:1s, but I have a few things to discuss if you have 15-20m time?11:36
dimiternwe can do it later/tommorow as well11:38
dimiternvoidspace, frobware, babbageclunk: I can successfully deploy multi-nic lxd containers on xenial with maas 1.9.1 beta411:40
jammgz: ping11:43
voidspacedimitern: that's awesome11:55
=== babbageclunk is now known as babbagelunch
dimiternvoidspace: maybe try upgrading your maas to see if it will fix the lxd issue?11:56
dimiternnow trying the same on 2.0.011:58
voidspacedimitern: ah, that's 1.911:58
voidspacedimitern: I'm using the latest 2.0 from experimental 3 ppa.11:58
voidspacedimitern: not so awesome news then. I mean, great - but it's not using my code...11:59
voidspacedimitern: you won't be able to deploy containers on maas 2 with master - you'll need my branch11:59
dimiternvoidspace: ah, ok can you paste a link and I'll try with your branch?12:00
voidspacedimitern: https://github.com/voidspace/juju/tree/maas2-allocate-container-addresses12:04
dimiternvoidspace: thanks!12:05
jamfwereade: have you seen the "test timed out waiting for the machiner to start" ? failure in CI?12:09
jamhttp://reports.vapour.ws/releases/3919/job/run-unit-tests-mongodb3/attempt/53812:09
mupBug #1382131 changed: local provider configs cannot be trivially duplicated <config> <local-provider> <juju-core:Invalid> <https://launchpad.net/bugs/1382131>12:10
jamfwereade: the very first thing that comes to mind is knowing *what* dependency is missing would be useful12:10
frobwaredooferlad: ping12:17
cheryljfwereade: Just sent an email about the pinger bug - can you take a look?12:19
urulamacherylj: are you changing the pinger?12:20
urulamacherylj: that's the only way to keep WS open so that apache doesn't close it by itself12:20
cheryljurulama: this is for fixing bug 157223712:21
mupBug #1572237: juju rc1 loses agents during a lxd deploy <lxd-provider> <juju-core:In Progress by ericsnowcurrently> <https://launchpad.net/bugs/1572237>12:21
cheryljurulama: it's to make sure the pinger restarts if it dies12:22
urulamacherylj: ok, thanks. just didn't want to get any surprises seeing GUI stop working all of a sudden :)12:22
cheryljhehe :)12:22
* dimitern is out for ~2h12:24
simonklbanyone know if someone is working on https://bugs.launchpad.net/juju-core/+bug/1565872 ?12:28
mupBug #1565872: Juju needs to support LXD profiles as a constraint <adoption> <juju-release-support> <lxd> <juju-core:Triaged> <https://launchpad.net/bugs/1565872>12:28
simonklband what is the best option for developing docker charms for 2.0 at the moment?12:28
fwereadejam, the infrastructure is there but I never got to exposing it -- call .Report() on an Engine and you'll get all sorts of useful diagnostic goodness12:33
fwereadecherylj, ack12:33
mupBug #1574607 opened: Multiple Interfaces lead to stalled charm download over wrong API endpoint <api> <networking> <juju-core:New> <https://launchpad.net/bugs/1574607>12:34
fwereadecherylj, it's a real race, in stuff my eyes completely skipped over because I thought I recognised it12:40
mupBug #1574632 opened: Data Race in apiserver/presence/pinger.go <blocker> <ci> <race-condition> <regression> <unit-tests> <juju-core:Triaged> <https://launchpad.net/bugs/1574632>13:04
* voidspace lunches13:12
mupBug #1574637 opened: TestScale fails intermittently <ci> <intermittent-failure> <test-failure> <juju-core:Triaged> <https://launchpad.net/bugs/1574637>13:16
mupBug #1574637 changed: TestScale fails intermittently <ci> <intermittent-failure> <test-failure> <juju-core:Triaged> <https://launchpad.net/bugs/1574637>13:22
GaryxAnyone have any idea of when juju 2.0 RC1 or next Beta will rear it's head in the repo's?13:25
mupBug #1574637 opened: TestScale fails intermittently <ci> <intermittent-failure> <test-failure> <juju-core:Triaged> <https://launchpad.net/bugs/1574637>13:34
babbageclunkGaryx: I think cherylj might be able to tell you that.13:42
alexisbsinzui, fwereade, cherylj yay for CI for catching issues13:43
alexisbfwereade, cherylj what do we htink the turn around time will be for fixing the race?13:44
Garyxbabbageclunk: thanks13:45
Garyxcherylj: any rough eta, not asking for a time and date ;)13:45
alexisbGaryx, we are working on the next beat atm13:45
alexisbtrying to get it out today13:45
alexisbbeta613:45
Garyxalexisb: thanks for the info :)13:46
mupBug #1574649 opened: environs/config LatestLtsSeries is not isolated from the host during testing <juju-core:New> <https://launchpad.net/bugs/1574649>13:46
fwereadealexisb, more than 0 hours and hopefully less than 2; surely less than 413:47
alexisbfwereade, ack thanks13:47
mupBug #1574564 changed: Juju tools 1.25.5 not found in https://streams.canonical.com/juju/tools/releases/ <juju-core:Invalid> <https://launchpad.net/bugs/1574564>13:58
mupBug #1574677 opened: Designation of 'local' for both users and controllers is confusing <docteam> <juju-core:New> <https://launchpad.net/bugs/1574677>14:43
* dimitern is back15:01
katcoericsnow: standup time15:02
dimiternvoidspace: I'm trying your branch now on 2.0 with lxd15:22
mupBug #1570917 changed: upgrade-juju: success but then deploy fails <upgrade-juju> <juju-core:Invalid by rogpeppe> <https://launchpad.net/bugs/1570917>15:22
voidspacedimitern: cool, happy path test (single NIC) just about ready to land15:22
voidspacedimitern: I'll get that up for review and then do more extensive tests15:23
voidspacedimitern: once it lands there's container support on master - so good to get it in15:23
voidspacemind you, master is blocked so it doesn't really matter I guess15:23
dimiternvoidspace: sounds good15:23
voidspacea review would be good though15:23
dimiternvoidspace: I was having issues with my maas2 - tgt was misbehaving again and nothing could be commissioned or deployed successfully, now it seems it works again15:24
voidspacedimitern: I think you'll hit this bug, but we'll see: https://bugs.launchpad.net/juju-core/+bug/156889515:27
mupBug #1568895: Cannot add MAAS-based LXD containers in 2.0beta4 on trusty <juju-core:Triaged> <https://launchpad.net/bugs/1568895>15:27
dimiternvoidspace: nope, it's not that I think, as the bug is about trusty and this is xenial15:28
voidspacedimitern: I had the exact same error with both15:28
dimiternvoidspace: if you grep for 'tgtadm: out of memory' in /var/log/maas/rackd.log and find anything, you might be hitting my issue15:31
voidspacedimitern: will do15:31
voidspacedimitern: frobware: babbageclunk: I have to pick my daughter up from the childminders' at 5pm - wife is stuck at the doctors15:35
voidspaceso will be late to meeting, sorry15:35
dimiternvoidspace: ok, np15:36
dimiternvoidspace: do you have a dual-nic hardware node, and the second one is usb2eth?15:37
voidspacedimitern: me, no15:37
dimiternvoidspace: and the node details page showing something like 'eno1' and 'enxxaabbccddeef0' ?15:37
voidspacedimitern: my testing is kvm only15:37
dimiternvoidspace: ah, ok15:38
voidspacedimitern: if you're talking about that bug report it is from frobware15:38
dimiternvoidspace: nope, it's about dealing with long NIC names (e.g. the second NIC on all of my NUCs shows up as e.g. 'enx00e10000163d'15:39
voidspaceah15:39
dimiternand that's a wee bit too long to prepend 'br-' to it15:39
voidspacehah15:39
dimiternbridge script fails and I can't access the node :/15:40
voidspacedimitern: frobware: babbageclunk: AllocateContainerAddresses with happy path test http://reviews.vapour.ws/r/4700/15:40
voidspacereview appreciated even though it can't land yet - so I can make changes15:40
voidspacein the meantime working on more tests15:40
dimiternvoidspace: that seems to happen if you commissioned the node with xenial instead of trusty15:40
dimiternvoidspace: looking15:40
mupBug # changed: 1556183, 1556252, 1557146, 1557148, 155767915:43
alexisbdimitern, stop landing stuff in master please15:50
dimiternalexisb: it wasn't blocked today, and I was not aware of restrictions :/15:51
alexisbdimitern, it is blocked15:52
dimiternalexisb: I see now it is, but this morning it wasn't - I haven't $$__JFDI__$$'ed anything since you mailed me15:52
alexisbthanks dimitern15:53
frobwaredimitern, voidspace: I find it odd that in 2016 the name of the iface is so constrained.15:55
voidspacedimitern: frobware: back, joining networking meeting16:03
voidspacefrobware: I'm nearly at the point where nothing crazy about networking can surprise me...16:04
natefinch_anyone care to help debug why I can't log in after  upgrding  to Xenial?  http://pastebin.ubuntu.com/16050493/16:07
perrito666natefinch_: sure16:08
perrito666natefinch_: expand, "cant login"16:09
natefinch_well, I get to the greeter screen, enter creds, it looks like its switching to the desktop, briefly displays "ubuntu has encountered a problem" andf then bumps me back to the greeter screen16:10
perrito666ah, I see, well first of all you seem to be using the floss driver for nvidia, is that intentional?16:11
natefinch_(please excuse my typing, i'\m on a tiny bluetooth keyboard via my tablet)16:11
natefinch_I don't care what driver I use so long as it works.... which this noe obviously does not16:11
perrito666if I had to guess, I would say that that your X is crashing after loggin in16:12
natefinch_I had to twiddle my modprobe.d blacklists to even get x to load16:12
natefinch_perrito666 - probably16:12
bogdanteleaganatefinc_, did you blacklist all the gpu drivers?16:13
perrito666natefinch_: check if the greeter doesnt provide an option for a session withouth the effects16:13
natefinch_I can switch to a console easily enough, but that's it16:14
perrito666natefinch_: I am sure ubuntu greeter has an option to change the session you are going to log in16:21
perrito666click on the ubuntu logo next to the user/pass prompt16:21
natefinch__evidently just reinstalling the nvidia driver fixed it16:22
perrito666lol16:23
dimiternlol16:31
dimiternCRITICAL: We failed, but the fail whale is dead. Sorry....16:31
redirup16:34
redirum, power outage.16:34
redirshutting down for a few minutes until it comes back.16:34
perrito666dimitern: what was the dirty hack to workaround the issue with tests complaining about xenial?17:05
dimiterndimitern: ha :) well, put a script returning "trusty", named 'distro-info' somewhere in $PATH, before /usr/bin17:06
mupBug #1574773 opened: bootstrap --to [lxd|lxc]:N fails to validate if machine N exists <landscape> <juju-core:New> <https://launchpad.net/bugs/1574773>17:22
mgzahasenack: that's a fun bug17:24
ahasenackmgz: thx! :)17:25
ahasenackand, let me fix the title17:25
ahasenackit's not bootstrap --to17:25
ahasenackit's deploy --to17:25
ahasenackobviously17:25
mgzahasenack: ah, looks like bug 136512417:26
mupBug #1365124: "juju deploy --to <non-existent-machine> <charm-name>" juju still tries to deploy the service. <deploy> <placement> <juju-core:Triaged> <https://launchpad.net/bugs/1365124>17:26
ahasenackjust --to N worked to validate N17:26
ahasenackit failed when I added the lxc/lxd prefix17:27
mgzbut you say that's fixed?17:27
ahasenackyeah, look at the paste in the bug17:27
ahasenack ERROR cannot deploy "ubuntu-to-machine" to machine 1: machine 1 not found17:27
ahasenackwhen I used --to 117:27
ahasenackbut --to lx[dc]:1 passed that check17:27
mgzwell, close one bug, open another17:27
ahasenackmgz: that bug you linked to is for juju 1.18!17:28
ahasenackor rather, was reported on juju 1.1817:28
mgzyup :)17:28
mgzahasenack: don't supose you have a 1.X env alive at present? I only have 2.0 up currently17:29
mgzjust want to series target17:29
ahasenacknope, I made it a point to move myself to juju 217:30
mgzahasenack: I'll stand something up and test here17:30
mupBug #1365124 changed: "juju deploy --to <non-existent-machine> <charm-name>" juju still tries to deploy the service. <deploy> <placement> <juju-core:Triaged> <https://launchpad.net/bugs/1365124>17:56
mupBug #1574783 opened: Juju 2 status should show model name - discrepancy between output formats <juju-core:New> <https://launchpad.net/bugs/1574783>17:56
tvansteenburghcan anyone point me to the code that retrieves group names from launchpad when i do `charm whoami`?18:01
natefinch__marcoceppi: ^18:03
=== natefinch__ is now known as natefinch
tvansteenburghnatefinch: pretty sure it's go code, i just can't find it18:04
mupBug #1574783 changed: Juju 2 status should show model name - discrepancy between output formats <juju-core:New> <https://launchpad.net/bugs/1574783>18:08
mupBug #1574783 opened: Juju 2 status should show model name - discrepancy between output formats <juju-core:New> <https://launchpad.net/bugs/1574783>18:20
mupBug #1574783 changed: Juju 2 status should show model name - discrepancy between output formats <juju-core:New> <https://launchpad.net/bugs/1574783>18:26
mupBug #1574798 opened: Surprising charm downgrade <landscape> <juju-core:New> <https://launchpad.net/bugs/1574798>18:26
mupBug #1574809 opened: "to: lxc:0" ignored in bundle <kanban-cross-team> <landscape> <juju-core:New> <https://launchpad.net/bugs/1574809>18:56
redir_and we're back20:12
=== redir_ is now known as redir
mrammDoes GOOSE support the OpenStack Identity API V3?20:14
mrammand can you configure/use it from juju?20:16
mupBug #1574844 opened: juju2 gives ipv6 address for one lxd, rabbit doesn't appreciate it. <landscape> <juju-core:New> <rabbitmq-server (Juju Charms Collection):New> <https://launchpad.net/bugs/1574844>20:17
katcomramm: it does. you just have to request a /v3/ endpoint in your accounts.yaml (i think that's the right file... might be clouds.yaml)20:19
mrammThanks!20:19
katcoperrito666: ericsnow: meeting time20:32
perrito666katco: tx20:36
alexisbwallyworld, I have 5 minutes before my next call21:55
wallyworldme too, be right there21:55
redirkatco: yst?23:01
alexisbthumper, I lost you23:06
katcoredir: kinda23:13
redirk. about to tanzanitestand but was looking for some juju1 assist. I'll hit you up tomorrow.23:13
redirkatco: ^23:13
katcoredir: ok tc, cya tomorrow23:14
mupBug # changed: 1537937, 1557747, 1564397, 1566130, 1568374, 1571832, 1571916, 1572237, 1572781, 1573148, 1573149, 1573259, 1573382, 1573659, 157463223:30
mupBug #1573410 opened: trusty juju 1.25.5 having issues deploying xenial lxc containers <canonical-bootstack> <juju-core:New> <https://launchpad.net/bugs/1573410>23:30

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!