wgrant | Ah, the Soyuz OOPS report is a bit happier today. | 02:02 |
---|---|---|
wgrant | lifeless: :( | 02:27 |
wgrant | Archive:+index's soft timeouts have gone away. | 02:27 |
wgrant | Our exception counts are 20% lower than pre-release, soft timeouts down by more than 60%. | 02:28 |
wgrant | Yay. | 02:28 |
lifeless | wgrant: cool | 02:45 |
lifeless | 219 / 432 BugTask:+index | 02:46 |
lifeless | 90 / 241 Distribution:+bugs | 02:46 |
lifeless | next up | 02:46 |
lifeless | I think Monday is 'remove substring search' day | 02:46 |
wgrant | Heh. | 02:56 |
wgrant | Hmm | 02:58 |
wgrant | Unbreak checkwatches, or split it into its own report? | 02:58 |
wgrant | The former will take quite a while, I suspect. | 02:58 |
wgrant | Particularly since it uses OOPSes somewhat differently to the rest of the world. | 02:58 |
lifeless | wgrant: unbreak | 03:03 |
lifeless | splitting it into its own report makes the daily report analysis harder, while not making the checkwatches component easier. | 03:03 |
lifeless | its a pessimism - just like having a soyuz specific report | 03:03 |
wgrant | Right. | 03:05 |
wgrant | But checkwatches is designed to use OOPSes for warnings. | 03:05 |
lifeless | we should get an oops when we need to take an action | 03:06 |
lifeless | checkwatches seems to be in a little bit of a grey area | 03:06 |
wgrant | Hmm. | 05:05 |
wgrant | We seem to have a major performance issue with update-bugtask-targetnamecaches. | 05:05 |
wgrant | It is again taking >18 hours. | 05:05 |
wgrant | I think I will delete it. | 05:06 |
wgrant | :( things still use it. | 05:07 |
wgrant | Perhaps selecting distinct targets and updating changed ones will be better and easy. | 05:07 |
wgrant | Well, that looks like it should take about 10 seconds. | 05:11 |
wgrant | Yeah. | 05:12 |
LPCIBot | Project devel build (436): FAILURE in 5 hr 37 min: https://hudson.wedontsleep.org/job/devel/436/ | 06:20 |
lifeless | wgrant: what uses it? | 08:51 |
wgrant | lifeless: I guess search is the only thing that can't calculate it on-the-fly. | 09:00 |
wgrant | But tables and titles use it too. | 09:00 |
lifeless | grah | 09:03 |
lifeless | well, fix that. | 09:03 |
lifeless | no good reason for it to be used; we load the related objects always anyway. | 09:03 |
wgrant | Fix what? The non-search uses? | 09:04 |
lifeless | yes | 09:04 |
lifeless | the search use I'm going to put behind a FF tomorrow | 09:05 |
wgrant | Aha. | 09:05 |
wgrant | Then turn it off and see if anyone notices? | 09:05 |
lifeless | hey, why does the cache updater touch every bug? surely just 'bugs changed since the last run' is sufficient. | 09:05 |
wgrant | product renames, I suspect. | 09:05 |
lifeless | wgrant: well, turn it off for lp devs, test the improvement, then wider etc. | 09:06 |
wgrant | :( | 09:06 |
lifeless | wgrant: products have a datestamp | 09:06 |
wgrant | A 'when my display name was last changed' datestamp? | 09:06 |
lifeless | wgrant: doesn't matter | 09:06 |
lifeless | actually no | 09:07 |
lifeless | product doesn't have a mod date | 09:07 |
lifeless | *fail* | 09:07 |
wgrant | That's what I thought. | 09:07 |
wgrant | Most stuff doesn't. | 09:07 |
wgrant | The only mandatory fields are owner and datecreated. | 09:07 |
lifeless | ok, so product and sourcepackagename [blech - this is crazy to normalise such fields] - need a last_modified | 09:07 |
lifeless | and distribution | 09:08 |
lifeless | or | 09:08 |
lifeless | just ditch all use of it | 09:08 |
lifeless | its really uninteresting | 09:08 |
lifeless | I think we can remove all uses by friday, with care and caution | 09:08 |
wgrant | Yeah. | 09:08 |
wgrant | We could disable it until then. | 09:09 |
lifeless | new data is updated by hand, right ? | 09:09 |
lifeless | I mean, new bugs have a valid cache value | 09:09 |
wgrant | Yes. | 09:09 |
wgrant | I'm not sure if task moves are. | 09:09 |
wgrant | But I presume so. | 09:09 |
lifeless | woo 12365 hath landed | 09:10 |
wgrant | Yep. | 09:10 |
lifeless | hmm | 09:10 |
lifeless | Does staging have the render time on it yet | 09:10 |
wgrant | It should... | 09:10 |
wgrant | Even the FF should have propogated by now. | 09:11 |
lifeless | nope | 09:11 |
lifeless | https://staging.launchpad.net/+feature-rules | 09:11 |
lifeless | wgrant: loggerhead really needs some time | 09:16 |
lifeless | wgrant: jam has prepped patches to fix the biggest issues; they need review and landing | 09:17 |
wgrant | Ah, great. I'll have a look tomorrow. | 09:17 |
lifeless | that would be awesome | 09:17 |
=== almaisan-away is now known as al-maisan | ||
=== al-maisan is now known as almaisan-away | ||
LPCIBot | Project db-devel build (362): FAILURE in 5 hr 18 min: https://hudson.wedontsleep.org/job/db-devel/362/ | 15:43 |
LPCIBot | Launchpad Patch Queue Manager: [rs=buildbot-poller] automatic merge from stable. Revisions: 12367 | 15:43 |
LPCIBot | included. | 15:43 |
LPCIBot | Project devel build (437): STILL FAILING in 14 min: https://hudson.wedontsleep.org/job/devel/437/ | 16:01 |
thumper | morning | 20:15 |
thumper | tethering with Rachel's phone is working | 20:15 |
thumper | so I'm off to town to get a haircut :) | 20:16 |
lifeless | mwhudson: what is 'mkdir -p /var/tmp/vostok-archive | 20:17 |
lifeless | ' | 20:17 |
lifeless | for | 20:17 |
mwhudson | lifeless: i think it's a DocumentRoot for apache | 20:20 |
mwhudson | that stuff is all stillborn though :/ | 20:20 |
lifeless | mwhudson: also https://code.launchpad.net/~mwhudson/lp-production-configs/no-more-launchpad-loggerhead/+merge/24366 | 20:21 |
lifeless | mwhudson: land or delete IMO | 20:21 |
mwhudson | oh oops, that should be landed | 20:22 |
mwhudson | lifeless: do we still use the production-devel and production-stable branches? | 20:23 |
lifeless | no | 20:24 |
lifeless | we deploy from stable | 20:25 |
mwhudson | lifeless: so the config-manager/production-{stable,devel} files can go? | 20:25 |
lifeless | we have a vestigial process to use production-stable if we have something that has to be embargoed all the way | 20:25 |
lifeless | we haven't used it in 6 months though | 20:25 |
mwhudson | i guess i don't need to care about those files referencing launchpad-loggerhead, anyway | 20:25 |
lifeless | mwhudson: also, while you are around | 20:26 |
lifeless | whats the deal with the different theme in lp's loggerhead | 20:26 |
mwhudson | at the time the idea was just to make the launchpad loggerhead blend in with the lp colour scheme a bit more | 20:26 |
lifeless | mwhudson: it seems to be a different colour scheme than lp itself has though ? | 20:27 |
mwhudson | it may have been less different once | 20:27 |
mwhudson | the 'codehosting orange' matches up, at least | 20:28 |
lifeless | codehosting oragne? | 20:28 |
* mwhudson realizes that that probably wasn't useulf | 20:28 | |
mwhudson | lifeless: the colour the h1 is in on https://code.launchpad.net/~mwhudson/lp-production-configs/no-more-launchpad-loggerhead | 20:29 |
mwhudson | is used in the lp loggerhead theme | 20:29 |
lifeless | oh | 20:29 |
lifeless | I'd always thought the h1's were arbitrary | 20:29 |
lifeless | nice | 20:43 |
lifeless | best overall 99th percentile yet, I think | 20:44 |
lifeless | 2.63 - https://devpad.canonical.com/~lpqateam/ppr/lpnet/latest-daily-categories.html | 20:44 |
lifeless | 6M renders | 20:44 |
michaelh1 | Morning. How can I register my application with login.lp.net so that I can get group membership information? | 21:00 |
* mwhudson listens carefully | 21:00 | |
lifeless | losas do that | 21:12 |
michaelh1 | lifeless: sorry, I don't understand | 21:42 |
lifeless | michaelh1: you need to talk to a losa | 21:50 |
lifeless | the operators of login.ubuntu.com | 21:50 |
lifeless | michaelh1: login.lp.net is just a theme on l.u.c. | 21:50 |
* thumper is futzing around with lazr.restful again | 21:52 | |
thumper | tests failed on Friday | 21:52 |
thumper | due to the fix I did | 21:52 |
thumper | the tests need to be updated, as it is working now as designed | 21:52 |
lifeless | wgrant: around? | 22:32 |
wgrant | lifeless: Yup. | 22:33 |
wgrant | mwhudson: Except that codehosting orange's reign ended yesterday. | 22:33 |
lifeless | wgrant: I'm wondering if I could ask you to do the rollback for https://bugs.launchpad.net/launchpad/+bug/632847 | 22:33 |
_mup_ | Bug #632847: Bug page OOPS when viewed in deactivated project context <404> <lp-bugs> <oops> <qa-needstesting> <Launchpad itself:Fix Committed by jcsackett> < https://launchpad.net/bugs/632847 > | 22:33 |
mwhudson | wgrant: hence 'at the time' :) | 22:34 |
wgrant | What's broken? | 22:34 |
lifeless | wgrant: correlated subquery on every bug search by a logged in user ? | 22:35 |
lifeless | wgrant: back to product for disabled products - something like 5% selectivity. | 22:35 |
lifeless | wgrant: its madness and will break performance | 22:35 |
wgrant | Ah, I hadn't actually looked at the diff yet. | 22:35 |
wgrant | Have you tested the performance hit on qastaging? | 22:36 |
wgrant | Yes, it looks bad, but I'd like to have something more concrete before I roll something back. | 22:36 |
lifeless | ok | 22:41 |
lifeless | I'm going to make you an admin of ~registry on qastaging | 22:41 |
lifeless | and now I've removed ~launchpad from it | 22:42 |
lifeless | OOPS-1870QS55) | 22:44 |
lifeless | wgrant: I've added another comment | 22:45 |
wgrant | Thanks. | 22:45 |
wgrant | mwhudson: What do you think about https://code.launchpad.net/~mwhudson/lp-production-configs/remove-production-max_workers_per_machine/+merge/19671? | 22:46 |
* thumper wipes brow | 22:55 | |
thumper | phew | 22:55 |
thumper | icky tests fixed | 22:55 |
wgrant | lifeless: Rolling it back. | 22:56 |
wgrant | But it'll be a couple of hours. | 22:56 |
wgrant | Since the queue is up to three items. | 22:56 |
lifeless | wgrant: thanks | 22:58 |
lifeless | hmm | 22:58 |
lifeless | I can't seem to login to qastaging with a slightly old lp lib | 22:59 |
lifeless | launchpad.Launchpad.login_with('hello-world', 'https://qastaging.launchpad.net/') | 22:59 |
lifeless | -> 404 | 22:59 |
wgrant | That's no service root. | 22:59 |
wgrant | api. | 22:59 |
lifeless | thanks | 23:03 |
thumper | lifeless: 86 queries/external actions issued in 1.78 seconds | 23:17 |
thumper | lifeless: this is just for devs yeah? | 23:17 |
thumper | on the main page | 23:17 |
lifeless | yes | 23:17 |
lifeless | its controlled by a feature flag | 23:18 |
lifeless | thumper: do you like it? | 23:18 |
thumper | yep | 23:18 |
Ursinha | morning people | 23:21 |
huwshimi | Ursinha: Hello | 23:22 |
Ursinha | :) | 23:22 |
Ursinha | huwshimi, why aren't you on our team's channel? :) | 23:23 |
huwshimi | Ursinha: Oh | 23:23 |
Ursinha | I understand, we all live in the past :P | 23:23 |
lifeless | you have a team channel? | 23:23 |
huwshimi | Ursinha: yeah no-one is ever awake there :) | 23:24 |
Ursinha | lifeless, yes sir | 23:24 |
wgrant | lifeless: Hum, targetnamecache is used for sorting too. | 23:26 |
wgrant | I wonder if anyone ever uses that. | 23:26 |
wgrant | lifeless: Do you have a problem with removing targetnamecache from the UI, turning off the cron job (or maybe taking it weekly or something), and hoping that nobody notices that sort order doesn't update when they rename their products? | 23:45 |
lifeless | when is it used for sorting | 23:45 |
wgrant | When you order bug search results by location. | 23:46 |
lifeless | is that used? | 23:47 |
wgrant | Occasionally by Ubuntu. | 23:47 |
wgrant | Where targetnamecache never changes. | 23:47 |
wgrant | It could also be used in a project group context, I suppose. | 23:48 |
lifeless | sure it can | 23:48 |
lifeless | retarget source package | 23:48 |
lifeless | but you mean 'where the thing targeted changes value | 23:48 |
wgrant | Well, yes. | 23:48 |
lifeless | wgrant: so, I suggest a similar discussion to mine about search | 23:48 |
lifeless | blog + launchpad-users + microblog links | 23:49 |
lifeless | and/or | 23:49 |
lifeless | web log analysis | 23:49 |
lifeless | wgrant: we can efficiently support this, if its useful | 23:49 |
wgrant | Right. | 23:50 |
lifeless | if its not useful we should drop the column | 23:50 |
lifeless | wgrant: to answer your question | 23:50 |
lifeless | a) happy to switch from the targetname cache in the UI to the raw data; that or actually transition to not loading the related things at all, but that seems improbable to me | 23:50 |
lifeless | b) fixing the cron job is pretty straight forward, if tedious | 23:51 |
lifeless | wgrant: I agree we need to fix the short term pain | 23:53 |
lifeless | wgrant: its up to you to choose how, I think. | 23:53 |
wgrant | I will remove it from the UI and rewrite the script to do a batch update. Requires no model changes, should be a lot quicker, and won't take long to do. | 23:54 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!