#metabrainz

/

      • alastairp
        ok, great
      • 2021-04-26 11655, 2021

      • alastairp
        from a quick glance 1409 looks great, I like it!
      • 2021-04-26 11622, 2021

      • _lucifer
        but i think i might need to merge the latest master into this to bring in Mr_Monkey's spotify player fix
      • 2021-04-26 11630, 2021

      • alastairp
        _lucifer: if you have time, could you have a quick glance at https://github.com/metabrainz/listenbrainz-server… ? I see it's on master on your fork, and has a few conflicts.
      • 2021-04-26 11650, 2021

      • alastairp
        ruaok tested the BU rate limiter last week (I thought it was buggy) and found no problems, so this sh
      • 2021-04-26 11654, 2021

      • alastairp
        should be ready to merge
      • 2021-04-26 11619, 2021

      • _lucifer
        sure, give me 15 mins
      • 2021-04-26 11632, 2021

      • alastairp
        no rush on this one, I won't get to it until tomorrow
      • 2021-04-26 11654, 2021

      • nawcom has quit
      • 2021-04-26 11600, 2021

      • nawcom joined the channel
      • 2021-04-26 11657, 2021

      • nawcom has quit
      • 2021-04-26 11612, 2021

      • nelgin
        sudo docker-compose run --rm musicbrainz fetch-dump.sh search
      • 2021-04-26 11612, 2021

      • nelgin
        sudo docker-compose run --rm search load-search-indexes.sh
      • 2021-04-26 11614, 2021

      • nawcom joined the channel
      • 2021-04-26 11628, 2021

      • nelgin
        When I run the 2nd command I get
      • 2021-04-26 11628, 2021

      • nelgin
        load-search-indexes.sh: 'annotation' has data already
      • 2021-04-26 11628, 2021

      • nelgin
        To delete it first, add the option '--force'.
      • 2021-04-26 11611, 2021

      • nelgin
        Ah, that's probably becuase of the failed one earlier.
      • 2021-04-26 11623, 2021

      • BrainzGit
        [bookbrainz-site] MonkeyDo merged pull request #600 (master…snyk-upgrade-8e1070bd73d2e0c7d088068ca2ecde58): [Snyk] Upgrade @babel/runtime from 7.12.5 to 7.13.10 https://github.com/bookbrainz/bookbrainz-site/pul…
      • 2021-04-26 11602, 2021

      • travis-ci joined the channel
      • 2021-04-26 11602, 2021

      • travis-ci
        Project bookbrainz-site build #3886: passed in 4 min 39 sec: https://travis-ci.org/bookbrainz/bookbrainz-site/…
      • 2021-04-26 11602, 2021

      • travis-ci has left the channel
      • 2021-04-26 11639, 2021

      • shivam-kapila
        Freso: Please put me off the review list for today. Sorry
      • 2021-04-26 11651, 2021

      • yvanzo
        ruaok: pong
      • 2021-04-26 11620, 2021

      • ruaok
        yvanzo: hiya. if I need to remove an RMQ queue on prince, which credientials do I do that with? guest cannot do it.
      • 2021-04-26 11656, 2021

      • yvanzo
        let me check
      • 2021-04-26 11616, 2021

      • sumedh has quit
      • 2021-04-26 11630, 2021

      • yvanzo
        ruaok: you need to set 'management' tag on 'lb' user, from prince: docker exec -it rabbitmq-prince rabbitmqctl set_user_tags lb management
      • 2021-04-26 11609, 2021

      • ruaok
        is that a one time task?
      • 2021-04-26 11619, 2021

      • yvanzo
        yes, see docker server configs
      • 2021-04-26 11629, 2021

      • ruaok
        yvanzo: thanks!
      • 2021-04-26 11649, 2021

      • ruaok
        yvanzo: thanks, queues cleared up.
      • 2021-04-26 11611, 2021

      • ruaok
        zas: I cleaned up the two queues with the weird names. and right now the unique queue will be collecting listens. it may trip an alert overnight, but we can safely ignore that. I'll continue this work in the morning.
      • 2021-04-26 11627, 2021

      • zas
        ok
      • 2021-04-26 11606, 2021

      • nawcom has quit
      • 2021-04-26 11646, 2021

      • _lucifer
        alastairp: ruaok: the sentry sdk logging is failing.
      • 2021-04-26 11610, 2021

      • nawcom joined the channel
      • 2021-04-26 11612, 2021

      • alastairp
        in which context?
      • 2021-04-26 11630, 2021

      • _lucifer
        the job fails to execute
      • 2021-04-26 11644, 2021

      • _lucifer
      • 2021-04-26 11610, 2021

      • alastairp
        so, within spark, then?
      • 2021-04-26 11627, 2021

      • _lucifer
        yes but interesting the errors reach sentry
      • 2021-04-26 11635, 2021

      • _lucifer
      • 2021-04-26 11655, 2021

      • sumedh joined the channel
      • 2021-04-26 11623, 2021

      • _lucifer
        so should be something else i think.
      • 2021-04-26 11645, 2021

      • alastairp
        right, so it's not clear if sentry is causing these errors, or if it's an innocent bystander in the stacktrace?
      • 2021-04-26 11607, 2021

      • _lucifer
        let me try readding all deps but just changing sentry.
      • 2021-04-26 11621, 2021

      • BrainzGit
        [musicbrainz-server] mwiencek opened pull request #2075 (master…caa-fixes): CAA fixes https://github.com/metabrainz/musicbrainz-server/…
      • 2021-04-26 11626, 2021

      • nawcom has quit
      • 2021-04-26 11642, 2021

      • nawcom joined the channel
      • 2021-04-26 11651, 2021

      • _lucifer
        alastairp, it seems like its sentry. it is trying to call an `attemptId` method which is not present and hence errors out. sentry catches this error occuring in itself and reports it.
      • 2021-04-26 11650, 2021

      • alastairp
        cool. maybe related to us being on an older version of spark? can we just run plain sentry with no spark integration?
      • 2021-04-26 11657, 2021

      • _lucifer
        i'll open a PR to try running sentry in test setup since that's on 2.. that might be helpful.
      • 2021-04-26 11619, 2021

      • _lucifer
        yes, i think regular sentry should work.
      • 2021-04-26 11623, 2021

      • alastairp
        (though if it doesn't work with the version of pyspark listed in the sentry requirements, that sounds like a sentry bug)
      • 2021-04-26 11647, 2021

      • _lucifer
        yeah, if it works in test. i'll open an issue with sentry.
      • 2021-04-26 11623, 2021

      • _lucifer
        also, i have updated the ratelimiting PR.
      • 2021-04-26 11658, 2021

      • iliekcomputers
        if someone wants to donate to help india's fucked covid situation:
      • 2021-04-26 11600, 2021

      • iliekcomputers
      • 2021-04-26 11607, 2021

      • iliekcomputers
      • 2021-04-26 11610, 2021

      • BrainzGit
        [listenbrainz-server] amCap1712 opened pull request #1413 (master…test-sentry): Test sentry in test setup https://github.com/metabrainz/listenbrainz-server…
      • 2021-04-26 11611, 2021

      • ephemer0l joined the channel
      • 2021-04-26 11619, 2021

      • _lucifer
        oh well test cannot find sentry at all https://github.com/metabrainz/listenbrainz-server…
      • 2021-04-26 11623, 2021

      • alastairp
        but that looks like the PR that I opened earlier today
      • 2021-04-26 11612, 2021

      • alastairp
        maybe test.sh spark doesn't rebuild images (same issue we had with the js tests in Actions?) and it was using a cached image?
      • 2021-04-26 11634, 2021

      • _lucifer
        yup, just checked it neither builds the image nor sets the exit status correctly
      • 2021-04-26 11648, 2021

      • _lucifer
        so the failing build also appears as passing
      • 2021-04-26 11610, 2021

      • _lucifer
        alastairp: https://github.com/metabrainz/listenbrainz-server… any idea what could be wrong here. regarding wrong error status?
      • 2021-04-26 11621, 2021

      • _lucifer
        could it be because this is using up instead of run?
      • 2021-04-26 11641, 2021

      • alastairp
        I'm not sure, I expect that if you `up` a container and then it finishes with a status, maybe it exits with that status?
      • 2021-04-26 11650, 2021

      • alastairp
        I'm not sure, sorry
      • 2021-04-26 11600, 2021

      • Sophist-UK joined the channel
      • 2021-04-26 11618, 2021

      • alastairp
        if we use run in the other tests then maybe we should just switch to it here too
      • 2021-04-26 11606, 2021

      • sumedh has quit
      • 2021-04-26 11605, 2021

      • Freso
        <BANG>
      • 2021-04-26 11605, 2021

      • Freso
        It’s World Intellectual Property Monday!
      • 2021-04-26 11605, 2021

      • Freso
      • 2021-04-26 11605, 2021

      • Freso
      • 2021-04-26 11605, 2021

      • Freso
        (I couldn’t decide which to use, so have both.)
      • 2021-04-26 11605, 2021

      • Freso
        This week is also World Immunization Week. If you have the opportunity, please go get vaccinated!
      • 2021-04-26 11613, 2021

      • Freso
        Anyway, on to business.
      • 2021-04-26 11622, 2021

      • Freso
        I haven’t received any mailed in reviews, so the usual suspects up are: ruaok, Mr_Monkey, Freso, alastairp, _lucifer, reosarevok, zas, yvanzo, bitmap, shivam-kapila, CatQuest. As usual, anyone else who want to give review, just let me know (explicitly, please) and I’ll add you.
      • 2021-04-26 11622, 2021

      • Freso
        For now… ruaok: Go!
      • 2021-04-26 11612, 2021

      • Freso
        Or… Mr_Monkey: Go!
      • 2021-04-26 11615, 2021

      • Mr_Monkey
        Hi !
      • 2021-04-26 11618, 2021

      • ShivamAwasthi joined the channel
      • 2021-04-26 11648, 2021

      • ruaok appears
      • 2021-04-26 11654, 2021

      • Mr_Monkey
        Last week i reviewed lots of PRs for Bb and LB — we had an LB PR marathon on Wednesday with the team.
      • 2021-04-26 11617, 2021

      • Mr_Monkey
        I also worked on fixing issues in the Youtube and Spotify players on LB
      • 2021-04-26 11640, 2021

      • Mr_Monkey
        Which were partly the result of setting up sentry to report front-end errors
      • 2021-04-26 11616, 2021

      • CatQuest
        :D
      • 2021-04-26 11620, 2021

      • MRiddickW joined the channel
      • 2021-04-26 11639, 2021

      • Mr_Monkey
        I spent the rest of my time refactoring code on LB , mainly making Error boundaries catch async errors and using React context as a sort of API service injection.
      • 2021-04-26 11651, 2021

      • Mr_Monkey
        That's it for me! ruaok go !
      • 2021-04-26 11600, 2021

      • ruaok
        sorry for my delayed arrival....
      • 2021-04-26 11632, 2021

      • ruaok
        last week I participated in the PR bonanza and also go the speed/searching improvements for listens PR done.
      • 2021-04-26 11633, 2021

      • Freso
        You're here now. :)
      • 2021-04-26 11651, 2021

      • ruaok
        that will greatly improve the speed of the LB experience.
      • 2021-04-26 11602, 2021

      • Mr_Monkey
        and fix bugs !
      • 2021-04-26 11612, 2021

      • ruaok
        I also signed the contract for a new Unicorn, but it hasn't been counter signed yet, so I still can't talk about it.
      • 2021-04-26 11619, 2021

      • CatQuest
        :O
      • 2021-04-26 11647, 2021

      • ruaok
        now I'm back on the MBID mapping pipeline element, which I hope to finish in the next few days.. or at least before the end of the month.
      • 2021-04-26 11601, 2021

      • ruaok
        and some minor GSoC stuff.. but that was it. fin.
      • 2021-04-26 11603, 2021

      • ruaok
        zas?
      • 2021-04-26 11609, 2021

      • zas
        hey
      • 2021-04-26 11622, 2021

      • zas
        usual maintenance & upgrades
      • 2021-04-26 11643, 2021

      • zas
        preparation for Picard 2.6.2 due tomorrow
      • 2021-04-26 11634, 2021

      • zas
        also reviewed/coded part of huge upcoming Picard patch https://github.com/metabrainz/picard/pull/1792 thanks to rdswift
      • 2021-04-26 11652, 2021

      • zas
        next, alastairp ?
      • 2021-04-26 11600, 2021

      • alastairp
        hi
      • 2021-04-26 11644, 2021

      • alastairp
        most of what I did has already been said - lots of PRs in LB finished and merged, still a bunch to go though
      • 2021-04-26 11638, 2021

      • alastairp
        I also started looking at a new project that ruaok and I discussed last week, to get a huge bunch of tags of genres into MB
      • 2021-04-26 11639, 2021

      • alastairp
        and I also spent some time looking into what's necessary to finally merge our similarity database into AB.
      • 2021-04-26 11601, 2021

      • alastairp
        I've realised that with lots of my pending tasks that perfect is the enemy of good, and so instead of agonizing over if something is 100% complete, it's better to just get it out and breaking so that people can report it to us
      • 2021-04-26 11605, 2021

      • Freso
        🎉
      • 2021-04-26 11622, 2021

      • alastairp
        so I'm going to try and be less pedantic about some of my pending tasks going forward
      • 2021-04-26 11632, 2021

      • alastairp
        Freso: thanks for stepping up, next!
      • 2021-04-26 11628, 2021

      • ruaok
        +100 alastairp !
      • 2021-04-26 11631, 2021

      • Freso
        o/
      • 2021-04-26 11651, 2021

      • alastairp
        ruaok: I was sure that you'd like that ;) it's an interesting realisation
      • 2021-04-26 11635, 2021

      • Freso
        Off the clock, I updated https://github.com/murdos/musicbrainz-userscripts… to prevent double 1200px links on /cover-art pages. :)
      • 2021-04-26 11645, 2021

      • ruaok
        good enough is far better than what other sites/projects (esp banks) do.
      • 2021-04-26 11602, 2021

      • Freso
        (Technically I did this the week before, but I merged it last week after the MBS release.)
      • 2021-04-26 11632, 2021

      • CatQuest
        ...
      • 2021-04-26 11633, 2021

      • Freso
        On the clock, not much happened. Uni work and personal things took way too much time and effort/energy than I had anticipated. I did poke a bit at LB-733 and partially as a result of that, got a bunch more mails sent in relation to that earlier today before my class.
      • 2021-04-26 11634, 2021

      • BrainzBot
        LB-733: Warn spammers & prepare to block accounts https://tickets.metabrainz.org/browse/LB-733
      • 2021-04-26 11652, 2021

      • CatQuest
        yay!
      • 2021-04-26 11638, 2021

      • Freso
        I still have at least one more cluster of accounts I haven’t fully untangled, so I expect I’ll be poking even more at that this week.
      • 2021-04-26 11618, 2021

      • ruaok
        thank you!
      • 2021-04-26 11622, 2021

      • Freso
        (Also, first account deletion as a result of this today.)
      • 2021-04-26 11652, 2021

      • Freso
        Thanks to ruaok and reosarevok for answering questions. :)
      • 2021-04-26 11606, 2021

      • Freso
        fin, I guess.
      • 2021-04-26 11617, 2021

      • Freso
        _lucifer: Go!
      • 2021-04-26 11655, 2021

      • ShivamAwasthi has quit
      • 2021-04-26 11603, 2021

      • Freso
        If _lucifer is here. 👀
      • 2021-04-26 11612, 2021

      • _lucifer
        Hi all! I spent most of the time last week on refactoring the spotify reader and oauth code to make it easier to add new integrations.
      • 2021-04-26 11626, 2021

      • Freso
        (People still up: reosarevok, yvanzo, bitmap, shivam-kapila, CatQuest. As usual, anyone else who want to give review, just let me know (explicitly, please) and I’ll add you.)
      • 2021-04-26 11658, 2021

      • _lucifer
        other than that, I reviewed 2-3 PRs and looked at some spark stuff ealrier today.
      • 2021-04-26 11604, 2021

      • _lucifer
        that's it for me.
      • 2021-04-26 11611, 2021

      • _lucifer
        reosarevok: next?
      • 2021-04-26 11620, 2021

      • reosarevok
        Hi!
      • 2021-04-26 11640, 2021

      • reosarevok
        This week I mostly worked on two things
      • 2021-04-26 11643, 2021

      • reosarevok
        One is MBS-10711
      • 2021-04-26 11643, 2021

      • BrainzBot
        MBS-10711: Convert report lists to react-table https://tickets.metabrainz.org/browse/MBS-10711
      • 2021-04-26 11606, 2021

      • reosarevok
        Where the reports components now have a lot more shared bits rather than implementing a new table 20 times
      • 2021-04-26 11624, 2021

      • reosarevok
        The other was adding a ton of genres (as in, more than a hundred) because of some forum discussions
      • 2021-04-26 11635, 2021

      • BrainzGit
        [bookbrainz-site] MonkeyDo merged pull request #571 (master…Branch3): feat(Collections): added a 'No.of Entities' column to Collections table https://github.com/bookbrainz/bookbrainz-site/pul…
      • 2021-04-26 11636, 2021

      • reosarevok
        Other than that, mostly smaller bits here and there
      • 2021-04-26 11647, 2021

      • reosarevok
        CatQuest: what about you?
      • 2021-04-26 11648, 2021

      • Mr_Monkey
        Excuse me.