#metabrainz

/

      • alastairp
      • 2022-05-25 14540, 2022

      • MRiddickW joined the channel
      • 2022-05-25 14524, 2022

      • alastairp
        ansh: can we make a few changes to the way you run CB on wolf?
      • 2022-05-25 14513, 2022

      • ansh
        okay, what changes?
      • 2022-05-25 14516, 2022

      • BrainzGit
        [bookbrainz-site] 14MonkeyDo merged pull request #829 (03master…master): MEB-135: Standardizing MetaBrainz Logos https://github.com/metabrainz/bookbrainz-site/pul…
      • 2022-05-25 14533, 2022

      • alastairp
        https://github.com/metabrainz/critiquebrainz/blob… can you change the two -p arguments here to '-p ansh_critiquebrainz' and '-p ansh_critiquebrainz_test' (or similar)
      • 2022-05-25 14557, 2022

      • alastairp
        this is so that we know at a glance who is running the services
      • 2022-05-25 14513, 2022

      • ansh
        Okay, i'll make these changes
      • 2022-05-25 14525, 2022

      • alastairp
        this will have the effect of recreating your volumes, so you'll need to reimport the CB/MB/BB databases if you've already done that
      • 2022-05-25 14558, 2022

      • alastairp
        the other thing is the docker-compose port for the webserver: https://github.com/metabrainz/critiquebrainz/blob…
      • 2022-05-25 14531, 2022

      • alastairp
        like I suggested for BB the other day, could you change this to `- "127.0.0.1:8200:8200"` and use ssh forwarding to access the service?
      • 2022-05-25 14500, 2022

      • ansh
        Okay
      • 2022-05-25 14520, 2022

      • alastairp
        👍 thanks
      • 2022-05-25 14533, 2022

      • ansh
        Is it okay if I do it tomorrow?
      • 2022-05-25 14546, 2022

      • alastairp
        no problem. can you shut down critiquebrainz before you finish for the day?
      • 2022-05-25 14514, 2022

      • ansh
        Yes, i'm just testing a script, After that, i'll shut it down
      • 2022-05-25 14523, 2022

      • alastairp
        perfect
      • 2022-05-25 14553, 2022

      • zas
        IA has problems, hard to upload any cover art atm
      • 2022-05-25 14540, 2022

      • pprkut_ is now known as pprkut
      • 2022-05-25 14554, 2022

      • lucifer
        alastairp: is LB#2009 ready? i'd like to release it today. the other discussed improvements can come later i think.
      • 2022-05-25 14554, 2022

      • BrainzBot
        Allow a maximim number of listens per import: https://github.com/metabrainz/listenbrainz-server…
      • 2022-05-25 14511, 2022

      • alastairp
        I'm just finishing docs for the changes
      • 2022-05-25 14516, 2022

      • alastairp
        will have it in 10-15m
      • 2022-05-25 14526, 2022

      • lucifer
        ah cool, thanks!
      • 2022-05-25 14530, 2022

      • zas
        bitmap: weird, I'm trying to upload a cover art for https://musicbrainz.org/release/6e733ff4-2067-464… but it always fails requesting the signature, though I could upload other cover art images. https://analytics1.archive.org/stats/s3.php#3h shows a higher than usual rate of 503s
      • 2022-05-25 14552, 2022

      • zas
        it finally worked after a large number of attempts
      • 2022-05-25 14516, 2022

      • KevlarNoir has quit
      • 2022-05-25 14543, 2022

      • bitmap
        reosarevok: hey, I'll look at the dump issue first
      • 2022-05-25 14550, 2022

      • reosarevok
        Thanks!
      • 2022-05-25 14528, 2022

      • alastairp
        lucifer: updated
      • 2022-05-25 14504, 2022

      • KevlarNoir joined the channel
      • 2022-05-25 14513, 2022

      • BrainzGit
        [brainzutils-python] 14alastair opened pull request #92 (03master…pep508): Use PEP508-style requirements, supported by both pip and install_requires https://github.com/metabrainz/brainzutils-python/…
      • 2022-05-25 14525, 2022

      • alastairp
        lucifer: ^ fixed, turns out it wasn't actually a problem
      • 2022-05-25 14556, 2022

      • lucifer
        alastairp: oh great!
      • 2022-05-25 14536, 2022

      • alastairp
        yeah, we were looking in the wrong direction - trying to make setuptools work with the old pip-style urls
      • 2022-05-25 14542, 2022

      • alastairp
        but we could just go the other way instead
      • 2022-05-25 14548, 2022

      • lucifer
        makes sense
      • 2022-05-25 14530, 2022

      • alastairp
        so maybe we upgrade this syntax in everything else too... I'll try and use it in CB when upgrading to this version
      • 2022-05-25 14534, 2022

      • BrainzGit
        [brainzutils-python] 14alastair merged pull request #92 (03master…pep508): Use PEP508-style requirements, supported by both pip and install_requires https://github.com/metabrainz/brainzutils-python/…
      • 2022-05-25 14523, 2022

      • lucifer
        sounds good. indeed, we can update in projects when they upgrade to new BU.
      • 2022-05-25 14526, 2022

      • BrainzGit
        [brainzutils-python] release 03v2.5.1 has been published by 14github-actions[bot]: https://github.com/metabrainz/brainzutils-python/…
      • 2022-05-25 14556, 2022

      • chinmay
        Thanks CatQuest :)
      • 2022-05-25 14532, 2022

      • bitmap
        zas: I see a lot of caa errors in sentry for different users/releases too. we were getting 'read timeout' from the IA servers. but it seems quiet since 13 minutes ago
      • 2022-05-25 14509, 2022

      • skelly37 joined the channel
      • 2022-05-25 14520, 2022

      • BrainzGit
        [critiquebrainz] 14alastair opened pull request #428 (03master…mb-schema-27): Upgrade to MB schema 27 and new version of mbdata and sample DB https://github.com/metabrainz/critiquebrainz/pull…
      • 2022-05-25 14525, 2022

      • BrainzGit
        [bookbrainz-site] 14MonkeyDo merged pull request #846 (03master…docker-setup): chore: Docker setup improvements https://github.com/metabrainz/bookbrainz-site/pul…
      • 2022-05-25 14532, 2022

      • alastairp
        lucifer: I updated that constant
      • 2022-05-25 14531, 2022

      • lucifer
        alastairp: thanks, lets merge then?
      • 2022-05-25 14539, 2022

      • alastairp
        yes! 👍
      • 2022-05-25 14535, 2022

      • lucifer
        huh tests are failing
      • 2022-05-25 14537, 2022

      • BrainzGit
        [critiquebrainz] 14alastair merged pull request #428 (03master…mb-schema-27): Upgrade to MB schema 27 and new version of mbdata and sample DB https://github.com/metabrainz/critiquebrainz/pull…
      • 2022-05-25 14544, 2022

      • lucifer
        E ImportError: cannot import name '_parse_int_arg' from 'listenbrainz.webserver.views.api' (/code/listenbrainz/listenbrainz/webserver/views/api.py)
      • 2022-05-25 14504, 2022

      • alastairp
        urgh, my fault
      • 2022-05-25 14512, 2022

      • alastairp
        pycharm said it was unused so I deleted it
      • 2022-05-25 14519, 2022

      • alastairp
        I'll put it back
      • 2022-05-25 14522, 2022

      • BrainzGit
        [musicbrainz-server] 14mwiencek opened pull request #2541 (03master…mbs-12400): MBS-12400: fix art_type dump clobbering https://github.com/metabrainz/musicbrainz-server/…
      • 2022-05-25 14553, 2022

      • lucifer
        ah, maybe tests are still importing it with/without use
      • 2022-05-25 14502, 2022

      • BrainzGit
        [critiquebrainz] release 03v-2022-05-25.0 has been published by 14github-actions[bot]: https://github.com/metabrainz/critiquebrainz/rele…
      • 2022-05-25 14520, 2022

      • lucifer
        i see, tests are importing it from the wrong place
      • 2022-05-25 14508, 2022

      • lucifer
        it seems tests import it from api.py which had that import (though unused there) so it coincidentally worked, actually that method is in api_tools.py
      • 2022-05-25 14516, 2022

      • alastairp
        yeah
      • 2022-05-25 14519, 2022

      • alastairp
        I'm just updating it now
      • 2022-05-25 14544, 2022

      • alastairp
        do you only see it in color_api and feedback_api?
      • 2022-05-25 14555, 2022

      • lucifer
        let me check
      • 2022-05-25 14519, 2022

      • alastairp
        that's the only place I saw it, I committed and pushed. let's see if tests work
      • 2022-05-25 14554, 2022

      • lucifer
        yes i see it only in those two files
      • 2022-05-25 14545, 2022

      • alastairp
        github runners are taking a bit of time to become available
      • 2022-05-25 14539, 2022

      • Lotheric has quit
      • 2022-05-25 14525, 2022

      • bitmap
        akshaaatt: did you determine what is conflicting exactly? bootstrap has noConflict methods which may be of use to you
      • 2022-05-25 14517, 2022

      • Lotheric joined the channel
      • 2022-05-25 14511, 2022

      • alastairp
        lucifer: passed
      • 2022-05-25 14531, 2022

      • Lotheric has quit
      • 2022-05-25 14553, 2022

      • lucifer
        alastairp: great, lets merge
      • 2022-05-25 14559, 2022

      • Lotheric joined the channel
      • 2022-05-25 14511, 2022

      • BrainzGit
        [listenbrainz-server] 14alastair merged pull request #2009 (03master…max-number-listens): Allow a maximim number of listens per import https://github.com/metabrainz/listenbrainz-server…
      • 2022-05-25 14527, 2022

      • alastairp
        should we revert our if statement in the writer too?
      • 2022-05-25 14535, 2022

      • alastairp
        or is that in a branch?
      • 2022-05-25 14504, 2022

      • lucifer
        in a separate branch
      • 2022-05-25 14516, 2022

      • alastairp
        cool, I'll make a release then?
      • 2022-05-25 14523, 2022

      • lucifer
        yes please
      • 2022-05-25 14527, 2022

      • BrainzGit
        [listenbrainz-server] release 03v-2022-05-25.0 has been published by 14github-actions[bot]: https://github.com/metabrainz/listenbrainz-server…
      • 2022-05-25 14515, 2022

      • alastairp
        maximim -> maximum
      • 2022-05-25 14525, 2022

      • alastairp
        CB released with new mbdata, no more exceptions on event pages
      • 2022-05-25 14536, 2022

      • lucifer
        oh lol. but docs seem to be correct so fine.
      • 2022-05-25 14540, 2022

      • lucifer
        nice :D
      • 2022-05-25 14510, 2022

      • alastairp
        merging ansh's new display features for events/places and labels/releases
      • 2022-05-25 14531, 2022

      • Guest7859 has quit
      • 2022-05-25 14531, 2022

      • Guest7859 joined the channel
      • 2022-05-25 14549, 2022

      • Guest7859 is now known as rdswift
      • 2022-05-25 14507, 2022

      • atj
      • 2022-05-25 14531, 2022

      • zas
        atj: I will :)
      • 2022-05-25 14549, 2022

      • reosarevok
      • 2022-05-25 14521, 2022

      • alastairp
      • 2022-05-25 14524, 2022

      • alastairp
        nice work ansh!
      • 2022-05-25 14537, 2022

      • alastairp
        lucifer: should I go ahead and release?
      • 2022-05-25 14532, 2022

      • alastairp
        you did it!
      • 2022-05-25 14534, 2022

      • alastairp
        thanks :)
      • 2022-05-25 14507, 2022

      • lucifer
        alastairp: yes, sounds good
      • 2022-05-25 14530, 2022

      • MRiddickW has quit
      • 2022-05-25 14537, 2022

      • alastairp
        lucifer: what sounds good?
      • 2022-05-25 14544, 2022

      • lucifer
        to do a release
      • 2022-05-25 14510, 2022

      • alastairp
        oh, which site are we talking about? :)
      • 2022-05-25 14524, 2022

      • lucifer
        from the context, i thought CB
      • 2022-05-25 14535, 2022

      • lucifer
        also we have already done LB.
      • 2022-05-25 14536, 2022

      • alastairp
        yes, now after reading again that's what I realised
      • 2022-05-25 14553, 2022

      • alastairp
        yeah, right. I wasn't sure if you restarted LB but then I just did
      • 2022-05-25 14558, 2022

      • alastairp
        uh
      • 2022-05-25 14559, 2022

      • lucifer
        i have updated prod containers if that's what you meant
      • 2022-05-25 14500, 2022

      • alastairp
        so I asked
      • 2022-05-25 14507, 2022

      • lucifer
        ah makes sense
      • 2022-05-25 14509, 2022

      • alastairp
        and then I looked at docker ps and saw you had.
      • 2022-05-25 14517, 2022

      • alastairp
        and then the CB screenshot was somewhat unrelated
      • 2022-05-25 14525, 2022

      • alastairp
        so my fault for saying too many things, sorry
      • 2022-05-25 14530, 2022

      • lucifer
        no worries :)
      • 2022-05-25 14548, 2022

      • lucifer
        i'll also update beta/test so that the same issue doesn't happen there either
      • 2022-05-25 14559, 2022

      • chinmay
        LB server question:
      • 2022-05-25 14559, 2022

      • chinmay
        I know I have brought this up before, idk if I'm missing something during the setup (I've followed the docs line by line), but I can't see any listens on my LB server.
      • 2022-05-25 14559, 2022

      • chinmay
        It was set up about 2 weeks ago and I've listened to tens of songs since. It shows Now Playing but doesn't show any listens.
      • 2022-05-25 14559, 2022

      • chinmay
        What could I be doing wrong?
      • 2022-05-25 14518, 2022

      • chinmay
      • 2022-05-25 14543, 2022

      • lucifer
        how are you submitting listens locally? using spotify reader?
      • 2022-05-25 14553, 2022

      • chinmay
        Spotify reader
      • 2022-05-25 14500, 2022

      • chinmay
        And lastfm
      • 2022-05-25 14505, 2022

      • chinmay
        to import
      • 2022-05-25 14515, 2022

      • lucifer
        what is set as the api url in your config.py?
      • 2022-05-25 14548, 2022

      • chinmay
        localhost:8100
      • 2022-05-25 14528, 2022

      • chinmay
      • 2022-05-25 14515, 2022

      • lucifer
        i see, can you try running `./develop.sh manage recalculate_all_user_data`?
      • 2022-05-25 14549, 2022

      • chinmay
        on it
      • 2022-05-25 14536, 2022

      • chinmay
        it worked! 🥹
      • 2022-05-25 14559, 2022

      • lucifer
        awesome. its a regression in LB server, i'll look into fixing it.
      • 2022-05-25 14509, 2022

      • chinmay
        was getting frustrating to test stuff out
      • 2022-05-25 14521, 2022

      • chinmay
        oh alright
      • 2022-05-25 14540, 2022

      • skelly37 has quit
      • 2022-05-25 14504, 2022

      • KevlarNoir has quit
      • 2022-05-25 14557, 2022

      • skelly37 joined the channel
      • 2022-05-25 14516, 2022

      • skelly37 has quit
      • 2022-05-25 14503, 2022

      • qookie joined the channel
      • 2022-05-25 14525, 2022

      • Lotheric_ joined the channel
      • 2022-05-25 14512, 2022

      • Lotheric has quit
      • 2022-05-25 14518, 2022

      • skelly37 joined the channel
      • 2022-05-25 14537, 2022

      • skelly37 has quit
      • 2022-05-25 14502, 2022

      • skelly37 joined the channel
      • 2022-05-25 14506, 2022

      • KevlarNoir joined the channel
      • 2022-05-25 14508, 2022

      • reosarevok
        bitmap, yvanzo: should we postpone the next release by a week? (I mean, other than hotfixing the issue bitmap is fixing)
      • 2022-05-25 14526, 2022

      • reosarevok
        Since I was off and yvanzo was/is off, I can't really see much to release anyway
      • 2022-05-25 14535, 2022

      • bitmap
        yes, I think that makes sense
      • 2022-05-25 14539, 2022

      • fatihemreyildiz joined the channel
      • 2022-05-25 14510, 2022

      • bitmap
        should I hotfix that now and initiate a new dump then?
      • 2022-05-25 14537, 2022

      • reosarevok
        Either that or just hotfix and let the dump happen on Saturday