hazmat | anybody around.. | 14:44 |
---|---|---|
hazmat | i'd liike some vetting of my approach for deploy-to | 14:44 |
hazmat | in core | 14:44 |
fwereade_ | hazmat, heyhey | 15:41 |
fwereade_ | hazmat, didn't know you were planning to grab that, but I'm delighted if you are :) | 15:42 |
hazmat | fwereade_, twas assigned to me friday, and i did say i'd take it up.. so yeah.. diving in. | 15:47 |
hazmat | fwereade_, do you have a minute for a g+? | 15:47 |
* hazmat preps a diff | 15:47 | |
fwereade_ | hazmat, sure | 15:47 |
hazmat | fwereade_, here's a diff of the approach.. http://paste.ubuntu.com/5686584/ | 15:47 |
hazmat | fwereade_, https://plus.google.com/hangouts/_/1a13f8d6e06fde8d8ce9f9f4a1219b62f75ee409?authuser=0&hl=en | 15:49 |
thumper | oh ffs | 22:06 |
thumper | trunk tests failing for me | 22:06 |
thumper | gah, in two places... | 22:07 |
thumper | hi davecheney | 23:34 |
thumper | davecheney: how are you doing? | 23:34 |
thumper | davecheney: as usual, I have questions :) | 23:34 |
davecheney | thumper: and you believe I have answers ? | 23:41 |
thumper | davecheney: for some at least :) | 23:41 |
davecheney | this is becoming increasingly untrue | 23:41 |
thumper | davecheney: firstly... | 23:41 |
thumper | davecheney: can I get you to pull trunk, | 23:41 |
davecheney | science says that you cannot be just an observer in a system, you are also a participant | 23:41 |
thumper | davecheney: and cd into the bzr package and run the tests | 23:41 |
davecheney | thumper: sure | 23:41 |
davecheney | two secs | 23:41 |
thumper | davecheney: I'm guessing it will fail | 23:41 |
davecheney | yeah, i saw that proposal | 23:42 |
davecheney | well, read the title | 23:42 |
davecheney | ... testing | 23:43 |
thumper | if it fails, then go `bzr merge lp:~thumper/juju-core/fix-bzr-test` and rerun the test | 23:43 |
davecheney | getting regex match | 23:44 |
davecheney | this whole multiple series thing is getting me down | 23:45 |
davecheney | either we all use one series | 23:45 |
davecheney | or we need build bots on every series | 23:45 |
thumper | davecheney: so you're tests pass/ | 23:45 |
thumper | ? | 23:45 |
davecheney | they do not pass | 23:45 |
davecheney | match == fail | 23:45 |
thumper | ah.. | 23:45 |
davecheney | bzr_test.go:69: c.Assert(string(output), Matches, "(?s).*revision-id: "+revid+"\n.*message:\n.*my log message\n.*added:\n.*myfile .*") | 23:46 |
davecheney | ... value string = "" + | 23:46 |
thumper | ok, can you merge my branch and rerun | 23:46 |
thumper | I'm happy to merge a test failure like this with one approval :) | 23:46 |
thumper | s/test failure/test failure fix/ | 23:46 |
davecheney | thumper: tests pass | 23:48 |
davecheney | LGTM, please submit | 23:48 |
thumper | https://codereview.appspot.com/8500043/ | 23:48 |
thumper | wanna mark that | 23:48 |
thumper | and I'll submit | 23:48 |
davecheney | done | 23:49 |
thumper | submitting | 23:49 |
thumper | davecheney: worker/uniter/context.go around line 142 | 23:50 |
thumper | davecheney: it seems to me that we are closing the outWriter pipe before waiting for the process to finish | 23:50 |
thumper | davecheney: this looks suspicious to me | 23:50 |
thumper | davecheney: am I missing something, or does it look weird to you too? | 23:51 |
thumper | my first assumption is that I'm missing something | 23:51 |
davecheney | thumper: that is the subject of literally months of arguing | 23:51 |
thumper | ah... wat? | 23:51 |
davecheney | beforewarned that digging in there involves interfacing deeply with roger and willam | 23:51 |
davecheney | thumper: are you fixing a test failure bug ? | 23:52 |
thumper | what was the arguing? | 23:52 |
davecheney | that one comes up over and over again | 23:52 |
davecheney | thumper: about the correct way to fix it | 23:52 |
thumper | davecheney: no, I was looking around the serialisation of hook execution on a machine | 23:52 |
thumper | so reading the uniter code | 23:53 |
* davecheney finds the bug | 23:53 | |
davecheney | (s) | 23:53 |
thumper | I'm wonering why if this was after a long time of arguing, why there is no comment there | 23:53 |
davecheney | because most of our arguments end in a stalemate | 23:55 |
* thumper cracks open a can of worms | 23:56 | |
thumper | davecheney: do you know the area of code where multiple hooks can be executed at one time? | 23:56 |
thumper | it seems to be related to subordinates | 23:56 |
davecheney | and this causes deadlocks on the apt cache etc ? | 23:58 |
davecheney | isn't this william's 'serialized hook execution' | 23:58 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!