#metabrainz

/

      • davic has quit
      • 2021-03-17 07630, 2021

      • BrainzGit
        [bookbrainz-site] INNOVATIVEGAMER opened pull request #575 (master…IG-Work-2): Feat (Core | BB-333) Implement a RelationshipType enum https://github.com/bookbrainz/bookbrainz-site/pul…
      • 2021-03-17 07649, 2021

      • c1e0 joined the channel
      • 2021-03-17 07629, 2021

      • mckean joined the channel
      • 2021-03-17 07616, 2021

      • mckean_ has quit
      • 2021-03-17 07606, 2021

      • BrainzGit
        [bookbrainz-site] endurance21 opened pull request #576 (master…feature-mailer): Adding node-mailer to the codebase https://github.com/bookbrainz/bookbrainz-site/pul…
      • 2021-03-17 07643, 2021

      • BrainzGit
        [bookbrainz-site] endurance21 closed pull request #576 (master…feature-mailer): Adding node-mailer to the codebase https://github.com/bookbrainz/bookbrainz-site/pul…
      • 2021-03-17 07606, 2021

      • BrainzGit
        [bookbrainz-site] endurance21 opened pull request #577 (master…feature/mailer): Feature/mailer https://github.com/bookbrainz/bookbrainz-site/pul…
      • 2021-03-17 07612, 2021

      • BrainzGit
        [musicbrainz-server] mwiencek opened pull request #1989 (production…mbs-11468): MBS-11468: ISE on "Edit cover art" edits https://github.com/metabrainz/musicbrainz-server/…
      • 2021-03-17 07648, 2021

      • dseomn_ joined the channel
      • 2021-03-17 07618, 2021

      • dseomn has quit
      • 2021-03-17 07619, 2021

      • dseomn_ is now known as dseomn
      • 2021-03-17 07656, 2021

      • Rohan_Pillai joined the channel
      • 2021-03-17 07602, 2021

      • RikkoM joined the channel
      • 2021-03-17 07624, 2021

      • dseomn_ joined the channel
      • 2021-03-17 07613, 2021

      • dseomn has quit
      • 2021-03-17 07613, 2021

      • dseomn_ is now known as dseomn
      • 2021-03-17 07629, 2021

      • astroanax has quit
      • 2021-03-17 07623, 2021

      • astroanax joined the channel
      • 2021-03-17 07624, 2021

      • c1e0 has quit
      • 2021-03-17 07630, 2021

      • c1e0 joined the channel
      • 2021-03-17 07634, 2021

      • adhi001 joined the channel
      • 2021-03-17 07605, 2021

      • akashgp09 joined the channel
      • 2021-03-17 07651, 2021

      • VS joined the channel
      • 2021-03-17 07626, 2021

      • Rohan_Pillai has quit
      • 2021-03-17 07649, 2021

      • VS has quit
      • 2021-03-17 07642, 2021

      • akashgp09 has quit
      • 2021-03-17 07652, 2021

      • BrainzGit
        [musicbrainz-server] reosarevok opened pull request #1990 (production…MBS-11470): MBS-11470: Ensure to_json_object(type) does run in Area::Edit https://github.com/metabrainz/musicbrainz-server/…
      • 2021-03-17 07615, 2021

      • yvanzo
        bitmap: would that be useful to support auth in replication mode? (or possibly to proxy auth to mb.o?)
      • 2021-03-17 07635, 2021

      • reosarevok
        yvanzo: there's two more ISEs connected to the TO_JSON stuff, in case you want to review. Not sure whether to hotfix, or wait a bit more to see if more appear :D
      • 2021-03-17 07646, 2021

      • reosarevok
        (you're off today for your moving, right?)
      • 2021-03-17 07640, 2021

      • Rohan_Pillai joined the channel
      • 2021-03-17 07610, 2021

      • yvanzo
        (will be mostly off during daytime mostly, I’m not loading a truck or something like that yet)
      • 2021-03-17 07649, 2021

      • yvanzo
        reosarevok: approved both, feel free to release right away, mainly for the release edits.
      • 2021-03-17 07643, 2021

      • yvanzo
        reosarevok: can we update the SC blog post before that?
      • 2021-03-17 07630, 2021

      • reosarevok
        Before releasing or moving? :D
      • 2021-03-17 07639, 2021

      • BrainzGit
        [musicbrainz-server] reosarevok opened pull request #1991 (master…MBS-11467): MBS-11467: Use sort name to order lists of artists for tags https://github.com/metabrainz/musicbrainz-server/…
      • 2021-03-17 07642, 2021

      • reosarevok
        Yes, in any case (although releasing can be done at the same time)
      • 2021-03-17 07642, 2021

      • BrainzGit
        [musicbrainz-server] reosarevok merged pull request #1990 (production…MBS-11470): MBS-11470: Ensure to_json_object(type) does run in Area::Edit https://github.com/metabrainz/musicbrainz-server/…
      • 2021-03-17 07657, 2021

      • BrainzGit
        [musicbrainz-server] reosarevok merged pull request #1989 (production…mbs-11468): MBS-11468: ISE on "Edit cover art" edits https://github.com/metabrainz/musicbrainz-server/…
      • 2021-03-17 07623, 2021

      • yvanzo
        reosarevok: “Potential new changes” is just for draft, right?
      • 2021-03-17 07627, 2021

      • reosarevok
        Yes
      • 2021-03-17 07643, 2021

      • reosarevok
        It was for us to decide whether to keep them (and put them in changes) or remove them
      • 2021-03-17 07634, 2021

      • reosarevok
        Building images for hotfixing
      • 2021-03-17 07637, 2021

      • yvanzo
        reosarevok: we all agree with having the change for no label at least, collection merging seems more blurry.
      • 2021-03-17 07651, 2021

      • yvanzo
        +1
      • 2021-03-17 07600, 2021

      • yvanzo
        I mean, thanks :)
      • 2021-03-17 07610, 2021

      • reosarevok
        Why is it blurry? The code is written and only needs adding a redirect table and making the code add the redirect
      • 2021-03-17 07625, 2021

      • reosarevok
        I mean, we can just not do the redirecting, I don't care much, but it seems sensible
      • 2021-03-17 07638, 2021

      • yvanzo
        because of the comment in your blog post draft
      • 2021-03-17 07644, 2021

      • reosarevok
        "has implementation in review but we could add a collection_gid_redirect rather than losing the GID" ?
      • 2021-03-17 07651, 2021

      • yvanzo
        reosarevok: if it's not blurry for you, please clarify implementation’s plan :) and yes redirect seems to be a must have.
      • 2021-03-17 07601, 2021

      • reosarevok
        Ok :)
      • 2021-03-17 07602, 2021

      • reosarevok
        I'll do
      • 2021-03-17 07620, 2021

      • yvanzo
        About bumping Perl version, let’s wait for zas’ input about possible Ubuntu version (20.04?) for MBS containers by May.
      • 2021-03-17 07659, 2021

      • reosarevok
        zas: would really like to have input today so we can post :)
      • 2021-03-17 07631, 2021

      • yvanzo
        At worst, we could use raise to 5.22 (version in xenial) which is what we are using already.
      • 2021-03-17 07625, 2021

      • yvanzo
        Given that xenial EOL is April, I doubt we will stick with it anyway, so probably 5.26 (version in bionic) at least.
      • 2021-03-17 07617, 2021

      • reosarevok
        https://ci.metabrainz.org/job/musicbrainz-docker-… wtf, is this yarn changing keys again or something?
      • 2021-03-17 07636, 2021

      • reosarevok
        Huh, guess not since beta built fine
      • 2021-03-17 07647, 2021

      • reosarevok retries prod
      • 2021-03-17 07601, 2021

      • Rohan_Pillai has quit
      • 2021-03-17 07613, 2021

      • c1e0 has quit
      • 2021-03-17 07652, 2021

      • yvanzo
        Error 18 obviously ;)
      • 2021-03-17 07609, 2021

      • reosarevok
        Of course
      • 2021-03-17 07615, 2021

      • reosarevok
        Updating beta
      • 2021-03-17 07602, 2021

      • yvanzo
        reosarevok: about formatting: some ticket references are followed with a dash, some other with a colon, please unify.
      • 2021-03-17 07657, 2021

      • reosarevok
        Sure :)
      • 2021-03-17 07605, 2021

      • yvanzo
        reosarevok: also <code> formatting is often missing, sometimes the affected code is not mentioned either.
      • 2021-03-17 07623, 2021

      • reosarevok
        Yes, I'm working on that now :) Also on reordering the entries by ticket number
      • 2021-03-17 07656, 2021

      • yvanzo
        I added actual release dates for the first tickets, see google doc. (They did not have any description in WP draft at that time).
      • 2021-03-17 07604, 2021

      • reosarevok
        Ok, will look into it too
      • 2021-03-17 07612, 2021

      • yvanzo
        I’m not sure it’s worth mentioning it might not be relevant to mirrors anymore. Maybe bitmap will just want to split the list instead, or not take it into account at all.
      • 2021-03-17 07655, 2021

      • reosarevok
        Ok, let's see
      • 2021-03-17 07621, 2021

      • reosarevok
        The 2019 post had a link to a Jira fix version with all the tickets
      • 2021-03-17 07626, 2021

      • reosarevok
        That probably doesn't work this time though?
      • 2021-03-17 07632, 2021

      • reosarevok
        Since some are already out in MBS
      • 2021-03-17 07647, 2021

      • reosarevok
        Should I add a label or something? Or just trust that the blog is enough as a list...
      • 2021-03-17 07649, 2021

      • yvanzo
        About ACs, what is changed, what is postponed?
      • 2021-03-17 07647, 2021

      • reosarevok
        "The change was released as an optional extension to the main MusicBrainz server schema on Dec 16, 2020, but it will be made the default in this schema change." does something like that seem reasonable for the recording date stuff?
      • 2021-03-17 07620, 2021

      • reosarevok
        bitmap edited the blog for the AC ticket to:
      • 2021-03-17 07621, 2021

      • reosarevok
        MBS-11456: Add MBIDs and redirect tables for artist credits. Adds a gid column to the artist_credit table, and a new artist_credit_gid_redirect table. The MBIDs will allow public identification of artist credits outside of MusicBrainz, and open the door to useful features in the future.
      • 2021-03-17 07622, 2021

      • BrainzBot
        MBS-11456: Add MBIDs to artist credits https://tickets.metabrainz.org/browse/MBS-11456
      • 2021-03-17 07642, 2021

      • reosarevok
        Beta done
      • 2021-03-17 07606, 2021

      • yvanzo
        but there were more potential changes being discussed yesterday, right?
      • 2021-03-17 07628, 2021

      • reosarevok
        Updating prod
      • 2021-03-17 07609, 2021

      • reosarevok
        Yes, but I think that was made *after* the discussion - we thought we'd just do the bare minimum for now
      • 2021-03-17 07629, 2021

      • yvanzo
        reosarevok: we can set more than one fixVersion per ticket iirc.
      • 2021-03-17 07658, 2021

      • yvanzo
        reosarevok: I got that, I just want to sum up the other issues we wanted to address, so as to keep track of them.
      • 2021-03-17 07625, 2021

      • reosarevok
        Oh, I'll try the multiple version thing
      • 2021-03-17 07657, 2021

      • reosarevok
        Well, the other discussion seemed to be mostly whether we want them to remain immutable like now, or not
      • 2021-03-17 07627, 2021

      • reosarevok
        And "not" seemed to introduce enough complications that, given there's two months until May 17, it's smarter to delay and check later
      • 2021-03-17 07650, 2021

      • reosarevok
        Possible changes related to that were dropping ref_count and changing created to last_updated
      • 2021-03-17 07630, 2021

      • yvanzo
        reosarevok: about MBS-1424, there are more changes than originally intended, see https://github.com/metabrainz/musicbrainz-server/…
      • 2021-03-17 07631, 2021

      • BrainzBot
        MBS-1424: Add a 'First release date' field to recordings https://tickets.metabrainz.org/browse/MBS-1424
      • 2021-03-17 07610, 2021

      • reosarevok
        Oh, ok
      • 2021-03-17 07645, 2021

      • sumedh joined the channel
      • 2021-03-17 07646, 2021

      • reosarevok
        yvanzo: so, does "This adds materialized tables recording_first_release_date and release_first_release_date which are updated via triggers whenever the earliest date changes." seem correct? That's what I understood but just in case :)
      • 2021-03-17 07626, 2021

      • reosarevok
        Prod updated
      • 2021-03-17 07618, 2021

      • yvanzo
        reosarevok: correct :)
      • 2021-03-17 07636, 2021

      • c1e0 joined the channel
      • 2021-03-17 07657, 2021

      • akashgp09 joined the channel
      • 2021-03-17 07658, 2021

      • yvanzo
        reosarevok: Ok thanks, does not seem worth adding another ticket about ACs for now.
      • 2021-03-17 07617, 2021

      • reosarevok
        Nice
      • 2021-03-17 07624, 2021

      • reosarevok
        I think the blog post is kinda-sorta done
      • 2021-03-17 07652, 2021

      • reosarevok
        Saved draft and closed - can you take a quick look and see if I missed something? :)
      • 2021-03-17 07603, 2021

      • mckean has quit
      • 2021-03-17 07645, 2021

      • reosarevok
        Waiting for daily cron to finish before I update the cron container
      • 2021-03-17 07604, 2021

      • reosarevok
        Probably not even needed, but we might as well keep it updated
      • 2021-03-17 07625, 2021

      • Rohan_Pillai joined the channel
      • 2021-03-17 07627, 2021

      • reosarevok
        Tagged and updated blog in the meantime
      • 2021-03-17 07603, 2021

      • yvanzo
        reosarevok: "main MusicBrainz Server schema" vs "main schema" is a bit unclear, maybe replace with "schema for master replication servers" and "schema for slave replication servers"?
      • 2021-03-17 07638, 2021

      • reosarevok
        Feel free to :)
      • 2021-03-17 07648, 2021

      • mckean joined the channel
      • 2021-03-17 07605, 2021

      • ruaok
        mooooin!
      • 2021-03-17 07647, 2021

      • c1e0_ joined the channel
      • 2021-03-17 07634, 2021

      • c1e0 has quit
      • 2021-03-17 07647, 2021

      • ruaok
      • 2021-03-17 07615, 2021

      • ruaok
        I'll be watching this carefully. monetizing APIs is hard IMHO, especially if you still have open options.
      • 2021-03-17 07636, 2021

      • yvanzo
        reosarevok: Maybe have bitmap’s double-check wording for these already released changes ^ but the rest looks nice, thanks! :)
      • 2021-03-17 07651, 2021

      • yvanzo
        off now
      • 2021-03-17 07618, 2021

      • Rohan_Pillai has quit
      • 2021-03-17 07641, 2021

      • Rohan_Pillai joined the channel
      • 2021-03-17 07656, 2021

      • sumedh has quit
      • 2021-03-17 07657, 2021

      • Mr_Monkey
        Mornin' !
      • 2021-03-17 07642, 2021

      • RikkoM has quit
      • 2021-03-17 07631, 2021

      • Mr_Monkey
        I was trying to update test.LB but there were some uncommitted changes in docker-server-configs on lemmy.
      • 2021-03-17 07631, 2021

      • Mr_Monkey
        I committed the change but did not see it is not on the master branch and now I don't know what I should be doing.
      • 2021-03-17 07631, 2021

      • Mr_Monkey
        Can I ask that we collectively avoid leaving docker-server-configs in a dirty state on production nodes (barring emergencies)? It makes it harder for other people to deploy things.
      • 2021-03-17 07604, 2021

      • Mr_Monkey
        Now, who was working on the branch `add-spark-dumps-key`, and what should I do? Revert the commit, I suppose, but then what to do with the changes?
      • 2021-03-17 07650, 2021

      • Mr_Monkey
        I assume ruaok, alastairp or _lucifer ?
      • 2021-03-17 07602, 2021

      • iliekcomputers
        another thing i've picked up doing is prepending my name on branches (param-new-feature) so it's easy to know that it's my branch.
      • 2021-03-17 07618, 2021

      • Mr_Monkey
        Good touch
      • 2021-03-17 07633, 2021

      • ruaok
        that's me
      • 2021-03-17 07612, 2021

      • Mr_Monkey
        Can I leave it with you? (I can revert my commit at least)
      • 2021-03-17 07630, 2021

      • ruaok
        I have a feeling that we were active at the same time. :(
      • 2021-03-17 07658, 2021

      • Mr_Monkey
        Uh oh. That's another issue then :)
      • 2021-03-17 07613, 2021

      • ruaok
        nope, I failed to clean up. sorry. I'll take care of it.
      • 2021-03-17 07632, 2021

      • Mr_Monkey
        But if we're doing everything on master (pushing the changes to the node file there and pulling it on Lemmy) we won't run into these issues
      • 2021-03-17 07625, 2021

      • ruaok
        sometimes that isn't possible since we don't have a duplicate production env.
      • 2021-03-17 07652, 2021

      • ruaok
        and I was working on adding a new dump type that I can't debug on my own machine.
      • 2021-03-17 07657, 2021

      • Mr_Monkey
        Hm, I see.
      • 2021-03-17 07630, 2021

      • ruaok
        server configs on lemmy now on master. affected branch deleted.
      • 2021-03-17 07636, 2021

      • Mr_Monkey
        Merci !
      • 2021-03-17 07618, 2021

      • ruaok
        I think we need a better way to deal with version number upgrades, which is a similar problem.
      • 2021-03-17 07635, 2021

      • ruaok
        those tend to stick around uncommited.
      • 2021-03-17 07641, 2021

      • CatQuest
        wait you can add review of a *place*???
      • 2021-03-17 07653, 2021

      • CatQuest
        you implemented that already?
      • 2021-03-17 07601, 2021

      • iliekcomputers
        CatQuest: yeah, that was a GSoC project ferbncode mentored ~two years ago I think
      • 2021-03-17 07632, 2021

      • sumedh joined the channel