#musicbrainz-devel

/

      • ianmcorvidae
        bah

        • 2012-09-19 26334, 2012

      • ianmcorvidae
        it's not picking up any languages

        • 2012-09-19 26335, 2012

      • ianmcorvidae
        not sure why

        • 2012-09-19 26346, 2012

      • ocharles
        nikki: the 'brianfreud' account, right?

        • 2012-09-19 26334, 2012

      • ocharles
        nikki: also, that stuff hasn't been released

        • 2012-09-19 26339, 2012

      • ocharles
        so yes, he'll still be able to edit :)

        • 2012-09-19 26347, 2012

      • ocharles wonders where the hell that code has gone

        • 2012-09-19 26314, 2012

      • ocharles
        http://codereview.musicbrainz.org/r/2224/ i don't even have a shipit

        • 2012-09-19 26306, 2012

      • ocharles
        ianmcorvidae: if you can figure out how to do logging that'd be great, but iirc it was the whole unix socket zero down time stuff making it tricky

        • 2012-09-19 26315, 2012

      • ianmcorvidae
        yeah

        • 2012-09-19 26329, 2012

      • ianmcorvidae
        probably getting rid of -keep-stderr would at least make it end up in the nginx error log like it does on rika

        • 2012-09-19 26339, 2012

      • ianmcorvidae
        but that's somewhat gross :)

        • 2012-09-19 26350, 2012

      • ocharles
        indeed

        • 2012-09-19 26310, 2012

      • ianmcorvidae
        well, bah

        • 2012-09-19 26318, 2012

      • ianmcorvidae gives up on translations on beta for now

        • 2012-09-19 26332, 2012

      • ianmcorvidae
        (mostly because it's 3:30am and I should sleep :P)

        • 2012-09-19 26345, 2012

      • ocharles
        ianmcorvidae: re deployment crash course: setup vpn, run 'fab beta', ???, profit!

        • 2012-09-19 26305, 2012

      • ianmcorvidae
        yeah, I've got deployment working

        • 2012-09-19 26312, 2012

      • ianmcorvidae
        no need for VPN, I have .ssh/config :P

        • 2012-09-19 26330, 2012

      • ocharles
        great

        • 2012-09-19 26342, 2012

      • ocharles
        so fabric can read ProxyCommand now?

        • 2012-09-19 26346, 2012

      • ocharles
        oh, hobbes is public

        • 2012-09-19 26338, 2012

      • nikki
        ocharles: I thought the problem was that the interface wasn't unsetting email addresses properly, not that the code to prevent people from editing was broken

        • 2012-09-19 26356, 2012

      • ocharles
        it failed beta testing, so it was reverted

        • 2012-09-19 26317, 2012

      • ocharles
        the problem was thta it still used the session and stuff, so it didn't do the job properly

        • 2012-09-19 26353, 2012

      • nikki
        didn't you say the session would expire though?

        • 2012-09-19 26323, 2012

      • ocharles
        not if people keep refreshing the session

        • 2012-09-19 26336, 2012

      • nikki
        :/

        • 2012-09-19 26357, 2012

      • ocharles
        if we want to hotfix release it or something I'm ok with that, but someone needs to review my code

        • 2012-09-19 26359, 2012

      • nikki
        why did you tell me it would work then? :/

        • 2012-09-19 26330, 2012

      • ocharles
        probably because I thought it was shipped

        • 2012-09-19 26335, 2012

      • ocharles
        I can't keep 60 tickets in my head, sorry

        • 2012-09-19 26350, 2012

      • ocharles
        when this ships then I'm confident that it will work

        • 2012-09-19 26347, 2012

      • MBJenkins
        * ollie: MBS-4599 hotfix: When editing a release label that no longer exists, fail

        • 2012-09-19 26348, 2012

      • MBJenkins
        * ianmcorvidae: i18n: update translations to current

        • 2012-09-19 26349, 2012

      • MBJenkins
        * ianmcorvidae: fix the ENCODING thing again

        • 2012-09-19 26308, 2012

      • ianmcorvidae
        huh

        • 2012-09-19 26319, 2012

      • ianmcorvidae
        well, apparently ocharles deploying things *did* make beta's translations work

        • 2012-09-19 26346, 2012

      • ianmcorvidae
        (that is, the two I enabled for testing)

        • 2012-09-19 26359, 2012

      • nikki switches to german :P

        • 2012-09-19 26328, 2012

      • ocharles tries italian

        • 2012-09-19 26334, 2012

      • nikki
        hmm.

        • 2012-09-19 26344, 2012

      • nikki
        the number of columns is wrong on the tracklist page

        • 2012-09-19 26302, 2012

      • ocharles
        very cool

        • 2012-09-19 26308, 2012

      • nikki
        e.g. http://beta.musicbrainz.org/release/11b1d97f-7c3c… in german

        • 2012-09-19 26319, 2012

      • nikki
        or is it just me?

        • 2012-09-19 26347, 2012

      • ocharles
        looks ok here

        • 2012-09-19 26350, 2012

      • nikki
        oh, I bet I know

        • 2012-09-19 26336, 2012

      • ianmcorvidae
        td.url-icons is what's breaking for me, which I think is the acoustid script

        • 2012-09-19 26343, 2012

      • ianmcorvidae
        which perhaps is looking for an untranslated header

        • 2012-09-19 26335, 2012

      • nikki
        oh right, it's the userscript I have for removing ratings

        • 2012-09-19 26311, 2012

      • ianmcorvidae
        translations breaking userscripts, yay :P

        • 2012-09-19 26336, 2012

      • MBJenkins
        Project musicbrainz-server_beta build #109: STILL FAILING in 2 min 16 sec: http://ci.musicbrainz.org/job/musicbrainz-server_…

        • 2012-09-19 26337, 2012

      • MBJenkins
        * ianmcorvidae: MBS-4614: change to using named parameters in the weird perl-style way

        • 2012-09-19 26338, 2012

      • MBJenkins
        * ianmcorvidae: MBS-4614: missed one place

        • 2012-09-19 26359, 2012

      • ocharles
        right, time to cheer up beta

        • 2012-09-19 26345, 2012

      • ianmcorvidae
        meh, apparently it won't reload DBDefs unless there are other code changes or somesuch

        • 2012-09-19 26357, 2012

      • nikki
        heh, and the extra menu plus the length of veröffentlichungsgruppe makes my compact menu wrap too

        • 2012-09-19 26309, 2012

      • ianmcorvidae
        anyway, if ocharles is looking into fixing those tests there should be another real deployment soon to turn on the other languages

        • 2012-09-19 26331, 2012

      • nikki
        does it work if you just touch one of the files without making any changes?

        • 2012-09-19 26352, 2012

      • nikki
        I seem to recall doing that at some point

        • 2012-09-19 26357, 2012

      • ianmcorvidae
        I'm not sure what it's looking for, I don't really want to poke anything except translations and such in the branch though :/

        • 2012-09-19 26349, 2012

      • ianmcorvidae
        that makes sure it rebuilds the translations, I know

        • 2012-09-19 26357, 2012

      • ianmcorvidae
        but I'm not sure what's going on that it won't reload DBDefs

        • 2012-09-19 26300, 2012

      • warp returns

        • 2012-09-19 26335, 2012

      • MBJenkins
        Project musicbrainz-server_beta build #110: STILL FAILING in 2 min 15 sec: http://ci.musicbrainz.org/job/musicbrainz-server_…

        • 2012-09-19 26336, 2012

      • MBJenkins
        ollie: MBS-5019: Constraints to ensure that _name tables never have the empty string

        • 2012-09-19 26335, 2012

      • warp
        STILL FAILING

        • 2012-09-19 26326, 2012

      • ianmcorvidae
        eh

        • 2012-09-19 26311, 2012

      • warp
        someone broke it on monday

        • 2012-09-19 26335, 2012

      • ocharles
        wait...

        • 2012-09-19 26342, 2012

      • ocharles
        why is is mbs-4264 merged?

        • 2012-09-19 26343, 2012

      • mb-chat-logger
        http://tickets.musicbrainz.org/browse/MBS-4264

        • 2012-09-19 26348, 2012

      • ocharles
        I thought I said that had to depend on search server data

        • 2012-09-19 26302, 2012

      • ocharles
        EINCORRECTREVERT!

        • 2012-09-19 26330, 2012

      • ianmcorvidae
        weird

        • 2012-09-19 26345, 2012

      • ocharles
        i dunno why I merged that

        • 2012-09-19 26312, 2012

      • warp
        ocharles: are you looking into the pgtap failures on beta?

        • 2012-09-19 26317, 2012

      • ocharles
        warp: all failures

        • 2012-09-19 26322, 2012

      • djce joined the channel

        • 2012-09-19 26347, 2012

      • warp
        ocharles: ok, I'll take the qunit one.

        • 2012-09-19 26315, 2012

      • ocharles
        sounds good

        • 2012-09-19 26334, 2012

      • warp
        oh meh. http://beta.musicbrainz.org/static/scripts/tests/… succeeds.

        • 2012-09-19 26320, 2012

      • ianmcorvidae
        phantomjs wackiness then, I guess :/

        • 2012-09-19 26316, 2012

      • warp
        the test runner needs to be upgraded to modern phantomjs anyway.

        • 2012-09-19 26331, 2012

      • ijabz
        ocharles re MBS-4264 no issue has been raised on search server about this

        • 2012-09-19 26332, 2012

      • mb-chat-logger
        http://tickets.musicbrainz.org/browse/MBS-4264

        • 2012-09-19 26351, 2012

      • ocharles
        ijabz: nope, it's ianmcorvidae's job to do that :)

        • 2012-09-19 26301, 2012

      • ocharles
        or he can just add the search server work, his call

        • 2012-09-19 26314, 2012

      • ijabz
        right, but ianmcorvidae you need to create issue even if you do the work if it requires changes on search server, released independently of mbserver

        • 2012-09-19 26302, 2012

      • djce joined the channel

        • 2012-09-19 26317, 2012

      • ijabz
        Why doesn't latest posts on http://musicbrainz.org/ show the latest post

        • 2012-09-19 26348, 2012

      • nikki
        probably cached

        • 2012-09-19 26302, 2012

      • dimonov joined the channel

        • 2012-09-19 26330, 2012

      • warp
        murdos: Index Builder Finished:11:53:32

        • 2012-09-19 26346, 2012

      • murdos
        fine

        • 2012-09-19 26359, 2012

      • warp
        what do I do now?

        • 2012-09-19 26316, 2012

      • murdos
        you can now setup the updater module as described in the README

        • 2012-09-19 26324, 2012

      • warp
        which README?

        • 2012-09-19 26343, 2012

      • murdos
        http://svn.musicbrainz.org/search_server/trunk/RE…

        • 2012-09-19 26337, 2012

      • murdos
        however the updateindex.sh script is build for slave database, so I think we will launch it a bit differently

        • 2012-09-19 26340, 2012

      • warp
        it's talking about index/updater/, but that should just be updater/ it seems.

        • 2012-09-19 26302, 2012

      • murdos
        right, it's a typo

        • 2012-09-19 26322, 2012

      • murdos
        IMO we should upate indexes in two different ways:

        • 2012-09-19 26330, 2012

      • marco-oliveira joined the channel

        • 2012-09-19 26346, 2012

      • murdos
        1. artist,label,work,annotation,release-group can updated every 5 minutes

        • 2012-09-19 26355, 2012

      • murdos
        2. release,recording each hour

        • 2012-09-19 26327, 2012

      • warp
        why the difference?

        • 2012-09-19 26349, 2012

      • murdos
        release and recording take ages to index

        • 2012-09-19 26334, 2012

      • murdos
        and if they're really too long, we'll have to keep the current strategy of rebuilding every 6 hours

        • 2012-09-19 26351, 2012

      • murdos
        warp: at that point, maybe we should verify that the search webap works and make beta use it?

        • 2012-09-19 26314, 2012

      • warp
        murdos: I'm running updateindex.sh now.

        • 2012-09-19 26314, 2012

      • ocharles
        ijabz: as nikki said, we cache those posts

        • 2012-09-19 26339, 2012

      • ocharles
        though there doesn't seem to be an expirey date on the key

        • 2012-09-19 26317, 2012

      • flamingspinach joined the channel

        • 2012-09-19 26335, 2012

      • demosdemon joined the channel

        • 2012-09-19 26330, 2012

      • stefans_ joined the channel

        • 2012-09-19 26301, 2012

      • murdos
        warp: I was not expecting that you'll run it so soon

        • 2012-09-19 26306, 2012

      • murdos
        you should use a modified version of updateindex: https://gist.github.com/3749448

        • 2012-09-19 26349, 2012

      • murdos
        so you can specify indexes to be build: ./updateindex.sh --indexes artist,label,work --verbose

        • 2012-09-19 26329, 2012

      • kurtjx joined the channel

        • 2012-09-19 26337, 2012

      • warp
        murdos: it is still running, can I safely ctrl-c that one?

        • 2012-09-19 26342, 2012

      • kurtjx joined the channel

        • 2012-09-19 26342, 2012

      • murdos
        I'm not sure ctrl-c is safe

        • 2012-09-19 26358, 2012

      • murdos
        what is it doing currently?

        • 2012-09-19 26306, 2012

      • warp
        Wed Sep 19 12:56:07 UTC 2012 INFO Started updating index: recording

        • 2012-09-19 26309, 2012

      • warp
        Wed Sep 19 12:56:07 UTC 2012 INFO Current index properties: schema_sequence=15, replication_sequence=62255, change_sequence=

        • 2012-09-19 26336, 2012

      • murdos
        ok, then you should rather wait

        • 2012-09-19 26342, 2012

      • warp
        looks like release took about 10 minutes.

        • 2012-09-19 26332, 2012

      • warp
        murdos: ok, I will let you know when it finishes.

        • 2012-09-19 26357, 2012

      • murdos
        ok. is search webapp running currently on beta (not the one for test.mb of course)?

        • 2012-09-19 26323, 2012

      • warp
        murdos: it is running on localhost:8081, but beta.musicbrainz.org is not configured to use it.

        • 2012-09-19 26352, 2012

      • nikki
        ooh. beta in more languages :D

        • 2012-09-19 26346, 2012

      • MBJenkins
        * ollie: Revert "Merge remote-tracking branch 'ianmcorvidae/mbs-4264-search-results-language-script' into beta"

        • 2012-09-19 26347, 2012

      • MBJenkins
        * ollie: Fix tests on beta

        • 2012-09-19 26322, 2012

      • ocharles
        sad panda.

        • 2012-09-19 26310, 2012

      • ocharles
        ok, pgtap fixed too. warp are you still looking at qunit?

        • 2012-09-19 26332, 2012

      • warp
        ocharles: yes.

        • 2012-09-19 26307, 2012

      • ocharles
        ok, cool

        • 2012-09-19 26319, 2012

      • warp
        ocharles: well, I'm yak-shaving. but that should eventually (today) lead to me fixing that qunit thing

        • 2012-09-19 26321, 2012

      • ocharles
        so we should have a blue light by the end of the day

        • 2012-09-19 26309, 2012

      • reosarevok joined the channel

        • 2012-09-19 26314, 2012

      • kurtjx_ joined the channel

        • 2012-09-19 26357, 2012

      • kurtjx_ joined the channel

        • 2012-09-19 26329, 2012

      • MBJenkins
        Project musicbrainz-server_beta build #112: STILL FAILING in 13 min: http://ci.musicbrainz.org/job/musicbrainz-server_…

        • 2012-09-19 26330, 2012

      • MBJenkins
        * ollie: Fix pgtap tests

        • 2012-09-19 26324, 2012

      • ocharles
        ok, just qunit to go now

        • 2012-09-19 26307, 2012

      • kurtjx joined the channel

        • 2012-09-19 26304, 2012

      • ocharles
        nikki: prod on http://codereview.musicbrainz.org/r/2280/

        • 2012-09-19 26332, 2012

      • warp
        ocharles: ok, done. I've upgraded our qunit and replaced my home-grown testrunner with https://github.com/twada/qunit-tap. so the patch is big but the changes are minor.