/srv/irclogs.ubuntu.com/2016/09/22/#juju-dev.txt

perrito666redir: love your abbreviations00:01
redirtyvm, perrito66600:03
redirsorry i am a lazy typist00:03
perrito666lol00:03
perrito666does Maas knows how to upgrade itself?00:03
mupBug # changed: 1312290, 1321212, 1339866, 1353242, 1357760, 138134000:49
rediraxw: updated pr from yesterday. It is a bit different so ptal https://github.com/juju/juju/pull/629701:36
axwredir: ok, looking01:36
axwredir: reviewed01:41
rediraxw: tx01:45
=== natefinch-afk is now known as natefinch
thumperah ha02:02
thumperthink I have found why cert updater tests fail intermittently02:02
thumperwell, getting to the bottom of it anyway02:02
thumperugh02:16
thumpermenn0: got a minute?02:16
menn0thumper: now I do02:37
thumperquick HO?02:38
menn0yep02:38
menn0axw: would you mind taking another look at this one please: https://github.com/juju/juju/pull/630203:05
menn0axw: i've added an upgrade step now03:05
axwmenn0: sure, looking03:05
axwmenn0: LGTM with QA, which I see you're doing now03:09
menn0axw: cheers03:10
menn0pop quiz... what's wrong with this in the bootstrap output?03:10
menn0Launching controller instance(s) on dev...03:10
menn0 - juju-0facb9-0                                                                    00%03:10
thumpermenn0: https://github.com/juju/juju/pull/630303:10
menn0looking03:11
=== rmcall_ is now known as rmcall
thumpermenn0: are you?03:42
menn0thumper: sorry, got distracted03:43
menn0thumper: done03:46
thumpermenn0: quick hangout do discuss something?03:52
menn0thumper: ok03:53
mupBug #1626304 changed: Unit disk fills with inprogress-n files <juju-core:Invalid> <https://launchpad.net/bugs/1626304>04:19
mupBug #1626304 opened: Unit disk fills with inprogress-n files <juju-core:Invalid> <https://launchpad.net/bugs/1626304>04:25
mupBug #1626304 changed: Unit disk fills with inprogress-n files <juju-core:Invalid> <https://launchpad.net/bugs/1626304>04:31
gennadiyhi, we have private openstack cloud and we want to use different subnetwork for deployed software. is it possible to do it with juju 2.0 ?05:23
menn0thumper: here's the formatting fix: https://github.com/juju/juju/pull/630405:26
* thumper looks05:27
thumperlgtm or at least good enough05:28
thumpergennadiy: not yet I don't believe05:47
thumpergennadiy: we will have spaces and subnets for openstack soonish, but I'm not sure on the timeline05:48
thumpernight all05:48
=== frankban|afk is now known as frankban
MmikeHi, lads. I need help in creating a password hash for the unit/machine agents that is stored in the mongodb database. On a fairly complex customer environment I lost /var/lib/juju directory. I was able to recreate everything, and jujud-s start without an issue, I just had to use passwords/hashes from some other unit - but now I ended up with same passwords for two units.08:16
MmikeIs there a simple way generate both the password and the hash?08:16
MmikeThis is juju 1 (1.20, we are prepping the customer for upgrade to 1.25)08:16
MmikeErm, hi to ladies too!08:16
frobwarevoidspace: ping - can we sync regarding the MAAS bug we saw yesterday08:45
voidspacefrobware: we can08:47
frobwarevoidspace: standup HO08:47
voidspacefrobware: the ipv6 bug I'm on has turned critical as well :-/08:47
frobwarevoidspace: in my experience that category is now quite low. :)08:48
voidspacefrobware: I'm in the HO08:48
voidspacefrobware: category?08:48
anastasiamacvoidspace: critical because the bug that u said is the same was critical ;)08:49
voidspacemgz: ping09:00
frobwarevoidspace: the logs do have TRACE info09:02
voidspacefrobware: ah yes, odd09:02
voidspacefrobware: maybe they hadn't loaded that far when I searched!09:02
voidspacefrobware: I see them now, sorry09:02
rogpeppe1anyone fancy a review of a little addition to juju/testing? (factored out from somewhere else we've been using it for while): https://github.com/juju/testing/pull/11109:17
=== rogpeppe1 is now known as rogpeppe
rogpeppeaxw: ^09:18
axwrogpeppe: I was a bit slow, but LGTM anyway09:22
rogpeppeaxw: tyvm - i always like to have someone from core sign off on juju/* PRs09:23
babbageclunkAm I in the wrong place for the core team meeting?10:05
babbageclunkjuju core team meeting? Is it not on today?10:08
babbageclunkfrobware, voidspace ^10:08
voidspacebabbageclunk: we're in a hangout on a critical issue - so we can't make it10:09
voidspacebabbageclunk: if there's no-one else there, then it probably aint happening :-(10:09
babbageclunkOh yeah, I thought you guys might still be working on that. Weird that no-one is there.10:10
=== mup_ is now known as mup
frobwarevoidspace: can you join the HO again?11:48
frobwarevoidspace: scrap that - heading out for a bit12:04
voidspacekk12:13
voidspacefrobware: so it's the provisionerTask that should be creating the machines and that's using a straightforward machines watcher12:39
voidspacefrobware: interestingly the "process machines with transient errors" call is from the provisioner task loop12:39
voidspacefrobware: and only happens when there is something on the retryChanges notify channel12:40
voidspacefrobware: I can't see in the logs a *failed* attempt that would cause that though12:40
voidspacefrobware: I'm still digging through the code and the logs12:40
voidspacetaking a break12:40
jamespageurgh13:28
rick_h_jamespage: that good eh?13:29
jamespagerick_h_, soooo13:29
* rick_h_ runs away13:29
jamespagerelations at the bottom of status13:29
jamespagemmm13:29
rick_h_jamespage: yes, what's up?13:30
jamespagethe side effect of that change is that if I type juju status, I just see the list of relations13:30
mgzjamespage, did we break you somehow?13:30
jamespagefor an openstack env with lots of them13:30
jamespagemgz, just feels odd13:30
rick_h_jamespage: yea, the thing was that folks doing status tend to either watch (so most improtant/changing stuff up top) or less.13:31
rick_h_jamespage: and having the things that have status/notes/etc together seemed appropriate13:31
jamespagehttp://paste.ubuntu.com/23215829/13:31
jamespagemaybe dropping them from the tabular view althogther might be better13:31
rick_h_jamespage: so that was a thought, but because it's directly part of the "topology" held that back13:32
rick_h_e.g. two status's are very different with the relations set for something vs not set13:32
jamespageside effect is if not using watch, then a juju status cli type results in most of the useful information about 3" above the top of my monitor13:32
rick_h_so status seems "all good" as far as machines deployed/applications running but still not be useful13:32
rick_h_jamespage: right, it's a trade off one way vs the other way13:32
rick_h_We emailed the juju list, asked for feedback, and wanted to try this out.13:33
jamespagerick_h_, yeah - sorry I did read that, but had my head in other stuff at the time13:33
rick_h_jamespage: understand, this is fine. Sometimes you don't know until you try it13:33
jamespagerick_h_, btw that pastebin has the version stuff we just added in13:34
rick_h_jamespage: <3 ty13:35
jamespagerick_h_, one comment is that its a bit lossy13:35
jamespagerick_h_, as its possible to get different versions on different units during transitions13:35
jamespageand if someone does something wonky by hand13:36
rick_h_jamespage: yea, that was something we debated about that feature13:36
rick_h_jamespage: we wanted to treat it more like resources where you get what it "should" be (by what most folks think it is) and a way to ask the units13:37
rick_h_jamespage: but that got shot down13:37
natefinchmaybe a flag for --relations or --no-releations depending on the default13:50
natefinchmaybe same for machines... machine info is fairly boring most of the time13:51
natefinchalso, we obviously need to do something about ports13:52
natefinchwow, the relation data is just static? There's no state there at all?  I agree, that shouldn't be in status at all.13:53
babbageclunkYou really want an ascii-art diagram of the relations.13:58
babbageclunk(only half joking)13:58
rick_h_natefinch: katco macgreagoir voidspace ping for standup14:00
rick_h_mgz: ^14:00
mgzta14:00
voidspaceomw14:01
alexisb_babbageclunk, you around still?14:26
mupBug #1544796 opened: Backup restore fails: upgrade in progress <2.0-count> <backup-restore> <blocker> <ci> <regression> <juju:Fix Released by cherylj> <juju-core:Triaged> <juju-core 1.25:New> <https://launchpad.net/bugs/1544796>14:35
mupBug #1626576 opened: credential v. credentials is confusing <juju-core:New> <https://launchpad.net/bugs/1626576>14:35
perrito666babbageclunk: could you tell me what do you mean by provider/maas/maas2instance.go L68 ?14:52
babbageclunkperrito666: Sure - looking at it now14:53
perrito666tx14:53
mupBug #1544796 changed: Backup restore fails: upgrade in progress <2.0-count> <backup-restore> <blocker> <ci> <regression> <juju:Fix Released by cherylj> <juju-core:Triaged> <juju-core 1.25:New> <https://launchpad.net/bugs/1544796>14:53
babbageclunkperrito666: I don't remember writing that at all, but from comparing with the v1 api version it looks like it's right - that code should rerequest the status from the controller.14:59
perrito666babbageclunk: heh, ok14:59
babbageclunkperrito666: :( hope that's not the source of the problems voidspace and frobware have been chasing.15:00
perrito666babbageclunk: the problem being?15:00
babbageclunkperrito666: marcoceppi's problem where the controller was taking a long time to deploy machines15:02
mupBug #1544796 opened: Backup restore fails: upgrade in progress <2.0-count> <backup-restore> <blocker> <ci> <regression> <juju:Fix Released by cherylj> <juju-core:Triaged> <juju-core 1.25:New> <https://launchpad.net/bugs/1544796>15:02
perrito666now mup, I find that hard to believe15:03
perrito666babbageclunk: mm, could be, I have a problem like that when the deployment fails in maas15:04
mupBug #1544796 changed: Backup restore fails: upgrade in progress <2.0-count> <backup-restore> <blocker> <ci> <regression> <juju:Fix Released by cherylj> <juju-core:Invalid> <juju-core 1.25:Triaged> <https://launchpad.net/bugs/1544796>15:05
natefincheasy review anyone? https://github.com/juju/juju/pull/630615:23
perrito666you are asking if the review is easy?15:23
natefinchno15:24
natefinchI'm asking you to go look at the code and review it :)15:24
katconatefinch: i'm tal15:25
perrito666I am not tal, its a hedious templating language but ill take a look anyway15:25
katcobeen wanting to try a GH review :)15:26
katcois there still no way to select multiple lines for a comment?15:28
rick_h_katco: no, don't think so15:29
mupBug #1544796 opened: Backup restore fails: upgrade in progress <2.0-count> <backup-restore> <blocker> <ci> <regression> <juju:Fix Released by cherylj> <juju-core:Triaged> <juju-core 1.25:Triaged> <https://launchpad.net/bugs/1544796>15:29
katconatefinch: review up15:33
katconot being able to highlight lines for context is unfortunate. one of my comments appears to be about 4 lines of code when i intended it to be about the entire function15:33
katcois there some sort of best practice around that?15:34
katcowow that bothers me way more than i thought. it kind of leaves the author guessing which lines are being discussed15:36
rogpeppenatefinch: juju is using Go 1.6 now, right?15:36
rick_h_rogpeppe: yes15:37
rogpepperick_h_: thanks15:37
rogpepperick_h_: just wanted to make double sure15:37
katcohow do you differentiate between a suggestion and a request in a github review?15:40
* katco thinks she better go read their tutorial15:41
rick_h_katco: if it says please?15:41
katcolol15:41
rogpeppetrivial (but important) review, please: https://github.com/juju/utils/pull/23915:42
rogpeppekatco: have they fixed it so you can comment on lines that aren't changed now?15:44
katcorogpeppe: you are asking the wrong person15:44
katcorogpeppe: i have no idea15:44
rogpeppekatco: :)15:44
rogpeppekatco: i'll have to try it15:44
katcorogpeppe: looks like no?15:44
rogpeppekatco: it always annoys me when i want to say "that code you added over there should be here", but "here" wasn't somewhere that was changed15:45
katcoyeah... wow that is not ideal either15:45
natefinchoh weird, yeah, it's not whether or not it's been changed, just whether or not it's in the window that shows what's changed15:46
katconatefinch: but if i expand the file that's changed i can't even comment outside the diff i don't think15:47
natefinchright yeas15:47
natefinchsorry, there's the initial preview with a little extra context, anything in that preview window you can comment on, changed or not.  Anything outside that, even when expanded, you can't.15:47
natefinchkatco: I made the suggestion in the email thread about marking issues that need to be addressed with :x: which makes a nice big visible red X in the text.  It's not perfect, but it seems sufficient.15:52
katconatefinch: i agree that would be an easy convention. i am always super hesitant to add conventions to teams bc they introduce a cognitive load, and only so much of that can be maintained.15:53
natefinchkatco: I know... I wish that feature existed in the review system, but I don't think it's a huge cognitive load, especially since we should all be doing reviews all the time, and should become pretty automatic.  Not really much more difficult than remembering to check "comment represents an issue" or whatever it's called.15:57
mupBug #1626626 opened: juju status outputs ERROR "" is not a valid tag <juju-core:New> <https://launchpad.net/bugs/1626626>16:09
natefinchkatco: trying to figure out how to properly replace the existing notfound error with my own notfound error... it's unfortunate we wrote our errors package in the way we did, just doing type assertions16:09
natefinchkatco: maybe Wrap is what I'm looking for16:10
natefinchkatco: yeah, wrap seems to have the correct behavior16:14
katconatefinch: i know i always confuse them.16:16
natefinchkatco: I had to actually write a little program to make sure that wrap didn't munge the old error's text in with the new one.16:19
natefinchoops... I'm going to have to get used to not doing commit --amend during reviews16:25
rick_h_natefinch: do you still have your PR in review?16:26
natefinchrick_h_: yep16:26
rick_h_natefinch: can you please create a second PR for that change but from your branch to the juju/develop branch ?16:27
katconatefinch: huh yeah, it's not really easy to tell what has changed in response to a review16:27
natefinchrick_h_: for the error message change when there's no current controller?16:27
rick_h_natefinch: yes please16:27
natefinchrick_h_: sure thing16:27
rick_h_natefinch: ty16:28
natefinchkatco: it might be easier if I didn't force push an amended commit.  That's something we'll ahve to look at16:28
natefinchbrb16:28
* rick_h_ goes for lunchables16:30
bdxhow's it going all?16:43
bdxis there a doc that details what ports and protocols need be allowed for client <-> controller, and agent <-> controller communication?16:45
bdxI'm having to write up an application network and security spec for a client16:46
=== frankban is now known as frankban|afk
bdxwhat I'm seeing is client <- tcp 22/17070 -> controller, and agent <- tcp 22/17070 -> controller17:09
bdxso 22 and 17070 are all that need be allowed between both sets?17:09
rick_h_bdx: sounds right17:11
mupBug #1626626 changed: juju status outputs ERROR "" is not a valid tag <juju-core:Invalid> <https://launchpad.net/bugs/1626626>17:45
bdxrick_h_: thanks18:01
bdxI know this is a long shot, and I think I know the answer, but may as well ask ... does juju have anyway to deploy time based instances?18:02
bdxon AWS18:03
rick_h_bdx: no, you have to script that18:04
bdxrick_h_: would it be a better practice to use the manual provider, and add machines that have been manually provisioned as time based?18:05
rick_h_bdx: maybe we're not thinking the same thing. What do you mean by time based?18:05
bdxrick_h_: something we currently use to save on our AWS bill - time based instances can be scheduled to turn off durring non use hours18:06
rick_h_bdx: oic, so it's an instance type? /me goes to look at aws docs18:07
rick_h_oh, not really an instance type, but a scaling type config on an instance layer18:08
bdxrick_h_: yea -> https://postimg.org/image/uc03ismnf/18:09
rick_h_bdx: interesting, a manual prover type situation may work?18:09
rick_h_the fun part will be how juju handles units showing up/going away like that18:09
rick_h_bdx: especially in relations, where juju epxects there to be these units talking to those units and they disappear18:09
bdxthe boxes I will be manually provisioning will be lxd staging silos18:10
rick_h_bdx: since nothing there will be updating the units in the relation lists in juju18:10
bdxso they wont even have any charm deployed to them18:10
rick_h_oic18:10
bdxI'm thinking I will just use juju to grant devs access to their respective instances that will each have the lxd non prod envs on them18:12
bdxseems to be the best option for my use case of containerizing our non-prod envs18:13
bdxstill seems slightly shiesty18:14
bdxha18:14
bdxI think it will work though18:14
alexisbanastasiamac, rick, tim and I will be on another call and will not be at the bug scrub20:51
anastasiamacalexisb: \o/ enjoy20:52
rick_h_oh wheeee20:52
mupBug #1587644 opened: jujud and mongo cpu/ram usage spike <canonical-bootstack> <canonical-is> <eda> <performance> <juju:Fix Released> <juju-core:Triaged> <juju-core 1.25:Triaged> <https://launchpad.net/bugs/1587644>21:03
anastasiamacand bug scrub time? :D21:05
mupBug #1587644 changed: jujud and mongo cpu/ram usage spike <canonical-bootstack> <canonical-is> <eda> <performance> <juju:Fix Released> <juju-core:Triaged> <juju-core 1.25:Triaged> <https://launchpad.net/bugs/1587644>21:15
=== hml_ is now known as hml
mupBug #1587644 opened: jujud and mongo cpu/ram usage spike <canonical-bootstack> <canonical-is> <eda> <performance> <juju:Fix Released> <juju-core:Triaged> <juju-core 1.25:Triaged> <https://launchpad.net/bugs/1587644>21:18
babbageclunkmenn0: hi! can I pick your brains some more please?21:22
menn0babbageclunk: sure21:22
menn0babbageclunk: hangout or IRC?21:23
babbageclunkmenn0: hangout might be quicker21:23
babbageclunkhttps://hangouts.google.com/hangouts/_/canonical.com/mongo-stuff?hl=en&authuser=121:23
babbageclunkmenn0: ^21:24
veebersthumper: would you know why when attempting a juju upgrade, this message states "juju.environs.sync sync.go:333 using agent binary 2.0-rc2-xenial-amd64 aliased to 2.0-rc2.2-xenial-amd64" why is it using patch version .2, I expected it to use .1. Any insight?22:11
thumpernope, sorry22:12
thumperwell22:12
thumperthe existing one probably uses .1 right?22:12
veebersthumper: the existing was rc1 binary, model version stated rc1.1, but this is using rc2 binary (and using lxd so I was expecting rc2.1)22:14
thumperhmm22:14
thumpernot sure sorry22:14
veebersthumper: any idea who might be able to shed some light?22:15
thumperheh, wallyworld, but he isn't around22:15
thumperI'm not sure who reviewed those changes, perhpas axw?22:15
veebersthumper: cool, I spoke with him a little yesterday. I'll pester him again today :-)22:16
axwveebers: if the client is .1, when you go to upgrade it'll auto-increment to .222:18
axwveebers: (because .1 is not in streams)22:18
veebersaxw: so it'll go from rc1.1 -> rc2.2 ?22:20
axwveebers: sorry, didn't notice the change in RC. that doesn't seem right.22:21
veebersaxw: oh odd, this run through (each time a fresh bootstrap etc.) it's aliased it to .322:26
axwveebers:that makes no sense. the client is just plain old "rc2", right?22:27
veebersaxw: oh, I plugged '--version 2.0-rc2.2' onto upgrade-juju (previous run was --version 2.0-rc2.1)22:27
veebersaxw: boostrapped with rc1 binary, using rc2 binary to execute the juju-upgrade22:27
axwveebers: ok, that sounds a bit funky. if you specify a version we shouldn't be auto-incrementing anything22:28
alexisbaxw, ping22:30
axwalexisb: coming22:30
alexisbthanks axw22:49
axwnp22:49
axwalexisb: apparently it was wallyworld who added the default security group thing :/22:52
axwi'll ping martin anyway22:52
alexisbthat must have been post the provider work22:53
alexisbdid his PR state why it was added?22:53
alexisbI wonder if we can track it ot a bug22:54
alexisbit probably was a user request22:54
veebersaxw: I'm going to file a bug re: the juju-upgrade --version thing, I was able to manually recreate it too23:14
axwveebers: thnaks23:14
anastasiamacalexisb: sorry.. installing bluejeans plugin23:17
veebersaxw: fyi: https://bugs.launchpad.net/juju/+bug/162678423:36
mupBug #1626784: upgrade-juju --version increments supplied patch version <juju:New> <https://launchpad.net/bugs/1626784>23:36
axwmwhudson: apparently there's a meeting option: http://bluejeans.force.com/KnowledgeSearch/articles/Knowledge_Base/Mute-Participants-on-Entry-to-your-meeting/p23:42
axwalexisb: can you set that please? ^^23:42
mwhudsonaxw: ah awesome23:43
mwhudsonaxw: it was also the usual thing of "i know, i'll use the internal mic!" :(23:43
mupBug #1626304 opened: Charm GC in 1.25.6 leaves breakage behind <juju:In Progress by menno.smits> <juju-core:In Progress by menno.smits> <https://launchpad.net/bugs/1626304>23:49

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