#metabrainz

/

      • ephemer0l has quit
      • 2021-04-13 10332, 2021

      • ephemer0l joined the channel
      • 2021-04-13 10321, 2021

      • flamingspinach has quit
      • 2021-04-13 10337, 2021

      • flamingspinach joined the channel
      • 2021-04-13 10332, 2021

      • mckean has quit
      • 2021-04-13 10334, 2021

      • mckean joined the channel
      • 2021-04-13 10314, 2021

      • RikkoM joined the channel
      • 2021-04-13 10315, 2021

      • RikkoM
        okno_: (and maybe others) I'm Ritiek and I've been somewhat involved with the musicbrainz_rs project lately. I've submitted an application on the GSoC forum to this project. I realize it's pretty late given the official application deadline is today, but better late than never! I'd be interested if anyone has any feedback to share on the application.
      • 2021-04-13 10311, 2021

      • yyoung
        Hi yvanzo, I haven't received your gist comments yet, but I've made some changes to the schedule according to our last discussion, please have a look at the proposal on the forum so we can finalize the proposal today, thanks!
      • 2021-04-13 10332, 2021

      • flamingspinach has quit
      • 2021-04-13 10352, 2021

      • flamingspinach joined the channel
      • 2021-04-13 10327, 2021

      • MRiddickW joined the channel
      • 2021-04-13 10321, 2021

      • BrainzGit
        [listenbrainz-server] amCap1712 opened pull request #1386 (master…deploy-image): Automatically build and push image to Docker Hub when new tag is pushed https://github.com/metabrainz/listenbrainz-server…
      • 2021-04-13 10321, 2021

      • BrainzGit
        [musicbrainz-server] reosarevok opened pull request #2058 (master…MBS-11581): MBS-11581: Don't try to display / offer darkened art as RG cover art https://github.com/metabrainz/musicbrainz-server/…
      • 2021-04-13 10312, 2021

      • reosarevok
        yvanzo: if you want to do some reviewing today, I'll be around :)
      • 2021-04-13 10325, 2021

      • navap-
        reosarevok: Well that was fast... it's like you knew the ticket was going to be created 😆
      • 2021-04-13 10301, 2021

      • reosarevok
        I hadn't started work on it before you created, in any case :) It's just hopefully a small patch, unless I missed something
      • 2021-04-13 10349, 2021

      • navap-
        It's been so long since I've looked at MBS code. The perl looks so foreign to me now :(
      • 2021-04-13 10321, 2021

      • sumedh joined the channel
      • 2021-04-13 10337, 2021

      • sumedh has quit
      • 2021-04-13 10346, 2021

      • yef has quit
      • 2021-04-13 10334, 2021

      • yef joined the channel
      • 2021-04-13 10335, 2021

      • yef has quit
      • 2021-04-13 10335, 2021

      • yef joined the channel
      • 2021-04-13 10300, 2021

      • endurance21
        hey! everyone
      • 2021-04-13 10356, 2021

      • endurance21
        I have a query,if i am not wrong bookbrainz dont directly store users/editors email adress it rather stores editors metabrainz id . how can one collect the email adress of user using this metabrainz id , if possible ?
      • 2021-04-13 10320, 2021

      • endurance21
        Mr_Monkey: :)
      • 2021-04-13 10307, 2021

      • CatQuest
        RikkoM: you should post to our https://community.metabrainz.org about your proposal too
      • 2021-04-13 10349, 2021

      • CatQuest
        hey navap 🙋
      • 2021-04-13 10330, 2021

      • sumedh joined the channel
      • 2021-04-13 10354, 2021

      • texke joined the channel
      • 2021-04-13 10301, 2021

      • RikkoM
        CatQuest: I did post it in the GSoC subsection. Is it not visible? Or is there another place I should post it too?
      • 2021-04-13 10302, 2021

      • Mr_Monkey
        endurance21: That's a good question, and I don't have a 100% sure answer for you. Most likely we will have to establish a connection to the MB database, and armed with the metabrainz_user_id column fetch the users we're interested in. The BB and MB databases are hosted in the same postgres instance so configuration shouldn't be too tricky.
      • 2021-04-13 10347, 2021

      • endurance21
        okay ! sounds promising !
      • 2021-04-13 10359, 2021

      • endurance21
        thanks :)
      • 2021-04-13 10312, 2021

      • BrainzGit
        [musicbrainz-server] reosarevok opened pull request #2059 (master…edit-preview-relationship-warning): Fix Catalyst warning: don't try using undef to access hash https://github.com/metabrainz/musicbrainz-server/…
      • 2021-04-13 10319, 2021

      • reosarevok
        bitmap, yvanzo: while I'm fixing warnings, https://github.com/metabrainz/musicbrainz-server/… fixes one and adds a filter for artist works lists that is extra useful now that relationships view is paged, can we get that one reviewed?
      • 2021-04-13 10356, 2021

      • sumedh has quit
      • 2021-04-13 10357, 2021

      • BrainzGit
        [musicbrainz-server] reosarevok opened pull request #2060 (master…no-parent-format-jsonld-warning): Fix Catalyst warning: ensure parent format id is not undef https://github.com/metabrainz/musicbrainz-server/…
      • 2021-04-13 10344, 2021

      • flamingspinach has quit
      • 2021-04-13 10302, 2021

      • flamingspinach joined the channel
      • 2021-04-13 10316, 2021

      • BrainzGit
        [musicbrainz-docker] alastair opened pull request #190 (master…search-readme): Improve documentation for setting up solr https://github.com/metabrainz/musicbrainz-docker/…
      • 2021-04-13 10343, 2021

      • BrainzGit
        [listenbrainz-server] mayhem merged pull request #1347 (master…dump-user-feedback): Dump user feedback https://github.com/metabrainz/listenbrainz-server…
      • 2021-04-13 10303, 2021

      • reosarevok
        Take that stupid feedback and dump it where nobody can find it!
      • 2021-04-13 10333, 2021

      • ruaok
        well, if FTP is removed and we dont add HTTP capabilities to our download, then we're set, reosarevok
      • 2021-04-13 10312, 2021

      • RikkoM has quit
      • 2021-04-13 10345, 2021

      • alastairp
        _lucifer: 3 minutes with cache sounds great! Is that just unit tests, or both unit and integration?
      • 2021-04-13 10322, 2021

      • _lucifer
        just unit tests.
      • 2021-04-13 10344, 2021

      • _lucifer
        but it should be straightforward to run both of them together i think.
      • 2021-04-13 10356, 2021

      • _lucifer
        let mw try that as well.
      • 2021-04-13 10331, 2021

      • alastairp
        that was kind of the branch that I pushed for you a while back
      • 2021-04-13 10343, 2021

      • BrainzGit
        [musicbrainz-server] reosarevok opened pull request #2061 (master…empty-review-markdown-warning): Fix Catalyst warning: don't pass undef review text to markdown https://github.com/metabrainz/musicbrainz-server/…
      • 2021-04-13 10332, 2021

      • MRiddickW has quit
      • 2021-04-13 10337, 2021

      • _lucifer
        yeah right. but i got caught up in other things after that so wasn't able to progress on it.
      • 2021-04-13 10316, 2021

      • ruaok
      • 2021-04-13 10331, 2021

      • ruaok
        sigh. I guess an all out war on the FSF will be coming.
      • 2021-04-13 10339, 2021

      • ruaok won't shed a tear
      • 2021-04-13 10329, 2021

      • ruaok
        alastairp: user dump feedback deployed and working as expected.
      • 2021-04-13 10340, 2021

      • alastairp
        \o/
      • 2021-04-13 10352, 2021

      • ruaok
        pristine___: user feedback dumps are now happening on a regular basis. will you have time to work on this spark side of things soon?
      • 2021-04-13 10316, 2021

      • alastairp
        _lucifer: yeah, no worries. we can always deploy it as-is with the 4 test processes for now
      • 2021-04-13 10324, 2021

      • alastairp
        did we merge or close the final jenkins fix?
      • 2021-04-13 10338, 2021

      • _lucifer
        neither, i thinks its open.
      • 2021-04-13 10341, 2021

      • alastairp
        I wouldn't mind still adding in the additional --filter to `docker images ls`
      • 2021-04-13 10343, 2021

      • alastairp
        then merge it
      • 2021-04-13 10347, 2021

      • alastairp
        so that we have it in the git history
      • 2021-04-13 10312, 2021

      • Mineo
        outsidecontext, zas: I've merged https://github.com/metabrainz/picard/pull/1790, but didn't do anything with the associated jira ticket. I don't know if what's currently in master will all make it into 2.7 or a later release. I'll leave that to you guys :-)
      • 2021-04-13 10349, 2021

      • _lucifer
        alastairp: updated Jenkins PR with filter on image name.
      • 2021-04-13 10323, 2021

      • reosarevok
        alastairp: do you know how I can choose where CB is built? I want to fix a couple bugs I found, but the docker images seem to go to /var by default and I don't have enough space in that partition
      • 2021-04-13 10343, 2021

      • alastairp
        reosarevok: yes, you can change your docker root
      • 2021-04-13 10319, 2021

      • reosarevok
        Oh, so not in CB conf but globally for docker?
      • 2021-04-13 10323, 2021

      • reosarevok
        That's probably sensible too
      • 2021-04-13 10335, 2021

      • alastairp
      • 2021-04-13 10313, 2021

      • reosarevok
        Ok, so create /etc/docker/daemon.json and put a "data-root" in there
      • 2021-04-13 10314, 2021

      • reosarevok
        Let's see
      • 2021-04-13 10336, 2021

      • _lucifer
        alastairp, should i merge both jenkins and github actions?
      • 2021-04-13 10308, 2021

      • alastairp
        go for it!
      • 2021-04-13 10312, 2021

      • alastairp
        one sec,
      • 2021-04-13 10317, 2021

      • alastairp
        let me look at the gh actions one again
      • 2021-04-13 10330, 2021

      • alastairp
        what did you do with the junit-xml output?
      • 2021-04-13 10355, 2021

      • _lucifer
        i haven't added that yet.
      • 2021-04-13 10318, 2021

      • _lucifer
        we can add that in the same PR or i can open a new one. what do you say?
      • 2021-04-13 10326, 2021

      • reosarevok
      • 2021-04-13 10349, 2021

      • reosarevok
        Getting that with ./develop.sh test up
      • 2021-04-13 10304, 2021

      • _lucifer
        there might be an old test db container running
      • 2021-04-13 10331, 2021

      • _lucifer
        which did not get cleaned up from previous test runs
      • 2021-04-13 10326, 2021

      • reosarevok
        Ok, pruned, trying again
      • 2021-04-13 10354, 2021

      • reosarevok
      • 2021-04-13 10357, 2021

      • outsidecontext
        Mineo: Thanks. It would go into the 2.6.1 release currently. It's probably fine, if we decide otherwise we can take it out
      • 2021-04-13 10358, 2021

      • reosarevok
        Ok...
      • 2021-04-13 10342, 2021

      • reosarevok
        I guess I wanted normal build, not test build anyway :)
      • 2021-04-13 10358, 2021

      • reosarevok
        I didn't realize that's not "debug/dev mode" but literally "run tests"
      • 2021-04-13 10303, 2021

      • _lucifer
        the db_test_1 logs are normal
      • 2021-04-13 10330, 2021

      • _lucifer
        just `./develop.sh up` for dev mode
      • 2021-04-13 10359, 2021

      • reosarevok
        Can I set this up to use my local MB DB copy?
      • 2021-04-13 10309, 2021

      • reosarevok
        Seems to be downloading its own dump atm...
      • 2021-04-13 10310, 2021

      • _lucifer
        yes.
      • 2021-04-13 10335, 2021

      • reosarevok kills it before it fills my drive with the dump :D
      • 2021-04-13 10345, 2021

      • prabal joined the channel
      • 2021-04-13 10354, 2021

      • _lucifer
        you'll need to add an external network in the docker-compose file and change the database uri accordingly
      • 2021-04-13 10301, 2021

      • _lucifer
        i have that somewhere let me find it
      • 2021-04-13 10320, 2021

      • alastairp
        _lucifer: I added some comments to the gh actions pr
      • 2021-04-13 10324, 2021

      • alastairp
        some things to check first
      • 2021-04-13 10353, 2021

      • reosarevok
      • 2021-04-13 10323, 2021

      • reosarevok
        That's what I do for MB when I want to access our live r-o DB via an ssh tunnel, which hopefully I can do here too :)
      • 2021-04-13 10317, 2021

      • yvanzo
        yyoung: posted comment to the gist
      • 2021-04-13 10347, 2021

      • _lucifer
      • 2021-04-13 10326, 2021

      • _lucifer
        reosarevok: something like the above patch should work
      • 2021-04-13 10340, 2021

      • prabal
        Mr_Monkey: I've updated my proposal, have a look at it whenever you're free :)
      • 2021-04-13 10329, 2021

      • yvanzo
        yyoung: The schedule looks a bit better but it should be more incremental, that is delivering a small feature that works every two weeks, instead of trying to create full-feature components independently and to connect them on week 9 and then realizing it doesn't work when it's too late.
      • 2021-04-13 10351, 2021

      • reosarevok
        _lucifer: ERROR: Network musicbrainzdocker_default declared as external, but could not be found. Please create the network manually using `docker network create musicbrainzdocker_default` and try again. - just do that?
      • 2021-04-13 10301, 2021

      • reosarevok is still bad at docker
      • 2021-04-13 10316, 2021

      • yvanzo
        okno_: There is a draft proposal you might want to review :) https://community.metabrainz.org/t/gsoc-2021-comp…
      • 2021-04-13 10335, 2021

      • _lucifer
        reosarevok: you should replace with `musicbrainzdocker_default` with the name of the network your musicbrainz db containers are running on
      • 2021-04-13 10356, 2021

      • reosarevok
        So, localhost? It's not a db container, it's local psql
      • 2021-04-13 10307, 2021

      • okno_
        @yvanzo I am on it :)
      • 2021-04-13 10357, 2021

      • reosarevok
        well, the one I access via ssh is technically a docker container, I guess, but I just access it via ssh tunnel and exposed port
      • 2021-04-13 10311, 2021

      • _lucifer
        no, i don't think you need to add this then. just changing the uri might be enough.
      • 2021-04-13 10330, 2021

      • reosarevok
        Let's see
      • 2021-04-13 10325, 2021

      • _lucifer
        ok i looked and it seems to be a bit more involved. alastairp or yvanzo might be able to better help you here.
      • 2021-04-13 10348, 2021

      • alastairp
        what's going on?
      • 2021-04-13 10305, 2021

      • alastairp
        you're on linux? and you have postgres running natively, and want to access it from within docker?
      • 2021-04-13 10356, 2021

      • alastairp
        or pg is running somewhere else, but you tunnel it to a port on localhost?
      • 2021-04-13 10317, 2021

      • ruaok
        _lucifer: how does the setup of the new spark cluster?
      • 2021-04-13 10340, 2021

      • _lucifer
        ruaok, i was reading up some more about it over the previous. we can probably do that today.
      • 2021-04-13 10306, 2021

      • ruaok
        I'm around for the next 5.5 hours if you need help.
      • 2021-04-13 10332, 2021

      • _lucifer
        nice, thanks!
      • 2021-04-13 10333, 2021

      • reosarevok
        alastairp: yeah
      • 2021-04-13 10346, 2021

      • reosarevok
        I tried MB_DATABASE_URI = "postgresql://musicbrainz:musicbrainz@localhost:65401/musicbrainz_db" but that doesn't seem to do the trick
      • 2021-04-13 10313, 2021

      • alastairp
        no, that won't work. because localhost is local to the container, not your host
      • 2021-04-13 10313, 2021

      • reosarevok
        alastairp: both are true - I have a sample DB locally and a full db via localhost port tunnel
      • 2021-04-13 10318, 2021

      • reosarevok
        Oh, I see
      • 2021-04-13 10329, 2021

      • yvanzo
        yeah, which setup is it?
      • 2021-04-13 10339, 2021

      • reosarevok
        I was trying to configure it for the tunnel
      • 2021-04-13 10344, 2021

      • reosarevok
        For now
      • 2021-04-13 10346, 2021

      • alastairp
        two options: mount the postgres socket into the cb container, that'll give you the sample db
      • 2021-04-13 10313, 2021

      • alastairp
        or you need to work out the docker ip address of your host (172.something.something maybe) and use that in the config
      • 2021-04-13 10326, 2021

      • flamingspinach has quit
      • 2021-04-13 10327, 2021

      • ruaok
        alastairp: _lucifer ignore sentry errors on user similarity stuff. testing some fixes.
      • 2021-04-13 10333, 2021

      • yvanzo
        reosarevok: the sample DB is in your system or in mb docker compose project?
      • 2021-04-13 10348, 2021

      • reosarevok
        My system
      • 2021-04-13 10322, 2021

      • reosarevok
        And the other db is on a ssh tunnel from pink but the only change I make to access it via DBDefs is specifying the port
      • 2021-04-13 10326, 2021

      • reosarevok
        (and localhost)
      • 2021-04-13 10334, 2021

      • reosarevok
        I'll try and find the ip
      • 2021-04-13 10342, 2021

      • flamingspinach joined the channel
      • 2021-04-13 10343, 2021

      • yvanzo
        possibly 172.17.0.1
      • 2021-04-13 10309, 2021

      • reosarevok
        Hmm
      • 2021-04-13 10311, 2021

      • reosarevok
        "As of version 18.03, you can use host.docker.internal as the host's IP."
      • 2021-04-13 10333, 2021

      • _lucifer
        alastairp: https://github.com/metabrainz/listenbrainz-server… this is locally reproducible by running `docker-compose -f docker/docker-compose.integration.yml pull` in the repository root
      • 2021-04-13 10347, 2021

      • reosarevok
        Guess not