#metabrainz

/

      • reosarevok
        I'd worry if it wasn't
      • 2021-05-10 13005, 2021

      • _lucifer
        i usually build images on bono to avoid such issues.
      • 2021-05-10 13032, 2021

      • ruaok
        yeah. smrt.
      • 2021-05-10 13051, 2021

      • ruaok
        builds fine on lemmy.
      • 2021-05-10 13056, 2021

      • ruaok shrugs
      • 2021-05-10 13015, 2021

      • shivam-kapila
        Lol everyone hates mac and there I am who wants to get a hang of it once
      • 2021-05-10 13054, 2021

      • sumedh joined the channel
      • 2021-05-10 13042, 2021

      • shivam-kapila
        Freso: please take me off the review list please. Exams.
      • 2021-05-10 13056, 2021

      • BrainzGit
        [listenbrainz-server] alastair opened pull request #1442 (master…upgrade-bu2): Upgrade to BU 2.0.0 https://github.com/metabrainz/listenbrainz-server…
      • 2021-05-10 13014, 2021

      • BrainzGit
        [listenbrainz-server] alastair closed pull request #1358 (master…bu-upgrade-metrics): Upgrade brainzutils and integrate metrics endpoints https://github.com/metabrainz/listenbrainz-server…
      • 2021-05-10 13002, 2021

      • BrainzGit
        [musicbrainz-server] yvanzo closed pull request #2053 (schema-change-2021-q2…mbs-10566): MBS-10566: Make allowed entity type constraints table-based https://github.com/metabrainz/musicbrainz-server/…
      • 2021-05-10 13052, 2021

      • sumedh has quit
      • 2021-05-10 13042, 2021

      • Mr_Monkey
        alastairp, _lucifer : If ENV is unused in this file now, should I just remove it from the `echo` call?
      • 2021-05-10 13042, 2021

      • Mr_Monkey
      • 2021-05-10 13058, 2021

      • alastairp
        good catch. yes please
      • 2021-05-10 13000, 2021

      • yvanzo
        reosarevok: about ACs, I’m missing how gid is generated for new entries.
      • 2021-05-10 13015, 2021

      • Mr_Monkey
        👍
      • 2021-05-10 13042, 2021

      • sumedh joined the channel
      • 2021-05-10 13019, 2021

      • reosarevok
        yvanzo: IIUC, generate_gid from Data::Utils
      • 2021-05-10 13044, 2021

      • reosarevok
      • 2021-05-10 13043, 2021

      • Mr_Monkey
        I'm updating test.LB with the Node upgrade PR
      • 2021-05-10 13037, 2021

      • c1e0 joined the channel
      • 2021-05-10 13054, 2021

      • BrainzGit
        [acousticbrainz-server] alastair opened pull request #399 (master…gh-actions): Run tests in github actions https://github.com/metabrainz/acousticbrainz-serv…
      • 2021-05-10 13010, 2021

      • flamingspinach has quit
      • 2021-05-10 13055, 2021

      • flamingspinach joined the channel
      • 2021-05-10 13027, 2021

      • _lucifer
      • 2021-05-10 13053, 2021

      • _lucifer
        alastairp, ruaok: i am now getting the above error in local development.
      • 2021-05-10 13003, 2021

      • alastairp
        _lucifer: yeah, there was a small period where we used the timesense 2.2-pg12 image, but then we downgraded back to pg11 to match prod
      • 2021-05-10 13022, 2021

      • ruaok
        unless you have important data in your DB, nuke it and start over.
      • 2021-05-10 13024, 2021

      • alastairp
        you'll have to delete the pg data volume and start again (I had the same issue this morning)
      • 2021-05-10 13030, 2021

      • _lucifer
        👍
      • 2021-05-10 13010, 2021

      • BrainzGit
        [listenbrainz-server] mayhem opened pull request #1443 (master…add-ftp-dump-date-check): Add ftp dump date check https://github.com/metabrainz/listenbrainz-server…
      • 2021-05-10 13044, 2021

      • Mr_Monkey
        Yeah, I noticed that too this morning
      • 2021-05-10 13038, 2021

      • yvanzo
        reosarevok: Thanks, I added it to the latest WIP commit as it seems to be required by Role::CoreEntity. Investigating lasting failures now.
      • 2021-05-10 13050, 2021

      • BrainzGit
        [acousticbrainz-server] alastair merged pull request #398 (master…downloads-http): Move download links to http https://github.com/metabrainz/acousticbrainz-serv…
      • 2021-05-10 13000, 2021

      • BrainzGit
        [acousticbrainz-server] release v-2021-05-10.0 has been published by github-actions[bot]: https://github.com/metabrainz/acousticbrainz-serv…
      • 2021-05-10 13017, 2021

      • BrainzGit
        [listenbrainz-server] MonkeyDo merged pull request #1436 (master…monkey-nodejs-upgrade): Upgrade NodeJS to version 16 https://github.com/metabrainz/listenbrainz-server…
      • 2021-05-10 13033, 2021

      • Mr_Monkey
        And voilà, NodeJS updated !
      • 2021-05-10 13015, 2021

      • alastairp
        thank you so much Mr_Monkey for dealing with that :)
      • 2021-05-10 13029, 2021

      • Mr_Monkey
        Thankfully it was pretty painless.
      • 2021-05-10 13038, 2021

      • reosarevok
        yvanzo: ok, I'm off for a while but back for the meeting time, hopefully
      • 2021-05-10 13035, 2021

      • ruaok
        alastairp: _lucifer : check your email.
      • 2021-05-10 13018, 2021

      • alastairp
        I mean, at least it comes from a bot and not a person
      • 2021-05-10 13028, 2021

      • ruaok
        that sounds like it arrived, good.
      • 2021-05-10 13048, 2021

      • ruaok
        wow, LB-884 is a red hot mess. Sigh.
      • 2021-05-10 13049, 2021

      • BrainzBot
        LB-884: Listens count not updated after old listens import https://tickets.metabrainz.org/browse/LB-884
      • 2021-05-10 13030, 2021

      • Mr_Monkey
        I updated test.listenbrainz.org again with the datepicker navigation PR, if anyone wants to try it (recent listens page).
      • 2021-05-10 13030, 2021

      • Mr_Monkey
        It works swimmingly!
      • 2021-05-10 13031, 2021

      • Mr_Monkey
        I had to hard refresh to get the newer css file, I'm going to try to solve that first.
      • 2021-05-10 13015, 2021

      • alastairp
        Mr_Monkey: ah, that was because we don't use webpack to generate css I believe?
      • 2021-05-10 13025, 2021

      • Mr_Monkey
        Correct
      • 2021-05-10 13006, 2021

      • Mr_Monkey
        And we don't have a hash in the name in a manifest file like we do for the js files
      • 2021-05-10 13025, 2021

      • shivam-kapila
        Unrelated but when en want to migrate to BS5 we might wanna switch to Sass as BS now come bundled ad Sass
      • 2021-05-10 13032, 2021

      • _lucifer
        ruaok: yup, the email arrived.
      • 2021-05-10 13035, 2021

      • _lucifer
        !m ruaok
      • 2021-05-10 13036, 2021

      • BrainzBot
        You're doing good work, ruaok!
      • 2021-05-10 13007, 2021

      • alastairp
        here's an interesting observation: I just loaded some local files into spotify. They expose the metadata of the currently playing song in now_playing, but the songs don't appear in the playback history on spotify api (I think this makes sense from their perspective). Do we want to try and do anything about these?
      • 2021-05-10 13032, 2021

      • ruaok
        how could we?
      • 2021-05-10 13042, 2021

      • alastairp
        I guess that's the question. we could special-case Spotify somehow: If it's now-playing and a local file, keep track of it and submit it ourselves when a new now-playing comes in
      • 2021-05-10 13005, 2021

      • alastairp
        however my feeling is that we shouldn't treat them as a special case, and just continue to ignore it as we currently do
      • 2021-05-10 13026, 2021

      • ruaok
        if users end up complaining that this is a problem, lets revisit.
      • 2021-05-10 13004, 2021

      • ruaok
        alastairp: _lucifer : I also just shuffled cron jobs around so that incremental dumps are imported BEFORE daily stats. lol.
      • 2021-05-10 13045, 2021

      • _lucifer
        lol, i thought that was the case always.
      • 2021-05-10 13026, 2021

      • ruaok
        heh, yeah. we should also collapse all those cron entries for stats into one action. We never plan to run the separately, so might as well do them in one swoop and compact our schedule.
      • 2021-05-10 13027, 2021

      • CatQuest
        ruaok: someone in #musicbrainz mentioned something about funkwhale/tagging/picard
      • 2021-05-10 13044, 2021

      • CatQuest
        (if interested :D)
      • 2021-05-10 13048, 2021

      • ruaok
        k, happy to answer questions.
      • 2021-05-10 13058, 2021

      • ruaok
        speaking of which, I just realized...
      • 2021-05-10 13035, 2021

      • ruaok
        Mr_Monkey: _lucifer : could we somehow use the new music services feature to add funkwhale as a local music service?
      • 2021-05-10 13045, 2021

      • CatQuest
        oooohhh cool
      • 2021-05-10 13021, 2021

      • ruaok
        I know it would't work out of the box, but if the user could specify the funkwhale IP:PORT and login credentials, then brainzplayer could play from funkwhale.
      • 2021-05-10 13022, 2021

      • _lucifer
        ruaok, i don't know about funkwhale. what should an integration with funkwhale do?
      • 2021-05-10 13031, 2021

      • ruaok
        one sec.
      • 2021-05-10 13048, 2021

      • _lucifer
        +1 to the collapsing the cron entries into one action.
      • 2021-05-10 13000, 2021

      • Mr_Monkey
        I think it would be doable ruaok
      • 2021-05-10 13019, 2021

      • ruaok
        it would be nerdvana.
      • 2021-05-10 13026, 2021

      • Mr_Monkey
        Haven't looked at the API on the funkwhale side for embedding or streaming content though
      • 2021-05-10 13035, 2021

      • Mr_Monkey
        Or searching for that matter
      • 2021-05-10 13039, 2021

      • ruaok
      • 2021-05-10 13053, 2021

      • ruaok
        ask to create an account and let me know when you did.
      • 2021-05-10 13025, 2021

      • ruaok
        I'll approve you. but in essence funkwhale is a web streaming front end to your private music collection. with aggregation of other music pods.
      • 2021-05-10 13027, 2021

      • _lucifer
        done.
      • 2021-05-10 13047, 2021

      • ruaok
        approved.
      • 2021-05-10 13004, 2021

      • outsidecontext
        awesome idea
      • 2021-05-10 13003, 2021

      • CatQuest
        it is, it even works in my stupid brpwser too
      • 2021-05-10 13010, 2021

      • CatQuest
        :D
      • 2021-05-10 13025, 2021

      • _lucifer
        i see. sounds awesome.
      • 2021-05-10 13030, 2021

      • ruaok
        the mismatch I see is that each funkwhale instance needs its own credentials. and that doesn't fit into our current system.
      • 2021-05-10 13057, 2021

      • CatQuest
        well it did last time.. now apparently i have connetion issues? idk
      • 2021-05-10 13008, 2021

      • BrainzGit
        [bookbrainz-site] akashgp09 opened pull request #628 (master…test-json): chore: stop git to track test.json file https://github.com/bookbrainz/bookbrainz-site/pul…
      • 2021-05-10 13032, 2021

      • _lucifer
        ruaok, the client id and client secrets are not stored in the current tables. we could make another table to store those creds. right now there is only youtube and spotify so it works in consul but if the number of services increase we would probably move to a table anywyas.
      • 2021-05-10 13018, 2021

      • _lucifer
        https://docs.funkwhale.audio/swagger/#/Auth%20and… looks promising. we could probably add a listen importer as well.
      • 2021-05-10 13054, 2021

      • alastairp
        automatic app registration! that's pretty neat
      • 2021-05-10 13050, 2021

      • alastairp
        Mr_Monkey: hi, got a moment?
      • 2021-05-10 13058, 2021

      • Mr_Monkey
        Go !
      • 2021-05-10 13003, 2021

      • alastairp
      • 2021-05-10 13046, 2021

      • alastairp
        just to check - in the end we went with `this.context` being the `GlobalAppContext` object, one item of which is the API service?
      • 2021-05-10 13055, 2021

      • Mr_Monkey
        Correct
      • 2021-05-10 13001, 2021

      • alastairp
      • 2021-05-10 13006, 2021

      • alastairp
        const { APIService } = this.context;
      • 2021-05-10 13019, 2021

      • alastairp
        ohhh, wait. this is object destructuring, isn't it
      • 2021-05-10 13024, 2021

      • Mr_Monkey
        It is
      • 2021-05-10 13029, 2021

      • alastairp
        APIService = this.context.APIService
      • 2021-05-10 13036, 2021

      • alastairp
        OK, never mind me. thanks
      • 2021-05-10 13057, 2021

      • Mr_Monkey
        👍 :)
      • 2021-05-10 13055, 2021

      • Freso
        shivam-kapila: Whenever you want to be back on it, feel free to add yourself again: https://wiki.musicbrainz.org/User:Freso/MetaBrain…
      • 2021-05-10 13050, 2021

      • ruaok
        _lucifer: outsidecontext already wrote a funkwhale plugin for submitting LB listens.
      • 2021-05-10 13025, 2021

      • ruaok
        and its been merged into their codebase. but yes, I would like to explore adding funkwhale support. but not right this second. too man loose ends still
      • 2021-05-10 13012, 2021

      • CatQuest
        too man 🤔
      • 2021-05-10 13020, 2021

      • ruaok
        that too.
      • 2021-05-10 13022, 2021

      • CatQuest
        :D
      • 2021-05-10 13051, 2021

      • _lucifer
        oh nice! so we are looking at a BrainzPlayer integration here. i have opened LB-889 for now, will look into it later and update with my findings.
      • 2021-05-10 13052, 2021

      • BrainzBot
      • 2021-05-10 13008, 2021

      • ruaok
        sweeeit!
      • 2021-05-10 13057, 2021

      • sumedh has quit
      • 2021-05-10 13003, 2021

      • Freso
        <BANG>
      • 2021-05-10 13003, 2021

      • Freso
        It’s International Monday of Argania!
      • 2021-05-10 13003, 2021

      • Freso
      • 2021-05-10 13003, 2021

      • Freso
        People up for reviews today include: zas, alastairp, _lucifer, bitmap, Mr_Monkey, yvanzo, ruaok, Freso, reosarevok, CatQuest – anyone else who might want to give review, let me know ASAP. :)
      • 2021-05-10 13003, 2021

      • Freso
        I haven’t received any mailed in reviews, so…
      • 2021-05-10 13004, 2021

      • Freso
        zas: Go!
      • 2021-05-10 13023, 2021

      • sumedh joined the channel
      • 2021-05-10 13056, 2021

      • Freso
        Or… alastairp, go!
      • 2021-05-10 13011, 2021

      • BrainzGit
        [bookbrainz-site] akashgp09 closed pull request #628 (master…test-json): chore: stop git to track test.json file https://github.com/bookbrainz/bookbrainz-site/pul…
      • 2021-05-10 13012, 2021

      • alastairp
        ahhh, I haven't pre-written my notes either :)
      • 2021-05-10 13015, 2021

      • alastairp
        OK, let's go
      • 2021-05-10 13054, 2021

      • yvanzo
        I can go now if you want to
      • 2021-05-10 13003, 2021

      • alastairp
        last week we took LB down to check the fan on gaga (I think that was last week?)
      • 2021-05-10 13024, 2021

      • alastairp
        it involved a bit of dev work to ensure that we could take the db down but leave the submissions api up.
      • 2021-05-10 13033, 2021

      • alastairp
        it worked well, we had a few hiccups but solved them
      • 2021-05-10 13049, 2021

      • reosarevok
        Oh, neat, wanna run our MBS schema change then?
      • 2021-05-10 13009, 2021

      • alastairp
        with ruaok and _lucifer a few days later we upgraded the timescale db on listenbrainz. this had a few small hiccups too regarding free disk space, but we worked around those too. we now have a pretty solid workflow to do this, which is great
      • 2021-05-10 13049, 2021

      • alastairp
        the end of this stream of tasks was to review ruaok's changes to improve the way that we count listens in LB. it's _muuuuuuch_ faster now
      • 2021-05-10 13051, 2021

      • alastairp
        !m ruaok
      • 2021-05-10 13051, 2021

      • BrainzBot
        You're doing good work, ruaok!
      • 2021-05-10 13005, 2021

      • alastairp
        I did some other small reviews of LB tasks too
      • 2021-05-10 13010, 2021

      • ruaok
        but with terrible listen counts apparently. lol
      • 2021-05-10 13039, 2021

      • zas
        btw, I'm there, I didn't notice it was meeting time already
      • 2021-05-10 13040, 2021

      • alastairp
        I migrated AB unit tests to github, and with zas's help set up AB downloads over http as web browsers are starting to remove support for FTP
      • 2021-05-10 13009, 2021

      • alastairp
        a few small other upgrade tasks that don't require more detail
      • 2021-05-10 13014, 2021

      • alastairp
        next? zas is here now!
      • 2021-05-10 13020, 2021

      • zas
        hey
      • 2021-05-10 13046, 2021

      • zas
        I did usual maintenance, upgrades, user support, etc
      • 2021-05-10 13011, 2021

      • zas
        also Picard PRs reviews & testing (massive ones in the queue)
      • 2021-05-10 13041, 2021

      • zas
        also deployed AB http(s) preliminary support
      • 2021-05-10 13059, 2021

      • zas
        question: should we deprecate ftp?
      • 2021-05-10 13024, 2021

      • zas
        that's it for me. yvanzo ?
      • 2021-05-10 13028, 2021

      • Freso
        IMHO, if it makes things easier for you, sure. If not, might as well keep.
      • 2021-05-10 13036, 2021

      • CatQuest
        this^^
      • 2021-05-10 13036, 2021

      • yvanzo
        Hi!