#metabrainz

/

      • lusciouslover joined the channel
      • ApeKattQuest joined the channel
      • HSOWA has quit
      • Goemon has quit
      • HSOWA joined the channel
      • nik[m] joined the channel
      • nik[m]
        Does InitDb.pl have a setting that allows you to change the postgres schema it initializes? If not, is this something I could contribute?
      • lucifer[m]
        monkey: BookBrainz was down, didn't find anything in `bookbrainz-web-prod` logs but restarted the container and things seem to work fine. would be great to add timestamps to the BB webserver logs.
      • pite has quit
      • mamanullah7[m]
      • s/i think//
      • Kladky joined the channel
      • lucifer[m]
        m.amanullah7: oauth tokens are meant to expire soon, update your code to refresh the token if expired and see if that helps. otherwise its a different issue and i'll take a look.
      • oh i see, you are getting the error during the refresh token flow. push your changes, i'll check locally.
      • mamanullah7[m]
        lucifer: Sure!
      • suvid[m]
        lucifer: i think i'm pretty much done with everything now
      • just have to implement submit listens function now
      • is there any already existing function i should use for that?
      • mamanullah7[m]
        <lucifer[m]> "oh i see, you are getting the..." <- lucifer: i've deleted the refresh_token func while refactoring! i'll add and will try once then will let u know!
      • yvanzo[m] joined the channel
      • yvanzo[m]
        Hi zas, we can shut down Solr 7 cluster as of now.
      • zas[m]
        yvanzo: ok, I'll proceed
      • yvanzo[m]
        zas: I’m working on stopping the indexer.
      • (and removing the triggers, and…)
      • yvanzo[m] is off for lunch
      • petitminion joined the channel
      • julian45[m]
        zas: once solr 7 instances are shut down, could the servers perhaps be briefly kept around for use as a [nomad](https://developer.hashicorp.com/nomad) testbed?
      • zas[m]
        We get 502s on mb right now
      • yvanzo[m]
        I just tried to removed SIR triggers from PG.
      • -d
      • zas[m]
        yvanzo: it's back, likely temporary due to triggers removal
      • julian45: those are VMs, if you want to conduct tests we can easily create new ones for this purpose
      • julian45[m]
        zas[m]: fair
      • yvanzo[m]
        zas: OK, now resuming triggers removal (90% left).
      • I canceled it again. Got many ERROR: canceling statement due to statement timeout
      • reosarevok[m]
        I guess we need to temporarily increase the timeout time?
      • yvanzo[m]
        Running again.
      • Same issue again.
      • It just removed 3 triggers, there are ~300 left.
      • I stopped it again.
      • I didn't expect it to cause that such a slowdown to the website and API.
      • Trying it again with a commit.
      • OK, same effect, aborted.
      • Need to announce the downtime as I don’t see any workaround.
      • zas: I just stopped the indexer for now.
      • zas[m]
        Can we still delete VMs?
      • yvanzo[m]
        Yes
      • Index messages will just queue up in RabbitMQ for nothing.
      • petitminion has quit
      • yvanzo[m] is off now
      • zas[m]
        I stopped them
      • lucifer[m]
        <suvid[m]> "is there any already existing..." <- take a look at api_tools.py or the existing listens importer, they have a function to submit a list of listens using rabbitmq
      • petitminion joined the channel
      • __BrainzGit
        [artwork-redirect] 14dependabot[bot] opened pull request #58 (03master…dependabot/pip/urllib3-2.5.0): Bump urllib3 from 2.4.0 to 2.5.0 https://github.com/metabrainz/artwork-redirect/...
      • Jigen joined the channel
      • nobiz has quit
      • ApeKattQuest has quit
      • iconoclasthero joined the channel
      • iconoclasthero
        it looks like there's been a good bit of work on title case caps and I found an edge case that probably should be addressed since it likely comes up a lot in songs: `Les Brers in A Minor, Pt 2` looks like "Minor" should stay caps from what I've seen but the auto correct is changing it to "minor"
      • it doesn't make sense anyway since Minor's not a preposition.
      • nobiz joined the channel
      • reosarevok[m]
        It's a key
      • For classical, "major" and "minor" must be lowercased
      • If this is not classical, then it's up to you
      • If they actually were inside a minor, then it'd be "a Minor", and also illegal.
      • iconoclasthero
        i appreicate the joke.
      • man, picard wants me to file a crash report... :(
      • and sorry, i got some bum info on that. my mistake.
      • reosarevok[m]
        Actually, I think the guess case code even checks it has "in" before
      • I'm sure there's some cases where it still fails, like "He Got in A minor Car Crash" or something...
      • But we did what we could :)
      • iconoclasthero
        what's a useful title for this crash report bug? https://bpa.st/NCQA
      • reosarevok[m]
        zas, outsidecontext ^
      • iconoclasthero
        no, i was mistaken, sorry.
      • pite joined the channel
      • pite_ joined the channel
      • pite has quit
      • Maxr1998_ joined the channel
      • Maxr1998 has quit
      • petitminion has quit
      • mamanullah7[m]
        Hey ansh: i was going through the `brainzplayerSettings.tsx` to add funkwhale in code i saw we are hardcoding priority in default!! thats not the issue but i user already selected their priority list if new externalservice is added by default it will not be added in user priority list until he select again! let me if i misunderstood anything>?
      • * let me know if i
      • <mamanullah7[m]> "Hey ansh: i was going through..." <- i was thinking of dynamically pushing the newly added in existing user priority list? is it worth it! to add remove from user priority if any services removed or added?
      • monkey, ansh or anyone who can suggest about this!!
      • yvanzo[m]
        bitmap: Do we have any kind of priorities for statements/users in our PG instance? Could it be used to preserve MB response time while deleting the old triggers and functions used for live indexing the now-deleted Solr 7 cluster?
      • bitmap[m]
        yvanzo: I don't think PG has this functionality. strange that it causes timeouts though, I wonder if that requires an exclusive lock on the table (the PG documentation doesn't mention it)