#metabrainz

/

      • Mineo has quit
      • 2021-03-08 06726, 2021

      • MajorLurker joined the channel
      • 2021-03-08 06742, 2021

      • Mineo joined the channel
      • 2021-03-08 06726, 2021

      • MajorLurker has quit
      • 2021-03-08 06712, 2021

      • davic has quit
      • 2021-03-08 06708, 2021

      • sampsyo has quit
      • 2021-03-08 06739, 2021

      • sampsyo joined the channel
      • 2021-03-08 06701, 2021

      • Rohan_Pillai joined the channel
      • 2021-03-08 06725, 2021

      • BrainzGit
        [musicbrainz-server] mwiencek opened pull request #1959 (master…mbs-11424): MBS-11424: Unbreak seeding multiple RG types https://github.com/metabrainz/musicbrainz-server/…
      • 2021-03-08 06755, 2021

      • BrainzGit
        [musicbrainz-server] mwiencek opened pull request #1960 (master…mbs-11420): MBS-11420: "Paste Credits" doesn't apply credited name https://github.com/metabrainz/musicbrainz-server/…
      • 2021-03-08 06715, 2021

      • sumedh joined the channel
      • 2021-03-08 06739, 2021

      • adhi001 joined the channel
      • 2021-03-08 06755, 2021

      • Rohan_Pillai has quit
      • 2021-03-08 06714, 2021

      • sumedh has quit
      • 2021-03-08 06742, 2021

      • sumedh joined the channel
      • 2021-03-08 06742, 2021

      • Rohan_Pillai joined the channel
      • 2021-03-08 06726, 2021

      • Rohan_Pillai has quit
      • 2021-03-08 06702, 2021

      • MRiddickW joined the channel
      • 2021-03-08 06709, 2021

      • sumedh has quit
      • 2021-03-08 06719, 2021

      • Rohan_Pillai joined the channel
      • 2021-03-08 06716, 2021

      • MajorLurker joined the channel
      • 2021-03-08 06752, 2021

      • MajorLurker has quit
      • 2021-03-08 06714, 2021

      • Rohan_Pillai has quit
      • 2021-03-08 06715, 2021

      • _lucifer_ is now known as _lucifer
      • 2021-03-08 06753, 2021

      • _lucifer
        ruaok: i think i fixed the string indices error. now it erring while inserting into table https://sentry.metabrainz.org/metabrainz/listenbr…
      • 2021-03-08 06716, 2021

      • _lucifer
        don't really have much idea about his part
      • 2021-03-08 06730, 2021

      • _lucifer
        *this
      • 2021-03-08 06718, 2021

      • _lucifer
        ok. fixed and deployed. sentry seems clear. but will need to check the db to confirm if it worked.
      • 2021-03-08 06740, 2021

      • sumedh joined the channel
      • 2021-03-08 06714, 2021

      • sampsyo has quit
      • 2021-03-08 06745, 2021

      • sampsyo joined the channel
      • 2021-03-08 06725, 2021

      • ruaok
        Mooin!
      • 2021-03-08 06745, 2021

      • ruaok
        _lucifer: i will check once I am in BCN again, this afternoon.
      • 2021-03-08 06702, 2021

      • ruaok
        Do you have stuff to work on in the meantime?
      • 2021-03-08 06715, 2021

      • _lucifer
        great, thanks!
      • 2021-03-08 06732, 2021

      • _lucifer
        yes need to resolve the merge conflicts on the follow feature PR
      • 2021-03-08 06750, 2021

      • ruaok
        Ok
      • 2021-03-08 06725, 2021

      • _lucifer
        also, amending tests that are using hardcoded values so that we can restrict searching listens to the last 6 months and not go to the medivial times :p
      • 2021-03-08 06752, 2021

      • yvanzo
        good time of the day
      • 2021-03-08 06752, 2021

      • CatQuest has quit
      • 2021-03-08 06727, 2021

      • CatQuest joined the channel
      • 2021-03-08 06727, 2021

      • CatQuest has quit
      • 2021-03-08 06727, 2021

      • CatQuest joined the channel
      • 2021-03-08 06700, 2021

      • MajorLurker joined the channel
      • 2021-03-08 06727, 2021

      • MajorLurker has quit
      • 2021-03-08 06729, 2021

      • pristine___
        ruaok: hey
      • 2021-03-08 06703, 2021

      • _lucifer
        Mr_Monkey: I have fixed the conflicts. The RecentListens.tsx file had a few conflicts. Can you please take a look at that file again?
      • 2021-03-08 06714, 2021

      • Mr_Monkey
        Hiya !
      • 2021-03-08 06741, 2021

      • Mr_Monkey
        Yes, I was actually looking at 1296 as we speak, thanks for cleaning it up !
      • 2021-03-08 06712, 2021

      • Mr_Monkey
        Do indices have to be manually deleted when one deletes a DB table ? Looking at https://github.com/metabrainz/listenbrainz-server…
      • 2021-03-08 06744, 2021

      • _lucifer
        no postgresql takes of that.
      • 2021-03-08 06751, 2021

      • _lucifer
        *takes care
      • 2021-03-08 06755, 2021

      • Mr_Monkey
        OK thanks.
      • 2021-03-08 06700, 2021

      • Mr_Monkey
        Here's one : in config.py.sample there's a `WEBSOCKETS_QUEUE = "follow_list"` left to change
      • 2021-03-08 06749, 2021

      • _lucifer
        yeah, that's intended. changing the name of the queue `follow_list` requires special consideration as ruaok mentioned.
      • 2021-03-08 06714, 2021

      • _lucifer
        changing the name of the key does not so i changed only that.
      • 2021-03-08 06724, 2021

      • Mr_Monkey
        OK ignore me :p
      • 2021-03-08 06739, 2021

      • Mr_Monkey
        I forgot we said that would go in another PR
      • 2021-03-08 06733, 2021

      • ruaok
        pristine___: hey
      • 2021-03-08 06751, 2021

      • pristine___
        ruaok: meanwhile you are creating dumps, I was thinking to work in artist similarity...
      • 2021-03-08 06712, 2021

      • ruaok
        That would be great!
      • 2021-03-08 06707, 2021

      • Mr_Monkey
        _lucifer: shall I merge 1296 then?
      • 2021-03-08 06722, 2021

      • _lucifer
        sure :D
      • 2021-03-08 06731, 2021

      • Mr_Monkey
        🚀
      • 2021-03-08 06738, 2021

      • BrainzGit
        [listenbrainz-server] MonkeyDo merged pull request #1296 (master…follow-remove): Remove old follow feature https://github.com/metabrainz/listenbrainz-server…
      • 2021-03-08 06741, 2021

      • Mr_Monkey
        And good riddance !
      • 2021-03-08 06741, 2021

      • _lucifer
        thanks!
      • 2021-03-08 06730, 2021

      • _lucifer
        ruaok: alastairp: iliekcomputers: Mr_Monkey: how about doing a release today? we have a ws bugfix, ui improvements for timestamp and follow feature removal. plus i can test my changes in spotify reader ;).
      • 2021-03-08 06725, 2021

      • Mr_Monkey
        I'm all for it
      • 2021-03-08 06700, 2021

      • iliekcomputers
        sounds good to me
      • 2021-03-08 06703, 2021

      • alastairp
        nice, sure thing
      • 2021-03-08 06743, 2021

      • alastairp
        _lucifer: depending on what your plans are, and what you've discussed with ruaok, you and I can have a bit of a chat about AB today too
      • 2021-03-08 06706, 2021

      • ruaok
        All good by me. If you want help doing that, I'll be fully online in 2-3 hours.
      • 2021-03-08 06740, 2021

      • _lucifer
        awesome, i'll get to it soon.
      • 2021-03-08 06733, 2021

      • _lucifer
        alastairp: we didn't discuss anythin in particular about AB. but I think the starting would to improve MeB admin interface?
      • 2021-03-08 06713, 2021

      • BrainzGit
        [bookbrainz-site] MonkeyDo merged pull request #553 (master…myBranch): fix(collection-form):Position of "Add another collaborator" button https://github.com/bookbrainz/bookbrainz-site/pul…
      • 2021-03-08 06756, 2021

      • alastairp
        ah, yeah. when we met in the office last Monday we talked about a lot of things, including the admin interface, and the plans for api keys and auth - that kind of merged into the oauth document that I started. later this afternoon we can straighten out all of the plans
      • 2021-03-08 06721, 2021

      • BrainzGit
        [bookbrainz-site] MonkeyDo merged pull request #558 (master…Fix/Issue-582): Add a scrollbar in the 'add to collection' modal https://github.com/bookbrainz/bookbrainz-site/pul…
      • 2021-03-08 06728, 2021

      • _lucifer
        awesome!
      • 2021-03-08 06716, 2021

      • ruaok
        alastairp: should we move ahead with meb/oauth stuff before AB gets a time slice?
      • 2021-03-08 06740, 2021

      • ruaok
        I will certainly need time to plan out key management features.
      • 2021-03-08 06747, 2021

      • travis-ci joined the channel
      • 2021-03-08 06747, 2021

      • travis-ci
        Project bookbrainz-site build #3671: passed in 4 min 24 sec: https://travis-ci.org/bookbrainz/bookbrainz-site/…
      • 2021-03-08 06747, 2021

      • travis-ci has left the channel
      • 2021-03-08 06716, 2021

      • alastairp
        ruaok: I think that prototyping the oauth could be a neat idea
      • 2021-03-08 06718, 2021

      • ruaok
        But easier to define things for meb could be tackled. E.g update node and python dependencies.
      • 2021-03-08 06751, 2021

      • ruaok
        alastairp: I'm fine with that, but am trying to address your lament that AB gets little love.
      • 2021-03-08 06705, 2021

      • ruaok
        Doing oauth is good for everyone, though.
      • 2021-03-08 06707, 2021

      • alastairp
        I'm planning on doing AB from next week anyway
      • 2021-03-08 06715, 2021

      • ruaok
        Ok, kewl.
      • 2021-03-08 06727, 2021

      • alastairp
        so I'm happy to get _lucifer working on it at a later stage
      • 2021-03-08 06738, 2021

      • ruaok
        _lucifer: you could work on the node upgrade when you have some spare time.
      • 2021-03-08 06718, 2021

      • alastairp
        for me, the oauth stuff is still full of a lot of unknowns, but I think having a prototype will help us work out what works and what needs to be revisited
      • 2021-03-08 06722, 2021

      • ruaok
        Ok, sounds like a rough plan for the week.
      • 2021-03-08 06723, 2021

      • _lucifer
        sure
      • 2021-03-08 06726, 2021

      • alastairp
        _lucifer: ever worked with oauth before? :)
      • 2021-03-08 06741, 2021

      • ruaok
        Agreed, on prototype.
      • 2021-03-08 06757, 2021

      • _lucifer
        not much, only what we have in the android app, alastairp
      • 2021-03-08 06748, 2021

      • alastairp
        feel free to add questions to that document. let's talk about it in ~2 hours
      • 2021-03-08 06759, 2021

      • _lucifer
        👍
      • 2021-03-08 06732, 2021

      • travis-ci joined the channel
      • 2021-03-08 06732, 2021

      • travis-ci
        Project bookbrainz-site build #3672: passed in 8 min 1 sec: https://travis-ci.org/bookbrainz/bookbrainz-site/…
      • 2021-03-08 06732, 2021

      • travis-ci has left the channel
      • 2021-03-08 06740, 2021

      • _lucifer
        alastairp: also, i just remembered this is pending as well, https://docs.google.com/document/d/1OWRDEdGbYqC6R… . i'll get to it in the coming weeks :D
      • 2021-03-08 06733, 2021

      • alastairp
        _lucifer: yes, right. as I said, I'll start looking at AB stuff next week too. There are 4-5 half-finished projects that I want to get released in the next 3 months
      • 2021-03-08 06710, 2021

      • alastairp
        as you can see, there are lots of projects, lots of ideas, and not many programmers :) you're a great asset to help us get through these plans...
      • 2021-03-08 06734, 2021

      • _lucifer
        :-D
      • 2021-03-08 06711, 2021

      • _lucifer
        alastairp: regarding the release, should i tag before or after merging master into production or does it not matter?
      • 2021-03-08 06757, 2021

      • alastairp
        I've not really thought about that much before. theoretically it should be the same
      • 2021-03-08 06714, 2021

      • alastairp
        (unless for some reason production has some other commits or weird commit order that master doesn't have)
      • 2021-03-08 06736, 2021

      • _lucifer
        makes sense
      • 2021-03-08 06720, 2021

      • BrainzGit
        [listenbrainz-server] release v-2021-03-08.3 has been published by github-actions[bot]: https://github.com/metabrainz/listenbrainz-server…
      • 2021-03-08 06753, 2021

      • alastairp
        .3? 😅
      • 2021-03-08 06751, 2021

      • _lucifer
        yeah, i pushed .0, 0.1, 0.2 earlier to test and fix some issues in spark reader so 😅
      • 2021-03-08 06747, 2021

      • _lucifer
        now that i think i should have used a different image tag for those images!
      • 2021-03-08 06739, 2021

      • MajorLurker joined the channel
      • 2021-03-08 06756, 2021

      • BrainzGit
        [musicbrainz-server] reosarevok merged pull request #1942 (master…MBS-9866): MBS-9866: Add bubbles explaining what IPI and ISNI are on add forms https://github.com/metabrainz/musicbrainz-server/…
      • 2021-03-08 06722, 2021

      • MajorLurker has quit
      • 2021-03-08 06700, 2021

      • iliekcomputers
        _lucifer: there is a schema change in master that you should apply as well
      • 2021-03-08 06722, 2021

      • iliekcomputers
      • 2021-03-08 06741, 2021

      • _lucifer
        iliekcomputers: yeah i have made the github release and pushed the images. but i am holding off deploying till someone else is available.
      • 2021-03-08 06725, 2021

      • alastairp
        cool. I'll go for lunch, and will be back in ~45 minutes if you want to deploy with me
      • 2021-03-08 06740, 2021

      • Rohan_Pillai joined the channel
      • 2021-03-08 06730, 2021

      • _lucifer
        sure, that works alastairp!
      • 2021-03-08 06743, 2021

      • sumedh has quit
      • 2021-03-08 06736, 2021

      • travis-ci joined the channel
      • 2021-03-08 06736, 2021

      • travis-ci
        Project bookbrainz-site build #3673: failed in 4 min 30 sec: https://travis-ci.org/bookbrainz/bookbrainz-site/…
      • 2021-03-08 06736, 2021

      • travis-ci has left the channel
      • 2021-03-08 06758, 2021

      • ruaok returns to a real computer.
      • 2021-03-08 06702, 2021

      • ruaok
        what needs attention?
      • 2021-03-08 06726, 2021

      • _lucifer
        can you check the user similarity tables to see if they worked as expected?
      • 2021-03-08 06733, 2021

      • ruaok
        ok, what else?
      • 2021-03-08 06708, 2021

      • _lucifer
        there is also a schema update to accompany today's release
      • 2021-03-08 06719, 2021

      • _lucifer
        but alastairp already volunteered to help with that :)
      • 2021-03-08 06739, 2021

      • ruaok
        k, happy to focus on other code bits instead. :)
      • 2021-03-08 06725, 2021

      • iliekcomputers
        ruaok: let's chat about the API endpoint if you're around.
      • 2021-03-08 06755, 2021

      • ruaok
        I am, but let me re read the critical bits of the PR on a larger screen. ;)
      • 2021-03-08 06709, 2021

      • iliekcomputers
        :D
      • 2021-03-08 06710, 2021

      • iliekcomputers
        sure
      • 2021-03-08 06713, 2021

      • alastairp
        _lucifer: I'm here. what's the plan?
      • 2021-03-08 06737, 2021

      • alastairp
      • 2021-03-08 06726, 2021

      • _lucifer
        this is for an earlier release. i can update it or open a new one
      • 2021-03-08 06730, 2021

      • _lucifer
        we updated dependencies so i think it makes sense to restart all services except cron. excluding spark reader as well because that is running a different branch currently.
      • 2021-03-08 06731, 2021

      • alastairp
        just approved 140, let's merge that and then do another one for this release
      • 2021-03-08 06739, 2021

      • _lucifer
        👍
      • 2021-03-08 06710, 2021

      • alastairp
        sounds fine to me to update all of those services. why do you not want to upgrade cron?
      • 2021-03-08 06738, 2021

      • alastairp
        when do you think that we'll be able to merge the spark reader branch?
      • 2021-03-08 06758, 2021

      • _lucifer
        no changes to cron so i thought no need to update that
      • 2021-03-08 06725, 2021

      • _lucifer
        hopefully sometime later this week. ruaok can probably provide a better estimate on that
      • 2021-03-08 06710, 2021

      • alastairp
        ok, no problem. If this is going to be a common occurrence going forward maybe we need to think about splitting some things into separate images. Let's not worry about it for now