#musicbrainz-devel

/

      • Sophist-UK
        That makes parallel usage unreliable as too requests within 0.1s would give a failed response.
      • ruaok
        but people could still do it. but if we catch them, that is really bad for their business.
      • $1000 donation, sign up for a proper plan and you're back.
      • otherwise, go fuck yourself.
      • ianmcorvidae
        well, people can do lots of things to circumvent us at present anyway
      • Sophist-UK
        Yes - and the license legalities would make the $5 for personal non-commercial usage only.
      • ruaok
        if your mission critical company is down, you'd open your wallet fast. :)
      • ianmcorvidae
        it's not like we protect the replication packets :P
      • ruaok
        ianmcorvidae: of course, but that doesn't use our resources.
      • warp
        note that github has very strict limits for unauthenticated access, which is somewhat related.
      • ruaok
        only some of OSU's
      • Sophist-UK
        Exactly - they are paying for the web service not the data.
      • ianmcorvidae
        well, true
      • what I mean is that anything we do here has an element of trust
      • ruaok
        for sure.
      • ianmcorvidae
        so saying "hey, don't do that" and yelling at anyone who breaks it is pretty consistent with what we do elsewhere :)
      • Sophist-UK
        And since you are using the same rate limiting mechanism as at present, just having two versions depending on whether you have a valid key or not - then the coding should be quick and easy.
      • ruaok
        ianmcorvidae: +1
      • Sophist-UK has an anecdote.
      • ianmcorvidae: you're ok to work up the details on point #1 going forward?
      • ianmcorvidae
        ruaok: re: attaching to oauth (or whatever)? sure
      • Sophist-UK
        My partner ran a pet food company. Credit card payments and shipments took 30 mins of staff time per order.
      • ruaok
        well, due diligence on the whole #1 bit.
      • ianmcorvidae
        okay
      • ruaok
        including a rough idea of how much it would take to implement this using stripe and not 3scale
      • ianmcorvidae
        well, defining "unlimited" or whatever we're doing will be the real challenge
      • Sophist-UK
        I gave them a new web site where orders went simultaneously to dispatch and credit card payments - on the basis that you only took action to stop the shipment if the credit card payment failed.
      • ruaok
        I think in this context we're ok with assuming that there will be some constants for us to work with.
      • Sophist-UK
        In other words, assume that customers are valid until you find an exception rather than be the security police.
      • ruaok nods
      • ruaok
        which has been our model forever anyways.
      • ianmcorvidae
        okay
      • Sophist-UK
        The Keep It Simple principle would seem to be relevant.
      • ianmcorvidae
        as long as I don't have to pull a Sprint and define the actual constant for "unlimited" or whatever we decide to call it, I can move forward :)
      • (telco picked at random :P)
      • Sophist-UK
        And I like the look of Stripe.
      • ruaok
        ianmcorvidae: no such thing. we're going to be honest with our users. :)
      • ianmcorvidae
        haha, fair enough :)
      • anyway I think I'm set for that
      • and I'll ponder #2 less formally :)
      • ruaok
        ok, certainly collect questions for either/both.
      • and if roadblocks come up, drop me a mail.
      • djce joined the channel
      • what else is on your radar for us to talk about, ianmcorvidae?
      • I'd like to chat about ninja tune for a sec...
      • did you see the CSV file?
      • ianmcorvidae
        yes, I did. haven't poked it further yet
      • ruaok
        with the umpteengazillion columns?
      • I wonder if we should import it into a table and then query JSON out of it or write a script.
      • but I would very much like to get that imported into geordi soonish.
      • and do a blog post with it.
      • Sophist-UK
        though stripe seem more expensive than e.g. braintree or chargify might be cheaper
      • Sophist-UK is disappearing
      • ttfn
      • ruaok
        see, there is a bit of a stumbling block wrt to other labels.
      • ttfn
      • ianmcorvidae
        hm
      • ruaok
        no one wants to be the first one to flinch.
      • and now we have someone who said, fuck it, here is our data.
      • ianmcorvidae
        script probably is easiest, does it have any actual *ID* or anything?
      • ruaok
        so I want to make noise about that.
      • ianmcorvidae
        it looks like no
      • ruaok
        there are ISRCs and catalog numbers.
      • ianmcorvidae
        catalog numbers may do
      • ruaok
        yeah.
      • ianmcorvidae
        it may not matter, depending how likely we are to get updates to this :P
      • marcooliveira joined the channel
      • ruaok
        we are!
      • ianmcorvidae
        updates that aren't just additions?
      • ruaok
        I've added you to the data@ alias, which I've given to ninja tune for future updates.
      • ianmcorvidae
        yeah, I saw the alias
      • ruaok
        unsure.
      • warp
        ruaok: perhaps except more excel sheets in the future, so geordi should have excel support? :P
      • expect
      • *
      • ruaok
        eww.
      • but that is quite likely to be the case, sadly.
      • ocharles
        ninja tune don't normally deliver XML
      • erm
      • excel*
      • so they should be able to deliver XML
      • warp afk to get some breakfast.
      • djce joined the channel
      • marcooli_ joined the channel
      • ruaok
        ha. we got a check from amazon.de. I bet that was mostly nikki's doing.
      • ianmcorvidae
        heh, the headers appear to be *wrong*
      • ruaok
        too bad I wasn't there to bring the check to the bank.
      • ianmcorvidae: they seem correct in the excel.
      • ianmcorvidae
        hm
      • lemme see
      • ruaok
        I didn't inspect the CSV. :(
      • mb-chat-logger
        New post: blog: Server Update, 2013-06-10 <http://blog.musicbrainz.org/?p=2000>;
      • ianmcorvidae
        I'm getting myself an actual thing for opening spreadsheets
      • much as I hate it, seems necessary :P
      • I seem to be missing "total parts"
      • from what I can figure out
      • ruaok
        that column is sparsely populated.
      • sometime is contains "1" (as text), not a number.
      • I've seen 2 a few times.
      • and some with "19" odd.
      • MBJenkins
        Project musicbrainz-server_beta build #470: STILL FAILING in 1 min 43 sec: http://ci.musicbrainz.org/job/musicbrainz-serve...
      • * reosarevok: Adding how tos to the menu
      • * reosarevok: area, not label
      • ianmcorvidae
        hm, something was being weird in looking at it, loading it in a real spreadsheet it seems fine
      • heh, combining artists with |
      • MBJenkins
        * mwtuea: MBS-6073: Inaccurate range of credits when grouped at the bottom of a release page
      • * mwtuea: Add test for MBS-6073
      • ianmcorvidae
        it does seem that their catalog numbers are good though
      • MBJenkins
        Oliver Charles: MBS-6364: Fix DuplicateArtists report now that comments cannot be NULL
      • ianmcorvidae
        ijabz: ping -- http://tickets.musicbrainz.org/browse/MBH-321 I'm thinking maybe can be fixed in the search server, would it be possible to make ?init=mmap finish serving any request currently in progress before re-initializing?
      • ijabz: as I mention on https://bitbucket.org/metabrainz/musicbrainz-se... we see a few ISEs in musicbrainz-server every time indexes get rotated because of the JSON stopping somewhere random in the middle
      • ocharles
        ianmcorvidae: presumably just having it finish serving requests does not solve the problem - the server is still in rotation while it's restarting, no?
      • i mean it does prevent it from serving half a response, which is definitely wanted
      • but people could still connect to a dead search server
      • ianmcorvidae
        sure, but an outright failure nginx will catch and route to the other one
      • djce joined the channel
      • I think
      • ocharles
        or do we HUP the search server, which blocks while it reloads indexes (queueing the connection)?
      • ianmcorvidae
        we call an endpoint on the search server
      • wget -O /dev/null 'http://localhost:8080?init=mmap'
      • is how we restart :)
      • ocharles
        ok
      • so not svc -d/svc -u
      • ianmcorvidae
        nope
      • ocharles
        ok
      • ianmcorvidae
        the search stuff is sorta weird, we have an MBH ticket for improving it
      • ocharles
        then whatever ?init=mmap does should cause the current requests to be finished, and any subsequent requests queued
      • ianmcorvidae
        my concern is I'm not sure how feasible running the loadbalancer from that process is
      • so, yeah, exactly that :)
      • (except asking ijabz, since I don't know enough about the search server to know how feasible and/or difficult that is)
      • MBJenkins
        Project musicbrainz-server_beta build #473: STILL FAILING in 11 min: http://ci.musicbrainz.org/job/musicbrainz-serve...
      • ocharles
        there, good wedge of PRs reviewed
      • warp, can you please merge those that have 2+ approvals?
      • (of your PRs)
      • ianmcorvidae checks if I have any
      • you do
      • ianmcorvidae
        looks like yes! I'll do the same
      • whoo shipping things
      • MBJenkins
        * Ian McEwen: MBS-2782/MBS-2848: don't expand (in Add Medium) or show (in Edit Medium) artist credits where all changes are visible; also, condense and visually separate expanded display
      • * Ian McEwen: MBS-6296: don't show non-functional alias editing links for non-location-editors (or logged-out users)
      • * Ian McEwen: MBS-6349: s/\.tracklist// to display artist credits
      • * Ian McEwen: MBS-2782/MBS-2848: use whitespace for visual hierarchy
      • * Ian McEwen: Fix rollup replication to work properly alongside GPG.
      • hawke_1 joined the channel
      • hawke joined the channel
      • warp
        ocharles: oh, did we release? I thought we were still in freeze.
      • ijabz
        ianmcorvidae hi, Im back
      • but only ruaok knows about how we rotate the indexes ectera
      • ianmcorvidae
        ocharles: shouldn't https://bitbucket.org/metabrainz/musicbrainz-se... be merged by now?
      • ijabz: it's done by moving them in, then calling init=mmap