#metabrainz

/

      • yano has quit
      • yano joined the channel
      • aerozol[m]
        Hey Jade, some good feedback re. the plain text email format: https://community.metabrainz.org/t/email-redesi...
      • lusciouslover has quit
      • lusciouslover joined the channel
      • dabeglavins has quit
      • Came up in my subscriptions, doesn’t require a reply but you might like to mayhem: https://www.reddit.com/r/selfhosted/comments/1g...
      • dabeglavins joined the channel
      • lucifer[m]
        zas: hi! please let me know when you are around to deploy some oauth stuff?
      • pite has quit
      • dabeglavins has quit
      • aerozol[m]
        Hey ansh worked a bit late but got the homepage mockups all ready for you to work on! I have left a few important notes as figma comments, let me know if you’d rather have the notes written out elsewhere, or if you would rather have a ticket with screenshots or something. Let me know if you have any questions
      • Link for the icons (use the linked ones, they are optimized for size) is at the top, in the yellow box
      • reosarevok[m]
        <aerozol[m]> "Don’t know if it’s on purpose..." <- That's an artist credit so I guess it might show on hover there?
      • huhridge joined the channel
      • huhridge has quit
      • huhridge joined the channel
      • huhridge has quit
      • yvanzo: hey, is there any way I can help with the solr stuff today? :)
      • huhridge joined the channel
      • LupinIII
        re recording alias
      • [12:29] reosarevok[m] The alternative I can see is showing it on hover only
      • I'd prefer that tbh
      • yvanzo[m]
        Hi reosarevok: Sure, I’m currently stuck on backup creation.
      • LupinIII
        oh i guess maybe wont show on mobile. but honestly an option for "alwaysshow/only with hover" where alwasy show is default would be stellar
      • lucifer[m]
        monkey: hi! are you still unable to download your data exports? i tried it last week and today again. it works fine for me every time. i do see some errors in sentry so it is clearly broken for some users but unable to reproduce it on my end so far.
      • LupinIII
        aerozol[m]: i *really* like the addition of the word "profile" in the sidebar. that's very usefull to a guy like me who from the lb frontpage just want to go to their listens and such
      • i always end up going to like "feed" first because idk the current doesn't make me think "my lsitens/myprofile"
      • BrainzGit
        [musicbrainz-server] 14reosarevok opened pull request #3412 (03master…MBS-13828): MBS-13828: Add report for empty release groups https://github.com/metabrainz/musicbrainz-serve...
      • reosarevok[m]
        yvanzo: ok, now available! How can I help with that?
      • yvanzo[m]
        atj, zas: SSH-ing to any SolrCloud 9 node takes more than 2 minutes. Is this response time expected? How can it be shortened?
      • mayhem[m]
        <lucifer[m]> "monkey: hi! are you still unable..." <- See https://github.com/metabrainz/listenbrainz-serv...
      • reosarevok[m]
        <LupinIII> "oh i guess maybe wont show on..." <- For what it's worth, very very few recordings currently have aliases, so it'll be very rare at least at first anyway :)
      • I think we can move it once more aliases exist if it's bothering people?
      • (maybe it won't)
      • mayhem[m]
        restarting the container fixed the problem, but the PR above is to mitigate this mysterious rollback required from happening again.
      • yvanzo[m]
      • I thought that the backups were not working, I was just not connecting to the same cloud.
      • lucifer[m]
        mayhem: i think that error is unrelated. the connection having problems in the error log is `ts_conn`, the connection in the code is `db_conn`
      • mayhem[m]
        fun
      • lucifer[m]
        i'll add the missing the rollback and move it to a finally block instead.
      • if you prefer to restart the container for these errors, i can leave that part in.
      • mayhem[m]
        lucifer[m]: yes please. I think all of our containers should have that idea -- a catch all, that if we catch that, exit the container. who knows what went wrong, when this happens, so why rely on the state of a possibly broken container?
      • lucifer[m]
        makes sense but then we don't need a catch all. the process/container will exit on an uncaught exception anyway.
      • mayhem[m]
        it didn't used to. :)
      • lucifer[m]
        hmm, weird.
      • https://github.com/metabrainz/listenbrainz-serv... - pushed the change. can you take a look before i merge?
      • mayhem[m]
        I think I liked the 5 second sleep in there. it prevents the container from cycling super fast if the startup fails.
      • lucifer[m]
        sure will add it back in
      • mayhem[m]
        could be 2 seconds...
      • lucifer[m]
        done.
      • mayhem[m]
        looks good. can't approve since its my PR.
      • atj[m]
        <yvanzo[m]> "atj, zas: SSH-ing to any..." <- Try `ssh -4`
      • yvanzo[m]
        Thanks, that resolved it.
      • BrainzGit
        [listenbrainz-server] 14mayhem merged pull request #3040 (03master…tiny-cron-cleanup): LB-1682: run-metadata-cache-seeder manage.py endpoint no longer exists https://github.com/metabrainz/listenbrainz-serv...
      • yvanzo[m]
        atj, reosarevok, zas: Added this tip to the docs
      • atj[m]
        It shouldn't be needed, not sure if it's a Hetzner IPv6 issue or something else.
      • zas[m]
        hmmm; it seems a misconfiguration
      • zas@rex:~$ ping6 -c2 2a01:4f8:c010:b00c::1... (full message at <https://matrix.chatbrainz.org/_matrix/media/v3/...>)
      • yvanzo[m]
        Is there any delay for you reosarevok?
      • zas[m]
        atj: ^^ we have 2a01:4f8:c010:b00c::2 in DNS, node has inet6 2a01:4f8:c010:b00c::1/64 scope global, firewall issue?
      • Ah got it, just a misconfiguration; almost all our nodes use :2, but not solr cloud ones
      • I'll fix the DNS
      • mayhem[m]
        <aerozol[m]> "Came up in my subscriptions..." <- thanks, responded.
      • reosarevok[m]
        yvanzo: what exactly is it that I should be doing? Is it running the upgrade instructions on that file on a local install of solr?
      • (the main reason I haven't been working on solr stuff so far is because there seem to be no actionable instructions on what needs to be done and tested and how)
      • yvanzo[m]
        reosarevok: I pointed you to the docs for setting up SSH.
      • reosarevok[m]
        Yes, I saw that, but just setting up ssh isn't much of a test, so I expect you want me to do something with it too :)
      • yvanzo[m]
        Read the backlogs, there are issues with SSH-ing.
      • Setting up ssh is a prerequisite to help further.
      • zas[m]
        Not really an SSH issue, there were issues with DNS IPv6 addresses of those nodes, it should be fixed now (though TTL is 1h). They were set as ending by :2 instead of :1. ssh used IPv6 first and fallback on IPv4 if it fails, explaining the delay.
      • reosarevok[m]
        I did ssh mb-solrcloud-1 with that config and it let me in immediately, so I guess zas fixed it
      • yvanzo[m]
        Yes he just did
      • mayhem[m]
        zas: I am writing a cron wrapper script that monitors the output of the cron job and if the cron job does not exit with status code 0, it posts the last X lines from the output to the LB telegram channel. I've got everything working, except for the "send to telegram" bit. Any thoughts on how to best do this?
      • zas[m]
        Hmmm; not really I guess that's possible using something like https://pypi.org/project/telegram-send
      • Hmmm it seems you can even use curl, see https://hackernoon.com/how-to-create-a-simple-b... for an example
      • mayhem[m]
        ok, send it directly. I wondering if you had something else in mind. cool, I'll investigate
      • do you have the bot token and a chat id I can use for this purpose, zas ?
      • reosarevok[m]
        yvanzo: ok, neat, what's the next step if any? :)
      • (if you're on top of things and don't need help that's of course fine, just say so!)
      • Maxr1998 has quit
      • Maxr1998 joined the channel
      • yvanzo[m]
        reosarevok: Please let me know what time you are available today, to be efficient in Q/A.
      • reosarevok[m]
        Well, I need to take 30 mins to eat at some point (although I can do that while stuff happens)
      • After that I'm available until 17 CET or so
      • yvanzo[m]
        You need a MB Docker instance with sample data and search server set up.
      • I'm going for lunch, I'll be back for 14 CET.
      • monkey[m]
        mayhem: might be worth storing those logs+message somewhere to retry in case the telegram sending fails (for example when we get a proper outage or loss of connectivity)
      • mayhem[m]
        that smacks of effort.
      • FAILURE_REPORT_RETRIES = 10... (full message at <https://matrix.chatbrainz.org/_matrix/media/v3/...>)
      • huhridge has quit
      • remember, these cron logs will still be on the server. we just won't get a notification for it
      • BrainzGit
        [listenbrainz-server] 14amCap1712 merged pull request #3036 (03master…mitigate-background-task-rollback-error): Background tasks container stuck in a rollback loop https://github.com/metabrainz/listenbrainz-serv...
      • allen joined the channel
      • monkey[m]
        mayhem: If you have some time, can you please look at LB#3042 ? It's short and we're getting a bunch of 500 errors on artist pages after the partial date fix
      • BrainzBot
        LB-1681: Fix 500 error on artist page: https://github.com/metabrainz/listenbrainz-serv...
      • mayhem[m]
        lol, sorry to have caused this
      • monkey: so, if we have a date that is 2022, it will be shown at 2022-01-01?
      • monkey[m]
        That's what it looks like to me, which is not ideal
      • mayhem[m]
        that is plain wrong.
      • kellnerd[m]
        Nope, that is just the sort key function.
      • mayhem[m]
        ah, ok, then in facts that isn't shown to the user.
      • but for sorting that should be fine.
      • monkey[m]
        I'll check what the front-end does, because we sort again on front end. Possibly this is only used for keeping a sort order for the cover art collage
      • pite joined the channel
      • Yes I suppose these default values are only so this formatting does not result in throwing an error datetime.strptime(release_date, "%Y-%m-%d")
      • huhridge joined the channel
      • But ansh would know better
      • Thanks for the catch kellnerd !
      • BrainzGit
        [listenbrainz-server] 14MonkeyDo merged pull request #3042 (03master…LB-1681): LB-1681: Fix 500 error on artist page https://github.com/metabrainz/listenbrainz-serv...
      • monkey[m]
        Thanks !
      • kellnerd[m]
        You are welcome, I just had a look at the PR between two other tasks and was asking myself the same question :)
      • yvanzo[m]
        bebop
      • reosarevok[m]
        Hi! It's indexing sample recordings
      • (for the last hour)
      • yvanzo[m]
        OK
      • On sample data?
      • reosarevok[m]
        Yeah
      • yvanzo[m]
        Can you please check if any change is needed to mbsssss for Solr 9.7.0?
      • More specifically with the configuration file that has been copied from Solr 9.6.1.
      • huhridge has quit
      • reosarevok[m]
        Got to url
      • yvanzo[m]
        Index building times would be useful to keep for future comparison.
      • (assuming that no other big app is running)
      • reosarevok[m]
        It wasn't, except for Chrome, which I guess is pretty large. But it crashed my laptop anyway so :D
      • I guess I'll just redo anything after url
      • (including)
      • yvanzo[m]
        reosarevok: Maybe use trille instead? Set up with `COMPOSE_PROJECT_NAME=reodombstuff` in `.env`
      • reosarevok: We don’t need the recording index, just the time to build it for comparison on the same machine.
      • These indexes are built with Solr 7, there is no upgrade path to Solr 9 but re-indexing.
      • reosarevok[m]
        Ah, ok, so I didn't need to index at all? :D
      • It took about an hour or so for the recording index
      • 1h 10 min approx
      • yvanzo[m]
        Did you check mbsssss?
      • reosarevok[m]
        Not yet
      • Let's see
      • yvanzo[m]
        I'm checking Maven dependencies for mb-solr
      • reosarevok[m]
        Heh, apparently indexing work and sending the laptop's image to my monitor at the same time is too much to ask 😅
      • It just gave up on the big screen entirely
      • yvanzo[m]
        Maybe there is a paper jam, try pulling your typewrite carriage 😉
      • huhridge joined the channel
      • reosarevok[m]
        yvanzo: most of the config docs seem the same, 9.7 adds a new option
      • reosarevok[m] sent a code block: https://matrix.chatbrainz.org/_matrix/media/v3/download/chatbrainz.org/axPkJRHMSyDNvUVTiOfjlYGm
      • Not sure if useful?
      • yvanzo[m]
        Probably not as the default is sensible.
      • reosarevok[m]
        Everything else seems exactly the same
      • So I expect no changes are needed
      • BrainzGit
        [mb-solr] 14yvanzo opened pull request #59 (03master…solr-9.7.0): Upgrade Solr version to 9.7.0 and other dependencies https://github.com/metabrainz/mb-solr/pull/59
      • yvanzo[m]
        Thank you.