the delete callback is added in the connect_to_rabbitmq method. delete_callback method is also wrapped by a action_wrapper decorator which contains another call to connect_to_rabbitmq, https://github.com/metabrainz/sir/blob/master/s...
yvanzo
lucifer: also added instructions to rabbitmq PR for docker server configs
lucifer
so if the connection to rabbitmq fails it could lead to recursing infinitely.
thanks again for the rabbitmq instructions! :D
bitmap
reosarevok: re: MBS-11852 - should it? I see we call load_for_edits, but that doesn't appear to load editors
maybe 48 hours would be an okay compromise, given your last comment there. I was just wanted the behavior to match the "few days" language we already use
ruaok: i increased the timeout temporarily with help of yvanzo's instructions, and deployed 1546. i have requested an entire batch of stats, working well currently half of stats processed and no timeouts :D
one job recordings all time stats took 40 mins to run 🤯, its probably the longest one we have currently methinks.
ruaok
reosarevok: yvanzo bitmap can one of you please kick the mb website container on .42? See telegram for details.
bitmap
ruaok: sure
kicked (it was on pink ftr)
yvanzo
Search has been completely updated with edits since yesterday.
Etua has quit
peterhil has quit
lucifer
ruaok: entire batch of stats processed without any delivery timeout :D
ruaok
!m lucifer
BrainzBot
You're doing good work, lucifer!
lucifer
:-D
akshaaatt[m]
Hi lucifer !
lucifer
hi!
akshaaatt[m]
In the mb android app's manifest, we have defined a few things
Like the instant play experience tag and the Google backup api key
I would like to understand if we use them?
lucifer
those are remanants of the past. i don't recall them being used in the current version of the app.
akshaaatt[m]
Yup
I was documenting the things so I'll be removing them then.
And after having a quick overview of the permission our app needs
I don't think we need READ_PHONE_STATE and ACCESS_WIFI_STATE
We are able to check whether the phone is connected to the internet using the ACCESS_NETWORK_STATE alone
lucifer
cool let's remove those then, but be sure to check if an older version has some permission differences
akshaaatt[m]
Yup
Since our minSdk version is 21, we are at par with the latest and best needs