#metabrainz

/

      • CatQuest
        reosarevok ಠ_ಠ
      • 2017-06-07 15821, 2017

      • CatQuest
        (here it is rain)
      • 2017-06-07 15827, 2017

      • CatQuest
        (well the plants like it.. i guess)
      • 2017-06-07 15806, 2017

      • Rotab
        i like it
      • 2017-06-07 15818, 2017

      • Rotab
        no pesky sun
      • 2017-06-07 15835, 2017

      • CatQuest
        but Rotab you're a datepalm anyway :>
      • 2017-06-07 15841, 2017

      • CatQuest
        a plant
      • 2017-06-07 15842, 2017

      • Rotab
        heh
      • 2017-06-07 15828, 2017

      • ruaok
        zas: once again I'm having issues with getting a docker container to stop.
      • 2017-06-07 15839, 2017

      • ruaok
        it was doing massive amounts of logging, now stopping the container times out.
      • 2017-06-07 15859, 2017

      • ruaok
        restarting docker daemon just restarts the crappy container.
      • 2017-06-07 15807, 2017

      • ruaok
        any ideas?
      • 2017-06-07 15815, 2017

      • zas
        which container on which host?
      • 2017-06-07 15827, 2017

      • ruaok
        lemmy:listenbrainz-influx-writer
      • 2017-06-07 15811, 2017

      • zas
        i can stop and restart it without issue
      • 2017-06-07 15808, 2017

      • ruaok
        meh, now it works. well thanks for your magic.
      • 2017-06-07 15852, 2017

      • zas
        well, better believe in magic, but i guess in the true world there is a good technical explanation
      • 2017-06-07 15814, 2017

      • ruaok
        if you have access to that explanation, please share!
      • 2017-06-07 15828, 2017

      • zas
        the process is writing a lot of logs ? perhaps a sync to disk was somehow hanging
      • 2017-06-07 15827, 2017

      • ruaok
        damnit.
      • 2017-06-07 15838, 2017

      • ruaok gets cockblocked by a 410 error
      • 2017-06-07 15847, 2017

      • zas
        lol
      • 2017-06-07 15821, 2017

      • CatQuest has left the channel
      • 2017-06-07 15836, 2017

      • zas
      • 2017-06-07 15809, 2017

      • ruaok
        thats fine. consult hadn't finished its bit yet.
      • 2017-06-07 15814, 2017

      • zas
        i think consul-template is started too soon, syslog-ng is not ready yet, hence the "error setting up syslog logger"
      • 2017-06-07 15832, 2017

      • zas
        a small delay may help
      • 2017-06-07 15844, 2017

      • ruaok
        who is user marc2k3 ?
      • 2017-06-07 15849, 2017

      • ruaok
        anyone in here?
      • 2017-06-07 15808, 2017

      • ruaok
        they are importing my data into LB from last.fm and it keep erroring.
      • 2017-06-07 15807, 2017

      • agentsim joined the channel
      • 2017-06-07 15858, 2017

      • ruaok
        zas: quick favor?
      • 2017-06-07 15817, 2017

      • Slurpee joined the channel
      • 2017-06-07 15829, 2017

      • ruaok
        please change the message from "Please use the API at betaapi.listenbrainz.org" to "Please use the API at beta-api.listenbrainz.org"
      • 2017-06-07 15834, 2017

      • ruaok
        one has dns that has propagated.
      • 2017-06-07 15842, 2017

      • zas
        k
      • 2017-06-07 15827, 2017

      • ruaok
      • 2017-06-07 15841, 2017

      • ruaok
        yet
      • 2017-06-07 15845, 2017

      • ruaok
      • 2017-06-07 15802, 2017

      • ruaok
        can I have two front gateways point to the same container?
      • 2017-06-07 15814, 2017

      • zas
        let me check
      • 2017-06-07 15839, 2017

      • agentsim has quit
      • 2017-06-07 15842, 2017

      • zas
        if ports are different yes
      • 2017-06-07 15854, 2017

      • zas
        btw, gateways has it, but the backend fails
      • 2017-06-07 15836, 2017

      • ruaok
        ports are not different though -- just vhost is different.
      • 2017-06-07 15816, 2017

      • zas
        i mean different ports on the backend host if they run on the same
      • 2017-06-07 15835, 2017

      • zas
        ah wait
      • 2017-06-07 15843, 2017

      • ruaok
        I only have one container and for now I want both vhosts to resolve to the same container.
      • 2017-06-07 15847, 2017

      • zas
        i understand you have ONE container on the backend
      • 2017-06-07 15853, 2017

      • ruaok
        that!
      • 2017-06-07 15808, 2017

      • zas
        it can't work, because the health check is based on the name
      • 2017-06-07 15811, 2017

      • ruaok
        this is future proofing for when we need to move this container to different host.
      • 2017-06-07 15816, 2017

      • zas
        "beta-api.listenbrainz-web"
      • 2017-06-07 15834, 2017

      • ruaok
        so, I *have* to run a different container?
      • 2017-06-07 15839, 2017

      • zas
        yes, for now, it would require a lot of changes in the way we do things to handle the case
      • 2017-06-07 15810, 2017

      • ruaok
        ok, can you undo the 410 hack then? I can't finish this today.
      • 2017-06-07 15816, 2017

      • agentsim joined the channel
      • 2017-06-07 15821, 2017

      • ruaok
        another docker rabbit hole.
      • 2017-06-07 15822, 2017

      • zas
        it is theorically possible, but here upstream is considered unavailable because consul is using health checks based on the name
      • 2017-06-07 15839, 2017

      • zas
        i disable 410, ok
      • 2017-06-07 15806, 2017

      • zas
        that's not a docker issue, but a consul/our stuff issue
      • 2017-06-07 15824, 2017

      • ruaok
        s/docker rabbit hole/deployment rabbit hole/
      • 2017-06-07 15851, 2017

      • ruaok
        back to 3 LB pre-beta issues.
      • 2017-06-07 15826, 2017

      • zas
        ruaok: can you re-add the beta-api service, but with an extra key/value: "upstream": ""beta.listenbrainz-web", under "proto": "uwsgi", i think i have a solution, but i need to test
      • 2017-06-07 15820, 2017

      • ruaok
        iliekcomputers: does the last.fm import on beta work for you?
      • 2017-06-07 15825, 2017

      • ruaok
        doesn't for me. no clue why.
      • 2017-06-07 15836, 2017

      • ruaok
      • 2017-06-07 15840, 2017

      • ruaok
        zas ^^
      • 2017-06-07 15850, 2017

      • zas
        yes, but remove the double ""
      • 2017-06-07 15855, 2017

      • ruaok
        :)
      • 2017-06-07 15815, 2017

      • zas
        not sure if it will work, but worth a try
      • 2017-06-07 15834, 2017

      • iliekcomputers
        The importer says it is working but nothing shows up on the user page 😕
      • 2017-06-07 15836, 2017

      • ruaok
        pushed
      • 2017-06-07 15845, 2017

      • ruaok
        iliekcomputers: yeah, same here. :(
      • 2017-06-07 15834, 2017

      • iliekcomputers
        I just did two last.fm imports in dev for LB-149 and they worked, weird
      • 2017-06-07 15835, 2017

      • BrainzBot
        LB-149: Ensure that all unique listens are successfully inserted into BigQuery https://tickets.metabrainz.org/browse/LB-149
      • 2017-06-07 15839, 2017

      • ruaok
        still getting 502 on that zas
      • 2017-06-07 15847, 2017

      • zas
        yes ! should work, wait i'll commit and deploy on kiki, i was testing on herb
      • 2017-06-07 15843, 2017

      • ruaok
        woot!
      • 2017-06-07 15850, 2017

      • ruaok
        thanks zas!
      • 2017-06-07 15851, 2017

      • zas
        it works on kiki
      • 2017-06-07 15802, 2017

      • zas
        so i can cancel the 410 cancel
      • 2017-06-07 15803, 2017

      • ruaok
        so, now the 410 block can come back.
      • 2017-06-07 15802, 2017

      • zas
        done
      • 2017-06-07 15822, 2017

      • zas
        with "upstream", one can use the upstreams from another service
      • 2017-06-07 15831, 2017

      • zas
        a bit hacky, but convenient
      • 2017-06-07 15841, 2017

      • ruaok
        thank you!
      • 2017-06-07 15842, 2017

      • ruaok
        iliekcomputers: the scraper.js doesn't seem to be using the right API endpoint, even though it was changed in the config.py file.
      • 2017-06-07 15820, 2017

      • ruaok
        BETA_URL = '''https://beta-api.listenbrainz.org'''
      • 2017-06-07 15825, 2017

      • ruaok
        from inside the container.
      • 2017-06-07 15841, 2017

      • ruaok
        didn't we fix that, iliekcomputers?
      • 2017-06-07 15852, 2017

      • ruaok
        or rather, didn't you fix that? :)
      • 2017-06-07 15845, 2017

      • iliekcomputers
        ruaok: that is used by alpha_importer
      • 2017-06-07 15858, 2017

      • ruaok
        rats.
      • 2017-06-07 15805, 2017

      • ruaok
        we need to have the scraper use it as well.
      • 2017-06-07 15824, 2017

      • ruaok
        can you whip up a quick PR for what while I debug why the listens are not getting in?
      • 2017-06-07 15807, 2017

      • iliekcomputers
        ruaok: okay
      • 2017-06-07 15811, 2017

      • iliekcomputers
        on it
      • 2017-06-07 15831, 2017

      • ruaok
        thx
      • 2017-06-07 15818, 2017

      • arbenina_ joined the channel
      • 2017-06-07 15847, 2017

      • ruaok spots two problems
      • 2017-06-07 15855, 2017

      • marc2k3 joined the channel
      • 2017-06-07 15854, 2017

      • marc2k3
        ruaok: I logged in listenbrainz beta yesterday and tried to import from alpha. That's all I've done. I've not tried importing anything from Last.fm
      • 2017-06-07 15816, 2017

      • ruaok
        ah, great, thanks for letting me know.
      • 2017-06-07 15852, 2017

      • ruaok
        did the import work as expected?
      • 2017-06-07 15859, 2017

      • marc2k3
        Checking now...
      • 2017-06-07 15840, 2017

      • marc2k3
        It looks incomplete compared to the most recent listens on the alpha site.
      • 2017-06-07 15816, 2017

      • ruaok
        ok, not surprising. thanks.
      • 2017-06-07 15809, 2017

      • drsaunders joined the channel
      • 2017-06-07 15827, 2017

      • github joined the channel
      • 2017-06-07 15827, 2017

      • github
        [listenbrainz-server] paramsingh opened pull request #193: Use config.BETA_URL in scraper.js (master...use-beta-api-url) https://git.io/vH6Os
      • 2017-06-07 15828, 2017

      • github has left the channel
      • 2017-06-07 15838, 2017

      • ruaok
        thanks iliekcomputers.
      • 2017-06-07 15812, 2017

      • iliekcomputers
        o/
      • 2017-06-07 15830, 2017

      • ruaok
        next problem: if influx rejects a write, we keep trying to insert it over and over again.
      • 2017-06-07 15847, 2017

      • ruaok
      • 2017-06-07 15803, 2017

      • ruaok
        sorry for the long line. I have zero ideas what are going on there.
      • 2017-06-07 15836, 2017

      • ruaok
        lets first look at the problem when influx rejects something. we need to log the error and then let the data go.
      • 2017-06-07 15836, 2017

      • LordSputnik
        anshuman73: about to go, be quick if you're coming!
      • 2017-06-07 15841, 2017

      • ruaok
        marc2k3: one more question: is this music part of your collection?
      • 2017-06-07 15843, 2017

      • ruaok
        album=\"Tectonics\",albumartist=\"Adam Freeland\",artist=\"Proper Filthy Naughty\" ?
      • 2017-06-07 15814, 2017

      • ruaok
        I think we have a bug where we never tested a multi-user setup only with single users and it is failing. lol.
      • 2017-06-07 15852, 2017

      • github joined the channel
      • 2017-06-07 15852, 2017

      • github
        [listenbrainz-server] mayhem closed pull request #193: Use config.BETA_URL in scraper.js (master...use-beta-api-url) https://git.io/vH6Os
      • 2017-06-07 15852, 2017

      • github has left the channel
      • 2017-06-07 15815, 2017

      • kyan joined the channel
      • 2017-06-07 15846, 2017

      • marc2k3
        Yep, that is a legit listen from me
      • 2017-06-07 15800, 2017

      • ruaok
        ok, great. :)
      • 2017-06-07 15807, 2017

      • iliekcomputers
        phew
      • 2017-06-07 15819, 2017

      • iliekcomputers
        ruaok: we could reduce the number of retries here: https://github.com/metabrainz/listenbrainz-server…
      • 2017-06-07 15836, 2017

      • ruaok
        Audiowerk, 3 Mile Island too marc2k3 ?
      • 2017-06-07 15809, 2017

      • ruaok
        well, with a 400 error, it will never work and we need to log the error and ditch the data.
      • 2017-06-07 15816, 2017

      • ruaok
        without any retries.
      • 2017-06-07 15825, 2017

      • marc2k3
      • 2017-06-07 15832, 2017

      • iliekcomputers
        okay
      • 2017-06-07 15857, 2017

      • arbenina_ has quit
      • 2017-06-07 15807, 2017

      • ruaok
        marc2k3: ok, great. then we may not have that problem.
      • 2017-06-07 15824, 2017

      • ruaok
        it appears that we have similar tastes in music. :)
      • 2017-06-07 15835, 2017

      • iliekcomputers
        the thing is we get 400 for the entire data, so we have to go down and break it into half, we don't know which listen is causing the error
      • 2017-06-07 15854, 2017

      • iliekcomputers
        we do ditch the data eventually already
      • 2017-06-07 15805, 2017

      • ruaok
        but we fill up the logs with data in the meantime. :(
      • 2017-06-07 15815, 2017

      • ruaok
        and the container just stops responding
      • 2017-06-07 15844, 2017

      • iliekcomputers
        Maybe we can stop logging in InfluxListenStore: https://github.com/metabrainz/listenbrainz-server…
      • 2017-06-07 15821, 2017

      • iliekcomputers
        that would mean we just log the error when we've given up on it here: https://github.com/metabrainz/listenbrainz-server…
      • 2017-06-07 15823, 2017

      • ruaok
        maybe only log when we actually drop a listen.
      • 2017-06-07 15834, 2017

      • ruaok
        great minds. :)
      • 2017-06-07 15841, 2017

      • iliekcomputers
        haha :)
      • 2017-06-07 15841, 2017

      • ruaok
        but, what the fuck is up with that error?
      • 2017-06-07 15854, 2017

      • iliekcomputers
        ruaok: I think we need to take a close look at how we handle measurement names, I think the problem might be somewhere there.
      • 2017-06-07 15837, 2017

      • ruaok
        I think you're right.