/srv/irclogs.ubuntu.com/2016/11/14/#juju-dev.txt

redirglad everyone is in good health and spirits.00:08
rediranastasiamac: I'm at a loss to help that person with https://github.com/CanonicalLtd/jujucharms.com/issues/37200:10
redirhe was in asking for help on Friday.00:10
redirBut got beyond my depth00:11
voidspaceshort review for anyone who fancies it: https://github.com/juju/persistent-cookiejar/pull/1912:49
mgzvoidspace: hm, that is shorter than I expected12:52
voidspacemgz: heh, good12:52
voidspacemgz: hey, do you know how easy it would be to get a custom build of juju into a ppa?12:52
voidspacemgz: all juju tests pass with that branch of persistent-cookiejar, FYI12:53
mgzit's not super hard, but also of somewhat questionable use12:53
voidspacemgz: hah, I uploaded a custom binary for the reporter of the bug this addresses - and they don't know what to do with it and are asking for a ppa12:54
voidspacemgz: I've added another comment to the bug, but I've told them I'd try to get it into a ppa12:54
voidspacemgz: maybe I'll ask rick what he thinks at standup12:54
rick_hvoidspace: yea this is part of the snap request13:06
rick_hTo start to make it universal for folks testing13:06
voidspacerick_h: I'm looking at snapping it now, but I don't think they would know what to do with that either13:20
rick_hvoidspace: right, but we can start to get docs together for it. snap install it, remove cleanly.13:23
rick_hvoidspace: can you link me to the bug? /me is curious who we're talking about here13:23
voidspacerick_h: https://bugs.launchpad.net/juju-wait/+bug/163236213:23
mupBug #1632362: error during juju-wait ERROR cannot load cookies: file locked for too long; giving up: cannot acquire lock: resource temporarily unavailable <eda> <oil> <juju:In Progress by mfoord> <Juju Wait Plugin:Invalid> <https://launchpad.net/bugs/1632362>13:23
voidspacerick_h: lustostag13:23
voidspacerick_h: he's not online right now13:24
rick_hvoidspace: oh, it this larry from OIL?13:25
rick_hvoidspace: hmm, /me thought he should be able to handle the binary.13:25
voidspacerick_h: Greg Lutostanski13:25
voidspacethe bug was reported by Larry, no response from him13:25
rick_hvoidspace: ah ok, larry is on the bug to and should be able to work with it as well.13:26
voidspacek, no response yet - just digging into snapcraft as our snapcraft.yaml in the development tree fetches from github13:26
voidspacebut grabbing coffee first13:26
voidspacebrb13:26
voidspace(if I find Greg or Larry online later I'll chat to them)13:26
rick_hvoidspace: yea, please reach out to cmars or balloons if there's any snap help there.13:30
voidspacenatefinch: o/14:58
perrito666bbl lunch14:58
rick_hjujuteam standup please15:00
rick_hvoidspace: ^15:00
voidspacekk15:00
* rick_h needs to check if folks setup nick highlight15:00
katcorick_h: really difficult to estimate; errors hiding other errors =/ http://pastebin.ubuntu.com/23476038/15:17
rick_hkatco: ok, np. just wanted to see how to set expectations re: next line of work15:18
katcorick_h: do we still need our 1:1 given we met tr?15:23
rick_hkatco: up to you, I have gotten out my notes so it's available time for yourself if you want it15:24
katcorick_h: i don't have anything new15:24
rick_hkatco: k, I'll leave you to awesome fun test land then, but poke me with a stick if anything comes up15:25
katcorick_h: will do15:25
mupBug #1641643 opened: Boostrapping node fails to complete cloud init does to failure of System Logging Service <cdo-qa> <juju-core:New> <https://launchpad.net/bugs/1641643>16:39
redirrick_h: someone, rock_, was in looking for help on https://github.com/CanonicalLtd/jujucharms.com/issues/372 but I didn't know where to point them. Thoughts?17:13
rick_hredir: the folks managing the store need to help. Uiteam17:17
redirperrito666 natefinch katco anyone have any idea on how to get access to https://goo.gl/tNVh06 in a kvm container https://goo.gl/NJV7gg ?18:11
perrito666redir: I know not, sorry :(18:11
redirperrito666: np, worth asking18:11
katcoredir: i suggest passing that function in as an argument and see what breaks. continue up the stack18:12
redirkatco: yeah that seems to mean passing state down through the hall of abstrat mirrors18:13
katcoredir: no don't pass state. *please* don't do that!18:13
katcoredir: just pass that one function18:13
redirwhich is on state18:14
redirI"ll look18:14
katcoredir: right, but you don't have to pass a reference to state, just the function18:14
katcoredir: remember functions are reified18:14
rediryeah, just changing untold layers of abstraction18:14
katcoredir: big systems necessitate this to be at all maintainable18:15
rediryes18:15
katcoredir: at any rate, that's what i do: fix the module by requiring injection, and that takes me up the stack to where i can pass it in18:16
katcoredir: good luck!18:16
redirthanks katco18:22
voidspacemgz: if you have a chance, I'd love your comments on my branch18:24
mgzvoidspace: sure thing, will do that now18:26
perrito666rick_h: perri.to19:26
natefinchreally? We stop people from putting capital letters in their application names?  Geez.19:38
thumpermorning folks19:57
rick_hmorning thumper20:02
thumpero/20:02
katcothumper: hey, hope everything is well down there20:11
thumperfine in Dunedin20:11
thumperalthough main highway down the island is shut at the top20:11
thumperdue to slips20:11
thumperlots of them20:11
thumpersomeone said 100000 slips20:11
katcowow20:11
thumperbig ones will take a long time to clear to make road passable again20:11
thumperand railway tracks were pushed off their footing, across the road and into the sea20:12
thumpersome cool photos20:12
rick_hthumper: going to need a hand with alexisb out. This seems :( and need to know if anyone on your side can look? https://bugs.launchpad.net/juju/+bug/164053520:12
mupBug #1640535: HA tests fail after the leader is deleted <ci> <ha> <regression> <juju:Triaged by rharding> <https://launchpad.net/bugs/1640535>20:12
thumperhmm...20:13
rick_hthumper: going to need a hand with alexisb out. This seems :( and need to know if anyone on your side can look? https://bugs.launchpad.net/juju/+bug/164053520:13
mupBug #1640535: HA tests fail after the leader is deleted <ci> <ha> <regression> <juju:Triaged by rharding> <https://launchpad.net/bugs/1640535>20:13
rick_hoops20:14
rick_hdammit, different irc client working different20:14
* rick_h goes to get the boy from school biab20:14
perrito666jam: are you around?20:40
rick_hperrito666: midnight there atm and he's on a couple of swap days20:49
rick_hperrito666: might be best to go async or if there's something someone else can help with20:49
perrito666rick_h: just a curiosity, I need someone old enough in the project :) and his name was on the code20:50
rick_hperrito666: gotcha20:50
voidspacemgz: you still around?21:53
mgzvoidspace: heya21:58
voidspacemgz: hey, thanks for the review21:58
voidspacemgz: I'll change that variable name, good call21:58
mgzthe other comment I think there's no change required21:59
voidspacemgz: but on the encode then replace dance - because only alphanumeric characters are valid for lock names21:59
voidspacemgz: yeah, I think it's needed21:59
voidspacemgz: but I agree it's not ideal21:59
voidspacemgz: anyway, thanks :-)21:59
voidspacewill do it now and land it, so we can get the change into the develop branch21:59
mgzmy comment was going to be base32 it instead, but given we just need the uniqueness, losing two of the encoded characters isn't really a big deal22:00
voidspaceright22:01
voidspaceand base32 would generate longer strings so we'd chop out more of the hash when we truncate it22:01
voidspace(40 char limit for lock names)22:01
perrito666who is admin for our github? I believe there is a way to set develop as the default branch when trying to merge22:14
rick_hperrito666: you can, but you have to do it as the default to pull and staging is meant to be that22:14
rick_hperrito666: moving it to develop as the default branch isn't the goal, it's just a pain point because we can't get blesses and merges to staging due to our test woes22:15
perrito666rick_h: oh, I was just talking about the papercut when PRing22:15
perrito666but I guess GH is not That configurable22:16
rick_hperrito666: it does have a single "default branch" that you can change, but I don't think it's wise for us to do that22:16
perrito666rick_h: correct, I thought it had a "branch to push" and "branch to pull"22:16
babbageclunkmenn0: ping?23:19
menn0babbageclunk: howdy23:19
babbageclunkmenn0: The logsink endpoint - does the client just keep the websocket open and keep sending logs to it as they come in?23:20
menn0babbageclunk: simplisticly speaking, yes23:21
babbageclunkmenn0: hmm, except it still gets shut down at migration, right?23:21
* menn0 checks23:21
menn0babbageclunk: sorry for the delay... a bit of drama here... looking now23:32
babbageclunkmenn0: No worries!23:33
menn0babbageclunk: so yeah, logsender is shut down during migrations23:33
menn0babbageclunk: there is a deque which accumulates when the logsender isn't active/connected23:35
babbageclunkmenn0: I've added code to the logsink handler to release the state when the connection finishes, but it's never happening.23:36
menn0babbageclunk: i'll probably need more context...23:39
babbageclunkmenn0: Might be easier to explain in person!23:41

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