#metabrainz

!m jasondk

wooo everyone go pin :3

yvanzo: hi! is sir-prod on the latest master?

lucifer: no, it is on the latest release + patch made by bitmap I guess

ah ok! can we update it to latest master + the patch then?

that patch should probably be pushed to master too and have a new release

yes makes sense. i have the patch around, will open a PR for it,

thank you

yvanzo: i do not have push access to sir repo. giving access to core-team should probably fix that.

[sir] 14amCap1712 opened pull request #122 (03master…bitmap-patch): Rearrange assignments in connect_to_rabbtimq https://github.com/metabrainz/sir/pull/122

lucifer: updated repo access, update PR to master and reviewed it, that bug is probably worth a SEARCH ticket too.

lucifer: I guess alastairp didn't finish the username renaming thing before going on holiday? :p

yvanzo: how do I test https://github.com/metabrainz/sir/pull/111 ?

(as in, what are you running to check whether the changes are working?)

Was thinking of starting to try looking into it

reosarevok: you can run local sir instance from musicbrainz-docker dev setup

see https://github.com/metabrainz/musicbrainz-docke...

Is there a way I can run only sir without having to run MBS on docker? Because that messed up my whole setup last time I tried and it's just a lot more annoying than running it locally :/

yvanzo: thanks! i'll open a ticket and update the commit message.

Uh-oh, 502 on the forum

zas: known issue?

reosarevok: which issue did occur with it?

reosarevok: alastairp started working on it. https://github.com/metabrainz/listenbrainz-serv... but yes its not complete yet.

yvanzo: IIRC it kept overwriting my DBDefs and it was also a lot (like, a lot) slower than just running locally

reosarevok: nope, I'll have a look

zas: seems it works now, but it didn't for a bit

yvanzo: I'm fine with keeping a separate install of docker, as long as it doesn't mess with my local install, I guess

Guess I should just reinstall docker without trying to reuse my local install, IIRC we tried that last time :)

reosarevok: DBDefs is set directly in the image for 'musicbrainz' service, it has to be changed at build time.

Yeah, IIRC it's just we were trying to reuse my existing setup

But if I'm not going to use this for MBS dev at all I guess it's fine to just install it normally separately. I'll try that

Yes, you don't need that 'musicbrainz-dev', just 'sir-dev'

yvanzo: does it not require a dockerized MB DB or anything?

param: happy birthday!

reosarevok: indeed, it is 'db' service in musicbrainz-docker

happy birthday, param!

Happy Birthday param !💯🔥

Happy Birthday, param 🎉

happy birthday param! 🎉🎉

Hi everyone,

I have recently joined the meta/musicbrainz community. For a few days, I have been facing troubles in the replication of Musicbrainz database (through Mbdata) and while I was looking for support, I noticed this chat, hoping that, by contacting users some of them might help me. Basically, following the guide there are no problems up to point 6,

everything is fine; however, on point 7, where I should import dumps in the form of .tar files, I keep running into this error:

mporting data from mbdump.tar.bz2

- Loading alternative_release_type to musicbrainz.alternative_release_type

Traceback (most recent call last):

File "/home/manuel/.local/bin/mbslave", line 8, in <module>

sys.exit(main())

File "/home/manuel/.local/lib/python3.9/site-packages/mbdata/replication.py", line 592, in main

args.func(config, args)

File "/home/manuel/.local/lib/python3.9/site-packages/mbdata/replication.py", line 253, in mbslave_import_main

load_tar(filename, db, config, config.schemas.ignored_schemas, config.tables.ignored_tables)

File "/home/manuel/.local/lib/python3.9/site-packages/mbdata/replication.py", line 245, in load_tar

cursor.copy_from(tar.extractfile(member), fulltable)

psycopg2.errors.UndefinedTable: relation "musicbrainz.alternative_release_type" does not exist

If I check in the Postgres musicbrainz database, all the tables are available. Any advice on how to proceed? Thank you so much in advance.

yvanzo: Updated PR, please review again.

hi mick_jj9: which version of mbdata are you using?

my mbdata version is 25.0.4

mick_jj9: sorry but no advice except please report the issue to https://github.com/lalinsky/mbdata/issues

maybe some other users will be able to help you at https://community.metabrainz.org/c/musicbrainz/...

@yvan

mick_jj9: nowadays it is better to use musicbrainz-docker to get a replated copy of the data.

yvanzo ok, thanks!

reosarevok, bitmap: https://github.com/metabrainz/musicbrainz-serve... is up for final review

ok

yvanzo: did you resolve the MB alerts earlier? wondering if it was consul-template again or something else

bitmap: no

I thought that was yet another sir alert. I’m not able to log in aretha atm.

btw, sir resumed processing messages from search.retry queue.

[listenbrainz-server] 14amCap1712 merged pull request #1584 (03master…cb-api): Add CritiqueBrainz functions to APIService https://github.com/metabrainz/listenbrainz-serv...

stopping test containers on aretha

oh, aretha. that's just json dumps again

it's xz + tar indeed

What's wrong with them?

they're just really big and take a long time to compress

the old williams alerts might've been tuned differently

Oh, ok

Was worried for a moment because of the flaky dump json tests

[critiquebrainz] 14amCap1712 merged pull request #378 (03master…fix-CORS-for-endpoints): Fix CORS issues for some endpoints https://github.com/metabrainz/critiquebrainz/pu...

lucifer: if you update sir's PR, please rebase it too as the GitHub auto-rebase actually injected a master merge commit :/

sure, will do.

jasondk: hi! is https://github.com/metabrainz/listenbrainz-serv... currently WIP or now ready ?

[musicbrainz-server] 14akshaaatt opened pull request #2237 (03master…getting_started): Updated favicons https://github.com/metabrainz/musicbrainz-serve...

yvanzo: fixed the sir PR.

lucifer: fixed Jira integration to sir project in Sentry, so it is possible to create ticket or link to existing one from Sentry again.

thanks! :D

[sir] 14yvanzo merged pull request #122 (03master…bitmap-patch): SEARCH-665: RuntimeError: maximum recursion depth exceeded in cmp https://github.com/metabrainz/sir/pull/122