#metabrainz

/

      • zas
        we have quest which has some diskspace, but it needs some preparation, I can have a look tomorrow (1AM here)
      • 2023-11-17 32147, 2023

      • bitmap
        no problem, it can wait until tomorrow
      • 2023-11-17 32159, 2023

      • zas
        actually there's not much to do, I'll try to prepare it right now
      • 2023-11-17 32148, 2023

      • zas
        hmmm, nope, I forgot this machine has no local net 10.2.2 so it is a bit trickier to setup
      • 2023-11-17 32116, 2023

      • zas
        I'll have a look tomorrow
      • 2023-11-17 32111, 2023

      • bitmap
        👍
      • 2023-11-17 32122, 2023

      • bitmap
        yvanzo: if you're around tomorrow morning and have time to prepare a copy of https://github.com/metabrainz/docker-server-confi… for trille -> quest that'd be great
      • 2023-11-17 32126, 2023

      • lusciouslover joined the channel
      • 2023-11-17 32158, 2023

      • lusciouslover has quit
      • 2023-11-17 32147, 2023

      • zas
        bitmap: I managed to have picard website (beta) on quest, see https://github.com/metabrainz/docker-server-confi…
      • 2023-11-17 32122, 2023

      • zas
        still experimental, but I think we are slowly moving towards getting rid of 10.2.2
      • 2023-11-17 32134, 2023

      • lusciouslover joined the channel
      • 2023-11-17 32139, 2023

      • zas
        I'll continue on this tomorrow, sleepy now
      • 2023-11-17 32138, 2023

      • lusciouslover has quit
      • 2023-11-17 32126, 2023

      • lucifer has quit
      • 2023-11-17 32138, 2023

      • lucifer joined the channel
      • 2023-11-17 32143, 2023

      • mayhem
        mooooin!
      • 2023-11-17 32154, 2023

      • s1b1 has quit
      • 2023-11-17 32133, 2023

      • nyaomi has quit
      • 2023-11-17 32153, 2023

      • nyaomi joined the channel
      • 2023-11-17 32114, 2023

      • s1b1 joined the channel
      • 2023-11-17 32133, 2023

      • genius3000 has quit
      • 2023-11-17 32145, 2023

      • genius3000 joined the channel
      • 2023-11-17 32151, 2023

      • antlarr has quit
      • 2023-11-17 32105, 2023

      • antlarr joined the channel
      • 2023-11-17 32114, 2023

      • trclst has quit
      • 2023-11-17 32138, 2023

      • trclst joined the channel
      • 2023-11-17 32148, 2023

      • relaxoMob
        Does the MusicBrainz API has any sort of API Terms of use like this, i can link to? Maybe just to the API doc? https://www.themoviedb.org/api-terms-of-use
      • 2023-11-17 32148, 2023

      • relaxoMob
        I am working on the privacy policy for my android app which uses the MusicBrainz API. Does this also apply to the API? https://wiki.musicbrainz.org/About/Privacy_Policy
      • 2023-11-17 32148, 2023

      • relaxoMob
        Which data is stored, in example in server logs? IP addresses?
      • 2023-11-17 32148, 2023

      • relaxoMob
        Anything else I need to mention in the app? Something like "this is not a part of MetaBrainz"?
      • 2023-11-17 32154, 2023

      • prout has quit
      • 2023-11-17 32106, 2023

      • prout joined the channel
      • 2023-11-17 32125, 2023

      • yvanzo
        O’Moin
      • 2023-11-17 32123, 2023

      • yvanzo
        Hi relaxoMob: Good questions. As a developer, make sure to follow https://musicbrainz.org/doc/MusicBrainz_API and more particularly “What can I do with”, “Who can use”, “Application rate limiting and identification”
      • 2023-11-17 32123, 2023

      • yvanzo
        Our privacy policy should be updated to include the API, so yes link to it but no it doesn’t cover it at the moment.
      • 2023-11-17 32149, 2023

      • yvanzo
        Basically, just link to https://metabrainz.org/privacy
      • 2023-11-17 32102, 2023

      • yvanzo
        API server “logs include: your IP address, your browser's "User-Agent" string, and which page you requested” as in https://wiki.musicbrainz.org/About/Privacy_Policy…
      • 2023-11-17 32140, 2023

      • yvanzo
        And yes, you can specify that the MusicBrainz API is provided by a third-party, the MetaBrainz Foundation.
      • 2023-11-17 32135, 2023

      • yvanzo
        bitmap, zas: available now, just reading the back logs.
      • 2023-11-17 32153, 2023

      • zas
        yvanzo: moooin
      • 2023-11-17 32129, 2023

      • BrainzGit
        [musicbrainz-server] 14reosarevok opened pull request #3094 (03master…MBS-13368): MBS-13368: Properly compare series relationships if series is target https://github.com/metabrainz/musicbrainz-server/…
      • 2023-11-17 32156, 2023

      • zas
        yvanzo: what are the exact dependencies for this stuff? I was thinking about quest, but this one isn't on 10.2.2.0/24 (but it is on 10.10.10.0/24) and not sure if we can run everything there, but for sure it has disk space and free to use right now;
      • 2023-11-17 32124, 2023

      • zas
        also, we have many services only listening on 10.2.2.0/24, it can be an issue if there are to be accessed from 10.10.10.0/24 on machines not having 10.2.2.x IP.
      • 2023-11-17 32119, 2023

      • yvanzo
        zas: It needs to have access to pg (jimmy/hendrix) and possibly redis (aphex) through consul, and to sshd (aretha) through local config.
      • 2023-11-17 32141, 2023

      • yvanzo
        s/possibly// (for sure)
      • 2023-11-17 32138, 2023

      • yvanzo
        (pg through consul too)
      • 2023-11-17 32122, 2023

      • zas
        well, consul will report IP/port as provided by serviceregistrator, and that's usually 10.2.2.x IP, which is an issue for quest
      • 2023-11-17 32146, 2023

      • zas
        so we need to find a way to expose needed services on 10.10.10.x too
      • 2023-11-17 32149, 2023

      • zas
        that's tricky
      • 2023-11-17 32154, 2023

      • zas
        atj: around?
      • 2023-11-17 32105, 2023

      • zas
        the experiment I did yesterday shows we can get gateways using backends on 10.10.10.x (because they are parts of this network, and backend service run on it too, plus serviceregistrator provides correct IP/port)
      • 2023-11-17 32110, 2023

      • zas
        would it be possible to use a mirror db (instead of directly accessing to jimmy/hendrix) ? Why does it need redis (which one?) ?
      • 2023-11-17 32116, 2023

      • yvanzo
        Those were the dependencies for mb production cron, I will check those for meb.o website too
      • 2023-11-17 32132, 2023

      • yvanzo
        zas: no, because it is also making changes to the DB.
      • 2023-11-17 32141, 2023

      • zas
        hmmm, then we need to get db accessible from 10.10.10.x, what needs to be done for it to happen? and it will not suffice... ok. Can't we run this stuff on hendrix and/or jimmy ? they have everything needed
      • 2023-11-17 32109, 2023

      • yvanzo
        It needs access to musicbrainz-redis-store-4 to expire some sessions, and access to musicbrainz-redis-cache-4 for caching.
      • 2023-11-17 32117, 2023

      • yvanzo
        hendrix is running the secondary DB iiuc, it could be the best choice.
      • 2023-11-17 32134, 2023

      • zas
        yes, and it's much simpler to set up
      • 2023-11-17 32118, 2023

      • yvanzo
        zas: where are the primary/secondary roles defined atm? (somewhere in ansible iirc)
      • 2023-11-17 32118, 2023

      • atj
        i'll be around in 30 mins or so
      • 2023-11-17 32125, 2023

      • atj
        why does the dump fail on trille?
      • 2023-11-17 32133, 2023

      • zas
        disk space
      • 2023-11-17 32106, 2023

      • zas
        trille doesn't have much of it, it wasn't meant to host huge dumps
      • 2023-11-17 32132, 2023

      • zas
        yvanzo: it isn't defined in ansible, and with recent changes bitmap did on pg, I think it is more dynamic (see https://github.com/metabrainz/syswiki/blob/master…)
      • 2023-11-17 32123, 2023

      • yvanzo
        relaxoMob: Actually, this is a web API, and our privacy policy already cover web access logs: https://metabrainz.org/privacy :)
      • 2023-11-17 32145, 2023

      • atj
        Primary and secondary is set in docker AFAIU
      • 2023-11-17 32156, 2023

      • atj
        the docker image, sorry
      • 2023-11-17 32150, 2023

      • yvanzo
        yes, found it, it is set by IP address: docker-postgres-cluster/Makefile
      • 2023-11-17 32117, 2023

      • mayhem
      • 2023-11-17 32150, 2023

      • mayhem
        Weird Indian pizzas. Lunchtime break at the Indian summit.
      • 2023-11-17 32157, 2023

      • yvanzo
        monkey: I need my breakfast now!
      • 2023-11-17 32105, 2023

      • yvanzo
        oops, mayhem ^
      • 2023-11-17 32128, 2023

      • yvanzo
        weird but at least no pineapple ;)
      • 2023-11-17 32132, 2023

      • atj
        how much disk space is required?
      • 2023-11-17 32158, 2023

      • yvanzo
        More than 100G
      • 2023-11-17 32110, 2023

      • yvanzo
        (disk space currently available on trille)
      • 2023-11-17 32131, 2023

      • atj
        is that compressed size?
      • 2023-11-17 32131, 2023

      • yvanzo
        + the current volumes, let me check
      • 2023-11-17 32109, 2023

      • yvanzo
        more than 250G, total
      • 2023-11-17 32124, 2023

      • yvanzo
        it contains a lot of already compressed archives.
      • 2023-11-17 32118, 2023

      • yvanzo
        (250G is what trille provides for those volumes and free disk space left)
      • 2023-11-17 32147, 2023

      • atj
        that's not too bad then
      • 2023-11-17 32135, 2023

      • yvanzo
        (I mean 250G is currently not enough but not too far.)
      • 2023-11-17 32114, 2023

      • yvanzo
        If we can have 350G, we should be fine until we split those into separate containers that can run on different servers.
      • 2023-11-17 32132, 2023

      • mayhem
        yvanzo: I was informed that these are not the weird pizzas we wanted. These are standard fare. Super tasty.
      • 2023-11-17 32116, 2023

      • monkey
        They did look pretty normal
      • 2023-11-17 32146, 2023

      • fletchto99 has quit
      • 2023-11-17 32130, 2023

      • fletchto99 joined the channel
      • 2023-11-17 32129, 2023

      • yvanzo
        zas: just noticed that there is currently no temperature data for jimmy and hendrix
      • 2023-11-17 32159, 2023

      • yvanzo
        (I was trying to compare their respective activity as hendrix is also running the search indexer)
      • 2023-11-17 32156, 2023

      • yvanzo
        but otherwise they use about the same amount of memory, so hendrix definitely seems to be a good choice, I will prepare PRs
      • 2023-11-17 32116, 2023

      • zas
        yvanzo: yes, it is because the hardware is very new, and it seems there's no support yet for temp sensors on those. I guess we'll be able to get it work after some time passed.
      • 2023-11-17 32130, 2023

      • yvanzo
        👍
      • 2023-11-17 32143, 2023

      • atj
        yvanzo: I will create a dedicated ZFS dataset for the dumps that will need to be bind mounted in the containers
      • 2023-11-17 32141, 2023

      • Maxr1998_ joined the channel
      • 2023-11-17 32139, 2023

      • Maxr1998 has quit
      • 2023-11-17 32104, 2023

      • bitmap
        yvanzo: the primary/secondary status is mainly stored in the database cluster (and you can query PG to determine whether a server is primary). ideally it isn't hard-coded anywhere, but there are a couple places still: the barman and repmgr configuration files
      • 2023-11-17 32127, 2023

      • bitmap
        the PRIMARY var in docker-postgres-cluster/Makefile actually isn't used, I'll remove it
      • 2023-11-17 32149, 2023

      • yvanzo
        bitmap: Ok thanks!
      • 2023-11-17 32112, 2023

      • bitmap
        btw, I nearly have a patch ready that will cause MB to direct a lot more read-only traffic to the standby...but if the current load average on jimmy is any indication, that hopefully shouldn't be any problem for running the dumps and whatnot
      • 2023-11-17 32130, 2023

      • yvanzo
        bitmap: Is there any reason for defining `RSYNC_FULLEXPORT_PORT` in `admin/config.incremental-json-dump.sh`instead of `script/services.sh`? Is it okay if I change that?
      • 2023-11-17 32101, 2023

      • bitmap
        yvanzo: yes, I think it makes senes to move them
      • 2023-11-17 32100, 2023

      • bitmap
        it makes it easier to change the configuration without restarting the container (if you are setting the env variable on container start)
      • 2023-11-17 32106, 2023

      • bitmap
        but restarting a cron container isn't an issue
      • 2023-11-17 32149, 2023

      • yvanzo
        It’s just that this has to be changed for moving stuff from trille to hendrix
      • 2023-11-17 32112, 2023

      • bitmap
        right, having it all in one repo makes things easier too.
      • 2023-11-17 32124, 2023

      • yvanzo
        So a new image is needed anyway.
      • 2023-11-17 32148, 2023

      • BrainzGit
        [musicbrainz-server] 14reosarevok opened pull request #3095 (03master…MBS-13366): MBS-13366: Also show medium data on Edit medium preview https://github.com/metabrainz/musicbrainz-server/…
      • 2023-11-17 32148, 2023

      • yvanzo
        bitmap: Also why do we have production cron on trille and the sshd for fullexport on aretha, while we have json dumper on aretha and the sshd for it on trille?
      • 2023-11-17 32144, 2023

      • bitmap
        heh
      • 2023-11-17 32118, 2023

      • bitmap
        well, json dumps are on aretha because it had the most space available for it
      • 2023-11-17 32121, 2023

      • bitmap
        and ftp is on aretha, which is where the full exports are synced to
      • 2023-11-17 32129, 2023

      • relaxoMob has quit
      • 2023-11-17 32108, 2023

      • bitmap
        the incremental json dumps are synced back to trille because that's where metabrainz.org is hosted
      • 2023-11-17 32135, 2023

      • bitmap
        the full json dumps are sent to ftp
      • 2023-11-17 32147, 2023

      • yvanzo
        Ok, I didn’t notice that meb.o had a direct dependency on this volume.
      • 2023-11-17 32127, 2023

      • yvanzo
        I can see it from `services.sh` now.
      • 2023-11-17 32134, 2023

      • yvanzo
        Thanks!
      • 2023-11-17 32140, 2023

      • relaxoMob joined the channel
      • 2023-11-17 32118, 2023

      • bitmap
        reosarevok: just remembered that for https://github.com/metabrainz/musicbrainz-server/… since prod and beta share a cache, we will have to deploy the {Data,Entity}::LinkAttributeType types to both prod and beta before we can put the rest of the patch on beta
      • 2023-11-17 32148, 2023

      • reosarevok
        Oh, ok. Do you see any reason why that'd be problematic?
      • 2023-11-17 32155, 2023

      • reosarevok
        (other than the annoyance of putting them on prod early)
      • 2023-11-17 32149, 2023

      • bitmap
        nope, I mainly just have to put it in a separate commit/PR that targets the production branch
      • 2023-11-17 32108, 2023

      • reosarevok
        Ok :) Feel free
      • 2023-11-17 32156, 2023

      • reosarevok
        btw, for https://github.com/metabrainz/musicbrainz-server/… maybe you can look into adding some basic tests for the whole duplicates setup :)
      • 2023-11-17 32120, 2023

      • reosarevok
        (seems no test explicitly tests compareRelationships but that might be because it's implicit elsewhere)
      • 2023-11-17 32157, 2023

      • bitmap
        heh
      • 2023-11-17 32115, 2023

      • bitmap
        not sure if I will get to it today but I can have a look
      • 2023-11-17 32123, 2023

      • reosarevok
        Oh, no need at all for it to be today
      • 2023-11-17 32128, 2023

      • relaxoMob has quit
      • 2023-11-17 32140, 2023

      • bitmap
        > console.log('it thinks it is the same');
      • 2023-11-17 32143, 2023

      • reosarevok
        It doesn't need to be even before this is merged IMO if we document needing them eventually, but we might just want them
      • 2023-11-17 32146, 2023

      • reosarevok
        Oh ffs, I didn't push :D
      • 2023-11-17 32159, 2023

      • reosarevok
        (I did change that locally)
      • 2023-11-17 32100, 2023

      • reosarevok
        On it
      • 2023-11-17 32126, 2023

      • reosarevok
        Done, thanks
      • 2023-11-17 32155, 2023

      • bitmap
        I'm glad I usually don't accidentally commit my console.logs, because I use much less professional messages
      • 2023-11-17 32135, 2023

      • relaxoMob joined the channel
      • 2023-11-17 32151, 2023

      • BrainzGit
        [musicbrainz-server] 14yvanzo opened pull request #3096 (03production…move-cron-to-hendrix): Drop hard-coded IP address for incrental JSON dump https://github.com/metabrainz/musicbrainz-server/…
      • 2023-11-17 32103, 2023

      • BrainzGit
        [musicbrainz-server] 14yvanzo merged pull request #3096 (03production…move-cron-to-hendrix): Drop hard-coded IP address for incrental JSON dump https://github.com/metabrainz/musicbrainz-server/…
      • 2023-11-17 32134, 2023

      • reosarevok
        aerozol, kellnerd, your input in https://tickets.metabrainz.org/browse/STYLE-2416 would be lovely
      • 2023-11-17 32135, 2023

      • BrainzBot
        STYLE-2416: Add "Broadcasted By" relationship for Labels or Places
      • 2023-11-17 32144, 2023

      • reosarevok
        (see my comment)
      • 2023-11-17 32102, 2023

      • reosarevok
        bitmap: well, I do too once I get frustrated enough, luckily I didn't here ;p
      • 2023-11-17 32110, 2023

      • bitmap
        yes, sometimes I have to grep for profanities before I commit :P
      • 2023-11-17 32156, 2023

      • kellnerd
        Hehe, my temporary log messages are usually only too short to be understood by anyone else :)
      • 2023-11-17 32159, 2023

      • mayhem
        Serious talk about doing the summit in India next year.... :-)
      • 2023-11-17 32153, 2023

      • kellnerd
        reosarevok: Looks great :) How about making https://musicbrainz.org/relationship-attribute/eb… (time) available for this relationship?
      • 2023-11-17 32120, 2023

      • reosarevok
        I considered it, but