#metabrainz

/

      • BenOckmore has quit
      • 2021-05-16 13606, 2021

      • thomasross joined the channel
      • 2021-05-16 13640, 2021

      • sumedh joined the channel
      • 2021-05-16 13629, 2021

      • MRiddickW joined the channel
      • 2021-05-16 13653, 2021

      • sumedh has quit
      • 2021-05-16 13620, 2021

      • v6lur joined the channel
      • 2021-05-16 13649, 2021

      • thomasross has quit
      • 2021-05-16 13630, 2021

      • sumedh joined the channel
      • 2021-05-16 13635, 2021

      • sumedh has quit
      • 2021-05-16 13608, 2021

      • sumedh joined the channel
      • 2021-05-16 13634, 2021

      • MRiddickW has quit
      • 2021-05-16 13638, 2021

      • texke` joined the channel
      • 2021-05-16 13621, 2021

      • MRiddickW joined the channel
      • 2021-05-16 13606, 2021

      • texke has quit
      • 2021-05-16 13615, 2021

      • texke` is now known as texke
      • 2021-05-16 13617, 2021

      • MRiddickW has quit
      • 2021-05-16 13657, 2021

      • CatQuest
        lol
      • 2021-05-16 13639, 2021

      • CatQuest
        but wait, then freso can't one the meeting with "Today isNorways National holiday *and* the International Day Against Homophobia and Transphobia" :(
      • 2021-05-16 13654, 2021

      • CatQuest
        boo!
      • 2021-05-16 13641, 2021

      • sumedh has quit
      • 2021-05-16 13626, 2021

      • sumedh joined the channel
      • 2021-05-16 13641, 2021

      • v6lur has quit
      • 2021-05-16 13652, 2021

      • sumedh has quit
      • 2021-05-16 13621, 2021

      • shivam-kapila
        Lol
      • 2021-05-16 13617, 2021

      • ruaok
        moooin!
      • 2021-05-16 13629, 2021

      • ruaok
        > looks like the error did not get logged. probably that's why sentry didn't see it so no report.
      • 2021-05-16 13643, 2021

      • ruaok
        _lucifer: any idea why that is? maybe my logger isn't setup right?
      • 2021-05-16 13621, 2021

      • sumedh joined the channel
      • 2021-05-16 13637, 2021

      • _lucifer
        ruaok, could be. but then it did log `INFO:root:metric writer starting!` correctly. do you know where consul template error logs go and how to view them? i know we run it consul with `syslog` so it should be in syslog. however, i couldn't find it there yesterday or maybe i was looking at the wrong place?
      • 2021-05-16 13604, 2021

      • ruaok
      • 2021-05-16 13614, 2021

      • ruaok
        _lucifer: ^^ nuthing in syslog :(
      • 2021-05-16 13656, 2021

      • ruaok
        but I verified that data is being written to redis. but sometimes values don't make it to finflux, and i suspect somethging in the metric writer is borking.
      • 2021-05-16 13624, 2021

      • _lucifer
        ruaok, right. that's what i saw.
      • 2021-05-16 13647, 2021

      • ruaok
        hmm, indeed very weird. the first log message appears in the log, so logging does appear to be correct.
      • 2021-05-16 13651, 2021

      • _lucifer
        consul should be logging something i believe when the process restarts. it does when we run without syslog directly in container.
      • 2021-05-16 13611, 2021

      • _lucifer
        maybe zas or bitmap know where to look?
      • 2021-05-16 13639, 2021

      • ruaok
        and I've got the most loose check for errors from influx, but it appears it never gets tripped: https://github.com/metabrainz/metric-writer/blob/…
      • 2021-05-16 13636, 2021

      • ruaok
        hmmm. maybe write_points doesn't throw exceptions?
      • 2021-05-16 13658, 2021

      • _lucifer
        yeah. also there was one another line in the log which did not make sense
      • 2021-05-16 13628, 2021

      • _lucifer
        * os: process already finished
      • 2021-05-16 13641, 2021

      • ruaok
        yeah, no clue on that either.
      • 2021-05-16 13643, 2021

      • _lucifer
        not sure why this ^ got printed.
      • 2021-05-16 13614, 2021

      • _lucifer
        i don't know about write_points either. will need to check docs.
      • 2021-05-16 13615, 2021

      • ruaok
        write _points returns bool. not useful, really.
      • 2021-05-16 13629, 2021

      • ruaok
        the docs are shitty, I'm reading the code.
      • 2021-05-16 13636, 2021

      • _lucifer
        :(
      • 2021-05-16 13644, 2021

      • ruaok
        I can try checking for false and attempt a re-try.
      • 2021-05-16 13601, 2021

      • ruaok
        afterall the influx server is on another continent, so errors more likely.
      • 2021-05-16 13654, 2021

      • _lucifer
        true.
      • 2021-05-16 13608, 2021

      • _lucifer
      • 2021-05-16 13624, 2021

      • _lucifer
        these are the current logs. any idea about those cron jobs?
      • 2021-05-16 13656, 2021

      • ruaok
        oh, I need to installl cron/down.
      • 2021-05-16 13608, 2021

      • ruaok
        because we need no cron in this container.
      • 2021-05-16 13621, 2021

      • _lucifer
        ah ok.
      • 2021-05-16 13658, 2021

      • ruaok
        the v1.x version of the python API is really piss poor.
      • 2021-05-16 13602, 2021

      • travis-ci joined the channel
      • 2021-05-16 13602, 2021

      • travis-ci
        Project bookbrainz-site build #3968: errored in 1 min 44 sec: https://travis-ci.org/bookbrainz/bookbrainz-site/…
      • 2021-05-16 13602, 2021

      • travis-ci has left the channel
      • 2021-05-16 13630, 2021

      • ruaok
      • 2021-05-16 13655, 2021

      • ruaok
        you know what? screw this. I will submit the data via the HTTP api and sod this API.
      • 2021-05-16 13608, 2021

      • _lucifer
        lol!
      • 2021-05-16 13607, 2021

      • _lucifer
      • 2021-05-16 13624, 2021

      • _lucifer
        the http api seems more informative in case of errors than the python library
      • 2021-05-16 13658, 2021

      • _lucifer
        definitely better off using HTTP then
      • 2021-05-16 13659, 2021

      • ruaok
        on it
      • 2021-05-16 13655, 2021

      • _lucifer
      • 2021-05-16 13629, 2021

      • ruaok nods
      • 2021-05-16 13650, 2021

      • ruaok
        influx just never threw any errors at all.
      • 2021-05-16 13614, 2021

      • _lucifer
        makes sense.
      • 2021-05-16 13613, 2021

      • ruaok
        > r = requests.post("http://%s:%d/write" % (config.INFLUX_SERVER, config.INFLUX_PORT), params=params, data=lines)
      • 2021-05-16 13632, 2021

      • ruaok
        hmm. requests still interprets data as KV data, not as text.
      • 2021-05-16 13642, 2021

      • _lucifer
        ruaok, maybe lines.encode() works?
      • 2021-05-16 13602, 2021

      • ruaok
        trying that now.
      • 2021-05-16 13657, 2021

      • ruaok
        huh.
      • 2021-05-16 13609, 2021

      • ruaok
        list += item seems to work as list.append(item)
      • 2021-05-16 13602, 2021

      • shivam-kapila
        Wut. My whole python had been a lie
      • 2021-05-16 13650, 2021

      • shivam-kapila
        Doesnt work for ints
      • 2021-05-16 13657, 2021

      • shivam-kapila
        Only for strings
      • 2021-05-16 13609, 2021

      • ruaok
        `line = r.lpop(REDIS_METRICS_KEY)` apparently sometimes fetches data, removes it from redis and returns None. how that hell a, I doing THAT wrong?
      • 2021-05-16 13649, 2021

      • _lucifer
        ruaok, so the data exists but redis returns None?
      • 2021-05-16 13630, 2021

      • ruaok
        seems so, but right now I am fighting docker since it no longer updating the container.
      • 2021-05-16 13644, 2021

      • _lucifer
        what's the issue?
      • 2021-05-16 13601, 2021

      • ruaok
        williams doesn't pull the updated images I push.
      • 2021-05-16 13647, 2021

      • _lucifer
        Error response from daemon: toomanyrequests: You have reached your pull rate limit. You may increase the limit by authenticating and upgrading: https://www.docker.com/increase-rate-limit ?
      • 2021-05-16 13635, 2021

      • ruaok
        where do I see that error?
      • 2021-05-16 13644, 2021

      • _lucifer
        when doing docker pull on williams
      • 2021-05-16 13611, 2021

      • ruaok
        right. clearly time for a nap.
      • 2021-05-16 13620, 2021

      • yvanzo joined the channel
      • 2021-05-16 13627, 2021

      • ruaok
        and maybe time for us to have our own docker image repo. this isn't workable anymore.
      • 2021-05-16 13637, 2021

      • _lucifer
        i was under the impression docker was logged in on prod machines.
      • 2021-05-16 13653, 2021

      • ruaok
        not necessarily.
      • 2021-05-16 13655, 2021

      • _lucifer
        we can use github container registry maybe?
      • 2021-05-16 13659, 2021

      • ruaok
        I'll just build from williams then.
      • 2021-05-16 13610, 2021

      • _lucifer
        or let's login on williams?
      • 2021-05-16 13609, 2021

      • ruaok
        we really need to make our start_services.sh more quiet. too much noise to see real error messages
      • 2021-05-16 13642, 2021

      • _lucifer
        yeah. for the time being you could try `. scripts/services.sh` and then just run the start service line for your particular service.
      • 2021-05-16 13601, 2021

      • _lucifer
        that's how i and alastairp do it usually.
      • 2021-05-16 13613, 2021

      • ruaok
        oh clever, I should get in the habit of doing that.
      • 2021-05-16 13647, 2021

      • ruaok
        `. scripts/services.sh start_metric_writer "prod"` like that?
      • 2021-05-16 13652, 2021

      • ruaok
        (doesn't output anything)
      • 2021-05-16 13650, 2021

      • _lucifer
        `sudo su; cd ~/docker-server-configs; . scripts/services.sh; start_metric_writer "prod";`
      • 2021-05-16 13603, 2021

      • ruaok
        ah, separate line.
      • 2021-05-16 13609, 2021

      • _lucifer
        something like this
      • 2021-05-16 13610, 2021

      • _lucifer
        yes
      • 2021-05-16 13638, 2021

      • _lucifer
        scripts/services.sh has the functions, we source them and then call the required one
      • 2021-05-16 13622, 2021

      • ruaok
      • 2021-05-16 13633, 2021

      • ruaok
        it almost always returns None.
      • 2021-05-16 13651, 2021

      • ruaok
      • 2021-05-16 13659, 2021

      • ruaok
        even when there is data.
      • 2021-05-16 13600, 2021

      • _lucifer
        but redis has the data?
      • 2021-05-16 13616, 2021

      • ruaok
        yes.
      • 2021-05-16 13627, 2021

      • ruaok
        see the llen past above.
      • 2021-05-16 13650, 2021

      • ruaok
        hmmm.
      • 2021-05-16 13614, 2021

      • _lucifer
        strange. what does this while loop do? fetch all the items at that key and decode them?
      • 2021-05-16 13619, 2021

      • ruaok
        I wonder if there is some other metric writer longering somehwere. because the data vanished even without my reader. lol
      • 2021-05-16 13627, 2021

      • _lucifer
        lol
      • 2021-05-16 13650, 2021

      • ruaok
        🤦
      • 2021-05-16 13643, 2021

      • v6lur joined the channel
      • 2021-05-16 13632, 2021

      • thomasross joined the channel
      • 2021-05-16 13630, 2021

      • Lotheric_ joined the channel
      • 2021-05-16 13636, 2021

      • Lotheric has quit
      • 2021-05-16 13606, 2021

      • HenryG has quit
      • 2021-05-16 13645, 2021

      • HenryG joined the channel
      • 2021-05-16 13638, 2021

      • chaban
      • 2021-05-16 13601, 2021

      • v6lur has quit
      • 2021-05-16 13635, 2021

      • ruaok
      • 2021-05-16 13653, 2021

      • sumedh has quit