#metabrainz

/

      • supersandro2000 has quit
      • 2020-09-09 25357, 2020

      • supersandro2000 joined the channel
      • 2020-09-09 25359, 2020

      • supersandro2000 has quit
      • 2020-09-09 25315, 2020

      • supersandro2000 joined the channel
      • 2020-09-09 25330, 2020

      • SomalRudra
        Congrtulations people 🎊
      • 2020-09-09 25321, 2020

      • sumedh joined the channel
      • 2020-09-09 25348, 2020

      • nelgin
        yvanzo how can I tell where the live index is in the replicated files? It's not stoppped for a couple of days.
      • 2020-09-09 25311, 2020

      • nelgin
        and it seems to be consuming a lot of disk space.
      • 2020-09-09 25353, 2020

      • sumedh has quit
      • 2020-09-09 25320, 2020

      • sumedh joined the channel
      • 2020-09-09 25346, 2020

      • diru1100
        Morning !! 🔥🔥
      • 2020-09-09 25357, 2020

      • shivam-kapila
        morning
      • 2020-09-09 25357, 2020

      • reosarevok
        I'm confused
      • 2020-09-09 25308, 2020

      • reosarevok
        https://musicbrainz.org/artist/a5c6e470-0808-474d… seems like the homepage icon *and class* have disappeared
      • 2020-09-09 25322, 2020

      • reosarevok
        That should have a "home-favicon" class...
      • 2020-09-09 25357, 2020

      • reosarevok
        Will check later
      • 2020-09-09 25339, 2020

      • MusicbrainzB0T joined the channel
      • 2020-09-09 25324, 2020

      • MusicbrainzB0T2 joined the channel
      • 2020-09-09 25325, 2020

      • MusicbrainzB0T3 joined the channel
      • 2020-09-09 25355, 2020

      • MusicbrainzB0T1 has quit
      • 2020-09-09 25326, 2020

      • MusicbrainzB0T has quit
      • 2020-09-09 25342, 2020

      • MusicbrainzB0T3 has quit
      • 2020-09-09 25311, 2020

      • gr0uch0mars joined the channel
      • 2020-09-09 25317, 2020

      • nelgin
        yvanzo ?
      • 2020-09-09 25301, 2020

      • v6lur joined the channel
      • 2020-09-09 25342, 2020

      • yvanzo
        hi nelgin, you can check the queue size in rabbitmq to monitor live indexing
      • 2020-09-09 25353, 2020

      • BrainzGit
        [listenbrainz-server] vansika opened pull request #1080 (master…distinct-similar-top-artist): [LB-703] Similar and top artist should be distinct for a user https://github.com/metabrainz/listenbrainz-server…
      • 2020-09-09 25354, 2020

      • BrainzBot
        LB-703: Top artists and similar artist should be distinct https://tickets.metabrainz.org/browse/LB-703
      • 2020-09-09 25338, 2020

      • yvanzo
        either publish the port 15672 (if your machine has a decent firewall) and use your browser, or use rabbitmqadmin command line.
      • 2020-09-09 25342, 2020

      • reosarevok
        yvanzo: looking into MBS-11085 and hoping we don't need a second hotfix :p
      • 2020-09-09 25343, 2020

      • BrainzBot
        MBS-11085: Cannot edit 8cm CD release with disc ID https://tickets.metabrainz.org/browse/MBS-11085
      • 2020-09-09 25323, 2020

      • yvanzo
        nelgin: sudo docker-compose exec mq rabbitmqadmin -u sir -p sir -V /search-index-rebuilder list queues
      • 2020-09-09 25324, 2020

      • reosarevok
        yvanzo: we probably do need a second hotfix :/
      • 2020-09-09 25301, 2020

      • yvanzo
        shhh, we saved a lot of bandwidth, that's the most important :D
      • 2020-09-09 25354, 2020

      • BrainzGit
        [musicbrainz-server] reosarevok opened pull request #1687 (master…MBS-11085): MBS-11085: Fix canHaveDiscID check https://github.com/metabrainz/musicbrainz-server/…
      • 2020-09-09 25356, 2020

      • BrainzBot
        MBS-11085: Cannot edit 8cm CD release with disc ID https://tickets.metabrainz.org/browse/MBS-11085
      • 2020-09-09 25316, 2020

      • reosarevok
        True! :D
      • 2020-09-09 25324, 2020

      • reosarevok
        yvanzo, bitmap ^
      • 2020-09-09 25348, 2020

      • reosarevok
        Tested locally, works fine, so I wouldn't even wait for bitmap if he's not around. yvanzo, if it works for you let's hotfix? :)
      • 2020-09-09 25349, 2020

      • reosarevok checks other recent reports in case we broke some other thing
      • 2020-09-09 25356, 2020

      • nelgin
        yvanzo
      • 2020-09-09 25356, 2020

      • nelgin
        | search.delete | 0 |
      • 2020-09-09 25356, 2020

      • nelgin
        | search.failed | 152 |
      • 2020-09-09 25356, 2020

      • nelgin
        | search.index | 230987 |
      • 2020-09-09 25356, 2020

      • nelgin
        | search.retry | 0 |
      • 2020-09-09 25357, 2020

      • nelgin
        +---------------+----------+
      • 2020-09-09 25333, 2020

      • nelgin
        Getting tons of scroll for the live index log, but none of the numbers seem to be moving.
      • 2020-09-09 25334, 2020

      • reosarevok
        yvanzo: added another (much more minor, but still) regression fix to that PR
      • 2020-09-09 25343, 2020

      • nelgin
        | search.index | 230984 |
      • 2020-09-09 25348, 2020

      • nelgin
        ok, seems to be going down, slowly.
      • 2020-09-09 25309, 2020

      • yvanzo
        how often do you replicate the database?
      • 2020-09-09 25326, 2020

      • yvanzo
        (the default is one time per day)
      • 2020-09-09 25357, 2020

      • yvanzo
        reosarevok: reviewing it
      • 2020-09-09 25301, 2020

      • reosarevok
        Thanks
      • 2020-09-09 25323, 2020

      • ruaok
        moooooin!
      • 2020-09-09 25320, 2020

      • reosarevok
        yvanzo: if it seems fine I can deal with the releasing, hopefully won't get stuck with anything :D
      • 2020-09-09 25331, 2020

      • Mr_Monkey
        Moin moin
      • 2020-09-09 25302, 2020

      • Gazooo794 has quit
      • 2020-09-09 25308, 2020

      • Gazooo794 joined the channel
      • 2020-09-09 25358, 2020

      • gr0uch0mars has quit
      • 2020-09-09 25314, 2020

      • alastairp
      • 2020-09-09 25336, 2020

      • ruaok
        interesting. thanks for sharing.
      • 2020-09-09 25355, 2020

      • alastairp
        sounds like it was a combination of 1) estimates for 4 months while everyone was living at home, 2) everyone using more than 6m^3 of water, after which prices triple, and 3) this new tax for recycling, which they decided to tie to water consumption
      • 2020-09-09 25322, 2020

      • sumedh has quit
      • 2020-09-09 25304, 2020

      • yvanzo
        reosarevok: cherry-pick'n'deploy but don't tag it yet, I feel like more patches may go through today ^^
      • 2020-09-09 25347, 2020

      • sumedh joined the channel
      • 2020-09-09 25311, 2020

      • reosarevok
        yvanzo: so, cherrypick onto prod, merge prod into beta, merge beta into master?
      • 2020-09-09 25338, 2020

      • reosarevok
        Or well, I guess I can merge into master first then cherrypick, but do you have any preference re: cherrypicking twice vs merging prod into beta :)
      • 2020-09-09 25350, 2020

      • yvanzo
        reosarevok: merge to master, merge to beta, merge to prod, merge to master
      • 2020-09-09 25308, 2020

      • yvanzo
        actually there is no other commit on master, so merge is fine
      • 2020-09-09 25340, 2020

      • BrainzGit
        [musicbrainz-server] reosarevok merged pull request #1687 (master…MBS-11085): MBS-11085 / MBS-11089: Fix canHaveDiscID check / Fix missing favicons https://github.com/metabrainz/musicbrainz-server/…
      • 2020-09-09 25342, 2020

      • BrainzBot
        MBS-11085: Cannot edit 8cm CD release with disc ID https://tickets.metabrainz.org/browse/MBS-11085
      • 2020-09-09 25342, 2020

      • BrainzBot
        MBS-11089: Homepage and blog favicons gone from sidebar https://tickets.metabrainz.org/browse/MBS-11089
      • 2020-09-09 25309, 2020

      • yvanzo
        cherry-picking a merge commit is not possible (or you need to specify parent), whereas merge is fine her.
      • 2020-09-09 25351, 2020

      • yvanzo
        and maybe merging back to master is not needed until we tag production branch.
      • 2020-09-09 25317, 2020

      • reosarevok
        Ok
      • 2020-09-09 25336, 2020

      • reosarevok
        So, merge master into beta, release beta, merge beta into prod, release prod, stop for now
      • 2020-09-09 25346, 2020

      • yvanzo
        +1
      • 2020-09-09 25342, 2020

      • nelgin
        yvanzo corryect, once per day.
      • 2020-09-09 25300, 2020

      • reosarevok
        yvanzo, bitmap: also, should we be updating jenkins?
      • 2020-09-09 25305, 2020

      • nelgin
        | search.index | 229996 |
      • 2020-09-09 25308, 2020

      • nelgin
        It's slowly going down.
      • 2020-09-09 25315, 2020

      • nelgin
        Took about an hour and 10 minutes to do 991.
      • 2020-09-09 25359, 2020

      • reosarevok
        Ouch
      • 2020-09-09 25302, 2020

      • BestSteve has quit
      • 2020-09-09 25352, 2020

      • BestSteve joined the channel
      • 2020-09-09 25335, 2020

      • reosarevok
        releasing beta
      • 2020-09-09 25301, 2020

      • zas
        yvanzo, reosarevok : what broke user scripts? -> https://community.metabrainz.org/t/tampermonkey/4…
      • 2020-09-09 25327, 2020

      • reosarevok
        zas: if they used lodash, then that
      • 2020-09-09 25339, 2020

      • reosarevok
        That's what happened to loujine's ones, anyway
      • 2020-09-09 25308, 2020

      • pristine___
        ishaanshah: you about?
      • 2020-09-09 25338, 2020

      • ishaanshah
        pristine___: Hi
      • 2020-09-09 25329, 2020

      • pristine___
      • 2020-09-09 25312, 2020

      • pristine___
        Is there a reason to have 10 as min??
      • 2020-09-09 25327, 2020

      • reosarevok
        releasing prod
      • 2020-09-09 25351, 2020

      • ishaanshah
        I dont think so, but we are going to change the cron tab soon
      • 2020-09-09 25306, 2020

      • ishaanshah
      • 2020-09-09 25311, 2020

      • ishaanshah
        Refer to this one
      • 2020-09-09 25332, 2020

      • sumedh has quit
      • 2020-09-09 25319, 2020

      • sumedh joined the channel
      • 2020-09-09 25327, 2020

      • BrainzGit
        [listenbrainz-server] vansika opened pull request #1081 (master…rec-crontab): Cronjob for recommendation scripts. https://github.com/metabrainz/listenbrainz-server…
      • 2020-09-09 25317, 2020

      • ishaanshah
        iliekcomputers: ping
      • 2020-09-09 25336, 2020

      • ruaok
        pristine___: your crontab is not future proof.
      • 2020-09-09 25355, 2020

      • iliekcomputers
        let's not create a new crontab for this
      • 2020-09-09 25357, 2020

      • ruaok
        those 4 steps, should they always be executed sequentially?
      • 2020-09-09 25321, 2020

      • pristine___
        ruaok: meaning? Yup.
      • 2020-09-09 25341, 2020

      • ruaok
        if so, it would be better to add one script that executes the four commands that is invoked by cron.
      • 2020-09-09 25359, 2020

      • ruaok
        otherwise if one step runs longer, then it may not finish before the next step kicks off.
      • 2020-09-09 25332, 2020

      • iliekcomputers
        because this is just sending over requests to the consumer in a queue, it should be ok
      • 2020-09-09 25335, 2020

      • ruaok
        and then what iliekcomputers said: no need for a separate crontab -- just add the invocation of the new script to the existing crontab
      • 2020-09-09 25310, 2020

      • ruaok
        iliekcomputers: agreed, but it makes me wig out. it feels very wrong.
      • 2020-09-09 25315, 2020

      • reosarevok
        yvanzo, bitmap: hotfixes out, all done AFAICT
      • 2020-09-09 25343, 2020

      • iliekcomputers
        ishaanshah: pong
      • 2020-09-09 25359, 2020

      • ishaanshah
        iliekcomputers: Hi
      • 2020-09-09 25312, 2020

      • ishaanshah
        I was working on showing listen count on artist map
      • 2020-09-09 25347, 2020

      • iliekcomputers
        ooh awesome!
      • 2020-09-09 25355, 2020

      • ishaanshah
        So, basically it's just adding another listen count field
      • 2020-09-09 25301, 2020

      • iliekcomputers
        right
      • 2020-09-09 25307, 2020

      • ishaanshah
        However we cache our result right
      • 2020-09-09 25323, 2020

      • iliekcomputers
        mhmm
      • 2020-09-09 25327, 2020

      • ishaanshah
        so it might take time for users' data to update
      • 2020-09-09 25338, 2020

      • ishaanshah
        which might lead to bugs on frontend
      • 2020-09-09 25303, 2020

      • ishaanshah
        One possible way is to add a dirty bit style column to DB which is set to 1 for all users
      • 2020-09-09 25322, 2020

      • ishaanshah
        and changes to 0 as data is recalculated
      • 2020-09-09 25343, 2020

      • ishaanshah
        or, wait for a week after deploying and then deploy the frontend
      • 2020-09-09 25303, 2020

      • iliekcomputers
        i'm not sure i understand
      • 2020-09-09 25316, 2020

      • iliekcomputers
        how are you calculating the listen count?
      • 2020-09-09 25319, 2020

      • yvanzo
        reosarevok: thanks!
      • 2020-09-09 25333, 2020

      • iliekcomputers
        from the top artists?
      • 2020-09-09 25337, 2020

      • ishaanshah
        yep
      • 2020-09-09 25303, 2020

      • ishaanshah
        add listen count for individual artists
      • 2020-09-09 25346, 2020

      • iliekcomputers
        right.
      • 2020-09-09 25305, 2020

      • iliekcomputers
        i'm ok with it taking time for the user's data to update tbh
      • 2020-09-09 25335, 2020

      • iliekcomputers
        it'll be done by next week right? i don't think we need to do extra engineering work to update everything immediately
      • 2020-09-09 25353, 2020

      • iliekcomputers
        just make the frontend backwards compatible, so that it works without listen counts as well
      • 2020-09-09 25320, 2020

      • ishaanshah
        yep, it will get updated in a week from when it is deployed
      • 2020-09-09 25346, 2020

      • ishaanshah
        cool, cool will do
      • 2020-09-09 25354, 2020

      • iliekcomputers
        awesome, thanks!
      • 2020-09-09 25303, 2020

      • iliekcomputers
        the full dump didn't fail this time
      • 2020-09-09 25311, 2020

      • ishaanshah
        yep, stats updated
      • 2020-09-09 25325, 2020

      • iliekcomputers
        i'm gonna try to merge and test your incremental dumps PR today
      • 2020-09-09 25317, 2020

      • ishaanshah
        Cool cool sounds good, I have some free time today, I will try to get the frontend done
      • 2020-09-09 25323, 2020

      • iliekcomputers
        nice
      • 2020-09-09 25346, 2020

      • iliekcomputers
        ping me if you need to chat about things we could work on in the future
      • 2020-09-09 25300, 2020

      • iliekcomputers
        ishaanshah: i was thinking
      • 2020-09-09 25319, 2020

      • iliekcomputers
        while you're looking at this, is there any way we could show the names of the artists as well?
      • 2020-09-09 25325, 2020

      • ishaanshah
        Yep, we could do it, basically have a custom tooltip which shows the top 5 artists maybe
      • 2020-09-09 25349, 2020

      • ishaanshah
        I thought it would be better to do it in a different PR though
      • 2020-09-09 25356, 2020

      • iliekcomputers
        yeah, not in this one, but we should keep it in mind
      • 2020-09-09 25300, 2020

      • iliekcomputers
        i was thinking this