#metabrainz

/

      • mara42 joined the channel
      • nawcom has quit
      • nawcom joined the channel
      • nelgin
        Thanks bitmap.
      • The other problem is that with Ubuntu 24.02 docker-compose produces the same issues I saw in #285 however if I remove the old docker-compose and install docker-compose-v2 I have to run it was "docker compose" which seems to break a lot of other things.
      • admin/create-amqp-extension: line 29: docker-compose: command not found
      • create-amqp-extension: cannot install: the Docker Compose service 'indexer' is not up
      • mara42 has quit
      • mara42 joined the channel
      • bitmap[m]
        nelgin (IRC): try with `DOCKER_COMPOSE_CMD='docker compose' admin/create-amqp-extension`
      • in general you should be able to prefix any of the commands with that environment variable to get them working
      • (and there are plans to make docker compose v2 the default soon, IIRC)
      • d4rk has quit
      • d4rk joined the channel
      • GautamShorewala[ has quit
      • mara42 has quit
      • mara42 joined the channel
      • mara42 has quit
      • d4rkie has quit
      • d4rkie joined the channel
      • mara42 joined the channel
      • mara42 has quit
      • mara42 joined the channel
      • mara42 has quit
      • mara42 joined the channel
      • nelgin
        I'll give that a try thanks. Also, how would I get that missing data if I didn't get it in the initial import?
      • Using the environment variable worked thank you.
      • groups: cannot find name for group ID 153
      • I'm seeing a bunch of these also
      • mara42 has quit
      • mara42 joined the channel
      • vardhan joined the channel
      • d4rkie has quit
      • d4rkie joined the channel
      • mara42 has quit
      • jasje[m]
        <sparksphereinspi> "Hey @jasje, I made the fix for..." <- I wasn’t facing any duplicate events issue but events just not loading after feb 19 + events disappearing when refreshing. Please look into these issues before initiating review.
      • mara42 joined the channel
      • nelgin
        WARNING: amqp could not commit tx mode on broker 1, reply_type=2, library_errno=4
      • Any ideas on this one during replication? There's a couple of issues, #142 but doesn't seem relevent since all my dockers are up and not #!50 since this isn't an upgrade.
      • hmm
      • =WARNING REPORT==== 27-Mar-2025::05:56:06 ===
      • closing AMQP connection <0.1358.0> (172.19.0.4:42368 -> 172.19.0.2:5672, vhost: '/search-index-rebuilder', user: 'sir'):
      • client unexpectedly closed TCP connection
      • sparksphereinspi
        I’ve fixed the issue on my local setup. Can you check if there facing any other issues?
      • s/facing//
      • mara42 has quit
      • mara42 joined the channel
      • mara42 has quit
      • outsidecontext[m has quit
      • lucifer[m]
        holycow23: ansh looking through the stats proposal, some of the stats are suggested to be implemented directly on MB/LB db. we shouldn't be doing that. all the stats should be generated on the spark cluster unless there is a very strong reason not to do so.
      • mara42 joined the channel
      • mara42 has quit
      • mara42 joined the channel
      • mayhem: for notifications, i think we should skip BU and CB integration. just integrate them in LB. we intend to get rid of CB frontend at some point and then for just LB there is no need of a separate BU library.
      • vardhan has quit
      • holycow23[m]
        <lucifer[m]> "holycow23: ansh looking through..." <- I’m not entirely sure yet, but my plan is to generate all of these items directly on the Spark cluster. They will leverage MB and LB for certain properties, though they will otherwise operate independently on Spark itself. If there’s anything specific that needs adjustment or further development, please let me know.
      • lucifer[m]
        holycow23: ah cool then, i misunderstood it from the proposal.
      • d4rkie has quit
      • d4rkie joined the channel
      • mara42 has quit
      • BrainzGit
        [musicbrainz-server] 14reosarevok merged pull request #3497 (03master…MBS-10563): MBS-10563: Add autoselect + sidebar for AWA URLs https://github.com/metabrainz/musicbrainz-serve...
      • mara42 joined the channel
      • mthax has quit
      • mthax joined the channel
      • vardhan joined the channel
      • mayhem[m]
        <lucifer[m]> "mayhem: for notifications, i..." <- agreed on CB, torn on BU. What if another python project comes along? move it then?
      • lucifer[m]
        [@mayhem:chatbrainz.org](https://matrix.to/#/@mayhem:chatbrainz.org) yes move it later then
      • SigHunter has quit
      • SigHunter joined the channel
      • suvid[m] has quit
      • mara42 has quit
      • mara42 joined the channel
      • mara42 has quit
      • mara42 joined the channel
      • mara42 has quit
      • mayhem: about the notifications document (https://docs.google.com/document/d/1jK2OreDzYq_...), the service that sends emails would be on MeB side right?
      • mayhem[m]
        yes.
      • lucifer[m]
        and it would call the MB mailing service?
      • mara42 joined the channel
      • mayhem[m]
        yes, that was my plan.
      • lucifer[m]
        so then the digest settings will also need to be stored in MeB?
      • mayhem[m]
        yes.
      • lucifer[m]
        instead of in each project that is. okay cool.
      • mayhem[m]
        though it was floated that people might want different settings for different projects.
      • lucifer[m]
        it will become simpler once we migrate rest of user data to MeB from MB but should be doable even now.
      • mayhem[m]
        but I have no idea how many people want this.
      • lucifer[m]
        i was thinking one digest email for all projects.
      • mayhem[m]
        agreed
      • lucifer[m]
        let's start simple i think and go with unified settings and later split it if needed.
      • mayhem: in that case do we need to store the email in the notification table, can't we just get it from users table too?
      • we don't have a users table at the moment but we will have it in the future.
      • when data has been migrated from MB to MeB.
      • mayhem[m]
        ideally we would not duplicate it, but look it up
      • lucifer[m]
        yup makes sense
      • vardhan_ joined the channel
      • mayhem[m]
        I see we already have two proposals for the subsonic project, which is great.
      • lucifer[m]
        i haven't seen any
      • mayhem[m]
        did we say that level of UI work (to setup a subsonic server) is ok?
      • lucifer[m]
        except the message in the LB channel just a few mins ago
      • mayhem[m]
        one was directly submitted to the gsoc site, the other will be posted on community soon.
      • lucifer[m]
        i see, i'll check the gsoc site but unless the contributor shows up here we aren't considering it right?
      • mayhem[m]
        correct, and I have not seen this person here yet.
      • but the proposal isn't terrible, so we'll see.
      • lucifer[m]
        do you want to email them and ask them to show up here in time?
      • vardhan has quit
      • mayhem[m]
        yeah, will do.
      • lucifer[m]
        cool
      • as for the UI, i think the only user facing part would be forms to create/edit/delete subsonic servers?
      • integrating in BP is frontend work but not UI specifically.
      • imo the proposal is very general, and does not reference existing LB implementations so my issue is how to know if all the content was not just generated by some llm/ai.
      • mayhem[m]
        <lucifer[m]> "as for the UI, i think the..." <- yes
      • <lucifer[m]> "integrating in BP is frontend..." <- ah ok. not sure I understand, but that sounds "its ok" enough. :)
      • lucifer[m]: the one on the gsoc site?
      • lucifer[m]
        mayhem[m]: i mean its typescript/react code but doesn't require much css
      • mayhem[m]: yup
      • monkey[m]
        Not much in terms of 'design'
      • lucifer[m]
        yup that
      • mayhem[m]
        ok, thanks for the clarification.
      • d4rk has quit
      • d4rk joined the channel
      • misnad[m] has quit
      • d4rk has quit
      • d4rk joined the channel
      • Mitraansh joined the channel
      • ApeKattQuest has quit
      • ApeKattQuest joined the channel
      • ApeKattQuest has quit
      • ApeKattQuest joined the channel
      • Mitraansh has quit
      • Mitraansh joined the channel
      • d4rk has quit
      • d4rk joined the channel
      • 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
      • mara42 has quit
      • mara42 joined the channel
      • minimal joined the channel
      • julian45[m] has quit
      • mara42 has quit
      • mara42 joined the channel
      • vardhan_ has quit
      • mara42 has quit
      • monkey[m]
        OMG. ansh : after hours spent on debugging the issues with DOMPurify, and hours again with tinycolor, I finally found the one stupid config line that allows us to import ESM modules when running the test suite (i.e. in Node):