#metabrainz

Who has wiki admin access? Let’s finally update that favicon

zas?

akshaaatt: It is probably fine to use the current sentry instance for mobile apps if needed.

bitmap, mayhem, zas: Is it fine ^ to use MeB Sentry instance for mobile apps? I have admin privileges so I can probably follow these steps: invite akshaaatt to a #mobile team and create a project for the app.

Anyone help with how to fix the track artist on track 5 or https://musicbrainz.org/release/e2d4719c-34d2-4...

Looks like the wrong recording is associated with that track, as it's listed as a cover recording, when this is the original album.

adhawkins: wrong channel :) but you should reach https://musicbrainz.org/user/IvanDobsky about this recording as he added the “cover” attribute yesterday.

Whoops, sorry. Meant to post it in #musicbrainz. Sorry yvanzo

No worry :)

yvanzo: seems fine by me to use MeB sentry for mobile apps, especially in terms of sending user information to MeB controlled servers instead of 3rd party ones

Thanks alastairp, I should have pinged you too as you wrote the Sentry admin doc :)

akshaaatt: sent invitation

Thanks a lot yvanzo!

alastairp: please approve the invitation if you can, I don’t have enough privileges for that.

alastairp: Also maybe invite him as Admin rather than Member.

Thank you!

done.

yvanzo: would you like to be a manager or owner too?

akshaaatt: let us know if you have any problems creating a project. I guess sentry knows all of the correct things to set up when making a mobile one

alastairp: I rarely use it. But shouldn’t previous team members be removed now?

yvanzo: I dont see why not...

mayhem: I didn’t set up Sentry, just asking.

yvanzo: what a great task to do during doc-week :)

It’s not really a doc thing.

it's not, but it's "organisation health" related I guess

alastairp: actually, wasn't there a doc with steps when on/off-boarding team members?

yvanzo: there is!

one thing about sentry is that it's not team-only. e.g. I added ansh when he was working on CB in order to see bug reports

mayhem, alastairp: i was thinking to improve accuracy of listens we could dump deleted listesn and remove those from spark.

*accuracy of stats reports

lucifer: is that because when listens are deleted we don't remove them from spark?

how do you know what a deleted listen is if it's deleted?

Yes

not until the next dump, which might not be for 15 days.

alastairp: Do they still need it? Do they have a contract with MeB to access users’ data.

remind me are, IP addresses listed in sentry? if so, it must contract-holders only. :(

mayhem: yes

yes, that's a good point. user ip's are in sentry, so it should be contractors only

ok, lets get rid of non-contractor accounts and in the future not add them.

I believe there is a way to get sentry to anonymise IPs, but I don't know if you can do that on a per-user basis

however in the meantime, I'll remove other users

done

thx

alastairp: Thanks

Sentry is already mentioned in ContractorOnboarding.md and it only mentions removing Admin flag.

here is our coordination doc this week:

that should definitely be updated until we can identify if we can block IPs from particular users

But it doesn’t seem possible to restrict access to PII based on role: https://docs.sentry.io/product/accounts/members...

hmm. I dont quite see how to share the document so that Meb contractors can edit, everyone else comment.

alastairp: Related backlogged issue: https://github.com/getsentry/sentry/issues/26300

yvanzo: I also saw https://github.com/getsentry/sentry-javascript/...

but it seems that you can only do this globally on an entire project. We could look and see if it's actually necessary for us to capture this information

ok, explicitly listed the team now. let me know if there are problems with the doc.

however it might be easier to just ensure that sentry-jira connection is properly working, and we can raise tickets in jira for other devs

please jump in and clean up from last year -- I just copied last year's doc as a starting point for this year.

Right, I don’t think that IP addresses are useful to musicbrainz-server since we have these in server logs already, but editor names are.

lucifer: I think we did a pretty good job on combining LB docs into user/dev/sysadmin last year, right?

monkey can you help me out to install bookbrainz locally

it will really help me as i am trying to figure out why i am getting this error after reading the documentation

bitmap: I dropped the seemingly unneeded JS bits of MBS-8668 now but I also noticed the error message it used to show when the rels were already dupes is also gone, maybe we can look into that at some point. Just to make sure that all the errors we *do* expect for rels do still appear.

MBS-8668: Adding duplicate entities to a series is still allowed https://tickets.metabrainz.org/browse/MBS-8668

yvanzo: the planning doc has the sysadmin/onboarding doc item from last year. From your reading of the onboarding doc just now do you think there is anything that could be added?

zas: IP unblock request in mail for ya

alastairp: I just updated it about Sentry account removal; It looks fine otherwise.

yvanzo: updated the MB section a bit, but I'm sure there's more stuff to add there :)

mayhem: handled

thx!

thanks yvanzo

lucifer: monkey: do I remember you talking in the last few weeks about the API documentation system that is used in BB and how it might be applied to LB?

bitmap: we changed this, right?

IIRC we always need to get import $c now?

alastairp: indeed. lucifer asked for the swagger config files in BB

Hi Thuraabtech, what error are you hitting?

 > [bookbrainz-webservice 1/10] COPY ./docker//rc.local /etc/rc.local:

 > [bookbrainz-webservice 4/10] COPY ./docker//webserver.command /etc/service/webserver/exec-command:

Try a pastebin or something similar, it will be easier to paste the whole output

i got this error when i used docker buildx build --platform=linux/amd64 -t getting-started . --network="host"

okay

when i tried running the container locally with this command

docker run -d -p 9200:9200 docker.elastic.co/elasticsearch/elasticsearch:5.6.8

i got a warinig

WARNING: The requested image's platform (linux/amd64) does not match the detected host platform (linux/arm64/v8) and no specific platform was requested

014a1f04e6130a687512ebb96354cae26125a7e6da12b7e37bc4421254c95654

i'm actually using mac m1 chip so that's why i think i am getting this warning

Sorry Thuraabtech I've never seen this issue, not sure I can help you much other than googling the error message :/

i searched it on stack over flow and it recommended me to use this command: docker buildx build --platform=linux/amd64 -t myorg/mytag:1.0.0 .

[+] Building 2.8s (27/31)

Anyone else here seen this type of docker issue before?

no, but I would guess that it is m1 related as Thuraabtech suspects

where are the BB build/run docs?

alastairp: Do you mean these? https://bookbrainz-dev-docs.readthedocs.io/en/l... https://bookbrainz-user-guide.readthedocs.io/en...

yes! thanks

Thuraabtech: which command were you running when you got your initial error?

initially i got warning when i was trying to run bookbrainz locally

docker run -d -p 9200:9200 docker.elastic.co/elasticsearch/elasticsearch:5.6.8

i used the following above command to run it locally

Thuraabtech: I don't see that command in the installation docs: https://bookbrainz-dev-docs.readthedocs.io/en/l...

were you following a different guide?

actually i am new to docker so i don't know how to run it locally , so i used google for the help

it looks like on apple silicon you need to include the `--platform=linux/amd64` flag in any command that you execute

I see that you tried this with the build step above

so in the case of the run command, you would need something like "docker run --platform=linux/amd64 -d -p 9200:9200 docker.elastic.co/elasticsearch/elasticsearch:5.6.8"

however, this is perhaps something that we can improve in the bookbrainz documentation and tools

Perhaps something that would need to be added to the docker-compose file?

akshaaatt: You should have received an email from Sentry, I created a #mobile team and a project for each app.

monkey: I think it might be time that we do something about bb's develop.sh (to allow people to add this flag)

I'm not sure if the `--platform` flag has a docker-compose equivalent

It does, just checked

(starting with compose v2)

yeah, that kind of indicated the same

yeah, that might be a good solution then