#metabrainz

/

      • ruaok
        I've made that clear before and it is still true.
      • bukwurm
        Leo_Verto: Ah, they're working.
      • Not sure why it was loading older page 🤔
      • KassOtsimine
        too bad the summit isn't about you personally then ¯\_(ツ)_/¯
      • thre are more people than you.
      • ruaok
        Freso: can you please step in?
      • KassOtsimine
        and fyi: YOU have not made my life that great either.
      • bukwurm
        LordSputnik: Should I let the edition mapping for the time be, and work on wrapping up of works and creators import?
      • KassOtsimine
        that's the last I wil lsay here
      • as I said. I am calm and patient
      • but even i can have my patience run out.
      • LordSputnik
        bukwurm: I'd go for that, yes
      • Wait to do editions later
      • bukwurm
        LordSputnik: Ok
      • LordSputnik
        I'd like to see importing working this week
      • bukwurm
        LordSputnik: Ok
      • LordSputnik
        It would be good to spend an hour or two with you tomorrow going through PRs and getting them closed up
      • bukwurm
        I'll commit the creator and works mapping in an hour or so.
      • LordSputnik: Sure thing 👍
      • LordSputnik
        Ok, thank you
      • Is normal meeting time ok?
      • bukwurm
        Yes
      • LordSputnik
        Ok good
      • bukwurm
        LordSputnik: I had updated the SQL PR, but kept the last updated field on source as nullable
      • LordSputnik
        Anything you want to quickly chat about now?
      • bukwurm
        As I felt that it's not necessary to always have a last_edited data
      • Is it ok?
      • LordSputnik
        Or are you happy to wait until tomorrow?
      • bukwurm
        LordSputnik: Just the SQL PR doubt :)
      • LordSputnik
        bukwurm: ok - might be useful to document what will happen when stuff gets imported again with the same origin ID and there's no last updated in the db
      • bukwurm
        I've changed it to timestamp, as was decided
      • LordSputnik
        Assume it needs reimporting?
      • bukwurm
        LordSputnik: Yea, that's what I had in mind
      • LordSputnik
        Ok, that sounds fine to me
      • Did we agree anything about the timestamp timezone in the end?
      • bukwurm
        LordSputnik: Yeah
      • I had a doubt what if the timestamp had a different timezone specified
      • But then realised we can change via some library and then check the timestamps
      • LordSputnik: I changed it to timestamp that same hour after the review :)
      • LordSputnik
        Ok I think I saw
      • bukwurm
        LordSputnik: Ok, so I think I'll make a PR of creators and works mapping today
      • LordSputnik
        That would be good, thank you
      • bukwurm
        And consumer side validators for them tomorrow
      • LordSputnik
        Ok, so by the time we meet tomorrow we should be able to import stuff?
      • bukwurm
        LordSputnik: It'll still need the bb-data connection wrapping up
      • I'll try to finish it too, but can't be sure
      • We'll be:
      • Read, parse and map, push data into queue
      • Consume, validate the data, reject/accept
      • The only part left would be to push it to the db
      • I have basic setup done for the pushing to the db part, so hopefully it won't take much time in any manner
      • outsidecontext has quit
      • LordSputnik
        bukwurm: ok, try to get as much done as you can
      • Monkey joined the channel
      • antlarr2 is now known as antlarr
      • bukwurm
        LordSputnik: 👍
      • yokel has quit
      • Monkey has quit
      • bitmap
        zas: ping
      • zas
        morning bitmap
      • bitmap
        moin zas
      • yokel joined the channel
      • about rabbitmq, I ran into an issue setting up the consul peer discovery. for some reason the consul health checks don't work
      • so perhaps we shoul skip the cluster stuff for now
      • zas
        what did you use for health checks ? defaults ?
      • bitmap
        the checks are set up by the plugin (not using registrator)
      • 15s ttl
      • zas
        ok, we can skip the cluster for now, we have to move everything before tomorrow 16:00 CEST
      • bitmap
        okay
      • zas
        9:00 AM for you i think
      • bitmap
        yup
      • zas
        there's also sentry on serge, who deployed it ?
      • bitmap
        gentlecat but it should be documented in syswiki
      • zas
        i think we can stop it, and backup bamed volumes
      • bitmap
        sure, it's not super critical
      • zas
        serge crashed again few hours ago
      • and cpu temp was super high, but we go for a full replacement to be sure
      • bitmap
        I guess backup /root/.ssh/sentry_deploy* and /root/sentry-onpremise to be sage
      • safe
      • zas
        what does it need to run ?
      • i'm thinking about deploying on a VM
      • bitmap
        i think it's all contained in the compose file it provides
      • the ssh key seems to just be for pulling from our private sentry-onpremise repo (for upgrades)
      • zas
        SENTRY_POSTGRES_HOST: "10.2.2.24" <--
      • it uses bowie ?
      • bitmap
        oh yeah, it does
      • zas
        but for its own db ? or it needs other stuff ?
      • bitmap
        just its own db
      • we talked about moving it before
      • zas
        we can now i think, using a dockerized postgres server for it, i don't know how much resources it needs
      • but basically if we can spawn a VM and move everything sentry needs on it it'd be great
      • it doesn't seem integrate in our usual stuff
      • hmmm it uses exim4 container as smtp relay it seems, this will require a change in gmail config
      • bitmap
        yeah I guess it's based on the official sentry compose files
      • I'll put the new rabbitmq on trille unless you'd rather somewhere else
      • zas
        but apps are finding sentry, how do they?
      • trille is ok, that's temporary
      • bitmap
        we forked their stuff and modified the compose file to add SERVICE_* tags
      • zas
        can we afford to have sentry down few hours?
      • bitmap
        that'd be fine
      • Leo_Verto
        yvanzo: ping
      • Leo__Verto joined the channel
      • yvanzo
        pong
      • ¿Leo_Verto ^ Leo__Verto?
      • Leo__Verto
        Matrix has a bit of lag, I'll use this
      • zas
        bitmap: for rabbitmq, you'll backup/restore rabbitmq-data, right? i think we can just do the same with sentry, we can stop caa-indexer and search-indexer
      • sshd-musicbrainz-fullexport <--- this one?
      • bitmap
        I was gonna try using the shovel plugin to move the data, but if we can stop the indexers, that'd be faster
      • zas
        we can stop them
      • bitmap
        I think sshd-musicbrainz-fullexport is unused, that was from when serge hosted ftp iirc
      • zas
      • i guess we can stop it right now
      • bitmap
        listenbrainz is also using rabbitmq, not sure what for
      • zas
        it does, that's why ruaok was whining ;)
      • i guess we don't use this rsyncd container either
      • gateways-redis isn't used atm (it is a failover)
      • bitmap
      • not sure moving the volume will work very smoothly, the hostname is hardcoded in a ton of places
      • meant to link to the start of the thread
      • zas
        hmmm
      • bitmap
        unless we set the container hostname on trille to the old container id on serge? dunno
      • zas
        when you say hardcoded in many places, where ?
      • ruaok: ping
      • bitmap
        the directory names, but also in a bunch of binary files I think. I guess there are command to change it, but it seems tricky
      • zas
        i don't understand, this hostname is what atm?
      • bitmap
        it's the hostname inside the container, which defaults to the container id
      • on serge it's 053a018a9e71
      • zas
        oh, ok. And rabbitmq depends on it ?
      • bitmap
        yeah, it won't find the data if we just move it
      • Monkey joined the channel
      • zas
        that's stupid ;) but i think we can set this "hostname" on docker command line
      • --hostname , -h
      • in fact, if rabbitmq depends on this, it should have been done before
      • using a generic name (rmq-1), instead of letting docker generates a random one
      • but you can just start the new container with -h 053a018a9e71
      • not fancy, but should work
      • bitmap
        I'll try
      • Monkey has quit
      • Monkey joined the channel
      • Dr-Flay_ joined the channel
      • Dr-Flay has quit
      • thomasross has quit
      • thomasross_ joined the channel
      • Dr-Flay joined the channel
      • Monkey_ joined the channel
      • Dr-Flay_ has quit
      • Monkey has quit
      • zas: it's moved now, I think