#metabrainz

/

      • minimal has quit
      • nawcom joined the channel
      • pite joined the channel
      • pite has quit
      • aerozol[m] joined the channel
      • aerozol[m]
        Getting an internal server error on MB, on prod and beta. Chrome, incognito mode
      • Itā€™s back, was just for a minute! Got a fright šŸ‘»
      • BrainzGit
        [listenbrainz-server] 14amCap1712 opened pull request #3002 (03masterā€¦stop-color-sync): Stop huesound color sync https://github.com/metabrainz/listenbrainz-serv...
      • lucifer[m] joined the channel
      • lucifer[m]
        mayhem: fyi, ^
      • BrainzGit
        [listenbrainz-server] 14amCap1712 merged pull request #3002 (03masterā€¦stop-color-sync): Stop huesound color sync https://github.com/metabrainz/listenbrainz-serv...
      • [listenbrainz-server] 14amCap1712 opened pull request #3003 (03masterā€¦fix-build): Fix build https://github.com/metabrainz/listenbrainz-serv...
      • [listenbrainz-server] 14amCap1712 merged pull request #3003 (03masterā€¦fix-build): Fix build https://github.com/metabrainz/listenbrainz-serv...
      • [listenbrainz-server] release 03v-2024-10-20.0 has been published by 14github-actions[bot]: https://github.com/metabrainz/listenbrainz-serv...
      • jasje[m] joined the channel
      • jasje[m]
        lucifer: is it possible to get last few failed api calls from my account to lb server?
      • reosarevok[m] joined the channel
      • reosarevok[m]
        yvanzo: when around, see if https://github.com/metabrainz/musicbrainz-serve... seems fine to you - if so we can release prod today and include it
      • mayhem[m] joined the channel
      • mayhem[m]
        mooooin!
      • mayhem[m] is finally feeling rested and close to 100%
      • reosarevok[m]
        mayhem: is MBS-13788 something you feel would make sense or do you think canonical access should remain separate?
      • BrainzBot
        MBS-13788: Access canonical release mbid with MusicBrainz API https://tickets.metabrainz.org/browse/MBS-13788
      • mayhem[m]
        it makes sense, but we need to manage the user's expectations. because the canonical data is a derived data set that isn't as stable as the other data returned by the API.
      • BrainzGit
        [listenbrainz-server] 14MonkeyDo merged pull request #3000 (03masterā€¦LB-1635): LB-1635: Navigation history broken when clicking on a user's stats https://github.com/metabrainz/listenbrainz-serv...
      • [listenbrainz-server] 14MonkeyDo merged pull request #3001 (03masterā€¦LB-1647): LB-1647: use anchor links in MissingData pagination https://github.com/metabrainz/listenbrainz-serv...
      • reosarevok[m]
        Ok, not something we'll probably work on super fast anyway, but good to know it might make sense
      • monkey: LB-1647 got marked as fixed but is still open, how do you usually cycle through states for LB? :) (trying to understand it in general, maybe it's correct)
      • BrainzBot
        LB-1647: Missing data page navigation buttons aren't links https://tickets.metabrainz.org/browse/LB-1647
      • monkey[m] joined the channel
      • monkey[m]
        Hmf, they don't always get closed automatically but I'm not sure why. I might have needed to set it as In Progress beforehand
      • monkey[m] is not an expert at the JIRA status maze
      • Yeah, I think the workflow doesn't like it if a ticket is resolved without having been set as in progress.
      • And the integration with GH is cool, but not smart enough to know how to set it in progress when a PR is raised
      • reosarevok[m]
        Hmm. Ours does
      • Well, sets as in review, but still
      • (in progress only if it's a draft)
      • mara42 joined the channel
      • monkey[m]
        šŸ¤”
      • I'll test my theory when I get a chance
      • mara42 has quit
      • mara42 joined the channel
      • mara42 has quit
      • mara42 joined the channel
      • mara42 has quit
      • mara42 joined the channel
      • mara42 has quit
      • mara42 joined the channel
      • BrainzGit
        [musicbrainz-server] 14yvanzo merged pull request #3389 (03productionā€¦disable-image-editing): Temporarily disable image editing https://github.com/metabrainz/musicbrainz-serve...
      • yvanzo[m] joined the channel
      • yvanzo[m]
        reosarevok, bitmap: Starting the release processā€¦
      • mara42 has quit
      • mara42 joined the channel
      • s-crypt has quit
      • s-crypt joined the channel
      • zas[m]
        atj: new server finally arrived, I did first steps (DNS, etc.) and open a PR for adding it to ansible. First bootstrap check shows an issue with missing binutils (24.04), so I guess you can take it from there. Current disk setup is software RAID1 + ext4. Not sure we need to bother with zfs on this one, but that's as you wish.
      • jasje[m]
        lucifer: is it possible to get last few failed api calls from my account to lb server?
      • atj[m] joined the channel
      • atj[m]
        zas: what was the error with missing binutils?
      • jasje[m]
        jasje[m]: Ive had few crashes on release versions of the app, cannot export logs as it was not my device. Errors were related to feed primarily.
      • zas[m]
        TASK [apt : Install packages] ******************************************************************************************************************************************************************************
      • fatal: [selda]: FAILED! => {"changed": false, "msg": "No package matching 'binutils' is available"}
      • But that's in -CD mode, so likely expected. Please just do the bootstrap
      • atj[m]
        hmm, looks like Ubuntu 24.04 has changed to the new sources.list format
      • so we need to use https://docs.ansible.com/ansible/latest/collect... to add repositories
      • mara42 has quit
      • zas: do you get a choice of FS when requesting the server or do they default to ext4? I'd rather we used XFS tbh
      • zas[m]
        Nope, that's the default and I prefer we stick to it on this server. Simpler.
      • mara42 joined the channel
      • minimal joined the channel
      • mara42 has quit
      • atj[m]
        ok, will bootstrap now and hope openssh doesn't break
      • mara42 joined the channel
      • the GID we normally use for docker is taken, will have to special case it
      • mara42 has quit
      • i can't remember why we hardcode the GID, do you know?
      • mara42 joined the channel
      • zas[m]
        Perhaps to ease copying stuff around (like volumes), I don't remember.
      • atj[m]
        bootstrap is done
      • shall I reboot?
      • zas[m]
        Yes
      • mara42 has quit
      • atj[m]
        @zas: rebooted. I added a few commits to your PR
      • yvanzo[m]
        Updating MB prod serversā€¦
      • zas[m]
        @atj: are you running the site playbook now?
      • mara42 joined the channel
      • mara42 has quit
      • atj[m]
        was just getting some food, running in check mode now
      • mara42 joined the channel
      • yvanzo[m]
        Updating MB beta serversā€¦
      • atj[m]
        zas: done
      • need to add one more commit
      • duckfromdisc[m] has quit
      • duckfromdisc[m] joined the channel
      • yvanzo[m]
        bitmap, reosarevok: MBS updateā€™s blog post is up for review: https://blog.metabrainz.org/?p=12121&previe...
      • reosarevok[m]
        Checking
      • (that link doesn't work for me but I can check the actual post in the editor)
      • "Both the Cover Art Archive and the Event Art Archive were hosted by the Internet Archive" are, hopefully :)
      • yvanzo[m]
        It isnā€™t available at the moment.
      • reosarevok[m]
        Well sure, but they're still hosted there :) Unless they lost them šŸ˜…
      • What about something like
      • BrainzGit
        [musicbrainz-server] 14JadedBlueEyes opened pull request #3390 (03masterā€¦MBS-13719-1): MBS-13719: Enable sending contact emails through new service https://github.com/metabrainz/musicbrainz-serve...
      • reosarevok[m]
        As it has been mentioned in the banner for the MusicBrainz website all of last week, images for events and releases are currently unavailable due to the Internet Archive (IA) needing to recover from a DDoS attack. See the IAā€™s Mastodon page for follow-up. Both the Cover Art Archive and the Event Art Archive are hosted by the IA, and since they are doing their best to restore services safely, they are not rushing and it might
      • still take several days for the images to come back. Therefore, todayā€™s server update temporarily disables editing images, since it's impossible to vote properly on edits to images the editors cannot see.
      • In any case I'd add something like that last clarification, because while it should be obvious why people shouldn't edit images... it also should be obvious we don't have control on when they'll be back and people keep asking, so let's be super clear :D
      • Should we also mention we will make the editing available as soon as possible once the services are back safely?
      • Should also be obvious but šŸ¤·ā€ā™‚ļø
      • yvanzo[m]
        Itā€™s mentioned it is temporary
      • I would just post it as-is.
      • reosarevok[m]
        Yes, my point was more that I assume we'll hotfix the server to bring them back if needed, not make people wait two weeks
      • But that's not clear from the post :)
      • yvanzo[m]
        "that will take as many days as necessary"
      • reosarevok[m]
        Yes, of course, but that just says the downtime at the IA will take as long as needed. Once that stops the images will be back, but editing won't, for that we need to hotfix or wait for a new release. I expect we'll hotfix, but we could also wait for the release and make sure that it's consistently back I guess.
      • Just trying to understand what the expected plan is (and maybe let users know)
      • bitmap[m]
        I'd +1 changing "were" to "are" in case people misinterpret that, but otherwise it looks good to go
      • yvanzo[m]
        ā€œuntil our submission access to the Internet Archive is restoredā€?
      • bitmap: That was intentional to underline that these archives are not currently available.
      • "were provided"?
      • "were available from"?
      • I'll be back later onā€¦
      • bitmap[m]
        yeah, I get what you meant but I guess there is a chance it could be misinterpreted. "were available from" sounds good to me
      • Jade[m] joined the channel
      • Jade[m]
        <BrainzGit> "[musicbrainz-server] JadedBlueEy..." <- Btw I've made this, I didn't see any feature flag system but let me know if I should add a toggle that turns back the old code
      • bitmap[m]
        Jade[m]: thank you! we normally just add an option to DBDefs.pm for feature flags, though I think this is fine without one
      • mara42 has quit
      • mara42 joined the channel
      • pite joined the channel
      • mara42 has quit
      • mara42 joined the channel
      • yvanzo[m]
        Back!
      • reosarevok, bitmap: I made a few changes and additions, does it look good now?
      • bitmap[m]
        yvanzo: thank you, LGTM
      • mara42 has quit
      • BrainzGit
        [musicbrainz-docker] release 03v-2024-10-21.0 has been published by 14yvanzo: https://github.com/metabrainz/musicbrainz-docke...
      • mara42 joined the channel
      • yvanzo[m]
        Release complete!
      • bitmap: Should we support disabling images editing through server configuration?
      • bitmap[m]
        yvanzo: possibly yes, I was expecting we would just revert the code and never have to use it again, but that might be optimistic
      • Maxr1998_ has quit
      • Maxr1998 joined the channel
      • mara42 has quit
      • yvanzo[m]
        bitmap: Does the maxima for workers/memory in seldaā€™s PR seem appropriate (for an AX102) to you?
      • reosarevok[m]
        <BANG>
      • Welcome to another MetaBrainz Monday Meeting!
      • I have one mailed in review this week, by outsidecontext:
      • '''
      • I'm on a business trip the entire week and hardly available. Last week not much did happen. I researched a bit about code signing certificates and did some code review.
      • mayhem, please ignore the SSL.com warnings about the certificate for now. Nothing bad will happen, we just need a new one for the next Picard release. I'll get back to you once I've returned from my trip and can concentrate on MB again.
      • '''
      • On my list for today: JadedBlueEyes, kellnerd, pranav, MonkeyPython, monkey, ansh, atj, akshaaatt, yellowhatpro, lucifer, derat, yvanzo, bitmap, theflash_, zas, rimskii, reosarevok, mayhem, jasje, ericd