cjwatsonStevenK: Help?  Upgrading dogfood results in http://paste.ubuntu.com/1018822/00:13
cjwatson'column "private" of relation "bug" does not exist'00:13
cjwatsonHmm.  Which is odd, given that there's no such COMMENT in devel00:15
cjwatsonUrgh, OK, ton of conflicts, never mind00:15
wgrantcjwatson: Ah yeah, it was used for some emergency pre-cowboy QA. reverting it all should work00:17
cjwatsonDoes anyone care about keeping a copy, or just 'bzr revert'?00:19
wgrantbzr revert00:20
cjwatsonOK, it's running again now, thanks00:26
wgrantcjwatson: Are you sure we have a new enough dpkg?01:13
wgrantcjwatson: This code is run on appservers too.01:13
cjwatsonI was just wondering that.  I think, though, that the only problem with the version on appservers is that it doesn't know that armhf matches any-arm.01:14
cjwatsonWhich is annoying but no worse than today.01:15
cjwatsonIt should know that armel matches any-arm, which is an improvement.01:15
wgrantAlso, shelling out randomly to bits of dpkg doesn't strike me as a truly marvellous idea, but perhaps my doubts in its security are unfounded today.01:15
cjwatsonThe arguments are strictly controlled here, and there doesn't exist a better interface.01:16
cjwatsonAnd dpkg-architecture is not particularly complex.01:16
cjwatsonEither trying to parse /usr/share/dpkg/*table or extending the previous broken parser is IMO a worse idea.01:17
cjwatsonIf there were a python-dpkg I'd use it :-)01:18
cjwatsonI've just checked dpkg-architecture's behaviour if given a pathological argument to -i (containing ;, `, newline).  It reads the argument unmolested and doesn't do anything silly with it.01:22
wgrantYeah, it seems OK.01:22
cjwatsonSo I do think this is safe.  But if you think there's a real security concern then of course feel free to revert.  I'm going to be away from ~8 hours from now until late Tuesday.01:23
wgrantShouldn't you be away until Wednesday? :)01:23
wgrantNo, I think it's OK, but I am wary of dpkg nowadays.01:23
cjwatsonWell, I mean technically I'm back home on Tuesday.  Probably won't be doing much that evening.01:23
wgrantdpkg-architecture seems to have avoided the horribleification that abounds in most of the rest of dpkg nowadays.01:24
cjwatsonIt might be worth patching in a bit more armhf understanding here, but I don't think it's desperately urgent.01:25
cjwatsonOnly two packages in quantal are using any-arm* at all.01:25
wgrantBut we now need to be sure to upgrade the appservers with any new dpkg stuff that we previously only had to push out to Soyuz.01:26
cjwatson(picolisp, scsh-0.6)01:26
cjwatsonHmm.  Can you think of a sane alternative, if that's particularly painful?01:26
wgrantIt's just a change from what we've done in the past.01:27
cjwatsonAlso, does http://paste.ubuntu.com/1018886/ look like a vaguely reasonable queue binary override API, based on our earlier conversation?01:28
cjwatsonIf that's usable then I think I have an approximately usable replacement for queue.01:29
wgrantcjwatson: That looks pretty good. Doesn't handle overrides differing between archs, but hopefully you don't care about that...01:30
cjwatsonNot really.  OTOH if we actually want that then we can override individual queue items.01:31
cjwatson(This is on PackageUpload.)01:31
wgrantYeah, but delayed copies etc.01:32
wgrantAll the builds in a single upload.01:32
cjwatsonI'm not sure I've ever tried overriding a delayed copy :-)01:32
cjwatsonMaybe I should given the chaos with kernel copies.01:32
cjwatsonYeah, for those I don't think we need arch-specific handling anyway.01:33
cjwatsonIf we do in future I could add an architecture key.01:33
cjwatsonqueue-api LoC +486 and scripts/ftpmaster-tools/queue + lib/lp/soyuz/scripts/queue.py = 862, never mind whatever tentacles that has elsewhere ...01:34
cjwatsonI'll see about getting that in for review on Wednesday, then.01:34

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