#metabrainz

/

      • yvanzo
        MeB accounts
      • 2024-02-08 03900, 2024

      • Satyaraj[m]
        > <@irc_yvanzo:moviebrainz.org> tweeted about it
      • 2024-02-08 03900, 2024

      • Satyaraj[m]
        * your twitter username?
      • 2024-02-08 03912, 2024

      • yvanzo
      • 2024-02-08 03926, 2024

      • monkey posted on BB twitter
      • 2024-02-08 03928, 2024

      • yvanzo
        (links)
      • 2024-02-08 03922, 2024

      • zas
        all alerts cleared (but load on kiss, soon)
      • 2024-02-08 03937, 2024

      • yvanzo
        lucifer: might be worth updating docker-server-configs then
      • 2024-02-08 03909, 2024

      • lucifer
        yvanzo: it was a temporary move, reverted.
      • 2024-02-08 03931, 2024

      • lucifer
        containers are now in sync with what is mentioned in docker-server-configs
      • 2024-02-08 03913, 2024

      • yvanzo
        👍
      • 2024-02-08 03936, 2024

      • zas
        everything looks good on my side, alerts are all gone, and stats look normal now
      • 2024-02-08 03954, 2024

      • bitmap
        nice, same here
      • 2024-02-08 03901, 2024

      • monkey
        👏
      • 2024-02-08 03949, 2024

      • bitmap
        I'll bbiab but can look into deploying MBS-13449 afterward, yvanzo
      • 2024-02-08 03950, 2024

      • BrainzBot
        MBS-13449: Reimplement transactional cache without database locks https://tickets.metabrainz.org/browse/MBS-13449
      • 2024-02-08 03945, 2024

      • zas
        We finally sorted out this mess. Now back to real work ;)
      • 2024-02-08 03908, 2024

      • yvanzo[m] joined the channel
      • 2024-02-08 03908, 2024

      • yvanzo[m]
        zas: Where is the script to double-check that there is no longer any duplicate MAC addresses?
      • 2024-02-08 03956, 2024

      • zas
        ansible -m shell -a 'docker ps --format {% raw %}{{.ID}}{% endraw %} | while read id; do docker inspect $id | jq ".[] .NetworkSettings .Networks .bridge .MacAddress"; done |sort |uniq -c |sort -n |grep -vE "^\s+1|null" || true' docker_servers
      • 2024-02-08 03906, 2024

      • zas
        I just checked, all clear
      • 2024-02-08 03918, 2024

      • yvanzo
        👏
      • 2024-02-08 03947, 2024

      • zas
        lucifer: on kiss & gaga it would be great to prune old containers & images
      • 2024-02-08 03919, 2024

      • zas
        bitmap: I just did on paco, because of jenkins (it creates images but never clear them, reclaimed 5gb)
      • 2024-02-08 03951, 2024

      • zas
        lucifer: I mean kiss & wolf (gaga's ok)
      • 2024-02-08 03940, 2024

      • mayhem[m] joined the channel
      • 2024-02-08 03940, 2024

      • mayhem[m]
        bitmap (IRC): just got mail from matthew@kite about a missing repllication packet on meb.org. could you please look into that?
      • 2024-02-08 03940, 2024

      • mayhem[m]
      • 2024-02-08 03941, 2024

      • bitmap
        mayhem[m]: it's expected since cron was stopped after packet 165383 from 15:00 UTC. 165384 will be generated in ~10 minutes
      • 2024-02-08 03959, 2024

      • mayhem[m]
        fabu, thanks!
      • 2024-02-08 03907, 2024

      • lucifer
        zas: done
      • 2024-02-08 03926, 2024

      • zas
        lucifer: thanks!
      • 2024-02-08 03937, 2024

      • mayhem[m] relays the message to the BBC/kite
      • 2024-02-08 03904, 2024

      • bitmap
        I should have timed that better and let cron run at 16:00, will consider that next time
      • 2024-02-08 03904, 2024

      • yvanzo
        lucifer: What time would work for you on tomorrow?
      • 2024-02-08 03928, 2024

      • lucifer
        yvanzo: i should be around most of the day.
      • 2024-02-08 03907, 2024

      • yvanzo
        Ok, I should be up early but leave early too.
      • 2024-02-08 03921, 2024

      • lucifer
        cool, ping me whenever you are around.
      • 2024-02-08 03933, 2024

      • yvanzo
        👍
      • 2024-02-08 03946, 2024

      • yvanzo
        bitmap: Do you mean downtime at :15?
      • 2024-02-08 03954, 2024

      • ApeKattQuest
        åssen gor'e?
      • 2024-02-08 03958, 2024

      • ApeKattQuest
        åssen går'e?
      • 2024-02-08 03933, 2024

      • lucifer
        yvanzo: can you point me to the branch or instructions to follow? I have some time now too and can test it out.
      • 2024-02-08 03946, 2024

      • bitmap
        yvanzo: sorry not sure I understand the question
      • 2024-02-08 03954, 2024

      • yvanzo
      • 2024-02-08 03941, 2024

      • yvanzo
        lucifer: Instructions are the same but using Git Bash
      • 2024-02-08 03919, 2024

      • lucifer
        yvanzo: can I use wsl 2 Ubuntu bash instead?
      • 2024-02-08 03937, 2024

      • yvanzo
        sure if you want
      • 2024-02-08 03954, 2024

      • lucifer
        cool
      • 2024-02-08 03925, 2024

      • yvanzo
        lucifer: Also git should be configured to not mess end of line: See https://docs.google.com/document/d/1bz32duq95jSfi…
      • 2024-02-08 03954, 2024

      • yvanzo
        bitmap: scheduling downtime at :15 would let replication packets go next time :)
      • 2024-02-08 03937, 2024

      • bitmap
        ah, right
      • 2024-02-08 03916, 2024

      • BrainzGit
        [musicbrainz-server] 14mwiencek merged pull request #3130 (03production…nix-advisory-locks): MBS-13449: Reimplement transactional cache without database locks https://github.com/metabrainz/musicbrainz-server/…
      • 2024-02-08 03959, 2024

      • bitmap
        I'm merging production into the beta branch now, is there anything else I should include (or is beta frozen?)
      • 2024-02-08 03946, 2024

      • bitmap just merges production only
      • 2024-02-08 03913, 2024

      • relaxoMob has quit
      • 2024-02-08 03914, 2024

      • reosarevok
        I'm not sure tbh
      • 2024-02-08 03930, 2024

      • reosarevok
        yvanzo would know
      • 2024-02-08 03912, 2024

      • bitmap
        I'm just deploying the cache changes for now
      • 2024-02-08 03933, 2024

      • reosarevok
        Sure
      • 2024-02-08 03930, 2024

      • yvanzo
        👍
      • 2024-02-08 03918, 2024

      • lucifer
        yvanzo: pg_amqp is failing to build
      • 2024-02-08 03926, 2024

      • lucifer
      • 2024-02-08 03930, 2024

      • yvanzo
        beta is frozen for a while, only #3130 was missing for a simultaneous beta/prod update
      • 2024-02-08 03932, 2024

      • bitmap
        ok, updating beta now
      • 2024-02-08 03934, 2024

      • yvanzo
        lucifer: is it for `docker-compose build --no-cache db`
      • 2024-02-08 03905, 2024

      • lucifer
        yvanzo: `docker-compose build`
      • 2024-02-08 03911, 2024

      • lucifer
        i can try building just db too.
      • 2024-02-08 03921, 2024

      • lucifer
        but yes this failure was for db image.
      • 2024-02-08 03906, 2024

      • yvanzo
        lucifer: it's failing here too
      • 2024-02-08 03946, 2024

      • huhridge has quit
      • 2024-02-08 03939, 2024

      • yvanzo
        probably due to some changes upstream in postgres:12 image
      • 2024-02-08 03953, 2024

      • lucifer
        ah :/
      • 2024-02-08 03904, 2024

      • lucifer
        they seemed to have moved to bookworm.
      • 2024-02-08 03916, 2024

      • lucifer
        not sure if that's new
      • 2024-02-08 03922, 2024

      • lucifer
        do you know if we use bookworm or bullseye in production?
      • 2024-02-08 03914, 2024

      • bitmap
        for the main cluster?
      • 2024-02-08 03923, 2024

      • petitminion joined the channel
      • 2024-02-08 03947, 2024

      • lucifer
        yes
      • 2024-02-08 03913, 2024

      • bitmap
        they use metabrainz/base-image which is based on ubuntu jammy
      • 2024-02-08 03938, 2024

      • lucifer
        ah interesting, and install postgres using apt?
      • 2024-02-08 03959, 2024

      • bitmap
        yeah from the PGDG repo
      • 2024-02-08 03909, 2024

      • bitmap
      • 2024-02-08 03949, 2024

      • bitmap
        I vaguely remember having to patch pg_amqp to add "int" to all of the broker_id arguments, but I don't remember what for.
      • 2024-02-08 03911, 2024

      • lucifer
        yup that is what these warnings show as well
      • 2024-02-08 03923, 2024

      • lucifer
        do we have a fork of pg_amqp?
      • 2024-02-08 03934, 2024

      • bitmap
        nope
      • 2024-02-08 03913, 2024

      • bitmap
        this might've been for the postgres-json-dump container, which uses the postgres:16.1-bookworm image
      • 2024-02-08 03935, 2024

      • lucifer
        the official repo looks more or less unmaintained so i guess it makes sense to have our own fork with the fixes
      • 2024-02-08 03900, 2024

      • yvanzo
        lucifer: you can append `.0` to the 2nd line FROM… for testing
      • 2024-02-08 03924, 2024

      • yvanzo
        (in `build/postgres/Dockerfile`)
      • 2024-02-08 03917, 2024

      • lucifer
        makes sense, trying
      • 2024-02-08 03925, 2024

      • bitmap
        beta updated, now updating prod
      • 2024-02-08 03907, 2024

      • Zhele has quit
      • 2024-02-08 03951, 2024

      • Zhele joined the channel
      • 2024-02-08 03930, 2024

      • bitmap
        prod is updated
      • 2024-02-08 03901, 2024

      • reosarevok
        Thanks!
      • 2024-02-08 03916, 2024

      • reosarevok
        So I guess we do a release on Monday
      • 2024-02-08 03952, 2024

      • reosarevok
        Should we look into AC stuff or EAA next?
      • 2024-02-08 03910, 2024

      • bitmap
        yes ;)
      • 2024-02-08 03953, 2024

      • reosarevok
        Putting both on beta at once seems a bit much
      • 2024-02-08 03925, 2024

      • petitminion has quit
      • 2024-02-08 03930, 2024

      • bitmap
        both will probably need an extra week or two in beta
      • 2024-02-08 03922, 2024

      • bitmap
        I guess it depends on how long the beta period is
      • 2024-02-08 03900, 2024

      • Satyaraj[m] has quit
      • 2024-02-08 03901, 2024

      • reosarevok
        The EAA thing in test does seem better now, anyway.
      • 2024-02-08 03908, 2024

      • reosarevok
        All the issues we found seem correct
      • 2024-02-08 03901, 2024

      • yvanzo[m] has quit
      • 2024-02-08 03906, 2024

      • relaxoMob joined the channel
      • 2024-02-08 03900, 2024

      • aerozol
        kellnerd: thank you for the blog check! I knew you would find things to improve 🫶
      • 2024-02-08 03940, 2024

      • aerozol
        !m yvanzo for keeping users updated on all channels 🙌
      • 2024-02-08 03940, 2024

      • BrainzBot
        You're doing good work, yvanzo for keeping users updated on all channels 🙌!
      • 2024-02-08 03957, 2024

      • aerozol
        !m team for sorting out docker!!
      • 2024-02-08 03957, 2024

      • BrainzBot
        You're doing good work, team for sorting out docker!!!
      • 2024-02-08 03909, 2024

      • aerozol
        (at least, from the chat I gather that it’s been sorted?)
      • 2024-02-08 03937, 2024

      • mayhem[m] has quit
      • 2024-02-08 03940, 2024

      • petitminion joined the channel
      • 2024-02-08 03938, 2024

      • bitmap
        looks to be sorted, thankfully
      • 2024-02-08 03932, 2024

      • aerozol
        amazing, what a frikkin headache
      • 2024-02-08 03957, 2024

      • BrainzGit
        [musicbrainz-docker] 14yvanzo opened pull request #268 (03master…fix-pg-amqp-build): Fix building pg_amqp with debian bookworm https://github.com/metabrainz/musicbrainz-docker/…
      • 2024-02-08 03938, 2024

      • yvanzo
        bitmap: Has production been updated to the next release? If so, we can announce it tomorrow.
      • 2024-02-08 03957, 2024

      • yvanzo
        Will have to fix building PG for mirrors though. ^
      • 2024-02-08 03957, 2024

      • bitmap
        I only merged the cache PR into what was already in production
      • 2024-02-08 03904, 2024

      • lucifer
        yvanzo: got the same error documented in the summit notes doc
      • 2024-02-08 03958, 2024

      • yvanzo
        bitmap: Ok thus the release on Monday, I marked the ticket in beta and updated the beta banner accordingly.
      • 2024-02-08 03916, 2024

      • bitmap
        thanks!
      • 2024-02-08 03912, 2024

      • yvanzo
        lucifer: as expected, let’s look closer into this tomorrow then :)
      • 2024-02-08 03922, 2024

      • atj
        just checked and all container IPs and MACs match, aside from app on mingus but that seems intentional
      • 2024-02-08 03942, 2024

      • lucifer
        yvanzo: sure, makes sense
      • 2024-02-08 03917, 2024

      • bitmap
        I noticed an issue in the cache, but I don't think it's a cause of any bugs...just some bogus keys being set
      • 2024-02-08 03927, 2024

      • bitmap
        MB:release:recently_invalidated:HASH(0x55e14656ac08)
      • 2024-02-08 03942, 2024

      • yvanzo
        atj, zas: I didn’t remove the containers on mingus because I wasn’t sure how to re-create these.
      • 2024-02-08 03944, 2024

      • bitmap
        it seems to occur when a release is merged
      • 2024-02-08 03955, 2024

      • bitmap
        I guess I could look into that tomorrow, I'd rather review derat's PR tonight
      • 2024-02-08 03910, 2024

      • atj
        yvanzo: no harm AFAICS
      • 2024-02-08 03922, 2024

      • bitmap
        yvanzo: does that seem odd to you? https://stats.metabrainz.org/d/000000058/redis-se…
      • 2024-02-08 03911, 2024

      • relaxoMob has quit
      • 2024-02-08 03928, 2024

      • relaxoMob joined the channel
      • 2024-02-08 03933, 2024

      • bitmap
        I can manually trigger a BGREWRITEAOF, but I'm not sure why it's not doing that automatically anymore (?). and the pattern seems to have had already changed before I deployed the cache PR
      • 2024-02-08 03958, 2024

      • bitmap
        well, I went ahead and ran that (at 23:04 UTC), as you can see in the graph
      • 2024-02-08 03938, 2024

      • huhridge joined the channel
      • 2024-02-08 03925, 2024

      • huhridge has quit