#metabrainz

/

      • CJ_
        gcilou, Agreed. I run into the same issue. :)
      • 2016-01-13 01341, 2016

      • Leo_Verto
        Ha
      • 2016-01-13 01345, 2016

      • Leo_Verto
        "published literature"
      • 2016-01-13 01314, 2016

      • Leo_Verto
        If it's on YouTube, it's official. Take that, LordSputnik! :P
      • 2016-01-13 01320, 2016

      • ruaok
        and there is also filtering to make people's voices lower.
      • 2016-01-13 01355, 2016

      • Leo_Verto
        gcilou, what sort of microphone did you use for the video?
      • 2016-01-13 01320, 2016

      • gcilou
        Just the built in on my computer Leo_Verto
      • 2016-01-13 01303, 2016

      • Leo_Verto
        Mhm, I think the sound quality's pretty good considering that
      • 2016-01-13 01305, 2016

      • CJ_
        It came out surprisingly clear.
      • 2016-01-13 01320, 2016

      • gcilou
        Yeah...I was in my closet on the floor. lol
      • 2016-01-13 01328, 2016

      • CJ_
        Leo_Verto, My complaint was not about the quality. Just that the volume was a bit low compared to the music. :)
      • 2016-01-13 01329, 2016

      • CJ_
        gcilou, :D
      • 2016-01-13 01330, 2016

      • ruaok
        creative. :)
      • 2016-01-13 01340, 2016

      • CJ_
        gcilou, Laptop?
      • 2016-01-13 01328, 2016

      • gcilou
        CJ_, MacBook Air
      • 2016-01-13 01309, 2016

      • CJ_
        Understood.
      • 2016-01-13 01309, 2016

      • opatel99
        Anyone want to test my preliminary Facebook Event Script?
      • 2016-01-13 01321, 2016

      • CallerNo6 joined the channel
      • 2016-01-13 01332, 2016

      • gcilou
        Leo_Verto, does BB contain non-published literature then?
      • 2016-01-13 01307, 2016

      • Leo_Verto
        No, but we had a discussion on how to describe BB
      • 2016-01-13 01323, 2016

      • Leo_Verto
        Whether it stores literature or publications/books
      • 2016-01-13 01358, 2016

      • gcilou
        Ah. Well I guess "officially" it has "published literature" haha
      • 2016-01-13 01359, 2016

      • Leo_Verto
        I'm not a native English speaker but apparently a lot of people consider literature to be only high literature or something
      • 2016-01-13 01330, 2016

      • gcilou
        Oh. I can see that, but for a database, it's not really BBs place to say what's "high" and whats "low" i guess.
      • 2016-01-13 01334, 2016

      • Leo_Verto
        Yeah, I think that's one of the reasons why it says "every single book, magazine, journal and other publication ever written" on BB's index these days
      • 2016-01-13 01310, 2016

      • gcilou
        I agree.
      • 2016-01-13 01345, 2016

      • bitmap
        zas: ping
      • 2016-01-13 01312, 2016

      • zas
        pong
      • 2016-01-13 01317, 2016

      • reosarevok
        opatel99: sure :)
      • 2016-01-13 01326, 2016

      • reosarevok has some past events he hasn't added yet after all
      • 2016-01-13 01304, 2016

      • zas
        bitmap: pingu is back, we need to plan few things about it and other servers
      • 2016-01-13 01340, 2016

      • bitmap
        ok. I haven't fully read the chatlogs yet
      • 2016-01-13 01345, 2016

      • zas
        first, i would like to see how you proceed for musicbrainz server updates (is this just running cookbooks)
      • 2016-01-13 01309, 2016

      • ruaok
        zas: baron install is scheduled for midnight.
      • 2016-01-13 01345, 2016

      • zas
        i think we can try uninstalling nginx from pingu, and re-run cookbooks to see if it is installed and started (trying to reproduce the dagon issue)
      • 2016-01-13 01316, 2016

      • zas
        then in this case, pingu mb install needs to be updated anyway
      • 2016-01-13 01335, 2016

      • zas
        since you did at least one release since it was taken out
      • 2016-01-13 01345, 2016

      • bitmap
        I use `fab -H $server production` from my local machine, which basically just updates server-configs on $server and provisions it
      • 2016-01-13 01350, 2016

      • bitmap
      • 2016-01-13 01306, 2016

      • zas
        oh, this is why i didnt find things on servers ;)
      • 2016-01-13 01334, 2016

      • bitmap
        lately I've been taking $server out of rotation before I do that, though
      • 2016-01-13 01319, 2016

      • bitmap
        because it does a `git pull` before it does svc -h, causing a bunch of ISEs if the .tt files reference new actions in the perl
      • 2016-01-13 01333, 2016

      • zas
        yes, prolly safer
      • 2016-01-13 01349, 2016

      • zas
        though fab could stop server before anything
      • 2016-01-13 01310, 2016

      • JonnyJD joined the channel
      • 2016-01-13 01345, 2016

      • zas
        second, it was in question to use ws servers and website server
      • 2016-01-13 01321, 2016

      • zas
        since now we have 5 servers, we could have 4 as ws servers, and one as website-only server
      • 2016-01-13 01348, 2016

      • bitmap
        I think it's a good idea
      • 2016-01-13 01351, 2016

      • zas
        i wonder if playing with nginx proxy setup would be enough, or if it would require more
      • 2016-01-13 01315, 2016

      • zas
        because in any case we still need to eventually have a backup website server
      • 2016-01-13 01327, 2016

      • zas
        for update downtime or failure
      • 2016-01-13 01312, 2016

      • bitmap
        right now we have separate unix sockets on each app server for the website/ws, but they don't appear to help much when the search servers freak out
      • 2016-01-13 01348, 2016

      • zas
        what i would like is in fact to have 5 identical servers, and just route /ws/ (or not) requests to different servers
      • 2016-01-13 01320, 2016

      • zas
        but i don't know much about mb server in fact, so it is perhaps not that easy
      • 2016-01-13 01315, 2016

      • bitmap
        sounds easy enough, I think
      • 2016-01-13 01342, 2016

      • Bookzombie has quit
      • 2016-01-13 01357, 2016

      • Bookzombie joined the channel
      • 2016-01-13 01329, 2016

      • reosarevok
      • 2016-01-13 01351, 2016

      • reosarevok
        (as in "why we want to do this", for anyone who's wondering :) )
      • 2016-01-13 01302, 2016

      • bitmap
        we'd remove this logic from the app servers:
      • 2016-01-13 01307, 2016

      • bitmap
      • 2016-01-13 01330, 2016

      • zas
        for now, let's take pingu back in play and let it run a bit (new RAM) to all is working. I will first uninstall nginx to test cookbook behavior
      • 2016-01-13 01302, 2016

      • bitmap
        okay
      • 2016-01-13 01330, 2016

      • zas
        interesting, remove nginx with apt-get on pingu didnt stop it before...
      • 2016-01-13 01330, 2016

      • zas
        apt-get remove --purge nginx -> not installed
      • 2016-01-13 01358, 2016

      • zas
      • 2016-01-13 01329, 2016

      • zas
        ok
      • 2016-01-13 01350, 2016

      • zas
        i execute provision.sh
      • 2016-01-13 01357, 2016

      • zas
      • 2016-01-13 01356, 2016

      • zas
        it looks like the cookbook did what it is supposed to do. So the dagon issue was not from here
      • 2016-01-13 01326, 2016

      • zas
        And /run/nginx.pid was created
      • 2016-01-13 01331, 2016

      • zas
        No issue
      • 2016-01-13 01326, 2016

      • zas
        bitmap: do you test the freshly update server somehow ?
      • 2016-01-13 01332, 2016

      • stanislas
        LordSputnik: ping
      • 2016-01-13 01343, 2016

      • zas
        updated*
      • 2016-01-13 01310, 2016

      • bitmap
        I just fetch some pages with w3m once the mbserver service is up
      • 2016-01-13 01321, 2016

      • bitmap
        from ernie
      • 2016-01-13 01352, 2016

      • bitmap
        that's strange, I guess the issue may have occurred during the dist upgrade, after rebooting
      • 2016-01-13 01340, 2016

      • zas
        bitmap: yes, prolly something we can hardly reproduce. I just wanted to be sure it wasnt a cookbook issue
      • 2016-01-13 01352, 2016

      • zas
        Ok test with w3m, looks to work
      • 2016-01-13 01302, 2016

      • zas
        bitmap: do you realize you are still using a text browser ... ? (nothing wrong, i'm a former ELinks dev) ;)
      • 2016-01-13 01330, 2016

      • bitmap
        heh :)
      • 2016-01-13 01350, 2016

      • zas
        i'll re-add pingu to the set now
      • 2016-01-13 01353, 2016

      • bitmap
        yes, I just got used to the controls in w3m and never cared enough to use anything else
      • 2016-01-13 01337, 2016

      • Leo_Verto
        opatel99, is your botbot-web PR ready to be merged?
      • 2016-01-13 01342, 2016

      • zas
        pingu is handling requests, we are running on 5 servers atm
      • 2016-01-13 01301, 2016

      • reosarevok
        Yay
      • 2016-01-13 01310, 2016

      • reosarevok
        Now what, wait a bit and check for errors?
      • 2016-01-13 01322, 2016

      • zas
        Yes.
      • 2016-01-13 01347, 2016

      • zas
        Then we'll set the website/ws split
      • 2016-01-13 01302, 2016

      • zas
        Mineo: can you have a look at https://github.com/musicbrainz/picard/pull/454 ? Just tell if it looks fine to you, i'll merge it if you agree.
      • 2016-01-13 01318, 2016

      • opatel99
        Leo_Verto: I don't see why not?
      • 2016-01-13 01340, 2016

      • Leo_Verto
        just wanted to be sure :)
      • 2016-01-13 01319, 2016

      • opatel99
      • 2016-01-13 01313, 2016

      • reosarevok
        opatel99: I don't seem to see an import button in https://www.facebook.com/events/936411456432893/ - where should I be looking? :)
      • 2016-01-13 01324, 2016

      • opatel99
        Right above the name
      • 2016-01-13 01349, 2016

      • opatel99
        I'll style and position it later to fit in with the other buttons
      • 2016-01-13 01329, 2016

      • Leo_Verto
        stanislas, I'm really confused about this commit https://github.com/bookbrainz/bookbrainz-ws/commi…. What branch is it on? It's not a PR, is it? Did you use git subtrees or something?
      • 2016-01-13 01359, 2016

      • opatel99
        reosarevok: Did you find it?
      • 2016-01-13 01339, 2016

      • reosarevok
        Nope
      • 2016-01-13 01329, 2016

      • reosarevok
        Which browser are you in?
      • 2016-01-13 01343, 2016

      • reosarevok
        (I'm in Chrome with Tampermonkey in case it matters)
      • 2016-01-13 01346, 2016

      • stanislas
        Leo_Verto: I am curious what is that. a) this commit is removed b) I am commiting only to my fork
      • 2016-01-13 01318, 2016

      • Leo_Verto
        It also refers to parent commits that don't exist
      • 2016-01-13 01330, 2016

      • Leo_Verto
        odd o_O
      • 2016-01-13 01343, 2016

      • stanislas
        Leo_Verto: So you have a point then. Where did you find this commit ?
      • 2016-01-13 01312, 2016

      • Leo_Verto
        It popped up in my GH notifications
      • 2016-01-13 01350, 2016

      • Leo_Verto
        And it has a different commit # in your fork
      • 2016-01-13 01338, 2016

      • Leo_Verto
        But the parent commit# remains the same
      • 2016-01-13 01310, 2016

      • stanislas
        Leo_Verto: btw. do you know how to get the user from a request in bbws/entity.py in get ? In put and others, it is request.oauth.user, but get doesn't require authentication.
      • 2016-01-13 01331, 2016

      • stanislas
        Leo_Verto; It doesn't appear in any bookbrainz/bookbrainz-ws branches
      • 2016-01-13 01311, 2016

      • stanislas
        * branch
      • 2016-01-13 01329, 2016

      • Leo_Verto
      • 2016-01-13 01337, 2016

      • Leo_Verto
        user.user_id
      • 2016-01-13 01349, 2016

      • opatel99
      • 2016-01-13 01351, 2016

      • Leo_Verto
        Ohh
      • 2016-01-13 01359, 2016

      • Leo_Verto
        Sorry, misunderstood your question
      • 2016-01-13 01328, 2016

      • reosarevok
        opatel99: yup, it does :)
      • 2016-01-13 01338, 2016

      • reosarevok
        Wonder what's the difference...
      • 2016-01-13 01328, 2016

      • reosarevok
        Also, you want to seed the URL also to the URL list, not just the edit note :) See how the iTunes one does it for the iTunes URL I guess :)
      • 2016-01-13 01321, 2016

      • gcilou
        reosarevok, I don't know if you saw this or not :) http://chatlogs.metabrainz.org/brainzbot/metabrai…
      • 2016-01-13 01311, 2016

      • reosarevok
        I can extend at least, sure!
      • 2016-01-13 01320, 2016

      • reosarevok
        Will check it tomorrow, no school so I have time :)
      • 2016-01-13 01324, 2016

      • reosarevok
        So extend today seems good
      • 2016-01-13 01329, 2016

      • stanislas
        Leo_Verto: I've meant the user that is currently doing the query.
      • 2016-01-13 01337, 2016

      • reosarevok
        gcilou: done :) Remind me to actually check the vid tomorrow too!
      • 2016-01-13 01314, 2016

      • gcilou
        Thanks :) I'll bug you about it tomorrow..... ;)
      • 2016-01-13 01355, 2016

      • LordSputnik
        stanislas: there's no way to get the user in GET /entity/:gid
      • 2016-01-13 01305, 2016

      • opatel99
        reosarevok: Can I get the event that the script doesn't work on? My suspicion is that React is messing with the class names.
      • 2016-01-13 01325, 2016

      • LordSputnik
        What you'll have to do for this feature is either take the user id as a query parameter in the URL, or a language_id
      • 2016-01-13 01326, 2016

      • stanislas
        LordSputnik: That's what I thought actually, there's no authentication = there's no user :)
      • 2016-01-13 01326, 2016

      • reosarevok
      • 2016-01-13 01331, 2016

      • LordSputnik
        Either one is acceptable to me ;)
      • 2016-01-13 01357, 2016

      • LordSputnik
        Leo_Verto: thanks for helping :)
      • 2016-01-13 01309, 2016

      • LordSputnik
        Leo_Verto: I think commits for forked repos may be accessible through the upstream repo URL, but I'm not sure
      • 2016-01-13 01317, 2016

      • stanislas
        LordSputnik: How to add a query parameter ?
      • 2016-01-13 01321, 2016

      • LordSputnik
        That's how I explained it to myself anyway :P
      • 2016-01-13 01357, 2016

      • opatel99
        reosarevok: Suspicion confirmed.
      • 2016-01-13 01316, 2016

      • LordSputnik
      • 2016-01-13 01315, 2016

      • stanislas
        LordSputnik: Ok, that looks fairly easy, thanks :)
      • 2016-01-13 01310, 2016

      • stanislas
        LordSputnik: And how to get the value ?
      • 2016-01-13 01305, 2016

      • ruaok
        have a good flight Leftmost
      • 2016-01-13 01325, 2016

      • Leftmost
        Thanks, ruaok. :)
      • 2016-01-13 01338, 2016

      • ruaok
        :)
      • 2016-01-13 01339, 2016

      • stanislas
        LordSputnik: no question
      • 2016-01-13 01339, 2016

      • Leftmost is getting ready to board right now.
      • 2016-01-13 01346, 2016

      • ruaok knew htat
      • 2016-01-13 01352, 2016

      • ruaok
        that even
      • 2016-01-13 01305, 2016

      • Leftmost
        You're stalking me? :)
      • 2016-01-13 01309, 2016

      • LordSputnik
        stanislas: check a few lines down from there - args.parse_args() parses the url, the you can access the values through the args variable