#metabrainz

/

      • jesus2099 has quit
      • 2017-06-21 17225, 2017

      • ruaok
        alastairp: > How can I test this myself? Is the rest of the system stable/bug-free enough that I can run it, or should I wait for you to fix conflicts and rebase/merge master?
      • 2017-06-21 17236, 2017

      • ruaok
        I think my merge has a merge conflict in it.
      • 2017-06-21 17255, 2017

      • alastairp
        I'm just considering all the changes that have been made in the last 2 weeks
      • 2017-06-21 17207, 2017

      • ruaok
        but you should be able to check out the branch, start with an empty influx DB, and then do a last.fm import.
      • 2017-06-21 17211, 2017

      • alastairp
        OK, cool
      • 2017-06-21 17214, 2017

      • ruaok
        yeah, its been a problem.
      • 2017-06-21 17224, 2017

      • alastairp docker-builds
      • 2017-06-21 17236, 2017

      • ruaok
        I'm vacillating between the pika-flask problem and the listen-counts issue.
      • 2017-06-21 17253, 2017

      • ruaok
        I will probably re-do the merge in a bit to see what went wrong
      • 2017-06-21 17224, 2017

      • henadel has quit
      • 2017-06-21 17217, 2017

      • alastairp
      • 2017-06-21 17230, 2017

      • alastairp
        I'm curious to know what was fixed in a new version of setuptools
      • 2017-06-21 17222, 2017

      • iliekcomputers
      • 2017-06-21 17213, 2017

      • iliekcomputers
        I added a link to the github issue and a failed build in the commit
      • 2017-06-21 17201, 2017

      • alastairp
        ahhhh, github doesn't show full commit messages in a nice way in pull requests
      • 2017-06-21 17206, 2017

      • alastairp
        nice one for using them!
      • 2017-06-21 17242, 2017

      • samj1912 joined the channel
      • 2017-06-21 17206, 2017

      • alastairp
        OK, I don't this to be taken in a mean way, but I just checked out, built, set up, started LB, did an import, and can now browse my listens
      • 2017-06-21 17209, 2017

      • alastairp
        awesome work guys
      • 2017-06-21 17220, 2017

      • alastairp
        I kind of didn't expect it to be that easy
      • 2017-06-21 17238, 2017

      • ruaok
        iliekcomputers has been relentlessly fixing shit I bring up.
      • 2017-06-21 17249, 2017

      • ruaok
        !m iliekcomputers
      • 2017-06-21 17249, 2017

      • BrainzBot
        You're doing good work, iliekcomputers!
      • 2017-06-21 17227, 2017

      • alastairp
      • 2017-06-21 17230, 2017

      • alastairp
      • 2017-06-21 17233, 2017

      • alastairp
        ya know
      • 2017-06-21 17235, 2017

      • alastairp
        that's good enough for me
      • 2017-06-21 17239, 2017

      • ruaok
        lol
      • 2017-06-21 17250, 2017

      • alastairp
        (completely serious here)
      • 2017-06-21 17204, 2017

      • ruaok
        :)
      • 2017-06-21 17220, 2017

      • alastairp
        OK, this is all up and running. I'm going to go home and continue looking at it from there
      • 2017-06-21 17232, 2017

      • ruaok
        great, thanks.
      • 2017-06-21 17239, 2017

      • alastairp
        hey, so
      • 2017-06-21 17252, 2017

      • alastairp
        UID=`id -u`
      • 2017-06-21 17252, 2017

      • alastairp
        GID=`id -g`
      • 2017-06-21 17252, 2017

      • alastairp
        docker-compose run --rm --user $UID:$GID web "$@"
      • 2017-06-21 17215, 2017

      • alastairp
        will stop docker creating all these files owned as root all over your home directory
      • 2017-06-21 17218, 2017

      • alastairp
        that's kind of nice
      • 2017-06-21 17229, 2017

      • reosarevok
        Thanks yvanzo_ :)
      • 2017-06-21 17231, 2017

      • alastairp
        and it's portable (over different linuxes... not sure if mac has `id`)
      • 2017-06-21 17220, 2017

      • ruaok
        alastairp: home dir? I'm confused.
      • 2017-06-21 17239, 2017

      • alastairp
        when you run docker on linux, and the process inside the container creates a file, it's owned by root
      • 2017-06-21 17256, 2017

      • alastairp
        (assuming you run your process in the container as root; which is the default)
      • 2017-06-21 17209, 2017

      • ruaok
        ah inside the container, ok.
      • 2017-06-21 17249, 2017

      • ruaok
        is that important though? I don't often care about the cleanliness of the container.
      • 2017-06-21 17234, 2017

      • alastairp
        not the container
      • 2017-06-21 17242, 2017

      • alastairp
        your local code checkout
      • 2017-06-21 17246, 2017

      • alastairp
        if you have . mapped as a volume
      • 2017-06-21 17211, 2017

      • alastairp
        e.g., if you have something inside the container which compiles less->css
      • 2017-06-21 17235, 2017

      • alastairp
        if you have a volume set up, the resulting css file will be in your code checkout, but owned by root
      • 2017-06-21 17203, 2017

      • alastairp
        anyway. I thought it was interesting
      • 2017-06-21 17210, 2017

      • Sophist-UK has quit
      • 2017-06-21 17235, 2017

      • Sophist-UK joined the channel
      • 2017-06-21 17232, 2017

      • Mineo joined the channel
      • 2017-06-21 17243, 2017

      • arbenina_ has quit
      • 2017-06-21 17218, 2017

      • Gentlecat
        Mineo: if you've seen my comment on a PR for sir, I'd appreciate if you could take a look
      • 2017-06-21 17232, 2017

      • Gentlecat
        since I'm a bit stuck on that part
      • 2017-06-21 17213, 2017

      • github joined the channel
      • 2017-06-21 17213, 2017

      • github
        [listenbrainz-server] mayhem opened pull request #203: Ditch Flask-pika for pika-pool (master...pika-pool) https://git.io/vQkEh
      • 2017-06-21 17213, 2017

      • github has left the channel
      • 2017-06-21 17224, 2017

      • ruaok
        iliekcomputers: ^^
      • 2017-06-21 17218, 2017

      • ruaok
        that works much better.
      • 2017-06-21 17200, 2017

      • ruaok
        iliekcomputers: good catch. I'll keep the clauses, but change error messages so we can tell what is going on in case of errror
      • 2017-06-21 17224, 2017

      • iliekcomputers
        👍🏽
      • 2017-06-21 17246, 2017

      • CaTTz has quit
      • 2017-06-21 17212, 2017

      • Gentlecat
        Mineo: also, if you could explain how paths were created and how/when they should be updated
      • 2017-06-21 17241, 2017

      • arbenina_ joined the channel
      • 2017-06-21 17209, 2017

      • Mineo
        Gentlecat, I'm looking at the PR at the moment
      • 2017-06-21 17228, 2017

      • Mineo
        the answer to your last question is one that I'm not sure you'll like ;-)
      • 2017-06-21 17249, 2017

      • Gentlecat
        I'm just collecting the facts :)
      • 2017-06-21 17215, 2017

      • Mineo
        you basically have to read through the *Index.java files in https://github.com/metabrainz/search-server/tree/… and figure out which SQL statements are used for which search field
      • 2017-06-21 17230, 2017

      • Mineo
        that's for all the paths where the data eventually ends up in a field
      • 2017-06-21 17234, 2017

      • Gentlecat
        so these might need an update during a schema change, right? which would make it easier to track down what exactly needs to be updated
      • 2017-06-21 17200, 2017

      • Gentlecat
        might need to do that for the last one actually
      • 2017-06-21 17237, 2017

      • Mineo
        the extrapaths paths are ones where the data is not stored in a search field, but in the generated XML by the wscompat module - we don't want to interact with the database in the wscompat module, so the data has to be fetched along with the rest
      • 2017-06-21 17230, 2017

      • arbenina_ has quit
      • 2017-06-21 17217, 2017

      • iliekcomputers
        ruaok: weird build failure, I think a restart is worth a try
      • 2017-06-21 17256, 2017

      • Mineo
        Gentlecat, and yes, those would need updates for schema changes where either the tables involved change or new data is supposed to show up in the index
      • 2017-06-21 17202, 2017

      • iliekcomputers
        or you could just merge considering one build passed and the other seemed to have failed because the db container seemed to be doing something weird
      • 2017-06-21 17247, 2017

      • Mineo has quit
      • 2017-06-21 17243, 2017

      • colbydray has quit
      • 2017-06-21 17246, 2017

      • samj1912 has quit
      • 2017-06-21 17218, 2017

      • CatQuest
        that's odd.. in one browser my profile has all the listens i imported (well most) and in the other browser same page.. none?
      • 2017-06-21 17239, 2017

      • CatQuest
        or wait, no
      • 2017-06-21 17248, 2017

      • CatQuest
        all my imports have been wiped :/
      • 2017-06-21 17240, 2017

      • Mineo joined the channel
      • 2017-06-21 17257, 2017

      • iliekcomputers
        CatQuest: the db was probably reset again
      • 2017-06-21 17259, 2017

      • ruaok
        yes, CatQuest, I wiped it again. I'll wipe it again before we're done, but I really appreciate you doing the import. its become a litmus test. :)
      • 2017-06-21 17243, 2017

      • CatQuest
        :D
      • 2017-06-21 17258, 2017

      • CatQuest is doing it again.. it's onabout half now
      • 2017-06-21 17259, 2017

      • CatQuest
        in the future people with much higher listens thna me wil be importing tino listenbrainz :)
      • 2017-06-21 17205, 2017

      • CatQuest
        into*
      • 2017-06-21 17211, 2017

      • CatQuest
        how the heck did tino beocme into?
      • 2017-06-21 17235, 2017

      • CatQuest
        into become tine i mean.. I need a nap :o
      • 2017-06-21 17241, 2017

      • CatQuest
        yes.. a nap
      • 2017-06-21 17243, 2017

      • CatQuest
        jeesh
      • 2017-06-21 17203, 2017

      • antgel joined the channel
      • 2017-06-21 17253, 2017

      • gcilou joined the channel
      • 2017-06-21 17203, 2017

      • lazka joined the channel
      • 2017-06-21 17224, 2017

      • lazka has quit
      • 2017-06-21 17232, 2017

      • ruaok
      • 2017-06-21 17247, 2017

      • Leo_Verto[m]
        huh, that's cool
      • 2017-06-21 17218, 2017

      • kepstin
        "the mind behind the brainz", eh? :)
      • 2017-06-21 17242, 2017

      • Leo_Verto[m]
        I love how you call it a "drug dealer business model"
      • 2017-06-21 17259, 2017

      • ruaok
        that's what we're doing, no?
      • 2017-06-21 17252, 2017

      • Leo_Verto[m]
        Get them hooked on the free music data and then make them pay for it, I guess that's scarily accurate
      • 2017-06-21 17257, 2017

      • iliekcomputers did not know the cake story :)
      • 2017-06-21 17200, 2017

      • iliekcomputers
        awesome
      • 2017-06-21 17235, 2017

      • ruaok
        :)
      • 2017-06-21 17220, 2017

      • Leo_Verto[m]
        yeah, I also didn't know that one. and it was actually a really nice cake!
      • 2017-06-21 17228, 2017

      • Leo_Verto[m]
        I wonder who ate it :P
      • 2017-06-21 17242, 2017

      • iliekcomputers
        they typo'ed ListenBrainz in the first line -- `ListenBrain` :(
      • 2017-06-21 17225, 2017

      • D4RK-PH0ENiX has quit
      • 2017-06-21 17217, 2017

      • Mineo has quit
      • 2017-06-21 17215, 2017

      • D4RK-PH0ENiX joined the channel
      • 2017-06-21 17257, 2017

      • D4RK-PH0ENiX has quit
      • 2017-06-21 17207, 2017

      • D4RK-PH0ENiX joined the channel
      • 2017-06-21 17240, 2017

      • D4RK-PH0ENiX has quit
      • 2017-06-21 17207, 2017

      • D4RK-PH0ENiX joined the channel