/srv/irclogs.ubuntu.com/2016/04/11/#ubuntu-website.txt

davidcalleMorning o/07:33
dholbachdavidcalle, the page generation refactoring is looking good - the new test case for the disappearing articles (on the 2nd import) passes and there are only two out of 37 tests left to fix now07:43
dholbachI hope to have something up to test for staging in the next hour07:43
davidcalledholbach: \o/07:44
dholbachdavidcalle, and it should help with the work of making Didier's full-tree imports easier07:47
=== yaili_ is now known as yaili
dholbachat some stage it might make sense to look into speeding up the test suite (480.910s with postgres, 450.254s with sqlite)08:26
dholbachall tests pass now, next step: testing things manually08:27
dholbachdavidcalle, is the JS redirect fix on prod already?09:22
davidcalledholbach: it is09:22
dholbachcool, I'll file a bug for us to update the definitions, so we can drop old redirects, if that works for you09:23
dholbachhttps://bugs.launchpad.net/developer-ubuntu-com/+bug/156876009:29
ubottuLaunchpad bug 1568760 in Ubuntu Developer Portal "Replace redirect pages with redirect logic in JS" [Undecided,New]09:29
davidcalledholbach: good idea, but I would wait for a proper admin interface before adding "legacy" redirects. Hopefully I'll have time for this later this week.09:38
dholbachsure09:38
dholbachI'll update the bug09:41
davidcalleThanks :)09:55
dpmdavidcalle, on the syntax you pasted for me for CLI instructions the other day, do you tend to add </br> to the end of each command?09:56
davidcalledpm: you don't, everything in <pre><code> uses linebreaks and indentations it finds in the source view.09:58
dpmok, I wasn't sure, thanks for confirming!09:58
davidcallenp09:58
dpmdavidcalle, do you have other examples of embedded TOC, such as the one in https://developer.ubuntu.com/en/start/ubuntu-for-devices/porting-new-device, or shall I use that one for the page I'm currently working on?11:13
davidcalledpm: that's more or less the one used when it needs one, same goes for design docs actually12:38
dpmdavidcalle, thanks, I ended up using it for the get-started page :)12:38
dholbachdavidcalle, the tests were passing, but two consecutive manual imports still showed the same issue... I'm pretty certain I found the issue, I need to think about a fix for this......12:51
davidcalleouch :( *hugs*12:52
dholbachafter the import we check if there are pages which were previously imported, but not part of the current import and delete them12:52
dholbachthere's an issue with timestamps I think12:53
dholbachmaybe articles which didn't need an update on the second import ... or something12:53
dpmdavidcalle, argh, I've got an issue whereby when adding the link plugin and saving the page, it undoes all your changes :(13:00
davidcalledpm: I have a trick for this! Don't use the link plugin :)13:00
dpmyeah, a bit late :)13:01
dholbachdpm, let's file a bug for this and see if a newer version fixes it and it not, let's report it upstream :-/13:02
dholbachdpm, is it just any page where this happens?13:02
davidcalledpm: more seriously, the issue I've seen is when copying a link plugin from on page to another. I don't use it because of this.13:03
dpmdholbach, I don't know, I've seen it just now on the page I'm editing, now twice. The first time I thought there might have been some cache issues, the second one I think I can track it to the link plugin. Either that, or pages are not getting saved at all, which I don't think it's the case13:04
dpmalso, viewing the history of a page and clicking on one of the history links generates an error page13:05
dpmperhaps because my page is not yet published?13:06
dholbachmaybe you can write down what you did in a bug report and we try to try to fix it as one of the next things?13:07
dholbachin any case it's something we would know how to work around, but probably not any of the people who edit dev.u.c just every now and then :/13:07
dholbachdavidcalle, I'm close to filing a bug for this issue and remove the tracking of 'stale pages' for now as a workaround :-)13:09
dholbachit'd let us bring the importer online13:09
dholbach... and would give me a break from this problem ;-)13:09
davidcalledholbach: what does it change to the process, if you remove the tracking of stale pages?13:17
dholbachdavidcalle, we would need to delete stale pages, if e.g. a doc in snappy's branch gets deleted or renamed13:18
dholbach(which doesn't happen all that often)13:18
dholbachit'd let us deploy now and give us breathing room to fix the issue properly :)13:19
davidcalleGotcha, yeah :)13:19
dholbachdavidcalle, https://code.launchpad.net/~dholbach/developer-ubuntu-com/fix-double-imports/+merge/290978 updated13:42
dholbachmhall119, davidcalle, can we deploy ^ on staging and get a semi-recent db dump loaded? is that a lot of work to get going?15:13
davidcalledholbach: db dump will take some time, but upload to staging when I'm out of the call and can fire the vpn :)15:41
* dholbach hugs davidcalle 15:42
dholbachwhich part takes time?15:42
davidcalledholbach: the format we got the dump in. It's basically a copy of the db and not an extract of the db content. I haven't figured a correct way to push it in the db. If we simply replace the db with it, the site doesn't connect (authentication error with the db).15:45
dholbachok, I see15:47
davidcalledholbach: deployment started, see you in 15 min :)16:03
davidcalleJuju crashed, starting again 0:-)16:05
dholbachouch16:07
davidcalledholbach: network is too slow right now apparently, it's going to take ages. I'm going to leave the server, let it run. If it fails, I'll do it again later, I need to drive home. I'll send you results :)16:13
dholbachno worries16:14
* dholbach hugs davidcalle 16:14
dholbachthanks a lot for your help with this16:14
* davidcalle hugs dholbach back16:14
dholbachyou're an absolute hero16:14
davidcalledholbach: have a nice eod o/ :)16:14
dholbachthanks!16:14

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