the software for sentry used to be called raven. then they made a new version called sentry-python. BU uses raven
2021-02-05 03616, 2021
alastairp
when I added sentry to the dataset hoster, I used sentry-python instead, because I knew that we were going to upgrade it
2021-02-05 03640, 2021
alastairp
for now I'd be happy to just match the versions, because we're going to fix it anyway
2021-02-05 03654, 2021
alastairp
I guess we should do something like what iliekcomputers and I discussed in the BU upgrade ticket... use a range in packages that are installed as dependencies (in this case datasethoster, I guess?) and an exact version in the repo (lb?)
2021-02-05 03655, 2021
_lucifer
yeah, i saw that. i was thinking of taking that up but i am not sure how to test it.
2021-02-05 03633, 2021
_lucifer
yeah, that makes sense, alastairp .
2021-02-05 03601, 2021
CatQuest
it's bandcamp friday!!!
2021-02-05 03654, 2021
atj
I know, I've spent too much money already...
2021-02-05 03602, 2021
CatQuest
\o/
2021-02-05 03604, 2021
zas
bitmap: did you see the 58s mean peak from cage few minutes ago?
2021-02-05 03644, 2021
bitmap
no, I wasn't watching
2021-02-05 03619, 2021
bitmap
there is definitely an issue with requests timing out connecting to PG
that triggers it but I'm seeing if the same happens with psql
2021-02-05 03632, 2021
ruaok
client.collections[COLLECTION_NAME].documents.search(...) is the function I want to mock, but I don't know how to do it when there are multiple levels, one of which is an array deref....
2021-02-05 03617, 2021
zas
bitmap: I don't get why only few servers seems impacted...
2021-02-05 03622, 2021
_lucifer
ruaok: maybe mock the search function instead of the client?
2021-02-05 03629, 2021
zas
and not all requests...
2021-02-05 03614, 2021
ruaok
how does one do that, _lucifer ?
2021-02-05 03633, 2021
bitmap
zas: no idea. same issue is happening with psql, so it's not a Perl or musicbrainz-server issue AFAICT
2021-02-05 03641, 2021
bitmap
I'll check if it happens *outside* the container
2021-02-05 03649, 2021
zas
ok
2021-02-05 03612, 2021
zas
I suspect a network issue... but I found nothing obvious
2021-02-05 03659, 2021
_lucifer
ruaok: what's the type of the documents? or location of the search function
2021-02-05 03643, 2021
ruaok
location is typesense.Client.collections[COLLECTION_NAME].documents.search
2021-02-05 03659, 2021
ruaok
and it it supposed to return a dict...
2021-02-05 03646, 2021
_lucifer
try mocking `typesense.documents.search`
2021-02-05 03652, 2021
bitmap
zas: interesting, running the same script outside the container doesn't trigger the iissue at all
2021-02-05 03604, 2021
zas
so that's a docker problem
2021-02-05 03617, 2021
bitmap
/home/bitmap/test_psql.sh on cage
2021-02-05 03607, 2021
bitmap
it hangs almost immediately (after a few tries) inside the musicbrainz-website-prod container
_lucifer: oh, that sounds very promising -- how did you arrive at this?
2021-02-05 03620, 2021
_lucifer
ruaok: i had to look where the search function was located i.e. the module or the class. because the mock is done at that level. I looked at the source code of typesense to find the search method. It was in `Documents` class in documents.py.
2021-02-05 03623, 2021
sumedh has quit
2021-02-05 03622, 2021
ruaok
ok, I guess I'll read the source of what I am trying to mock next time. thanks!
2021-02-05 03631, 2021
_lucifer
ideally, that shouldn't be needed but i couldn't find it in the api docs.
2021-02-05 03648, 2021
_lucifer
so had to go in the source
2021-02-05 03631, 2021
atj
zas: I'm curious what you changed?
2021-02-05 03606, 2021
alastairp
ruaok: mmm, mocks in class instances are super annoying
