spent time this weekend figuring out mbxml code, apart from that fixed a jira ticket and code reviews
fin
yvanzo: go
yvanzo
salute
I spent most of last week debugging a weird issue in tests renderer and coding the removal of illegitimate accounts.
Also did normal stuff: reviews, forums
finito, ListMyCDs? (psolanki seems to be missing)
ListMyCDs
I have little concrete to report. I've been toying with the code and it's been challenging but also rewarding. I've learned a lot.
go, ferbncode
ferbncode_
Hi :)
his week I mostly worked on the docker image for setting up a MB database with the dump imports. Also worked slightly on dump_manager.py related PR and worked on the final PR to get CB code completely off the ORM: https://github.com/metabrainz/critiquebrainz/pu.... That's all for my week. fin !
anyone left?
hibiscuskazeneko joined the channel
yvanzo
alastairp or Leo_Verto[m] maybe
Freso
Seems like we're through.
ruaok: spam review
ruaok
k
yvanzo: what is the general state of the spam stuff?
I'm guessing that bitmap had other things to focus on.
ruaok prods yvanzo
Freso
They were *just* here too...
ruaok
ok, we'll get back to spam related things in a minute.
bitmap
the PR he has up looks good, I think just tests need fixing
ruaok
instead, lets go with ws/2.5
yvanzo
It is almost complete, should be in production in two weeks.
ruaok
ah, ok. back to spam, I guess. :)
how about the script to remove old accounts?
yvanzo
The PR is for half of tickets, I already worked on the rest.
bitmap
I can hotfix it when it's complete
CatQuest
Spam Spam Sapm Glorius Spam ♫♪♬
ruaok
bitmap: yes, I'd like that when the other fires are put out.
yvanzo
We can probably push everything this week then.
ruaok
I'm guessing we won't really have much of a change to review the impact before I leave on vacation on thursday.
does the spammer account removal script have a dry-run option?
CatQuest
♫♪♬ Spam Spam Spam Glorious Spam ♫♪♬
ruaok
it might be nice to post an excerpt of the output and let people sanity check it.
Sophist-UK
Freso: If you want to try to discuss multi-level works, ping me.
Freso
Sophist-UK: I think it's the only topic for tomorrow, so... :)
ruaok
yvanzo: please let people sanity check the results before you run it.
CatQuest
+++111
yvanzo
The removal should not be effective before two weeks.
ruaok
sounds like we're too early to have much of a recap on the spam stuff.
ok, then we can review when I get back.
onward to WS/2.5
zas is having to spend a lot of time blocking asshats and in general combatting spammers.
we really need to move to an API key, there is no way around it MB has become too popular now.
Quesito
+1
ruaok
there is a toolkit that can make all of this pretty easy... Kong, right zas?
problem is that it only works with JSON.
zas
yes
I can explain if you want ruaok
ruaok
sure, go!
zas
Basically Kong is an API gateway, providing everything to manage an API-key based JSON API, including plugins for rate limiting, oauth, and tons of shit