#metabrainz

/

      • Freso
        (People still up: atj, skelly37, yuzie – anyone else who wish to give review, let me know ASAP!)
      • yvanzo
        Nearly completed diagrams, continued writing docs, looked into MBS dependencies.
      • Also reviewed a few PRs, and made a bit of maintenance of Jira/Tickets.
      • Fin. Go atj?
      • Or skelly37?
      • Or yuzie?
      • skelly37
        hello
      • Freso
        atj, skelly37, yuzie: Any of you around? :D
      • Ayy.
      • yvanzo
        Go skelly! :)
      • skelly37
        Last week I've been improving the single instance mode for Picard. Very annoying bug got removed, URLs support is almost ready. Passing commands to the existing instance was discussed
      • quite a productive week
      • Also I was trying to move imp -> importlib but got stuck for now
      • When it comes to personal life: well,finally got my first job (with some help of the gsoc experience, thanks y'all!) :D
      • fin, go atj
      • alastairp
        \o/
      • skelly37
        or yuzie
      • yuzie
        Hi!
      • Last week I add backend API for resetting user timezone and also modified the corresponding frontend UI a bit.
      • But I’m stuck with problem related to CORS policy.
      • That’s for me.
      • Freso
        I guess atj is not around, so I’ll wrap up after yuzie. If atj or anyone else are around and want to give review, let me know now!
      • Well, let’s wrap up then.
      • Thank you all for your reviews, and thank you all all for your time!
      • Stay hydrated, masked, and take care of yourselves!
      • </BANG>
      • lucifer
        yuzie: the endpoint seems to work fine for me. https://usercontent.irccloud-cdn.com/file/60C74...
      • can you share what you see in network and console logs?
      • yvanzo
        Thanks
      • yuzie
        Sure. Please wait a minute.
      • alastairp
        thanks everyone
      • skelly37
        Just a reminder for all the GSoC folks that midterm evaluation should be open in 15 minutes :)
      • Pratha-Fish 👀
      • alastairp
        Pratha-Fish: hi, do you want to get LB running?
      • Pratha-Fish
        alastairp: Sure, if you don't mind
      • I hope it isn't too late for you tho
      • alastairp
        nah, it's fine
      • Pratha-Fish
        Alright, then let's get on to it :)
      • alastairp
        OK, so the idea is to clone the LB repo, configure it so that you don't run over anyone else's version on the same server, and then connect it to the main MB mirror database so that you can run the necessary apps
      • lucifer: still around? got a moment
      • lucifer
        yup
      • alastairp
      • I looked into this, it was the same thing we ended up with a few weeks back - the service was trying to restart but failed to bind() to the port, causing an error and a restart
      • there are a _lot_ of error messages for it. I stopped and restarted the container
      • I can't remember what container we had this issue on last time. but it seems like our restarter script isn't sleeping between executes
      • Pratha-Fish: ok, go ahead and clone LB into wolf (I'm following along to make sure we make the correct changes)
      • lucifer
        i think it was the labs api container last time as wel
      • Pratha-Fish
        cloning
      • yuzie
      • ansh
        alastairp: Are there any changes required in CB#441?
      • BrainzBot
        CB-437: Add entity metadata to review get endpoints: https://github.com/metabrainz/critiquebrainz/pu...
      • lucifer
        yuzie: you need to API_URL in your config.py.
      • *need to set API_URL to localhost:8100 in your config.py
      • Pratha-Fish
        alastairp: done cloning
      • yuzie
        OH, I see. I set it to API_URL = 'https://api.listenbrainz.org'
      • alastairp
        ansh: I don't think so. I just wanted to take another look at the database query to understand what information the endpoint is returning
      • Pratha-Fish: great. first change - edit devlop.sh, line 13 in function invoke_docker_compose, change the argument after -p to `listenbrainz_prathfish` or something similar
      • doesn't matter what it is, just something different to the normal. your login name, or your name, or something
      • lucifer
        yuzie: setting it to api.listenbrainz.org will send the api request to LB production servers where this endpoint doesn't exist thus giving you a 404 error
      • when you change it to localhost:8100, it will go to your local container.
      • Pratha-Fish
        alastairp: done
      • reosarevok
        bitmap, yvanzo: not sure there's enough ready to merge for a beta yet?
      • ansh
        alastairp: Okay :) If there are any changes, can I make them tomorrow?
      • yvanzo
        Probably not
      • alastairp
        ansh: yes, of course. sorry for the delay
      • ansh
        Thanks :D
      • alastairp
        Pratha-Fish: cool, now you can run `./devlop.sh build` to build your listenbrainz containers
      • Pratha-Fish: these instructions are at https://listenbrainz.readthedocs.io/en/latest/d... btw
      • Pratha-Fish
      • oopsie
      • alastairp
        hmm
      • did we ever set you up with docker access? perhaps not, as you were just using jupyter
      • one sec
      • Pratha-Fish
        yep
      • alastairp
        OK, I added you to the group
      • are you doing this through a terminal in vscode, or separate ssh?
      • Pratha-Fish
        vscode terminal
      • alastairp
        ok, we need to convince vscode to restart itself and log in again
      • Pratha-Fish
        alrighty
      • alastairp
        is there a command (control-shift-p) called "Remote-SSH: kill VS Code Server on Host" ?
      • Pratha-Fish
        alastairp: yes, there is :D
      • alastairp
        ok, let's try that
      • Pratha-Fish
        Too late, VScode's restarting :D
      • alastairp
        the problem is that restarting vscode locally won't fix it
      • because when it reconnects, it'll connect to an existing long-running process on the server
      • we need to shut down and restart those ones
      • lucifer: hi, it doesn't look like we have a local dev setup for labs api, right?
      • lucifer
        alastairp: yup don't have it.
      • alastairp
        ah, but there _is_ a `docker/docker-compose-labs-api.yml`
      • did I make that?
      • yes, looks like I did. it's a local only file :)
      • lucifer
        ah no, i see it here as well
      • alastairp
        huh
      • lucifer
        its docker/docker-compose.labs.api.yml here
      • . vs -
      • alastairp
        yep
      • good, lots of fixes for Pratha-Fish to put in his PR ;)
      • lucifer
        however, its local as in you need a mb db and typesense server also available.
      • alastairp
        yeah, but the typesense server on wolf is connected to the musicbrainz network
      • so that should be perfect
      • lucifer
        yup makes sense
      • alastairp
        Pratha-Fish: any luck? I see a few new vs code services started (though I also see some old ones running)
      • Pratha-Fish
        I've restarted it for the second time, apparently it's still not working
      • The kill SSH command didnt do much either TBH
      • alastairp
        docker commands still fail?
      • yeah. I remember looking into this when ansh tried it. it seems like there's no great solution
      • Pratha-Fish: anyway, perhaps you could just ssh in with a separate terminal?
      • Pratha-Fish
        sure
      • alastairp
        I'll kill all of your servers on wolf meanwhile
      • Pratha-Fish
        alastairp: should I port forward while connecting? (just in case)
      • alastairp
        they should all be shut down and restarted, maybe your terminal there works too now
      • not yet
      • we need to decide what to do about ports
      • Pratha-Fish
        hm
      • alastairp: Alright, I reconnected on vscode, and ran the command there. It worked :D
      • alastairp
        great. has it finished?
      • Pratha-Fish
        yep
      • alastairp
        great
      • so actually, this part of LB is still a bit developer-unfriendly
      • do you know about docker and docker-compose?
      • Pratha-Fish
        Not really
      • alastairp
        ok
      • we have this helper script, develop.sh. in the background it uses `docker-compose`
      • if you look at the file `docker/docker-compose.yml`, you'll see that it defines a bunch of services
      • these are all of the programs that LB needs to run. the flask webserver, a database server, redis, some other stuff
      • Pratha-Fish
        I see it in the docker folder
      • alastairp
        and when you run `./develop.sh up` it starts all of these services and makes them available on a port
      • however, the mapper, which we're looking at is separate from this infrastructure at the moment
      • take a look, we have a config file `docker/docker-compose.labs.api.yml`
      • Pratha-Fish
        I see it
      • alastairp
        this says "run a service called labs_api", the `build` section tells us how to set up the code, the `command` tells us what to run to start it, and `ports` tells us which ports it should be available on in the server
      • the `networks` section tells us that we want to be able to connect to the musicbrainz database which is running on wolf
      • bitmap
        reosarevok: not atm but I'll try to review stuff tonight so we have more to look at tomorrow
      • Pratha-Fish
        I see
      • reosarevok
        Okay, sounds good
      • alastairp
        so, we can do 2 things: either run all of LB, or just run the mapper
      • as we just need to fix the mapper, let's do that instead
      • we have another little helper script which allows us to run the mapper, it's `./docker/start-labs-api.sh`
      • which works like the develop.sh script
      • skelly37 has quit
      • so let's edit this file, and again change the argument to `-p` to the same that you put in develop.sh
      • Pratha-Fish
        Done
      • alastairp
        (this is called the docker-compose "project", it's a prefix that docker-compose uses to keep track of its processes. normally it's OK to leave it on "listenbrainz" but because we're on a shared server, it'll cause conflicts if multiple people run it at the same time)
      • ok, let's try and run it
      • Pratha-Fish
        What does the -p represent here though? I am guessing "profile"
      • alastairp
        ./docker/start-labs-api.sh build
      • ^ project :)
      • Pratha-Fish
        ah
      • skelly37 joined the channel
      • Looks like the ```./docker/start-labs-api.sh build``` command is throwing an error