/srv/irclogs.ubuntu.com/2014/10/14/#juju-dev.txt

=== jcw4|afk is now known as jcw4
* thumper tries to remember menno's git trick02:19
=== Ursinha_ is now known as Guest97591
=== Beret- is now known as Beret
=== meetingology` is now known as meetingology
=== bigjools_ is now known as bigjools
=== mup_ is now known as mup
=== psivaa_ is now known as psivaa
=== Guest97591 is now known as Ursinha
=== Ursinha is now known as Guest20083
=== viperZ28_ is now known as viperZ28
=== uru is now known as urulama
=== rharper is now known as rharper_brux
dimiternmorning all06:50
dimiternany willing reviewers around?06:51
dimiternI need a review on http://reviews.vapour.ws/r/167/ (req by hazmat) and this trivial one http://reviews.vapour.ws/r/168/06:51
hazmatg'morning06:53
=== gnuoy` is now known as gnuoy
rogpeppe2two trivial PRs: one to update depdendencies.tsv (https://github.com/juju/juju/pull/906), the other to fix a sporadic test failure i just saw (https://github.com/juju/juju/pull/907)07:34
rogpeppe2dimitern: ^07:34
dimiternrogpeppe2, looking in a moment07:36
=== rogpeppe2 is now known as rogpeppe
rogpeppedimitern: ta07:36
dimiternrogpeppe, both LGTM, second one with a question07:44
dimiternrogpeppe, can you review those two of mine I posted earlier? ^^07:45
rogpeppedimitern: yes, that character is produced by Duration.String07:45
rogpeppedimitern: that's why the test was failing07:45
dimiternrogpeppe, fancy :)07:45
rogpeppedimitern: can i still merge with $$merge$$ or do i now have to go through vapour.ws ?07:46
dimiternrogpeppe, you can *only* merge the PR, RB is only for reviewing07:48
rogpeppedimitern: ah, cool07:48
dimiternrogpeppe, so can you? :)07:54
rogpeppedimitern: am doing. a bit of an unequal swap, if I may say so :)07:55
dimiternrogpeppe, well, at least the trivial one, not the FwNone one if you prefer07:55
dimiternhazmat, hey07:56
dimiternhazmat, have a look at http://reviews.vapour.ws/r/167/ if you want - it's what your asked for I think07:56
hazmatdimitern, did we decide anything last week re going back to github?07:57
hazmatfor reviews07:57
dimiternhazmat, no, in fact we decided to automate the RB workflow by creating RB reviews and closing them as PRs appear07:58
dimitern..and get merged07:58
hazmatdimitern, cool, i'm still in meetings atm. i'll take a look though07:59
dimiternhazmat, cheers07:59
rogpeppedimitern: reviewed the trivial one08:02
dimiternrogpeppe, tyvm08:03
=== jacekn_ is now known as jacekn
mattywmorning all08:17
TheMuemorning btw08:38
voidspaceTheMue: morning08:41
voidspacerestarting computer08:41
voidspacebrb :-)08:41
dimiternmorning TheMue, voidspace, mattyw08:48
dimitern:)08:48
voidspacedimitern: morning08:48
mattywdimitern, morning08:48
TheMuevoidspace: done any steps regarding the installation of MAAS on gremlin? there is a controller running, but I don't know user and/or pw09:33
TheMuevoidspace: looks like a fresh installation, the boot images are not yet imported (according to a message shown when trying to log in)09:35
voidspaceTheMue: not me - probably elmo :-)09:44
voidspaceTheMue: I think elmo talked to jam about it09:45
voidspaceTheMue: jam is not around at the moment it seems09:45
voidspaceTheMue: I'm going to install locally on my machine09:45
TheMuevoidspace: ok, I'll try to see if the superuser is created via cli09:45
voidspaceTheMue: but into a kvm image (probably bridged)09:45
voidspaceTheMue: cool09:45
TheMuevoidspace: kvm inside parallels? or extra machine?09:46
rogpeppedimitern: interesting, it appears that the utf-8 microsecond symbol has appeared after go 1.3 (in tip)09:48
dimiternrogpeppe, so it fails earlier09:48
voidspaceTheMue: on my desktop - which is native ubuntu09:49
dimiternrogpeppe, how about adding "u" to the [charset] in the regex?09:49
rogpeppedimitern: i'm about to do that09:49
TheMuevoidspace: ok09:49
voidspaceTheMue: with plenty cpu / ram / hard drive09:49
voidspaceTheMue: unlike my laptop....09:49
TheMuevoidspace: ah, you are the one who took the orange box with him09:49
voidspacehaha, I wish09:50
TheMueyeah, nice toy09:50
voidspaceTheMue: did you create a MaaS superuser on gremlin?10:27
voidspaceTheMue: ah, I see you did10:28
voidspaceor someone did10:28
TheMuevoidspace: someone10:32
TheMuevoidspace: not me10:32
voidspaceTheMue: it's the same user(name) we had before10:33
voidspaceTheMue: it was probably still elmo - I told him what username and password we used10:33
TheMuevoidspace: interesting10:33
TheMueaaah10:33
voidspacehe wasn't impressed with our high-security password :-)10:33
TheMuehehe10:33
TheMueabsolutely10:33
TheMuethat's a typical superuser problem, instead of heaving administrator groupd and adding individual users to it10:34
voidspaceright10:34
TheMuewhen it's MY machine it's no problem, but when sharing it ...10:34
voidspaceTheMue: interesting - I added a new VM (kvm instance) and it has automatically shown up in the MaaS cluster10:37
voidspaceI need to add power details10:37
voidspaceat least when I manually started it it came up10:37
TheMuevoidspace: nice10:37
voidspaceTheMue: what's the format for power address and power id?10:38
voidspaceI can look it up10:38
TheMuevoidspace: hmm, here it maybe enough to add the maa user to the libvirt group10:38
voidspaceyep, needed anyway I think10:38
TheMuevoidspace: http://www.teale.de/tealeg/computing/cloud/kvm_maas_juju_openstack.html#sec-5-410:39
voidspacedone10:39
voidspaceTheMue: we still need to generate a key for the maas user I think10:41
TheMuevoidspace: seems so10:41
voidspacehmmm... although it's running on the host10:41
voidspacebut it will still use ssh+qemu I think10:41
TheMuevoidspace: btw, did you start the boot image import? otherwise I'll do it now10:41
voidspaceI didn't10:41
TheMuevoidspace: ok, I start it10:41
voidspaceI'm downloading a utopic iso image locally10:41
voidspacemy internet connection is horrible :-/10:42
TheMuevoidspace: hmm, and here I've got troubles with the boot images link, I get a "not found"10:43
TheMuevoidspace: ah, the link misses the "MAAS" part of the URI, interesting10:44
voidspaceTheMue:  sudo -u maas virsh -c qemu+ssh://maas-admin@localhost/system list --all10:59
rogpeppetwo trivial PRs for review: https://github.com/juju/utils/pull/44 and https://github.com/juju/charm/pull/6211:05
rogpeppeanyone know what can cause an "Extant directories unknown" error in the 'bot ?11:08
rogpeppehttp://juju-ci.vapour.ws:8080/job/github-merge-juju/920/console11:08
rogpeppeah, i know.11:08
rogpeppewell, perhaps.11:09
* rogpeppe tries something.11:09
luca?11:12
voidspaceTheMue: can you see the node?11:25
voidspaceTheMue: I see one node with the status Ready11:26
voidspaceand I need coffee11:26
TheMuevoidspace: strange, have the headline "0 nodes in gremlin MAAS" here in the UI11:33
voidspaceTheMue: how odd11:38
TheMuevoidspace: I'm logged in with our admin user11:41
voidspaceTheMue: I'm logged in as maas-admin11:59
voidspacemaas-test-controller - 1 node in this MAAS12:00
voidspaceour node is called "useless-weight"12:00
voidspacewhich is a pretty cool name12:01
TheMuevirsh list --all shows me maas-test-controller running and maas-test-1 shut off12:06
voidspacethat's correct12:07
voidspacethe node is not running12:07
TheMuebut I don't see any "useless-weight"12:07
TheMueonly 0 nodes :,(12:08
voidspaceI've logged out and back in again and I still see it12:08
voidspacethat's very odd12:08
jcw4rogpeppe: I figured out what was causing that Extant directories issue12:08
voidspaceTheMue: now I am home my power supply works fine...12:09
voidspacewhich is good, but also odd12:09
TheMuevoidspace: oh, one moment, do you logged into the maas controller on gremlin or inside the image maas-test-controller?12:12
voidspaceTheMue: I am logged into the maas controller (web view) on gremlin12:13
voidspaceMy local url being http://localhost:8080/MAAS/12:14
voidspaceand I'm running ssh forwarding12:14
voidspacessh gremlin -L 8080:10.124.0.11:8012:14
dimiternvoidspace, you could run a vpn connection to access gremlin's internal ip directly - that's what I use12:15
TheMuevoidspace: ah, so we're using different controllers12:15
voidspacedimitern: this is easier12:15
voidspaceTheMue: ah, yes12:15
voidspacemaas-controller image is a different maas12:15
voidspacemaas-test-controller12:16
voidspaceheh, that's why12:16
voidspaceTheMue: I'm using the MaaS that's installed *on gremlin*12:16
TheMuevoidspace: me too12:18
TheMuevoidspace: but then you don't need port forwarding, you can use gremlin/MAAS directly12:19
voidspaceTheMue: if I use the VPN ?12:19
voidspacethis works and is easy enough12:20
TheMuevoidspace: the forwarding was only needed for a MAAS controller inside a KVM12:21
TheMuevoidspace: I start the vpn and then can talk to port 80 on gremlin directly12:21
voidspaceright, I'm not using a vpn12:22
voidspaceso I need the port forwarding12:22
TheMuevoidspace: yeah, but you're talking to .11 (that's the KVM I installed on Friday), not to .10 (which is gremlin itself running a controller too)12:23
voidspaceoh really!12:23
TheMue:D12:23
voidspacehah12:23
voidspaceso the kvm image appeared there automatically12:23
voidspaceI didn't manually add the node12:23
voidspacehow funny12:23
voidspaceI just copied the port forwarding from my laptop12:24
voidspaceI didn't check the ip address12:24
TheMue*lol*12:24
TheMueso I still would like to know which node you see if only one KVM instance is running, the controller itself *stunning*12:25
voidspac_TheMue: installing MaaS locally into a KVM image has seized the keyboard / mouse12:28
voidspac_TheMue: and the key combination it wants to release it doesn't appear to exist on my keyboard (I've remapped some of the keys...)12:28
voidspac_TheMue: so even when only the controller is running the cluster knows about the other node12:29
voidspac_TheMue: it shows it as in a Ready state12:29
voidspac_TheMue: I'm going on lunch12:31
voidspac_back in a bit12:31
dimiternwaigani, hey, are you around for a review?12:33
dimiternhttp://reviews.vapour.ws/r/167/12:37
waiganihey dimitern, sure just abut to head for lunch though12:37
dimiternwaigani, sure, np - when you can, have a look please :)12:38
rogpeppejcw4: oh yes?12:41
jcw4rogpeppe: it's a godeps issue I believe12:41
rogpeppejcw4: entirely probable...12:42
alexisbjcw4, it is before 6am for you, thats dedication12:42
jcw4(coupled with a juju-release-tools weirdness)  lol12:42
jcw4alexisb: gotta maximise overlap12:42
jcw4and right back at you alexisb12:43
jcw4:)12:43
jcw4rogpeppe: godeps seems to follow second level imports at the HEAD revision of first level imports12:43
jcw4rogpeppe: before updating the first level imports to the dependencies.tsv revision12:44
rogpeppejcw4: when updating, godeps doesn't follow imports at all AFAIR12:44
alexisbjcw4, I am in Brussels so it is normal time for me12:44
jcw4rogpeppe: yeah, I think it does12:44
jcw4alexisb: whew12:44
jcw4:)12:44
jcw4rogpeppe: it does a go get -d ....12:45
jcw4which seems to be a little recursive12:45
rogpeppejcw4: go get -d shouldn't follow imports12:45
jcw4hmm12:45
rogpeppejcw4: (that's the point of the -d flag)12:45
jcw4rogpeppe: when I was testing yesterday it seemed to12:45
rogpeppejcw4: ah, you're right12:46
rogpeppejcw4: i misread/misremembered what that flag did12:46
jcw4rogpeppe: of course then the juju-release-tools check that no *extra* packages are downloaded, it causes that Extand directories error12:47
jcw4Extant even12:47
rogpeppehmm, i wonder what the best way is of just fetching a repo without its dependencies12:47
jcw4rogpeppe: I can't think of a way to work around this other than to relax the juju-release-tools check12:48
mgzrogpeppe, jcw4: I should probably email the list explaining,12:48
mgzbut I think we can just fix12:48
jcw4mgz: I'm on the edge of my seat waiting for the rest of that comment12:49
mgzthe issue is go get pulls in tip of master and all dependencies of tip of master, which somewhat conflicts with the concept of having clear and well defined deps12:49
jcw4mgz: right12:49
mgzso, can either not use go get in godeps, or make the tarball creation delete extranious things go get went and got and trust the build to blow up if they did actually matter12:50
rogpeppemgz: yeah. i think godeps should probably avoid using go getr12:50
jcw4well I'm guessing we're more likely to 'fix' godeps than go get12:50
mgzrogpeppe: I had a brief go at that last night, it's a little annoying12:50
rogpeppemgz: yeah, there's a fair amount of logic in the go tool that would need duplication12:51
mgzbut is possible to add a Clone() to VCS and do our own mapping of how to get launchpad.net/ github.com/ etc12:51
rogpeppejcw4: well, go get is working as advertised, so it doesn't really need fixing12:51
mgzthe dep collection is at least already there and could be resused12:52
rogpeppejcw4: except i guess to add another flag to specify that deps should not be downloaded12:52
mgzyeah, that's sort of what we want12:52
jcw4rogpeppe: I wonder if we used tags instead of head revisions in the import url12:52
mgzjcw4: we still have the problem that go get always gets master12:53
rogpeppejcw4: go get does what it does. no way to specify tags to go get12:53
mgznot tags12:53
jcw4rogpeppe, mgz I thought if the import path ended with .something go get looked for a something tag to check out12:53
rogpeppejcw4: i don't think so12:53
jcw4hrm - I would expect go get would want to be more deterministic too12:54
mgzrogpeppe: I think, as a fix for now, I'm going to delete unknown dirs, and trust that things will break12:55
rogpeppemgz: i'm not sure that's quite in the spirit of that check12:55
rogpeppemgz: isn't the point of that check to make sure that we include godeps info for all our deps?12:55
mgzyes12:55
rogpeppemgz: ah, i see12:56
rogpeppemgz: sgtm12:56
rogpeppemgz: if we're really dependent on it, then it should break12:56
jcw4+112:56
mgzick... the script is so nice and safe at present13:03
mgzmaking in rmtree stuff is a little scary13:03
jcw4mgz: lol13:03
jcw4mgz, rogpeppe would using gopkg.in-like urls for dependencies avoid this issue?13:04
rogpeppejcw4: no13:04
mgznah13:04
jcw4:(13:04
rogpeppemgz: please let me know when i can try to re-land that branch...13:12
mgzrogpeppe: will do13:12
mgzrogpeppe: done change, writing some tests before proposing/landing13:20
=== mgz is now known as wgz
=== mgz_ is now known as mgz
rogpeppemgz: did you manage to get those changes in?15:35
marcoceppi_what's the configuration option to disable apt-get upgrade on add machine and is it in 1.20 ?16:14
jrwrenmarcoceppi_: enable-os-upgrade: false    i'm pretty sure its not in 1.20.16:17
* marcoceppi_ raises fists to the sky, falls to his knees, and yells NOOOOOOOOOOOOOOOO16:17
jrwrenmarcoceppi_: There is master. It can be your friend.16:18
marcoceppi_sure, run a demo using tip of tip, what could go wrong16:18
mbruzekmarcoceppi_: https://bugs.launchpad.net/juju-core/+bug/135049316:19
mupBug #1350493: 1.20.x local provider not running apt-get update <charms> <regression> <juju-core:Fix Released by cox-katherine-e> <https://launchpad.net/bugs/1350493>16:19
thumpermorning folks19:41
jcw4hi thumper19:41
jcw4waigani: I have a question about state localID and docID19:42
waiganijcw4: go for it19:42
jcw4:)19:42
thumperwaigani: still working?19:42
* thumper looks around for fwereade19:42
jcw4does it make sense to do a prefix check on docID like there is in localID?19:42
waiganithumper: i need to talk to you about the watcher and localID/docID19:43
jcw4I'd like to be able to safely pass an environment prefixed id to docID19:43
waiganioh right19:44
jcw4I could just do docID(localID(id))19:44
jcw4but ...19:44
waiganino, ugly19:44
jcw4yep19:44
waiganiwhy do you need to docID() if it already has the prefix?19:45
waiganii'm guessing because you are not sure if it has it or not19:45
jcw4waigani: I have a method that might be called with either19:45
jcw4yep19:45
waiganii would argue that the code would be easier to read if you did the check in that method19:45
jcw4waigani: okay, that's plausible19:46
thumperwaigani: did you want a hangout?19:46
jcw4tx waigani19:46
waiganithat way, it is clear that the method handles an ambiguous id19:46
jcw4good point19:46
waiganiwhich, i for one, would be glad to spot when reading your method19:47
jcw4interesting19:47
jcw4so... if the method today does docID(id)19:47
waiganithen I'd assume that id is a localID19:47
waiganino further checking needed19:47
jcw4okay.  Thanks again19:48
waiganii.e. do the prefix check in your method19:48
waiganithumper: okay19:48
jcw4yeah19:48
waiganithumper: hangout channel?19:48
thumperwaigani: you mean standup hangout?19:49
waiganilol - yeah, sigh19:49
waiganiit's late19:49
=== jcw4 is now known as jcw4|afk
thumpercmars: hey, you around?20:58
=== perrito6` is now known as perrito666
=== thumper is now known as thumper-dogwalk
=== wallyworld_ is now known as wallyworld
=== uru_ is now known as urulama
=== thumper-dogwalk is now known as thumper
=== jcw4|afk is now known as jcw4

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