#musicbrainz-devel

/

      • ianmcorvidae
        if we had any clients using it we might know
      • 2013-11-18 32243, 2013

      • ianmcorvidae
        the tests work on production, the multiple-nonce protection may not
      • 2013-11-18 32244, 2013

      • ocharles
        we could easily push to graphite to get that info
      • 2013-11-18 32257, 2013

      • ianmcorvidae
        (that being the thing that's broken)
      • 2013-11-18 32201, 2013

      • ocharles
        to see if that path is at least executed
      • 2013-11-18 32239, 2013

      • ianmcorvidae
        we could put in something there for a week or something, but are people okay with us getting rid of that code if, as I expect, it's not used at all?
      • 2013-11-18 32206, 2013

      • ruaok
        sounds like a good approach
      • 2013-11-18 32212, 2013

      • ianmcorvidae
        okay
      • 2013-11-18 32228, 2013

      • ianmcorvidae
        segue, then: ocharles, do we have statsd running somewhere so we can shove something into the code to track this? :)
      • 2013-11-18 32237, 2013

      • ianmcorvidae
        (I assume a statsd thing would be the easiest approach)
      • 2013-11-18 32246, 2013

      • luks
        I think it would be easier to check in the DB if there are any mac tokens
      • 2013-11-18 32252, 2013

      • ianmcorvidae
        oh, that's a good point
      • 2013-11-18 32227, 2013

      • ruaok
        right then.
      • 2013-11-18 32233, 2013

      • ruaok
        that makes that easier.
      • 2013-11-18 32237, 2013

      • ocharles
        ianmcorvidae: we do have statsd somewhere, but you only need carbon-aggregator for this
      • 2013-11-18 32243, 2013

      • ocharles
        luks: oh, heh, that'd be much easier :)
      • 2013-11-18 32249, 2013

      • ruaok
        should we be done or should we be good and do a DR?
      • 2013-11-18 32255, 2013

      • ianmcorvidae
        select * from editor_oauth_token where mac_key is not null; returns no rows
      • 2013-11-18 32212, 2013

      • ruaok
        nuke it then. :)
      • 2013-11-18 32237, 2013

      • luks
        implementing it was a bad decision in the first place, nobody actually uses it
      • 2013-11-18 32256, 2013

      • ianmcorvidae
        okay
      • 2013-11-18 32200, 2013

      • ianmcorvidae
        I'll get rid of it then
      • 2013-11-18 32204, 2013

      • ianmcorvidae
        then I'll be able to run tests again :)
      • 2013-11-18 32207, 2013

      • ianmcorvidae
        and done, onward
      • 2013-11-18 32221, 2013

      • ruaok
        I think onward is… onward with our evenings.
      • 2013-11-18 32229, 2013

      • ruaok
        thanks for your time, everyone.
      • 2013-11-18 32231, 2013

      • nikki
        there was something I wanted to bring up today, but I still can't remember what :P
      • 2013-11-18 32238, 2013

      • ruaok
        that was an enlightening discussion on spam, TBH.
      • 2013-11-18 32249, 2013

      • ruaok
        </BANG>
      • 2013-11-18 32249, 2013

      • MBChatLogger
        MBChatLogger has changed the topic to: http://musicbrainz.org/#devel
      • 2013-11-18 32223, 2013

      • nikki
        can we make MBChatLogger stop doing that?
      • 2013-11-18 32237, 2013

      • navap
        (what's the point of that URL anyway)
      • 2013-11-18 32218, 2013

      • ruaok
        we need to ditch that old logger soon anyway.
      • 2013-11-18 32231, 2013

      • navap
        Question: Have we considered providing musicbrainz-server as an ubuntu package? I attended a talk where some people were handling dependencies by doing that
      • 2013-11-18 32253, 2013

      • reosarevok joined the channel
      • 2013-11-18 32253, 2013

      • ocharles
        navap: we're almost there
      • 2013-11-18 32257, 2013

      • ocharles
        at least in terms of dependencies
      • 2013-11-18 32211, 2013

      • ocharles
        it just needs libmusicbrainz-server-dev to be made
      • 2013-11-18 32232, 2013

      • ocharles
        all of the pieces that make up libmusicbrainz-server-dev exist on my launchpad
      • 2013-11-18 32248, 2013

      • ocharles
        that is in fact how we deploy musicbrainz to our own servers now
      • 2013-11-18 32201, 2013

      • ocharles
        and once we get libmusicbrainz-server-dev, then our servers can be better pruned of unused dependencies
      • 2013-11-18 32252, 2013

      • navap
        Well, that's good news then, I think
      • 2013-11-18 32200, 2013

      • derwin joined the channel
      • 2013-11-18 32248, 2013

      • ianmcorvidae
        hm, I guess for nofollow I'll need to provide a patch for Text::WikiFormat
      • 2013-11-18 32210, 2013

      • fractalizator2 joined the channel
      • 2013-11-18 32242, 2013

      • ocharles
        ianmcorvidae: I have maintainership for that
      • 2013-11-18 32251, 2013

      • ianmcorvidae
        ocharles: yeah, I know
      • 2013-11-18 32255, 2013

      • ocharles
        ok :)
      • 2013-11-18 32211, 2013

      • ocharles
        time to study, bye bye!
      • 2013-11-18 32225, 2013

      • ianmcorvidae
        I'm currently putting up an MBS review that monkeypatches it with a note that moving it over is better :)
      • 2013-11-18 32208, 2013

      • fractalizator2 joined the channel
      • 2013-11-18 32218, 2013

      • marcooliveira joined the channel
      • 2013-11-18 32241, 2013

      • misterswag joined the channel
      • 2013-11-18 32201, 2013

      • fractalizator2_ joined the channel
      • 2013-11-18 32227, 2013

      • misterswag joined the channel
      • 2013-11-18 32225, 2013

      • DWSR2 joined the channel
      • 2013-11-18 32233, 2013

      • alastairp
        nikki: hello?
      • 2013-11-18 32225, 2013

      • DWSR2
        Hey, getting Caught exception in MusicBrainz::Server::Controller::Root->begin "stash takes a hash or hashref at /usr/lib/perl5/Class/MOP/Method/Wrapped.pm line 162" when accessing my local mirror. Just did a clean import of the database and restarted jetty as well, seems to give me the same thing.
      • 2013-11-18 32248, 2013

      • snartal joined the channel
      • 2013-11-18 32247, 2013

      • marcooliveira joined the channel
      • 2013-11-18 32256, 2013

      • Freso
        ianmcorvidae: Sucks about that not breaking tests.
      • 2013-11-18 32224, 2013

      • ianmcorvidae
        yeah, I'm confused about that
      • 2013-11-18 32240, 2013

      • ianmcorvidae
        if it's updated I can still merge it though, I need to fix up one of my patches and there's a chunk of regressions bitmap was working on
      • 2013-11-18 32225, 2013

      • Freso
        ianmcorvidae: I just pushed the fix. Let me update the PR real quick.
      • 2013-11-18 32251, 2013

      • Freso
        ianmcorvidae: Updated.
      • 2013-11-18 32257, 2013

      • ianmcorvidae
        cool, thanks
      • 2013-11-18 32207, 2013

      • Freso
        ianmcorvidae: I also have https://bitbucket.org/metabrainz/musicbrainz-serv… open, btw... :p
      • 2013-11-18 32214, 2013

      • ianmcorvidae
        yeah, I know
      • 2013-11-18 32238, 2013

      • Freso
        :)
      • 2013-11-18 32248, 2013

      • Freso
        Anyway.
      • 2013-11-18 32256, 2013

      • Freso
        The Bandcamp one is the more urgent one.
      • 2013-11-18 32259, 2013

      • ianmcorvidae
        yeah
      • 2013-11-18 32221, 2013

      • ianmcorvidae
        amazing, down to only 11 PRs
      • 2013-11-18 32252, 2013

      • SultS joined the channel
      • 2013-11-18 32241, 2013

      • Freso
        I think it's Google's time to do some fire-fighting.
      • 2013-11-18 32255, 2013

      • Freso
      • 2013-11-18 32246, 2013

      • warp
        hah :)
      • 2013-11-18 32233, 2013

      • Freso wouldn't want to be a Google server admin right now :p
      • 2013-11-18 32226, 2013

      • DWSR2
        Freso: huh?
      • 2013-11-18 32201, 2013

      • warp
        DWSR2: youtube.com was showing a 500 Internal Server Error.
      • 2013-11-18 32215, 2013

      • DWSR2
        oh, maybe because I'm signed in it wasn't showing for me.
      • 2013-11-18 32229, 2013

      • DWSR2
        Anyway, anyone with a suggestion for my earlier problem?
      • 2013-11-18 32233, 2013

      • DWSR2
        Hey, getting Caught exception in MusicBrainz::Server::Controller::Root->begin "stash takes a hash or hashref at /usr/lib/perl5/Class/MOP/Method/Wrapped.pm line 162" when accessing my local mirror. Just did a clean import of the database and restarted jetty as well, seems to give me the same thing.
      • 2013-11-18 32247, 2013

      • DWSR2
        Tried restarting the entire box as well, didn't seem to do anything.
      • 2013-11-18 32207, 2013

      • Freso
        DWSR2: As warp said, YouTube was utterly and completely down a few minutes ago.
      • 2013-11-18 32221, 2013

      • Freso
        It had probably at least 10 minutes of downtime.
      • 2013-11-18 32232, 2013

      • marcooliveira joined the channel
      • 2013-11-18 32227, 2013

      • reosarevok_ joined the channel
      • 2013-11-18 32220, 2013

      • LordSputnik has left the channel
      • 2013-11-18 32250, 2013

      • rvedotrc1 joined the channel
      • 2013-11-18 32221, 2013

      • ianmcorvidae wonders if anyone who has opinions on editor stats is still around
      • 2013-11-18 32247, 2013

      • ianmcorvidae
        anyway, I'm thinking I'll add three more. At present we have count.editor, count.editor.deleted, and count.editor.valid (where valid = not deleted)
      • 2013-11-18 32238, 2013

      • ianmcorvidae
        the three I'd add would be count.editor.valid.{inactive,verified,active}, where the three steps are a.) no verified email, nothing done on-site, b.) verified email but done nothing, and c.) has taken at least one action, ever