/srv/irclogs.ubuntu.com/2017/03/03/#juju-dev.txt

thumperveebers: what do we need to do to add another repo into the bot landing?00:18
veebersthumper: as in a dep?00:24
thumperveebers: no, as in github.com/juju/description00:25
veebersthumper: I would have to find out00:26
veebersthumper: I believe we would need a new jenkins job and to update some scripts/cron for that. I would have to dig further to get a full answer00:29
blahdeblahthumper: Re: lp:1669046 - we have tickets in already to update all of our standalone controllers to 2.1.0.00:29
blahdeblahI'll create one shortly to get our HA controller env upgraded, but it may take a little longer - are there any special precautions/procedures for an HA upgrade?00:29
blahdeblahthumper: Also, if you need anything more on that 1.x to 2.x upgrade preparation, a reply to the ticket I created would be appreciated; gives me a place to log time worked on it.00:35
thumperblahdeblah: ack00:40
thumperhmm...00:50
thumpersublime not doing well with this global rename00:50
thumperdoing save all on 5000 different go files causes it's plugins to work overtime00:51
* thumper twiddles fingers waiting00:51
thumperoh bollocks00:53
wallyworldbabbageclunk: i just noticed we have remoteApplicationsC in the ignored collections in migration_internal_test.go00:54
thumpermanaged to somehow end up with submodules00:54
babbageclunkwallyworld: oh, whoops - I'll fix that00:55
wallyworldbabbageclunk: no wuckers. i noticed because i'm about to delete an entire collection and associated data model we won't be using anymore00:56
wallyworldno rush on it - get the spaces worker stuff done so i can test it out :-)00:57
babbageclunkwallyworld: ok - still trying to track down what I've broken.00:57
wallyworldjoy00:57
thumper11000 changes across 2800 files00:58
thumperyeah... do it00:58
babbageclunkwallyworld: I mean, I've worked out the line that's done it, but I don't understand why. Trying it again now01:00
wallyworldlet me know if you need a 2nd set of eyes01:00
babbageclunkwallyworld: will do, thanks01:01
blahdeblahthumper: So, re: the precautions/procedures for HA upgrade question?01:06
axwwallyworld: sorry, https://github.com/juju/juju/pull/7047 should have been closed. I have another branch which supersedes it01:10
wallyworldno worries, i thought that was the case but then i second guessed myself and reviewed anyway. but yeah, it didn't seem to match the latest discussion01:11
babbageclunkwallyworld: ok, worked it out - the connection's dropping because of a panic - nil pointer dereference. :( fixing!01:11
wallyworldi hate those01:11
wallyworldbabbageclunk: what that in a unit test?01:11
wallyworldthat's why i added the stacktrace to catacomg errors01:12
wallyworldcause i had a similar issue that was hard to diagnse where it was happening01:12
axwwallyworld: well actually it's the same branch but updated. pushing now, will address comments and ping you later01:12
wallyworldnp01:12
axwbleh nope, I'm wrong. closing and will create a new one01:16
thumperblahdeblah: shouldn't be any different01:22
blahdeblahthumper: Cool - so we don't need to upgrade each node separately or anything like that?01:23
thumpernope01:24
blahdeblahsweet01:26
blahdeblahthanks01:26
axwwallyworld: https://github.com/juju/juju/pull/7066. I'm running QA now01:39
wallyworldok, otp with tim01:39
babbageclunkwallyworld: Oops, ListSubnets doesn't like blank space names either - I think there are going to be a few of these.01:54
wallyworldyeah01:54
veebersdid something in 2.2 change for Azure? I see a _bunch_ of test failures in azure with errors like "'Failed': Code="AuthenticationFailed" Message="Authentication failed.""02:11
babbageclunkveebers: Sounds like authentication failed.02:15
babbageclunk:)02:15
veebersbabbageclunk: ah an interesting take on an error message :-\02:16
babbageclunkveebers: In non-Friday-afternoon-japes mode, not that I know of.02:16
veebersbabbageclunk: thing is this is happening during a bootstrap02:17
veebersbabbageclunk: ack. I still need to pack for the sprint, leaving the house early tomorrow :-*(02:17
babbageclunkveebers: ouch, I'm not leaving until Sunday afternoon.02:18
veebersbabbageclunk: ah, and arriving Sun evening right? I'm hoping that arriving Sat evening I lessen jetlag and hit the ground running on Monday02:19
babbageclunkveebers: yeah, shake off some of that jetlag on Bourbon St, I hear you.02:19
babbageclunk;)02:19
veebersbabbageclunk: ^_^02:20
babbageclunkwallyworld: ugh, segfault whackamole02:22
babbageclunkwallyworld: actually, this one isn't a segfault, just the subnets command (calling ListSubnets which I just fixed) now complains about blank spaces.02:24
babbageclunkwallyworld: rather than giving subnets a blank space, maybe we should be giving them a special space name? I feel like we're going to encounter knock-on effects from this.02:33
babbageclunkwallyworld: Although I'm not sure about what the special space name would be.02:34
wallyworldbabbageclunk: sorry have been otp till now02:34
babbageclunkwallyworld: no worries, mostly thinking out loud.02:35
wallyworldbabbageclunk: IIANM it is the intent to use "" as the default space rather than a const "default" or whatever02:35
wallyworldwe will just have to solve the whack a mole sadly02:35
wallyworldbabbageclunk: i need food, will be biab02:36
babbageclunkwallyworld: ok, lucky I hate those pesky moles.02:36
babbageclunkwallyworld: given that ListSubnets might now return a blank space tag, should I make a new facade version?02:47
babbageclunkwallyworld: review plz? https://github.com/juju/juju/pull/706703:09
wallyworldbabbageclunk: reviewed03:54
wallyworldaxw: reviewed - IMO we need to include Type in list storage output04:06
axwwallyworld: should be obvious from the pool?04:07
axwand/or provider ID?04:07
wallyworldperhaps, but not guaranteed to be obvious? the pool name might not reflect the type so well? same with provider id which could well be opaque04:08
wallyworldpool names  can be defined by the user04:08
axwwallyworld: true, but I would think the operator would know what they've set up04:08
wallyworldbut it is not the operator who always runs list-storage04:08
axwwallyworld: I'm reluctant because it's going to add a fair bit to the width04:09
wallyworldthink of a support person or other user who did not do the original deploy04:09
wallyworldthe width of the table?04:09
axwwallyworld: yeah04:09
wallyworldjuju status is not exactly un-wide04:09
axwwallyworld: indeed, but one of the things we did in 2.0 was to cut it down04:09
axwcut down size of instance IDs, etc.04:10
axwto avoid overflowing terminal width by default04:10
wallyworldsure. but "volume" or "fiesystem" is not that much extra04:10
axwwallyworld: actually you can't tell from the pool alone anyway, since you couldh ave a filesystem on EBS or whatever04:11
wallyworldtrue04:11
axwI'll look at adding it in04:11
wallyworldI do think we need it TBH. ty04:11
wallyworldwe can always remove if people complain, but i honestly don't think they will04:12
axwwallyworld: https://github.com/juju/juju/pull/7066/commits/fa75bb44923f8aafaf70a8c7615d64dafb494ea004:51
wallyworldlooking04:51
wallyworldaxw: awesome, thanks04:52
axwnp04:52
blahdeblahAnyone who's worked on the juju2 user permissions subsystem around?  Just want to ask some questions before I log some seemingly rather silly bugs.06:12
blahdeblahAsking anyway... :-)06:13
anastasiamacblahdeblah: 2.0.x or 2.1.x?06:13
anastasiamacthere were some changes...06:13
blahdeblahanastasiamac: 2.1.0 stable!06:13
anastasiamacblahdeblah: ask away.. there may not b answers that make ur day :)06:14
blahdeblahSo, Q1: is it expected that superusers are exempt from being disabled?  (Because in my testing, they are.)06:14
blahdeblahTo me this is rather unexpected.06:14
anastasiamacblahdeblah: yes. i believe it's so tat someone can destroy and kill06:16
anastasiamacblahdeblah: maybe that logic needs to be adjusted to not disable if it's the last one06:16
anastasiamacblahdeblah: we have a discussion at sprint next week about user mngmt... so i can bring it up, especially if it's bugged up :)06:17
blahdeblahOK, so then the bug I will logging is that when you disable a superuser, it gives no indication that your command did absolutely nothing.06:18
blahdeblah(although it doesn't do nothing; it does actually mark the user as disabled, and it shows this way in list-users --all)06:18
blahdeblahanastasiamac: Q2: Is it expected that non-superusers which are disabled and then re-enabled lose all of their grants to various models?  To me this is also rather unexpected.06:23
blahdeblahWhen I disable a user and then re-enable that same user immediately (say, because I typoed the user name and disabled the wrong one), I don't want to have to work out which models were affected.06:24
anastasiamacblahdeblah: i *think* this is expected as you want to b explcit about ur grants when re-enabling a user06:25
anastasiamacblahdeblah: but again, it something to discuss06:25
blahdeblahNo, I don't.  I want them to go back exactly as they were. :-)06:26
anastasiamack \o/ file it and i'll see what we do06:26
blahdeblahOK - thanks06:26
axwwallyworld: do you recall if CI/builders are using go 1.7 to build already? or still on 1.6?06:57
wallyworldaxw: still 1.6 AFAIK07:01
axwbugger07:01
wallyworldi could be wrong but i think it's 1.607:03
wallyworldis there a 1.7 feature you want to use?07:03
axwwallyworld: yeah, http request context07:03
axwwallyworld: 1.8 would be better, since that has graceful shutdown/close methods on server07:03
wallyworldis 1.8 officially out yet? last i heard it was still rc?07:04
axwwallyworld: https://blog.golang.org/go1.807:05
wallyworldi/m 2 weeks out of date07:05
axwwallyworld: also I suspect sort.Slice could be useful in a few parts of the codebase...07:05
wallyworldindeed07:05
wallyworldwe'll have the right people necxt week to discuss07:06
wallyworldi'll add it to the agenda07:06
axwwallyworld: cool, thanks07:07
axwIIANM we can use whatever we like now, and are not dependent on archives07:07
wallyworldyeah, i think so too07:09
=== frankban|afk is now known as frankban
mupBug #1669729 opened: Cannot update machine data <canonical-bootstack> <juju-core:New> <https://launchpad.net/bugs/1669729>10:02
SimonKLBany idea why i can't switch to a model that i have never connected to using a new user in juju?10:36
SimonKLBi can see the model in `juju models`10:36
SimonKLBbut `juju switch aws:default` just spits out 'ERROR "aws:default" is not the name of a model or controller'10:37
SimonKLBlooks like there is some problem at the controller level even10:38
babbageclunkSimonKLB: is your controller called aws?10:38
SimonKLBbabbageclunk: yea10:39
SimonKLBadded a new user, got the register base64 string, added it to the new user and named the controller aws10:39
babbageclunkSimonKLB: maybe something's getting confused between that and the aws provider10:39
SimonKLBill retry with a more unique name10:40
babbageclunkSimonKLB: It's a guess, sorry.10:40
babbageclunkSimonKLB: Could also try passing --debug to  `juju switch`10:41
SimonKLBbabbageclunk: do you have to re-do the add-user process, because now the second time i use the register string i get 'ERROR secret key for user "X" not found'10:41
SimonKLBor can i genenrate the string again without having to do add-user?10:42
babbageclunkSimonKLB: I don't think you can reuse the register string (I'm not certain of that though) - try adding another user?10:43
SimonKLByea will do10:43
SimonKLBbabbageclunk: same thing using "mycontroller" as the name10:45
SimonKLBnothing stands out in the --debug output10:45
babbageclunkSimonKLB: you can see the model in `juju models`?10:46
babbageclunkSimonKLB: did you grant access to the user?10:46
SimonKLBbabbageclunk: yea i granted access10:47
SimonKLBhttp://paste.ubuntu.com/24101356/10:47
babbageclunkSimonKLB: Try `juju switch mycontroller:admin/default`?10:48
SimonKLBdoh...10:48
SimonKLBthanks man haha10:48
babbageclunkSimonKLB: hmm, I don't think the admin/ should be necessary, since just default is not ambiguous.10:49
SimonKLBany idea why its named admin/default there and not elsewhere?10:49
SimonKLBwell it worked using admin/ but not without :O10:49
babbageclunkSimonKLB: It's admin/default because it's owned by admin.10:50
SimonKLBaaah10:50
SimonKLBmake sense10:50
babbageclunkSimonKLB: It's still pretty confusing though - can you file a bug? Or I will.10:51
SimonKLBlaunchpad or github?10:51
perrito666morning10:51
SimonKLBnvm launchpad it is!10:54
babbageclunkSimonKLB: yes launchpad please!10:54
babbageclunkmorning perrito666!10:55
perrito666babbageclunk: coming to the sprint?10:55
babbageclunkperrito666: yup yup, assuming I get into the country10:55
perrito666babbageclunk: yes, we are all under the same assumption, I dont thing the place for the sprint was very wisely chosen :p10:55
perrito666we go to uk after the brexit and now to the US, who is pickig the destinatios?10:56
babbageclunkperrito666: political storm-chasing10:57
perrito666heh10:57
SimonKLBbabbageclunk: https://bugs.launchpad.net/juju/+bug/166974511:06
mupBug #1669745: Model name different for model owner and non-owner might confuse new users <juju:New> <https://launchpad.net/bugs/1669745>11:06
babbageclunkSimonKLB: Nice write-up, thanks! I've tagged it with usability.11:07
=== frankban is now known as frankban|afk
=== marcoceppi_ is now known as marcoceppi
=== frankban|afk is now known as frankban
rogpeppe1if anyone cares to take a look, i've just put a PR up for review of a change to gopkg.in/juju/worker.v1. my aim is to use it to replace a bunch of the state/workers.RestartWorker stuff: https://github.com/juju/worker/pull/217:15
=== rogpeppe1 is now known as rogpeppe
=== ejat_ is now known as ejat
=== Dmitrii-Sh_ is now known as Dmitrii-Sh
=== frankban is now known as frankban|afk
rediri saw perrito666 new ride today: https://goo.gl/NDtqUN21:20

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