akshaaatt[m]: looks awesome!! awesome work :D i'll try to review it later today.
2021-06-07 15858, 2021
akshaaatt[m]
lucifer I found AcousticBrainz Android Client and made a pull request on it š
2021-06-07 15842, 2021
lucifer
akshaaatt[m]: oh nice. i intended to add that functionality to the app last year but there are some issues with the ffmpeg dependency so not done that yet.
alastairp: thoughts on adding a manage.py commnad to read an sql script to initialize the db and execute it with psycopg2.
2021-06-07 15831, 2021
lucifer
i am working on simplifying meb.org dev setup and currently it uses subprocess to execute the scripts through psql
2021-06-07 15846, 2021
lucifer
*to clarify, only db and extensions initialisation and dropping is using psql)
2021-06-07 15815, 2021
akashgp09_ has quit
2021-06-07 15839, 2021
nelgin has quit
2021-06-07 15855, 2021
nelgin joined the channel
2021-06-07 15833, 2021
alastairp
lucifer: the init_db methods in LB do that
2021-06-07 15841, 2021
lucifer
oh nice! it makes sense to do that in other projects as well methinks.
2021-06-07 15841, 2021
lucifer
plus, we could add a command that accepts a file name to execute for apply db migrations in prod, that way we don't need to include psql client in the images.
2021-06-07 15817, 2021
akshaaatt[m]
lucifer I had a chat with timusus, the guy with the KTagLib, regarding overwriting/saving artwork to the files and apparently we'll have to add that feature on our own since he's not planning on working on it as of now. On Musicbrainz tagger, I've added cover art supports from our APIs but yeah we lag the part about saving them locally.
2021-06-07 15848, 2021
akshaaatt[m]
Lack*
2021-06-07 15849, 2021
akshaaatt[m]
Btw a French guy reached out to me regarding our app and he's really happy with it. He was suggesting some features like default select the directory on tagger. After that I went through playstore for any other apps which provide such functionality and found that our app is not being indexed on playstore. We need to better write the description, add some relevant terms in the title like "MusicBrainz - Tag, Listen and Search" something
2021-06-07 15849, 2021
akshaaatt[m]
like that and even add a precise description.
2021-06-07 15846, 2021
akshaaatt[m]
Maybe the other team members can help us prepare really good content to go with the playstore which is our policy friendly and everything.
2021-06-07 15845, 2021
Pokey has quit
2021-06-07 15849, 2021
[Pokey] joined the channel
2021-06-07 15807, 2021
lucifer
akshaaatt[m]: indeed, the app description hs much scope of improvements. regarding app indexing, its a conscious decision to not do that. the app is not on feature parity with the website and probably never will be because edits are only possible through the api so we probably don't want to redirect users using the website to the app.
2021-06-07 15831, 2021
lucifer
*edit are _not_ possible through the api
2021-06-07 15805, 2021
akshaaatt[m]
Right
2021-06-07 15840, 2021
akshaaatt[m]
Do you have any recommended tutorial for me where I could learn about JNI and how these C++ libraries work? Because I am proficient with C++ but I still am not sure how these libraries are communicating with the Android internal hardware.
2021-06-07 15858, 2021
akshaaatt[m]
If I get the hang of it I could start adding more features to our app
do you know if its expected that autocommit mode will not work if we try to execute multiple lines at once? maybe that's why we read a line at once.
2021-06-07 15856, 2021
alastairp
I can't remember why, sorry
2021-06-07 15822, 2021
lucifer
the docs do not mention anything of this sort but it looks like it still executes inside a transaction if i execute the entire file. but if i copy paste the function i linked above, it works fine.
2021-06-07 15815, 2021
lucifer
i'll try to debug this later and see if i can figure what the issue is.
2021-06-07 15841, 2021
lucifer
do you want to discuss what i should work on next in AB?
i think we have discussed this ticket before. let me see if i can find that
2021-06-07 15812, 2021
alastairp
I'd be tempted to just merge this PR as-is, then get the integration branch up to date with current master
2021-06-07 15852, 2021
lucifer
i think we should that. we can continue doing the fixups in that branch then.
2021-06-07 15858, 2021
lucifer
*should do
2021-06-07 15805, 2021
alastairp
then try and make sure that we understand the workflow of how this branch works
2021-06-07 15849, 2021
alastairp
if it's more or less good, then we could probably merge/release as-is and continue work as necessary. I recall that this PR just sets the groundwork anyway, we'll need additional features to actually make use of it
2021-06-07 15804, 2021
lucifer
makes sense
2021-06-07 15832, 2021
lucifer
i'll merge this PR then and start looking into the overall changes in the branch.
ah, great. yeah, we have to be careful about parsing these lines then
2021-06-07 15855, 2021
lucifer
added to docs only 3 weeks ago :)
2021-06-07 15850, 2021
lucifer
I'll add a comment to the function in our codebase as well.
2021-06-07 15849, 2021
lucifer
now that i think these functions could probably go into BU.
2021-06-07 15826, 2021
ruaok
moooin!
2021-06-07 15830, 2021
ruaok
zas: ping
2021-06-07 15852, 2021
ruaok
the AX51-NVMe servers are ready at Hetzner -- we can't act on the williams replacement yet, but we can act on the prince replacement.
2021-06-07 15810, 2021
ruaok
we should schedule the move for prince for.. tomorrow. you ok with that?
2021-06-07 15804, 2021
ritiek joined the channel
2021-06-07 15808, 2021
zas
Yup, ok for me
2021-06-07 15835, 2021
[Pokey] is now known as Pokey
2021-06-07 15825, 2021
ruaok
what time should we say>?
2021-06-07 15840, 2021
ruaok
and can you just confirm that we are replacing prince tomorrow?
2021-06-07 15842, 2021
zas
Let me check, lunch atm
2021-06-07 15800, 2021
ruaok
ok
2021-06-07 15813, 2021
zas
prince is running bookbrainz, rabbitmq, redis
2021-06-07 15858, 2021
monkey
(and elasticsearch)
2021-06-07 15802, 2021
zas
yes
2021-06-07 15813, 2021
zas
we need to move all these to other servers, I guess it is feasible, but not sure how much it impacts services
2021-06-07 15835, 2021
zas
elasticsearch is used only by bookbrainz?
2021-06-07 15844, 2021
monkey
Yes
2021-06-07 15811, 2021
monkey
We can do without test.bookbrainz for a bit if it helps (also means only one instance of elasticsearch instead of 2)
2021-06-07 15802, 2021
zas
the main issue will be rabbitmq (it is used by lb, caa, sir)
2021-06-07 15808, 2021
zas
how easy is it to move this to another server? can it be done without downtime (of related services)?
2021-06-07 15827, 2021
zas
sir downtime is not a problem
2021-06-07 15835, 2021
zas
caa not sure (bitmap?)
2021-06-07 15850, 2021
zas
and lb is your stuff, so I guess you can tell
2021-06-07 15809, 2021
lucifer
i think it should be easier now, lb was preped last time rabbitmq was moved.
2021-06-07 15810, 2021
zas
there's also a redis instance, not sure what is using it (mb?)
2021-06-07 15805, 2021
zas
ruaok: let's wait for bitmap to be around, and we decide appointment time depending on this, I guess somewhere end of afternoon tomorrow (17?)
2021-06-07 15857, 2021
zas
the setup of the new machine should be fast (as most scripts should now work without much hassle), if everything goes smooth, we can expect the new server to be ready under one hour in any case
2021-06-07 15816, 2021
zas
but in case of hardware issue with new server ofc
2021-06-07 15820, 2021
zas
ah redis is bookbrainz-redis
2021-06-07 15828, 2021
zas
so I guess only used by bb
2021-06-07 15830, 2021
ruaok
what do we need bitmap for in this case? (curious, I'm fine waiting)
2021-06-07 15836, 2021
zas
yvanzo: can you check how hard it is to move rabbitmq for sir?
2021-06-07 15857, 2021
zas
ruaok: I'd prefer to have bitmap for caa/rabbitmq migration
2021-06-07 15807, 2021
ruaok
got it.
2021-06-07 15836, 2021
zas
we moved all the stuff to boingo?
2021-06-07 15847, 2021
zas
s/moved/move/
2021-06-07 15820, 2021
zas
load is low on both boingo & prince, so I see no problem with that
2021-06-07 15844, 2021
zas
monkey: can you start to move bb to boingo today?
2021-06-07 15810, 2021
monkey
Yes
2021-06-07 15828, 2021
lucifer
ruaok: any update from Youtube side regarding Ouath?
2021-06-07 15856, 2021
ruaok
Nothing yet. I wonder if the rerequest went through ok.
2021-06-07 15830, 2021
lucifer
š
2021-06-07 15808, 2021
benrbray joined the channel
2021-06-07 15815, 2021
benrbray has quit
2021-06-07 15854, 2021
lucifer
ruaok: is the mail container useful during meb.org local development?
2021-06-07 15858, 2021
ruaok
Naw, it's used to send mails about customers signing up etc. Not needed, really.
nvm, i see the dropped tables are not in the dropped schemas.
2021-06-07 15805, 2021
lucifer
brainzgit seems down
2021-06-07 15833, 2021
bitmap
zas: hey, we should be able to migrate caa queues without downtime. (basically, spin up a new CAA-indexer pointing to the new queue, point pg_amqp to the new queue, then bring down the old CAA-indexer once the old queue is empty.) I can work on that now
2021-06-07 15846, 2021
bitmap
maybe something similar is possible for sir, not sure about lb
2021-06-07 15846, 2021
bitmap
it'd be a bit easier for all projects if we had a multi-node cluster, but
2021-06-07 15815, 2021
yvanzo
bitmap, zas: I checked how to migrate rabbitmq from prince to boingo.
2021-06-07 15805, 2021
yvanzo
bitmap: To soften the migration, Iām using shovels like last time.
2021-06-07 15830, 2021
bitmap
won't that duplicate messages?
2021-06-07 15820, 2021
yvanzo
no
2021-06-07 15849, 2021
bitmap
I see, it consumes them first
2021-06-07 15822, 2021
bitmap
ok, I'll work on something else then :)
2021-06-07 15802, 2021
yvanzo
bitmap: just wanted to have you around in case caa get down :)
