<06julian45> hi folks, i just went to check out the new report for video relationships mentioned in the blog post, and i got what appears to be a db permissions error - i'm away from a computer so i hope this pastebin paste of the error will suffice: https://pastebin.com/9L80XwXf
aerozol[m]
Hmm, works for me when logged in, but not in a incognito window/not logged in. I checked some other reports and it seems we offer the same inelegant server error when not logged in, for all tested.
btw @bitmap and @mayhem I have started writing a blog post for the EAA, but give me a little bit to make it good. I went back and looked at the amazing CAA announcement post and am feeling inspired to make it a really interesting post -https://blog.metabrainz.org/2012/10/09/announcing-the-cover-art-archive/
is another attempt at testing, if this looks good, I will add more cases, and for other modules too.
Btw creating these functions I eventually corrected a couple mistakes in the functions I was testing, so I am hoping them to be fine 🤪
Kladky joined the channel
Regarding docs tests, we can't do them in a binary project. There must be some workarounds which i haven't explored yet. Will do them soon
rimskii[m] joined the channel
rimskii[m]
lucifer: Hi!
Srry was busy yesterday, forgot to ask
So I am trsting lookup with wolf db, but its not finding any spotify_track_id, even though they exist in db
rimskii[m] uploaded an image: (53KiB) < https://matrix.chatbrainz.org/_matrix/media/v3/download/matrix.org/YiEmjwKkmNThhBsWtKAhlpJI/Screenshot%202024-06-25%20at%2012.08.15.png >
but when I use listenbrainz labs api, it works for the same mbids
rimskii[m] uploaded an image: (87KiB) < https://matrix.chatbrainz.org/_matrix/media/v3/download/matrix.org/YzJdhAStklCJhoXzSReIDhXc/Screenshot%202024-06-25%20at%2012.08.31.png >
I was assuming maybe it because of this [`sql statement`](https://github.com/metabrainz/listenbrainz-server/blob/master/listenbrainz/labs_api/labs/api/spotify/utils.py#L27) , but not sure whats the problem exactly
s/am/was/, s/trsting/testing/
lucifer[m]
rimskii: i checked on spotify that track seems to have been removed from the album, its present on a different album but that is named different. so it doesn't match. the prod database probably has some old entries that haven't been removed.
that's my guess so far. will have to check in more detail later.
rimskii[m]
Okay
I’ve checked for like 10 tracks, so far it couldn’t find Spotify id track for any of them
ok maybe even more than 10 tracks
reosarevok[m]
yvanzo: STYLE-2560 seems like a fair suggestion, do you see any reason not to do that?
outsidecontext[m: It feels like every time is that time 😅
outsidecontext[m
yes, indeed. Apple really does not want developers to get bored, so they always keep them busy with stuff like this. and with deprecations and new requirements nobody asked for.
lucifer[m]
<rimskii[m]> "ok maybe even more than 10..." <- oh weird
i'll take a look into it rimskii
can you work on the indices of apple music metadata in meantime?
yvanzo[m]
Hi yellowhatpro, nice, that goes in the right direction, how can I comment the code?
yellowhatpro[m]
Hi, sorry I didn't understand. Commenting the code in?
mayhem[m]
outsidecontext: I read every last word of that agreement before accepting it. 🤥
yvanzo[m]
reosarevok: I quickly checked existing poster rels, it seems to be all good. 😄
reosarevok[m]
Ok, deprecated it then (from the bus)
outsidecontext[m
mayhem: sure sure :D Thanks
yvanzo[m]
yellowhatpro: For example, can you have texts with different kinds of punctuations following URLs in `test_extract_urls_from_text`?
Sophist-UK joined the channel
yellowhatpro[m]
Yupp, I'll the cases ✅✅
Thanks, I will add such cases for other modules too then.
* Yupp, I'll add the cases
yvanzo[m]
yellowhatpro: `web.archive.org` URLs should probably be excluded too.
Maxr1998_ has quit
yellowhatpro[m]
Aah yes, I was thinking to ask it to you whether to keep already archived URLs or not, guess we'll not
yellowhatpro: Still about tests, declaring URLs to be tested beforehand like you did in `test_should_exclude_url` is a good practice and can be generalized to other tests.
aerozol[m]
I have drafted the EAA blog post, but would like to have another look at it with fresh eyes tomorrow, if people can hold off publishing. I am also not sure if we want to wait until all of the relevant doc pages etc have been finished (nudge me if there is anything I can help with there). Anyone who wants to have a look: https://wordpress.com/post/blog.metabrainz.org/...
yvanzo[m]
yellowhatpro: You can use `concat!` or `format!` for reusing these in longer strings.
yellowhatpro[m]
yessirr will do ✅
yvanzo[m]
aerozol: Thank you! We probably want to wait for the documentation pages to be ready, or people will start wondering about broken link or just trying to find some documentation. Hopefully, only a few pages are missing, see CAA-154.
aerozol: Would it be worth a video tutorial like the one you made for how to add album covers?
aerozol[m]
yvanzo: I had the same thought! I’m a bit time poor but if it takes a few days to get the documentation finished I will try make a video on adding events + one on adding event art before then. I have a new (much better) microphone, looking forward to trying it!
rimskii[m]
lucifer: just tested spotify_track_from_mbid and it worked for one song!
rimskii[m] uploaded an image: (66KiB) < https://matrix.chatbrainz.org/_matrix/media/v3/download/matrix.org/OozuSvHsjeQbAVYycswilPdW/Screenshot%202024-06-25%20at%2015.36.59.png >
Thanks. Are you happy with the SSH `HostName %h.metabrainz.org` thing? It does save extra having to change two lines when we add another node to the cluster.
* Thanks. Are you happy with the SSH `HostName %h.metabrainz.org` thing? It does mean you only have to change 1 line when we add another node to the cluster.
yvanzo[m]
It is fine, I just digressed.
zas[m]
@yvanzo @atj: it would be great if we could get rid of old solr cluster soon, to cut related costs.
atj[m]
Yay, merged.
zas[m]: Is there a dependency on me for this? I'm on holiday and have just been killing time on a train this afternoon.
yvanzo[m]
Not for now, thanks.
Switching SIR to the new LB name.
Successfully processing messages again.
minimal joined the channel
bitmap[m]
aerozol: the blog post looks amazing, thank you so much! ❤️
zas[m]
sentry VM is very slow it seems
I can't reliably connect to it
btw, this VM is on an old (obsolete) plan, we may rescale it
Can we stop sentry for few minutes in order to rescale it?
bitmap[m]
yes, I noticed this yesterday but couldn't even ssh into the machine
doesn't help that sentry is a resource hog
zas[m]
Ok, I'll proceed, we don't have ssh access anyway :(
sentry rescaled to AMD CPX41 (2x cores, same RAM), rebooted, ssh access is now possible again