/srv/irclogs.ubuntu.com/2010/06/09/#bzr.txt

igcmorning00:33
lifelesshi igc00:40
lifelesslosa ping00:40
spmmorning igc00:40
spmhey lifeless, sup?00:40
lifelesspqm00:40
lifelessI rt'd a request to do an upgrade00:40
lifelessa weekortwo back00:40
lifelessits affecting developer troubleshooting00:40
lifelessdoyouhavetimetodoittoday?00:41
spmif it's sufficiently high priority - and the world doesn't break again - sure?00:41
spmwhat's the rt #?00:41
lifelessI'm not sure :)00:42
lifelessits somewhere in the cauldron00:42
spmheh, see if I can find it. sec.00:42
spmdeploy PQM trunk rev 236 please - in coordination with Robert ? tho who this robert dude is, dunno. never heard of him.00:42
lifelesssee00:42
lifelessthats the problem with young folk these days00:43
lifelessno short term memory00:43
spmold folk man, *old* :-)00:43
lifelessspm: darn, I *forgot* that bit00:43
spmahh. i see what's happened, looks like francis set the wrong priority button - tho it's new to me that RT has > 1 priority setting00:43
spivlifeless: I just had that error too00:43
pooliehi there00:44
spmhey poolie00:44
poolielifeless, i filed https://bugs.edge.launchpad.net/pqm/+bug/591507 for the pqm 'lost connection' problem00:44
ubot5Launchpad bug 591507 in PQM "bzr tests fail with "lost connection" (affected: 1, heat: 6)" [Undecided,New]00:44
lifelessthanks00:44
pooliesince three people have hit this in recent days, at least, and since the subunit output is not actually valid, i'm inclined to disable it until it's more debugged00:45
lifelesspoolie: well, we haven't deployed the candidate fix00:45
lifelesspoolie: which spm and I are doing right now00:45
spmindeedily00:45
lifelesspoolie: so perhaps we should try to fix it first :)00:46
poolieok, let's at least try that00:46
spmnovel :-)00:46
poolieah00:46
pooliesurely this is not going to make the streams actually valid though?00:46
poolieor does it change them into attachments?00:47
lifelesspoolie: changes them to attachments00:47
poolieoh great00:47
lifelesspoolie: it should be a step in the right direction. Will more steps be needed? Probably.00:47
lifelesspoolie: its been not deployed because of a priority confusion, apparently. '11:43 < spm> ahh. i see what's happened, looks like francis set the wrong priority button - tho it's new to me that RT has > 1 priority setting00:47
spmlifeless: pqm disabled in crontab00:47
lifeless'00:47
lifelessspm: ok, so grab the revno00:48
lifelessspm: as we don't have staging yet, this will be 'test by doing'00:48
lifelessspm: bzr revision-info is the output to save00:48
lifelessspm: then pull in trunk00:48
spmsimiar to you guys with bugs, we schedule our work based on priorities. by default stuff comes in and stays at zero; so tickets that are 'more important' need to be more apprioriately set. a LOT of our requests are wishlisty, so tend to be in the "I need a break, something easy" bucket.00:49
lifelessspm: yeah, I'm not critical; explaining to poolie, because figuring out why something important went stale helps us prevent it.00:50
spmlifeless: so current is revno 23200:50
spmnod00:50
lifelessspm: right, so the new commits are all simple:00:50
lifeless  Record unfixed typeerror note.00:51
lifeless  Make test suite run with bzr 2.2: set an explicit test usercode.00:51
lifeless  Merge and tweak Aaron's errors-as-attachments code.00:51
spm232 robertc@robertcollins.net-20100422043204-70jn37kx2whdm8fr <-- rev info00:51
lifeless  More tweaks to email attachment support.00:51
pooliespm, i don't understand how you can schedule your work based on priorities but also "it's new to me that RT has > 1 priority setting"00:51
lifelessspm: please pull in trunk00:51
poolieor do you mean there's more than one priority-type variable?00:51
lifelesspoolie: multiple fields, I think.00:51
spmpoolie: more than 1 priority type variablee. 'priority' and 'final priority'00:51
lifelessrt is... not the simplest00:51
spmfrancis had set the latter; the former is what we use00:51
spmlifeless: that is a major understatement :-)00:52
spmNow on revision 236.00:52
lifelessgreat, spin it up, I have a broken branch to send to it00:52
poolie'final justice' <http://bit.ly/acROW7>00:52
spmbzr only re-enabled in the crontab; I gather we don't need a UI restart?00:52
lifelessspm: UI is unaffected00:53
poolieyou're going to tell francis about that right?00:53
lifelesspoolie: #launchpad-code00:53
lifelesshe's usually around again for a bit soon, if he disappears off IRC without acking, I'll email00:54
spmI think it was an accident tbh; normally he does set priority.00:54
spmjust hit the wrong field by mistake00:54
lifelesscan we hide that field ?00:54
spmI have no idea. see "[09:51:48] <lifeless> rt is... not the simplest" :-D00:55
lifelesslol00:55
lifelessok so the current branch has a failing test00:56
lifelesseverything going well it will a) fail to land, b) give me a useful email attachment in gmail.00:56
lifelessI shall report in about an hour, I suspect.00:56
spmnp; should I be re-enabling pqm for U1 etc? ie they're safe here?00:57
spivlifeless: you should have tweaked the Makefile to do -f that_test  :P00:57
lifelessspiv: no00:58
lifelessspiv: its deliberately in the middle, want to be sure that under normal circumstances we get sensible stuff out00:58
lifelessspiv: and that would raise the risk of getting it wrong00:58
lifelessspm: unknown.00:58
spmlifeless: fair enough; in that case, i'll stick with disabled for the time being.00:59
lifelessspm: if its faulty, the most likely failure mode is errors going to cronmail00:59
lifelessspm: its unlikely to let things through inappropriately00:59
spmcoolio, given the U1 guys don't seem busy on pushing code in; going into holding shouldn't hurt.00:59
lifelessspm: can I ask a favour ?01:00
lifelessspm: file a bug/rt ticket as appropriate, for your team, to delete the unused button; its caused friction here, and IIRC rt is meant to be very customised01:00
spmlifeless: I can do that. I have no idea how easy that is tho; or the likyhood of it being done anytime soon.01:10
poolielifeless, do you know what 'unknown content code' in roland mas's problem report is likely to mean?01:14
poolieto mean, the earlier gzip errors probably indicate a file-level problem01:14
pooliewhich may ultimately be attributable to us for example not defending enough against a sudden machine crash01:15
lifelessok01:22
lifelessthe gzip errors indicate a group couldn't be decompressed01:22
lifeless-> bit (or more) error in disk content01:22
lifelessand either out-of-order disk operations(man I wish we had user space barriers), or entropy/cosmic ray01:23
lifelessthe invalid content code suggests a multibit error in gzip content not detected by the [weak] crc on the compressed block01:23
lifelessand thus garbage coming out of gzip01:24
poolieright01:25
lifelessso two symptoms, one problem01:26
lifelesswe can verify the packs01:26
lifelessmd5sum them01:26
lifelessand see if they match01:26
lifelessI've mailed a reply to that01:33
pooliethat was what i suggested too01:33
pooliethanks01:33
pooliewe could talk about the fixture stuff01:33
lifelesssure01:35
lifelessI was going to pop out and do lunchtimey shores01:35
pooliebut perhaps when i get home01:35
lifelesss/shores/chores/01:36
pooliewe're going to look at some ui stuff with ian for a few hours01:36
lifelesswell, if you have speaks & a mic we could do a group call01:36
lifelessI'm happy any which way01:36
pooliethe document is probably a bit rambly but the process was good01:36
lifelessok, -> local stuff01:43
lifelessSMS if things blow up and I need to rush home01:43
lifelessspm: ok, bzr causing funnies I think; tweaking the patch to deal03:00
spmkk03:00
lifeless File "/home/pqm/pqm/pqm/core.py", line 194, in send_mail_reply03:00
lifeless   message = Message()03:00
lifelessTypeError: 'LazyImporter' object is not callable03:00
lifelesswent to cron03:00
lifelessspm: please pull rev 23703:01
lifelessspm: tell me when you have, I'll retest03:01
spmlifeless: done, go for it.03:01
=== arjenAU2 is now known as arjenAU
lifelessspm: ok wow, rabbit hole goes deeper04:24
lifelessMessage, in email, is not what it should be04:25
lifelessspm: I need some interactive testing please04:25
spmoh?04:25
spmsure. gimme 5? just sorting some space issues on lardibartfast04:25
lifelesssure sure04:25
lifelessutter bollocks04:26
lifelessexcuse the vernacular04:26
lifelessspm: I've just done a push --overwrite; please pull --overwrite, and we'll try again.04:28
spmsure04:28
lifelessuhm, pushed ...04:28
lifelessnow04:29
spmow on revision 237. ?04:29
lifelessyeah04:29
lifelesslog -r -1 should say 'import from the right place'04:29
spmhrm. maybenot. we were there before. try again?04:29
spmwith a capital I and a Message in tehre as well, yarp04:30
lifelessok04:31
lifelesscan yuou aslo04:31
lifelessalso04:31
lifelessdo04:31
lifeless"python -c 'from email.message import Message'"04:31
spmpqm@balleny:~/pqm$ python -c 'from email.message import Message'04:32
spmpqm@balleny:~/pqm$ echo $?04:32
spm004:32
spmlooks good to me04:32
lifelessthanks04:32
lifelessboombs away04:32
spmnp04:32
lifelessspm: pull again please05:57
spmlifeless: rev 23806:02
lifelessthanks06:03
=== lifeless_ is now known as lifeless
=== almaisan-away is now known as al-maisan
spmlifeless: how's it going? have a u1 item in the queue - are we safe to let it play?07:29
lifelessspm: yes, its giving attachments now07:34
lifelesswe can iterate safely from here07:34
lifelessplease enable it all07:34
spmoki, re-opening pqm07:34
lifelessor bugger07:35
lifelessits rather smaller than expected07:35
spm?07:35
spmproblem?07:35
lifelessif their merge fails they won't get much diagnostics right now07:35
lifelessbut enable it07:35
lifelessit isn't going to crash now07:36
spmoki, it's enabled.07:36
lifelessright, the attachment isn't07:38
lifelessI need to dig deeper into the email api07:39
lifelessI shall do that around dinner07:39
spmkk07:39
spmupdating the RT as to where we're at07:40
lifelessspm: when are you disappearing tonight? 15 minutes?07:45
spm75.07:46
lifelessthis looks wrong: Content-type: multipart/mixed07:46
lifelessContent_type: text/plain07:46
spm6pm local.07:46
lifelessbloody broken API's07:46
lifelessspm: can you do python -c 'from email.mime.multipart import MIMEMultipart'07:46
lifelessplease07:46
spmworks fine07:47
lifelessoh man07:48
lifelessit wants headers spelt in lower case07:49
lifelessno, thats not it.07:49
lifelessgmm07:49
lifelesswho writes a dict interface that doesn't replace keys07:49
lifelessSo very not pythonic07:49
* lifeless sobs07:50
mgzdid you work out the email.Message vs. email.message thing from earlier?07:50
lifelessmgz: yes07:50
mgzyou need both.07:50
lifelessmgz: email/__init__ contains a bad lazy_import implementation07:50
lifelessmgz: no, can just grab from email.message.Message07:50
lifelessMessage appears ripe for user bugs in so many ways07:51
lifelessheaders aren't normalised07:51
mgzthis needs to run on python 2.4 right?07:51
lifelessmgz: no07:51
mgzah, okay, ignore me then.07:51
lifelessmgz: it needs to run outside the chroot on balleny07:51
Tibihi07:52
mgzmime isn't fun.07:52
lifelessmgz: email.* isn't fun; mime itself is pretty well established ;)07:52
lifelessmgz: check out message.Message.get_param for an eye watering docstring07:52
mgzsee, I'd blame some of that on mime/rfc 2231 making things overly painful07:55
lifelessif the email module was close to rfc2231 compliant, I'd agree07:56
lifeless:)07:56
lifelessspiv: can you tell poolie, if he doesn't see me first, that I'm 'bloody fixing bloody pqm'.08:27
=== radoe_ is now known as radoe
lifelesslosa ping10:42
mthaddonlifeless: hi10:42
lifelessmthaddon: hi, need to rollback the PQM change from earlier today; take 1 minute10:43
mthaddonok10:44
lifelessmthaddon: on balleny in the pqm dir, bzr pull -r 232 . --overwrite10:44
lifelessbzr revision-info should then report 232, and bzr st no changes10:44
lifelessturns out the email API is really tricky to get right and I'm doing more extended QA locally before we do the upgrade again10:45
mthaddonlifeless: https://pastebin.canonical.com/33182/ - I think we're good10:45
lifelessmthaddon: we should be, thanks. I'll send a test through to make sure its back to its ugly old self10:45
mthaddonk10:45
parthmthere is a bzr grep bug #591233 ... does bzrlib.option provide a way of having an option that allows '-' as start of argument? e.g. '-e "-blah"'10:47
ubot5Launchpad bug 591233 in bzr-grep "support -e to specify pattern (affected: 1, heat: 6)" [Wishlist,Confirmed] https://launchpad.net/bugs/59123310:47
lifelessparthm: no10:48
lifelessparthm: or rather, I'm pretty sure it doesn't.10:48
parthmlifeless: ok. thanks. i will look into it to see if there is an easy way to do it.10:48
lifelessit will parse it as option 'b' with param 'lah' and no parameters given to 'e'10:48
parthmlifeless: yes. thats what i got when i tried it.10:49
lifelessThats what I'd expect to happen; I don't think this is really a bug is it? just escape the - - -e "\\-blah"10:49
parthmlifeless: yes. thats the workaround i suggested in the bug. but posix grep supports -e ... so i suppose people used to posix grep would try that first.10:50
lifelessok10:51
parthmso its not critical, but nice to have.10:51
lifelessso, if you find a way to add this to bzr core10:51
lifelessI'd like to suggest that it be made obvious in two places10:51
lifelessthe docstrings for any objects that will sometimes-handle-leading-dash-and-sometimes-not10:51
lifelessand the help for commands that use such options - automatically in the this second case10:52
parthmargh. seems to be some problem with my network today. i keep dropping off.10:53
parthmlifeless: i will see if there is a good way to support this. maybe a new option type. will file a wishlist against bzr for now.10:53
lifeless21:51 < parthm> so its not critical, but nice to have.10:54
lifeless21:51 < lifeless> so, if you find a way to add this to bzr core10:54
lifeless21:51 < lifeless> I'd like to suggest that it be made obvious in two places10:54
lifeless21:51 -!- parthm [~chatzilla@122.167.117.28] has quit [Read error: Connection reset by peer]10:54
lifeless21:51 < lifeless> the docstrings for any objects that will sometimes-handle-leading-dash-and-sometimes-not10:54
lifeless21:52 < lifeless> and the help for commands that use such options - automatically in the this second case10:54
lifeless21:52 -!- parthm [~chatzilla@122.167.117.28] has joined #bzr10:54
parthmlifeless: makes sense. bug #59165710:59
ubot5Launchpad bug 591657 in Bazaar "bzrlib.option should allow an option that allows argument starting with dash (-) (affected: 1, heat: 6)" [Wishlist,Confirmed] https://launchpad.net/bugs/59165710:59
lifelessmthaddon: can you please tell me what python -V shows on balleny, outside the chroot ?11:48
mthaddonPython 2.5.211:49
lifelessthanks11:49
lifelessgnight y'all12:37
bilalakhtarPeople, please help. bzr is stuck on Fetching revisions:Inserting missing keys: . Is this fine?13:22
bilalakhtarok, its fine. got it13:23
alf__Hi all! When packaging using bzr builddeb is the versioned directory (upstream+debian) supposed to have debian/patches applied or not?13:54
james_wif it is dpkg-source v3 (quilt)13:54
alf__james_w: So, if I understand correctly, if it is v3 (quilt) it should have the changes applied *and* also have .pc/ versioned13:57
alf__james_w: and that is what I should push to launchpad?13:57
james_wyeah13:58
james_wit's not ideal, but it's the best we have for now13:58
alf__james_w: great, thanks13:58
=== IslandUsurper is now known as IslandUsurperAFK
=== mwhudson_ is now known as mwhudson
serviliohi! I have a branch with a couple of commits I'd like to remove is there a way to do this without using uncommit? or is there a plugin that automates this?17:13
=== beuno is now known as beuno-lunch
maxbservilio: Uh..... this is what uncommit is for.#17:27
serviliomaxb: yes, but there are ~20 commits before the ones I need to take out17:31
maxbIn that case you should just undo the effect of those commits on the tree (by doing a reverse-merge), and commit that17:32
serviliomaxb: I changed the history of the parent after that branch was made17:32
maxbThat sounds hideous17:32
servilioyes, it is17:32
maxbWell don't do that then :-)17:32
serviliobut it was a merge gone wrong I needed to take out ASAP17:33
servilioI am looking into cherry-picking now17:33
serviliolooks like that will be the solution17:33
servilionew branch, cherry-pick merge, push to the parent :D17:33
=== IslandUsurperAFK is now known as IslandUsurper
=== deryck is now known as deryck[lunch]
=== beuno-lunch is now known as beuno
taxiliancan anyone tell me if there is a good equivilent to webdav publishing like svn or mercurial can do with bzr?  I'm using bzr+ssh, but that requires creating a user account on the central repo machine for each user, which I'd prefer not to do18:36
taxilianor if there is a way I haven't found yet to run bzr over webdav...18:38
serviliotaxilian: have taken a look at bzr-webdav?18:40
taxilianlooked at a bit, but looks like it isn't stable18:40
taxilianmaybe it's just been too long since I looked18:40
serviliohaven't tried it myself, a co-worker did and it worked fine18:41
taxilianso it looks like it requires installing a plugin on each dev machine?18:41
taxilianwhat is the "recommended" way to publish a bzr repo?  sftp?18:42
servilioyes18:42
servilioyes was regarding the installation of the plugin in each dev system18:42
servilioregarding the "recommended" way, can't tell you sincerely, just a user here18:42
servilioI use bzr+ssh currently18:43
taxilianyeah; that's what I don't get about bzr.  I like bzr; I've been using it for awhile18:43
taxilianbut bzr+ssh is not ideal for a real solid source control system18:43
serviliodefine "real solid" :D18:43
serviliomeaning, state your requirements ;)18:43
taxilianwe used to use that with cvs, and that's one of the main things that everyone I've worked about liked about svn over cvs; it worked over http instead of needing ssh18:43
taxilianflexible permissions on various projects18:44
taxilianeasy to add and manage users18:44
taxilianwith ssh, you have to deal with system users18:44
taxilianand unix groups aren't real ideal for managing access control18:45
=== deryck[lunch] is now known as deryck
servilioI have been looking at a solution for access control for ssh access that wouldn't need system account, but would use ssh keys18:46
serviliobut haven't found any so far18:46
taxilianwell, you can use ldap, but that's a pain18:47
serviliothere is http://pypi.python.org/pypi/ClueBzrServer/18:47
servilioyes, a bit of a complicating solution LDAP is18:48
servilioand yesterday I found nappingcat http://pypi.python.org/pypi/nappingcat/18:48
serviliobut haven't had time to take a look at it18:48
taxilianhmm.  does look like cluebzrserver supports pushing; very interesting18:49
taxiliannot much information about nappiungcat18:50
lifelessmoin20:11
mtaylorlifeless: moin20:27
mtaylorlifeless: hey - quick question for you20:27
=== BasicPRO is now known as BasicOSX
netshine;-(22:08
netshinebzr: ERROR: Connection closed: Unexpected end of message. Please check connectivity and permissions, and report a bug if problems persist.22:08
netshinethere any known problems right now?22:08
lifelessnetshine: known problems with?22:14
netshinewish launchpad or bazaar ;-022:15
lifelessthat error usually indicates an ssh configuration issue22:16
lifelessdo you have your ssh key setup correctly?22:16
netshinei think so...22:18
netshinei just updated my pgp key to be sure..22:18
netshine:-022:18
netshinewait, ssh key?22:19
netshinenot pgp? about what you talking about :-022:19
lifelessssh22:19
lifelessI have to pop out for a bit22:19
netshine:-?22:19
lifelessif this is your first time using bzr with launchpad, you might try asking on #launchpad22:20
lifelessbye for now.22:20
netshine:-022:30
netshinelifeless, thanks: D22:30

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