| StevenK | wgrant: Finally, but the code is *HORRIBLE* | 04:37 |
|---|---|---|
| StevenK | wgrant: http://pastebin.ubuntu.com/5886428/ | 04:38 |
| wgrant | StevenK: That looks quite reasonable. | 04:41 |
| StevenK | wgrant: I haven't shown you the auditor code yet | 04:46 |
| wgrant | I have read approximately two lines of auditor code ever | 04:47 |
| StevenK | wgrant: http://pastebin.ubuntu.com/5886439/ | 04:47 |
| wgrant | Why are OBJ and OPER in ALLCAPS, and why is the SQL not formatted? :) | 04:49 |
| wgrant | Oh | 04:49 |
| wgrant | They're replaced, I see... | 04:49 |
| wgrant | Are you sure you can't do something more sensible? | 04:49 |
| StevenK | I'm not sure, no. | 04:50 |
| StevenK | wgrant: A quick googling shows even more hideous solutions, so it works | 04:55 |
| === tasdomas_afk is now known as tasdomas | ||
| StevenK | wgrant: So I should release auditor{,fixture,client} and push my changes for export-pu-auditor? | 05:48 |
| wgrant | StevenK: How is the test situation? | 05:51 |
| StevenK | wgrant: http://pastebin.ubuntu.com/5886563/ | 05:52 |
| wgrant | === modified file 'lib/lp/services/auditor/server.py' | 05:54 |
| wgrant | 'distinct' might be better called 'latest' | 05:55 |
| StevenK | wgrant: Blah, that requires changes to auditor{,fixture,client} | 05:58 |
| wgrant | Sure | 05:58 |
| wgrant | But 'distinct' really doesn't say anything at all | 05:58 |
| StevenK | wgrant: So it does address your concerns about a test? | 06:00 |
| wgrant | StevenK: I'm more wondering about testing the divergent postgresql and sqlite3 implementations | 06:00 |
| StevenK | I'm not sure how to test the postgresql implementation | 06:00 |
| StevenK | Requires having postgres around and configured so $USER can create a db | 06:01 |
| StevenK | wgrant: Not sure if I can reach into Django's innards and convince it to change the default db. Or if I can configure a postgres db in runtests.py with things like 'user': environ['USER'] | 06:07 |
| wgrant | I'm not sure | 06:07 |
| wgrant | But having a way to test that the production code works might be nice... | 06:07 |
| StevenK | I've tested it by hand, at least, but I know that isn't what you mean. | 06:08 |
| pindonga | hi, I'm trying to troubleshoot an issue with a user getting a different openid from launchpad than from Ubuntu SSO | 12:27 |
| pindonga | is it possible to check if this user has merged accounts recently? | 12:27 |
| pindonga | is this the right channel for that? | 12:27 |
| === tasdomas is now known as tasdomas_afk | ||
| === matsubara is now known as matsubara-lunch | ||
| === dpm_ is now known as dpm | ||
| === matsubara-lunch is now known as matsubara | ||
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!