#musicbrainz-devel

/

      • ianmcorvidae
        were we hoping to do that for /ws/2?

        • 2013-02-14 04534, 2013

      • warp
        well, it's going to take centuries for clients to start using them, so we should implement them in some fashion ASAP so clients who want to can already start :)

        • 2013-02-14 04543, 2013

      • ianmcorvidae
        heh, yeah

        • 2013-02-14 04503, 2013

      • warp
        even if they're mostly dummy values on our end at first.

        • 2013-02-14 04504, 2013

      • ianmcorvidae
        I guess for /ws/2 we'd have to have an upper limit of 1/s (i.e., the headers can only give you better than that)

        • 2013-02-14 04508, 2013

      • ianmcorvidae
        yeah

        • 2013-02-14 04512, 2013

      • ianmcorvidae
        anyway, point stands that we should discuss! :)

        • 2013-02-14 04549, 2013

      • warp
        I'll create a ticket if we don't have one yet.

        • 2013-02-14 04535, 2013

      • ianmcorvidae
        looks like no, everything on jira that mentions rate limiting is pretty much stuff for 3scale :)

        • 2013-02-14 04529, 2013

      • MBJenkins
        * mwtuea: MBS-3971: Display recording relationships at the bottom of the release page

        • 2013-02-14 04530, 2013

      • MBJenkins
        * mwtuea: Add back "has no relationships" notice

        • 2013-02-14 04531, 2013

      • MBJenkins
        * mwtuea: refactor relationship grouping code a bit

        • 2013-02-14 04541, 2013

      • warp
        ianmcorvidae: http://tickets.musicbrainz.org/browse/MBS-5874

        • 2013-02-14 04546, 2013

      • djce joined the channel

        • 2013-02-14 04511, 2013

      • Leftmost
        ianmcorvidae, I just need to change that to a detach_404, right?

        • 2013-02-14 04547, 2013

      • ianmcorvidae
        Leftmost: not sure -- it may just be 'not_found'

        • 2013-02-14 04510, 2013

      • Leftmost
        I see a bunch of /error_404s in the code.

        • 2013-02-14 04532, 2013

      • ianmcorvidae
        you should make sure I'm right first, too, but :P

        • 2013-02-14 04510, 2013

      • Leftmost
        It is a privacy leak, I think, but it's one that's copied over from an existing one in the code.

        • 2013-02-14 04510, 2013

      • ianmcorvidae
        hm, I think $c->detach('/error_404') would do it, yes

        • 2013-02-14 04512, 2013

      • ianmcorvidae
        yeah

        • 2013-02-14 04526, 2013

      • Leftmost
        I'll file a separate ticket for the other one and correct this one.

        • 2013-02-14 04536, 2013

      • ianmcorvidae
        anyway, try that and make sure it returns the same page for a nonexistent versus a private collection, I guess, that should be all

        • 2013-02-14 04502, 2013

      • Leftmost
        It doesn't, because non-existant collections break. Whoops.

        • 2013-02-14 04540, 2013

      • djce joined the channel

        • 2013-02-14 04519, 2013

      • sezuan joined the channel

        • 2013-02-14 04534, 2013

      • Leftmost
        And existant is a non-existent word.

        • 2013-02-14 04512, 2013

      • Leftmost
        Is codereview.mb.o having some issues?

        • 2013-02-14 04517, 2013

      • Leftmost
        Or is it me?

        • 2013-02-14 04530, 2013

      • warp
        seems fine here.

        • 2013-02-14 04532, 2013

      • Leftmost
        Bits of diffs keep wanting to not load, then it doesn't want to respond at all.

        • 2013-02-14 04519, 2013

      • Leftmost
        ianmcorvidae, all fixed up.

        • 2013-02-14 04554, 2013

      • MBJenkins
        * mwtuea: MBS-5591: pressing enter with <select> open submits dialog

        • 2013-02-14 04555, 2013

      • MBJenkins
        * mwtuea: warpify var statements

        • 2013-02-14 04528, 2013

      • ijabz joined the channel

        • 2013-02-14 04556, 2013

      • ZaphodBeeblebrox
        ¬_¬ warp

        • 2013-02-14 04556, 2013

      • warp joined the channel

        • 2013-02-14 04548, 2013

      • reosarevok joined the channel

        • 2013-02-14 04519, 2013

      • MBJenkins
        mwtuea: MBS-5836: validate entity types against link types in relationship editor

        • 2013-02-14 04559, 2013

      • sezuan joined the channel

        • 2013-02-14 04504, 2013

      • MBJenkins
        Project musicbrainz-server_beta build #324: SUCCESS in 29 min: http://ci.musicbrainz.org/job/musicbrainz-server_…

        • 2013-02-14 04505, 2013

      • MBJenkins
        * Ian McEwen: MBS-4947: generatesql update

        • 2013-02-14 04524, 2013

      • CatCat
        Now grooving to: ZZ Top ~ Rough Boy (4:52) (from Greatest Hits) (1986)

        • 2013-02-14 04529, 2013

      • CatCat
        erh wromng srry

        • 2013-02-14 04543, 2013

      • ZaphodBeeblebrox has left the channel

        • 2013-02-14 04557, 2013

      • voiceinsideyou joined the channel

        • 2013-02-14 04533, 2013

      • dukeleto joined the channel

        • 2013-02-14 04558, 2013

      • dukeleto joined the channel

        • 2013-02-14 04515, 2013

      • derr84|2 joined the channel

        • 2013-02-14 04556, 2013

      • dukeleto joined the channel

        • 2013-02-14 04545, 2013

      • voiceinsideyou joined the channel

        • 2013-02-14 04546, 2013

      • Leftmost joined the channel

        • 2013-02-14 04554, 2013

      • dukeleto joined the channel

        • 2013-02-14 04537, 2013

      • hawke_1 joined the channel

        • 2013-02-14 04541, 2013

      • hawke joined the channel

        • 2013-02-14 04516, 2013

      • warp sighs at code review.

        • 2013-02-14 04532, 2013

      • Leftmost
        It's my fault, isn't it? :-P

        • 2013-02-14 04538, 2013

      • warp
        no, code review just has some weird quirks which I never seem to get used to.

        • 2013-02-14 04512, 2013

      • Leftmost
        It's okay, you can blame that on me.

        • 2013-02-14 04525, 2013

      • warp
        for example: http://codereview.musicbrainz.org/r/2526/

        • 2013-02-14 04554, 2013

      • warp
        why does Oliver get one big block with all his comments in it, and I get three separate blocks. it does not make sense.

        • 2013-02-14 04511, 2013

      • JonnyJD
        luks: did you create libdiscid milestones manually or did launchpad pick that information up from my settings automatically (after some time)?

        • 2013-02-14 04534, 2013

      • JonnyJD
        I mean I set everything up and was hoping it would catch that information somehow, but I am suprised why that happens now and not 1-2 hours after I added that information.

        • 2013-02-14 04518, 2013

      • dukeleto joined the channel

        • 2013-02-14 04522, 2013

      • luks
        JonnyJD: sorry, I don't remember

        • 2013-02-14 04545, 2013

      • JonnyJD
        luks: No, I mean like 13 hours ago.

        • 2013-02-14 04527, 2013

      • luks
        what are you using launchpad for anyway?

        • 2013-02-14 04529, 2013

      • JonnyJD
        luks: libdiscid went from no releases/tags/milestones to all releases 13 hours ago.

        • 2013-02-14 04544, 2013

      • JonnyJD
        Just reminding ubuntu packaging guys when there are new releases

        • 2013-02-14 04548, 2013

      • luks
        ah

        • 2013-02-14 04508, 2013

      • luks
        I'm not sure that is of any help

        • 2013-02-14 04521, 2013

      • luks
        the package is primarily maintained in debian, so will be just copied from there

        • 2013-02-14 04522, 2013

      • JonnyJD
        would be nice when they just catch downloads and tags automatically and it certainly looks like it now

        • 2013-02-14 04514, 2013

      • JonnyJD
        yes, no clue what the actual synchronizing workflow is. In debian 0.3.0 is in experimental and it looks like they have a new maintainer now

        • 2013-02-14 04505, 2013

      • JonnyJD
        and having an outdated project in launchpad doesn't help. Deleting projects in launchpad is impossible as far as i see

        • 2013-02-14 04518, 2013

      • Leftmost joined the channel

        • 2013-02-14 04523, 2013

      • JonnyJD
        luks: anyways. Thanks for the information. I hope that launchpad project is working "by itself" now.

        • 2013-02-14 04525, 2013

      • warp
        ocharles: how/where do I review NES ?

        • 2013-02-14 04531, 2013

      • mhanney joined the channel

        • 2013-02-14 04522, 2013

      • mhanney
        please can anyone direct me to documentation for running musicbrainz with something other than the lightweight development server described in the INSTALL.md. I understand how nginx and fastCGI work, but never done it with perl before. Thanks.

        • 2013-02-14 04533, 2013

      • derr84 joined the channel

        • 2013-02-14 04515, 2013

      • warp
        mhanney: see admin/nginx/001-musicbrainz

        • 2013-02-14 04534, 2013

      • warp
        mhanney: and we run the server itself using daemontools, the daemontools run file contains something like this:

        • 2013-02-14 04501, 2013

      • warp
        mhanney: plackup -Ilib -s FCGI -E deployment --port 55901 --nproc 100 -pid fcgi.pid --keep-stderr=1

        • 2013-02-14 04534, 2013

      • warp
        I don't think our daemontools run scripts are in a git repository, they probably should be.

        • 2013-02-14 04538, 2013

      • hawke joined the channel

        • 2013-02-14 04510, 2013

      • hawke_1 joined the channel

        • 2013-02-14 04538, 2013

      • Ben\Sput joined the channel

        • 2013-02-14 04516, 2013

      • ruaok joined the channel

        • 2013-02-14 04517, 2013

      • ruaok joined the channel

        • 2013-02-14 04551, 2013

      • mhanney
        thank you derr84, awesome

        • 2013-02-14 04500, 2013

      • sezuan joined the channel

        • 2013-02-14 04545, 2013

      • Ben\Sput has left the channel

        • 2013-02-14 04540, 2013

      • voiceinsideyou joined the channel

        • 2013-02-14 04541, 2013

      • mhanney
        derr84, looks like a daemontools run file is in repo here admin/nginx/service/run

        • 2013-02-14 04529, 2013

      • mb-chat-logger joined the channel

        • 2013-02-14 04519, 2013

      • Freso joined the channel

        • 2013-02-14 04500, 2013

      • Freso joined the channel

        • 2013-02-14 04511, 2013

      • warp
        mhanney: oh indeed :)

        • 2013-02-14 04503, 2013

      • warp is not derr84 though. :P

        • 2013-02-14 04534, 2013

      • mhanney
        Thank you warp, sorry for the case of mistaken identity, Your help is very much appreciated.

        • 2013-02-14 04551, 2013

      • ruaok joined the channel

        • 2013-02-14 04534, 2013

      • sezuan joined the channel

        • 2013-02-14 04521, 2013

      • ruaok_ joined the channel

        • 2013-02-14 04516, 2013

      • ijabz joined the channel

        • 2013-02-14 04501, 2013

      • ruaok
        kepstin-work: ping

        • 2013-02-14 04511, 2013

      • kepstin-work
        hi

        • 2013-02-14 04534, 2013

      • ruaok
        I'm working on making a LATEST files or something for the change-mbids feed.

        • 2013-02-14 04545, 2013

      • ruaok
        what should the file contain?

        • 2013-02-14 04547, 2013

      • ruaok
        a number?

        • 2013-02-14 04549, 2013

      • ruaok
        a filename?

        • 2013-02-14 04555, 2013

      • ruaok
        full path?

        • 2013-02-14 04507, 2013

      • ruaok
        full path isn't quite future proof if it gets mirrored.

        • 2013-02-14 04510, 2013

      • kepstin-work
        either the number or the filename would be best, I think.

        • 2013-02-14 04522, 2013

      • ruaok
        I'm leaning towards filename.

        • 2013-02-14 04558, 2013

      • kepstin-work
        since the application has to parse out the number anyways to sequence the file downloads, I think I'd prefer just the number. But either works.

        • 2013-02-14 04547, 2013

      • ruaok
        ok, I'll do that.

        • 2013-02-14 04531, 2013

      • ocharles
        is having this be a http service off the plan atm?

        • 2013-02-14 04534, 2013

      • ocharles
        this latest stuff is much simpler if it's just a 3xx redirect

        • 2013-02-14 04529, 2013

      • kepstin-work
        ocharles: the benefit of the current solution is that it's trivial to mirror without special server support

        • 2013-02-14 04555, 2013

      • ocharles
        sure, but at the cost of users

        • 2013-02-14 04506, 2013

      • ocharles
        and given we have none of them right now... we don't exactly need to worry about scaling :)

        • 2013-02-14 04542, 2013

      • marcooliveira joined the channel

        • 2013-02-14 04546, 2013

      • kepstin-work
        well, the point of the latest file is that an application wants to find out "what is the sequence number of the most recent update packet?"

        • 2013-02-14 04522, 2013

      • kepstin-work
        just reading that from the body of a request is probably easier than parsing the number from the url returned in a redirect

        • 2013-02-14 04527, 2013

      • ruaok
        ocharles: I'm hoping someone will adopt this code.

        • 2013-02-14 04546, 2013

      • ocharles
        you wouldn't parse from the URL, you'd fetch the latest which is a json file in its own right, with a sequence number inside it

        • 2013-02-14 04541, 2013

      • ocharles
        ruaok: cool

        • 2013-02-14 04558, 2013

      • ocharles grabs all the changes and see's how his toy service holds up

        • 2013-02-14 04550, 2013

      • ocharles
        uff, still no robots.txt to let me crawl

        • 2013-02-14 04501, 2013

      • Prophet5 joined the channel

        • 2013-02-14 04536, 2013

      • ruaok
        ha. foiled.

        • 2013-02-14 04539, 2013

      • ruaok
        want one?

        • 2013-02-14 04544, 2013

      • ruaok
        what should it say?

        • 2013-02-14 04549, 2013

      • ruaok
        permit everything?

        • 2013-02-14 04516, 2013

      • ocharles
        for my needs, yes :)

        • 2013-02-14 04526, 2013

      • ocharles
        i ended up scripting `seq` and `curl` to do the same

        • 2013-02-14 04528, 2013

      • kepstin-work
        that would have to be at the top level of http://ftp.musicbrainz.org tho, which is hosted by osusl, right?

        • 2013-02-14 04540, 2013

      • ocharles
        kepstin-work: i'm downloading from http://changed-mbids.musicbrainz.org

        • 2013-02-14 04535, 2013

      • kepstin-work
        huh, is that gonna be the final location for them, or is this still subject to change?

        • 2013-02-14 04554, 2013

      • ruaok
        that isn't the final location.

        • 2013-02-14 04507, 2013

      • ruaok
        its mapped into ftp

        • 2013-02-14 04523, 2013

      • ruaok
        most people should be fetching from there, but ocharles should probably fetch from ch.mb.org

        • 2013-02-14 04521, 2013

      • ocharles
        whats-changed or whats-new is quite a fun name

        • 2013-02-14 04533, 2013

      • ocharles
        ftr, a 2 week catchup (downloading all those json files) takes around 6 minutes

        • 2013-02-14 04559, 2013

      • ocharles
        while that's quite a big delay, i'm not so sure that's extreme

        • 2013-02-14 04525, 2013

      • kepstin-work would expect it to go faster than that

        • 2013-02-14 04547, 2013

      • ocharles
        that's what it is

        • 2013-02-14 04547, 2013

      • marcooliveira joined the channel

        • 2013-02-14 04503, 2013

      • ocharles
        i imagine Keep-Alive will do you some good, but you're gonna roundtrip on every json packet