/srv/irclogs.ubuntu.com/2005/01/03/#launchpad.txt

=== lamont__r [~lamont@phantom.acmeps.com] has joined #launchpad
=== lamont__r is now known as lamont_r
=== jamesh [~james@203-59-217-65.dyn.iinet.net.au] has joined #launchpad
=== mdz [~mdz@69-167-148-207.vnnyca.adelphia.net] has joined #launchpad
=== carlos -> bed
dilysMerge to rocketfuel@canonical.com/launchpad--devel--0: Fixed and improved the import daemon and added my script to import pofile over the web to have a backup so I don't lose it (patch-1135, carlos.perello@canonical.com)03:29
=== stub [~stub@dsl-246.248.240.220.dsl.comindico.com.au] has joined #launchpad
=== SteveA [~steve@office.pov.lt] has joined #launchpad
SteveAhi folks11:21
carlosSteveA: morning11:28
dilysMerge to rocketfuel@canonical.com/launchpad--devel--0: Allow access to the changelog via the debug skin (patch-1136, stuart.bishop@canonical.com)11:32
SteveAallow access to the changelog via the debug skin?11:48
SteveAstub: what does that mean?11:48
carlosSteveA: could we add more entries to the error page (https://launchpad.ubuntu.com/errors)? 11:51
SteveAcarlos: sure11:52
carlosSteveA: we have many users now and it's hard to see an error if they report it after more than 5 minutes they get it11:52
SteveAI'm not coding yet -- still hacking11:52
SteveAwhat we should do is have errors mailed to a launchpad-errors mailing list11:53
SteveAthen we can all see them11:53
SteveAand refer to them in URLs11:53
SteveAand discuss them11:53
stubUrgh.... beeps off again :-(11:53
carlosSteveA: hmm, sounds better, yes11:53
SteveAhi stub11:54
stubSteveA: It means you can retrieve /changelog.html if you are on the debug port, which is good so people can see when stuff is rolled out to dogfood11:54
carlosSteveA: some users asked to get more information than just a "system error" so they can report the error easily11:54
SteveAcarlos: okay.  we'll make it send an email, and generate a UID11:54
SteveAthen the error page can print the UID, and it can be in the subject of the email11:54
SteveAI'm not coding yet, still catching up, I meant11:55
SteveAwhen I said earlier "I'm not coding yet -- still hacking"11:55
carlosSteveA: sounds good. Thanks11:56
carlosok11:56
SteveAcarlos: file a bug on it11:56
carlosSteveA: public or closed malone?11:56
carlos /s/closed/private/11:56
SteveAthe launchpad one11:57
SteveAso, the one on dogfood11:57
dilysMerge to rocketfuel@canonical.com/launchpad--devel--0: More changes to prevent that the daemon dies if we get an exception (patch-1137, carlos.perello@canonical.com)11:57
SteveAthat's where we file launchpad bugs11:57
carlosok11:57
carlosstub: could we get a production update?11:58
carlosstub: we really need it11:58
SteveAin this case, we must be careful not to use the transactional mail sender11:58
SteveAas the transaction will be aborted11:58
SteveAcarlos: can you note that in the bug report?11:58
carlossure11:59
SteveAthanks11:59
stubcarlos: I'll investigate if I can without screwing  lifeless when he is back12:01
carlosstub: when will be available?12:01
carlosI'm blocked with my current task until production update is done12:02
carlosstub: at least, could you execute the import daemon? elmo restarted the servers and it's not running anymore12:02
carlosTraceback (innermost last):12:04
carlos    * Module zope.publisher.publish, line 143, in publish12:04
carlos      publication.afterCall(request, object)12:04
carlos    * Module zope.app.publication.browser, line 64, in afterCall12:04
carlos      super(BrowserPublication, self).afterCall(request, ob)12:04
carlos    * Module zope.app.publication.zopepublication, line 167, in afterCall12:04
carlos      txn.commit()12:04
carlos    * Module transaction._transaction, line 293, in commit12:04
carlos      self._commitResources(subtransaction)12:04
carlos    * Module transaction._transaction, line 340, in _commitResources12:04
carlos      rm.tpc_vote(self)12:04
carlos    * Module transaction._transaction, line 629, in tpc_vote12:04
carlos      self._datamanager.prepare(transaction)12:04
carlos    * Module sqlos.transaction, line 157, in prepare12:04
carlos      obj.sync()12:04
carlos    * Module sqlobject.main, line 672, in sync12:04
carlos      raise SQLObjectNotFound, "The object %s by the ID %s has been deleted" % (self.__class__.__name__, self.id)12:04
carloshere we go again....12:04
carlos:-(12:04
carlosSQLObjectNotFound: The object POMsgID by the ID 8608 has been deleted12:04
carlosstub: the dogfood server is down atm12:16
stubcarlos: ok. I was going to do a rollout anyway.12:17
carlosok12:17
SteveAcarlos: is there a bug for showing only simple translations to new translators?12:37
=== cprov [~cprov@200.158.100.251] has joined #launchpad
carlosnot yet12:37
SteveAthese would omit translations that use %s and similar, and also those that use shortcut keys12:37
SteveAI hadn't realized about the short-cut keys before12:37
carlosIt's hard to detect the shourtcut keys, sometimes you will remove strings that are not really shortcuts12:38
carlosalso, it depends on the toolkit12:38
carlosI think that QT uses '&' as the shortcut char12:38
carlosand gtk uses '_'12:39
carlosbut we could try to do something about it12:39
carlosSteveA: we are using now public malone for Rosetta bugs, is it ok to use it for all Rosetta bugs/feature requests filed by us?12:40
SteveAthere's an additional issue that the shortcut key can change, but the change needs to be co-ordinated with other short-cut keys in the same application12:40
SteveAin a way, this should be kept separate from the actual translated string itself12:40
carlosSteveA: since some gtk versions, it's not a problem if you have two shortcuts with the same key12:41
carlos"since some gtk version _ago_"12:41
carlosnot sure about QT12:42
SteveAit is a problem for the UI12:42
SteveAand it makes the translated strings less reusable12:42
SteveAbut, that is a way future feature12:42
carlosright12:42
=== carlos -> breakfast
stubcarlos: We don't want security related bugs in here if we find any12:52
carlosstub: are you talking about the public malone?12:53
carlosstub: yeah, any bug report _but_ security bugs12:53
SteveAwell12:53
SteveAmaybe malone should have a "this is security-related" checkbox12:54
SteveAand reject all bugs that are filed with that checked, with an explanation12:54
SteveAI would file this bug, but I know I shouldn't ;-)12:54
=== debonzi [~debonzi@200.158.100.251] has joined #launchpad
carlosSteveA: well, the idea is that the security bugs will be hidden to the security team12:58
carlosthat's what I think it's going to be implemented12:58
dilysNew Malone bug #182: "Improve the error log for the production server", submitted by Carlos Perell Marn12:58
dilyshttps://dogfood.ubuntu.com/malone/bugs/18212:58
SteveAcarlos: yes.  but right now, it is easy to inadvertently file a security bug.01:00
carlosSteveA: are we able to restrict the access to some of the launchpad's features only to admins?01:25
carlosSteveA: we need an UI to manage the production server's language/country tables and it should be done only by Canonical people (daf and I)01:27
carlosso we can add and update easily plural forms information01:27
SteveAcarlos: yes, we can restrict things to admins01:31
SteveAcarlos: you need to do the following, then I will help you do this.01:32
carlosok01:32
SteveA1. write down the methods / attributes of which interfaces govern setting the language and country tables.01:32
SteveA2. write a stubbed out page that will become the edit form for one of these.01:32
=== salgado [~salgado@200-206-134-238.async.com.br] has joined #launchpad
SteveA3. we'll work out how to do the permissions and authorization code and so on.01:33
carlosok, I will do it "next year" ;-)01:34
carlosas soon as it's ready I will tell you it01:35
SteveAok01:35
dafshould we worry about the error reporting service being publically available?01:36
SteveAwell, perhaps01:36
dafin the future, it might be a security risk01:36
SteveAideally, it would not be01:36
dafsince people might see errors related to security bugs01:37
SteveAindeed01:37
dafI think it's ok for now, but something we should fix sooner or later01:37
SteveAwe could do that using apache -- certificatize it01:37
dafgood idea01:37
SteveAI don't really want the error reports going through much code01:37
SteveAof launchpad01:37
dafyou mean you want to keep it simple?01:37
SteveAyes01:38
carlosSteveA: if we take the approach you suggested, we could remove that page, right?01:38
SteveAno01:38
dafcarlos: which page?01:38
carlosthe errors one01:39
dilysMerge to rocketfuel@canonical.com/launchpad--devel--0: Fix generated changelog (patch-1138, stuart.bishop@canonical.com)01:39
dafwhat approach?01:39
carlosdaf: https://dogfood.ubuntu.com/malone/bugs/18201:40
dafah, I see01:41
dilysSystem error on launchpad.ubuntu.com: RuntimeError: Got translation for msgid u'foo' which is not in the template. (User: 3646, Manuel Antoni; Page: https://launchpad.ubuntu.com/rosetta/products/wordpress/wordpress-1.5/+translate)01:46
carlosdaf: ?01:47
carloswhat's that?01:47
daf^^^ we could do something like that for errors01:47
carlosstub: around?01:47
stubcarlos: yup02:04
carlosstub: could you execute this into the production database?:02:04
carlosupdate language set pluralforms=3, pluralexpression='n < 2 ? 0 : n == 2 ? 1 : 2' where code='gd';02:04
carlosupdate language set pluralforms=4, pluralexpression='n%100==1 ? 0 : n%100==2 ? 1 : n%100==3 || n%100==4 ? 2 : 3' where code='sl';02:04
carloswe are going to get an easier way to do that in the future, but if you could do it now... ;-)02:05
stubcarlos: Done02:10
carlosstub: thank you!02:10
carlosdaf: about the synaptic thing02:22
carlosdaf: at this moment, if we import a pofile that does not have the translations from Rosetta02:22
carloswe will "lose" them02:22
carlosbecause we don't have a way to know if they were removed after exporting the pofile02:23
carlosand we don't have (yet) a way to suggest previous translations so the data is still in the database but the user will feel that they are lost02:24
dafyes02:25
carlosonly rhythmbox remains to be imported in my laptop!! and I only got 1 error from about 500 po/pot files imported02:26
=== carlos hopes his HD will not die after 10 hours or more of intensive work
=== carlos -> lunch
dafgreat!02:30
dafwhat was the error?02:30
SteveAnice work tending the rosetta mailing list, carlos02:35
dafyes, seconded!02:46
dafcarlos has also been doing much of the mailman moderation for the list02:46
dafSteveA: if I move doctests from lib/canonical/rosetta/test/test_browser.py into lib/canonical/rosetta/browser.py, will they still get detected and run by the test runner?02:51
dafI'd like to move at least some of the tests to be inline02:51
SteveAyou need to add a DocTestSuite for rosetta.browser into test/test_browser.py02:54
dafah, ok02:54
dafdid you need to do something similar for canonical.launchpad.database.person?02:55
SteveAI think those are in the canonical/launchpad/doc/ directory02:56
SteveAbut, yeah02:56
SteveAI must have done02:56
SteveAfor the inline doctest in browsername02:56
SteveAso you can look into that to see how it is done02:56
dafthat's the one I was thinking of02:56
carlosSteveA: thank you02:57
carlosdaf: the error was a missing plural form, so it's not our fault directly02:57
dafSteveA: ah, yes, lib/canonical/launchpad/database/tests/test_rundoctests.py02:58
dafcarlos: good :)02:58
carlosalso, I went out of space in my laptop tonight and the import daemon died, I have tried to recover it from that kind of errors (lifeless already told me about that problem). I think that it should be able to recover from any error except for a database restart03:00
carloswell, more than recover... it just ignore any exception and retries it later03:00
dafa more resilient import daemon would be good03:02
dafok, I think I have both bugs with tabs in message IDs fixed03:03
dafI'll submit a merge, then go to lunch03:03
carlosdaf: nice!03:03
dafeven better, I was able to add test cases for both bugs03:03
dafassuming the merge goes in, I think the change is production-ready03:06
=== daf -> lunch
carloswill try to test it in my laptop after testing the synaptic import03:08
stubcarlos: I'm trying a production update03:41
stubdaf: Is your fix urgent, or can we dogfood it until boxing day?03:41
dafI would like it to go in today03:41
dafI had a conflict03:41
carlosstub: boxing day?03:41
dafcarlos: 26th December03:41
stubcarlos: Dec 26th03:42
carlos;-)03:42
carlosI have found a bug that should be fixed also today03:42
carlosstub: if you could wait...03:42
dafcarlos: which bug is that?03:42
carlosdaf: seems like some plural forms are not being imported03:42
carlosfrom the pofile03:42
dafdo we have a fix for it?03:42
carlosdaf: I just detected it03:43
carlosI'm working on it03:43
dafhmm03:43
carlosdaf: look at rhythmbox translations in my computer03:44
dafI think I'd prefer it if we don't wait for this fix03:44
carlosdaf, stub: is it ok to have two updates today? :-P03:45
dafwell, I'm thinking of two things (a) the bug I have a fix for is one that is blocking users and (b) we have a fix for it03:46
carlosyou are right, I'm just asking if it's possible to get a sync after I finish the fix and we have tested it enough03:47
stubAssuming this update works (still backing up the previous install - arch does like to create lots of files, doesn't it...), I'll do another rollout in about 12 hours.03:57
dilysMerge to rocketfuel@canonical.com/launchpad--devel--0: cleanups; fix bugs with tabs in message IDs (patch-1139, daf@canonical.com)04:02
dafstub: ^^^ there's my merge04:03
carloswow, the bug is sooooo stupid04:18
=== carlos should reenable the import/export tests as soon as possible
stubI can't work out how lifeless is maintaining the production config - I suspect it isn't :-(04:21
dafit isn't what?04:21
dafmaintained?04:21
stubmaintained anywhere04:21
stubAll this arch config and I can't find where the production customizations are - if I checkout launchpad--production--1.8, you don't get a configured version. 04:22
stubSo if I do a production rollout, I'm going to have to guess what the local changes are. Apart from the .zcml database configuration, I have no idea what local changes needed to be made on macquarie to get it all running happily.04:27
carlosthen, I suppose we should wait for lifeless ...04:28
carlosor try it and if it does not work, revert to the current codebase04:28
dafshouldn't we have staging serves for this kind of thing?04:29
carlosdaf: I don't understand you04:40
carlosWhat do you mean?04:40
dafa server we can test things on before making them live04:40
carlosthat's dogfood04:41
carlosright?04:41
dafno04:41
dafnot quite04:41
dafthe point of the staging server is to have a configuration as close as possible to the production server04:41
dafincluding production data if possible04:42
dilysMerge to rocketfuel@canonical.com/launchpad--devel--0: Fixed plural forms import from po files and added a new plural form expression into our database (patch-1140, carlos.perello@canonical.com)04:46
carlosdaf: https://launchpad.ubuntu.com/malone/bugs/904:47
daf:/04:48
carlosthat bug prevents to import many of the languages that only have singular forms (like ja, and tr)04:48
dafaren't the PO files broken?04:48
carlosno04:49
carloswell...04:49
carlosgettext thinks that they are ok04:49
carloswe could say that are broken, but we are able to recover from that situation easily04:49
dafby ignoring extra translations?04:49
carlosignoring the extra plural forms if we have more, and adding empty ones if we don't have enough04:50
dafor by importing them regardless?04:50
carlosdaf: we could import them04:50
carlosbut mark the extra ones as not active04:50
carlosso next time we export a po file, it will be "fixed"04:50
carlosand we are not losing information, because we have it stored into the database04:51
dafthat seems sane, I guess04:51
carlosSteveA, daf: we should fix the legal link from launchpad04:53
carlosI'm not too good redacting legal text in English04:54
carloscould you look at it?04:54
dafyes, we need to do that04:54
=== salgado is now known as salgado-lunch
=== stub [~stub@dsl-246.248.240.220.dsl.comindico.com.au] has joined #launchpad
=== carlos -> shower
=== mdz [~mdz@69-167-148-207.vnnyca.adelphia.net] has joined #launchpad
=== bob2 [rob@202.174.101.196] has joined #launchpad
=== carlos -> university
carloslater05:53
=== cprov [~cprov@200.158.100.251] has left #launchpad []
=== debonzi [~debonzi@200.158.100.251] has left #launchpad []
=== lamont_r [~lamont@rover3.mmjgroup.com] has joined #launchpad
=== stub [~stub@dsl-246.248.240.220.dsl.comindico.com.au] has joined #launchpad
carlosstub: around?10:51
carlosI'm moving from my laptop (hoary) to my desktop computer (warty) and the database creation is not working correctly, It gives me an error about the type "tsvector"10:52
carlosand I have installed the postgres' contrib package10:52
stubcarlos: make sure you still have the search path set correctly in your postgresql.conf file11:01
stubcarlos: And that you are building the database with 'make PYTHON=python2.4' and not just 'make'11:02
carlospython2.4?11:02
carlosI'm using warty in my desktop computer11:02
stubOh... sorry... say hoary ;)11:02
carlosnot hoary ;-)11:03
stubFirst point is still valid11:03
carlosohh11:03
carlosright11:03
carlosI forgot the search path...11:03
carlosstub: thank you11:04

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