#metabrainz

/

      • alastairp
        the main drive is resyncing (probably because of reallocated bad sectors)
      • 2018-08-22 23458, 2018

      • alastairp
        this is going to affect the speed of the dumps (and the dumps will affect the speed of the resync)
      • 2018-08-22 23417, 2018

      • alastairp
        12 days to finish the resync
      • 2018-08-22 23454, 2018

      • iliekcomputers
        okay, so they should be a bit faster on frank then.
      • 2018-08-22 23405, 2018

      • alastairp
        what disks does frank have?
      • 2018-08-22 23419, 2018

      • iliekcomputers
        let me check
      • 2018-08-22 23452, 2018

      • CatQuest
        I'm going out for a bit, but LordSputnik, pong me when you have a test ready for me :D
      • 2018-08-22 23439, 2018

      • iliekcomputers
      • 2018-08-22 23419, 2018

      • iliekcomputers
      • 2018-08-22 23425, 2018

      • alastairp
        right, normal spinning disks, then
      • 2018-08-22 23441, 2018

      • iliekcomputers
        yes
      • 2018-08-22 23443, 2018

      • iliekcomputers
        7200 rpm
      • 2018-08-22 23400, 2018

      • iliekcomputers was on 5400 rpm hdds until a few months ago
      • 2018-08-22 23404, 2018

      • iliekcomputers
        :/
      • 2018-08-22 23428, 2018

      • ruaok
        4TB is SSD is quite expensive. :(
      • 2018-08-22 23440, 2018

      • ruaok
        *of
      • 2018-08-22 23405, 2018

      • alastairp
        I can imagine
      • 2018-08-22 23436, 2018

      • iliekcomputers
        lolyea
      • 2018-08-22 23423, 2018

      • iliekcomputers
        another 30GB file done and being compressed rn
      • 2018-08-22 23432, 2018

      • alastairp
        mmm. faster??
      • 2018-08-22 23431, 2018

      • Nyanko-sensei joined the channel
      • 2018-08-22 23403, 2018

      • iliekcomputers
        this is the second one, I don't think so, I think it took around the same time.
      • 2018-08-22 23457, 2018

      • D4RK-PH0ENiX has quit
      • 2018-08-22 23420, 2018

      • clarjon1 joined the channel
      • 2018-08-22 23420, 2018

      • clarjon1 has quit
      • 2018-08-22 23401, 2018

      • Nyanko-sensei has quit
      • 2018-08-22 23406, 2018

      • Mr_Monkey has quit
      • 2018-08-22 23426, 2018

      • bhargav joined the channel
      • 2018-08-22 23450, 2018

      • bhargav is now known as Guest27381
      • 2018-08-22 23444, 2018

      • D4RK-PH0ENiX joined the channel
      • 2018-08-22 23409, 2018

      • bukwurm joined the channel
      • 2018-08-22 23411, 2018

      • Mr_Monkey joined the channel
      • 2018-08-22 23451, 2018

      • Guest27381 has quit
      • 2018-08-22 23422, 2018

      • bhargav_prakash joined the channel
      • 2018-08-22 23434, 2018

      • bukwurm
      • 2018-08-22 23447, 2018

      • bukwurm
      • 2018-08-22 23433, 2018

      • bukwurm
        I think postgres version is causing issue here.
      • 2018-08-22 23434, 2018

      • bhargav_prakash_ joined the channel
      • 2018-08-22 23444, 2018

      • bhargav_prakash_ has quit
      • 2018-08-22 23425, 2018

      • bukwurm
        Later tests which fail are:
      • 2018-08-22 23437, 2018

      • bukwurm
        `insert into "bookbrainz"."editor_type" ("id", "label") values ($1, $2) returning "id" - relation "bookbrainz.editor_type" does not exist[0m[90m error: relation "bookbrainz.editor_type" does not exist`
      • 2018-08-22 23413, 2018

      • bukwurm
        I wonder if transaction is aborted then how partial tables are created and remaining tests pass
      • 2018-08-22 23425, 2018

      • Mr_Monkey has quit
      • 2018-08-22 23415, 2018

      • bhargav_prakash has quit
      • 2018-08-22 23447, 2018

      • bhargav_prakash joined the channel
      • 2018-08-22 23432, 2018

      • Mr_Monkey joined the channel
      • 2018-08-22 23437, 2018

      • bukwurm
        Postgres version is 9.6.6, so something else is perhaps the matter.
      • 2018-08-22 23407, 2018

      • LordSputnik
        CatQuest: won't be today I'm afraid, had guests over last night and I'm out tonight - but the code is working now and I just need to commit everything and update the server, so maybe I can do that tomorrow night
      • 2018-08-22 23409, 2018

      • CatQuest
        :(
      • 2018-08-22 23431, 2018

      • AC`97_ joined the channel
      • 2018-08-22 23417, 2018

      • AC`97_ has quit
      • 2018-08-22 23427, 2018

      • Mr_Monkey
        bukwurm Thanks, I'll have a look!
      • 2018-08-22 23451, 2018

      • bhargav_prakash has quit
      • 2018-08-22 23419, 2018

      • outsidecontext has quit
      • 2018-08-22 23423, 2018

      • outsidecontext joined the channel
      • 2018-08-22 23407, 2018

      • Mr_Monkey
        bukwurm & LordSputnik : It looks like a CI-breaking change was introduced to bookbrainz-sql with this commit, and it looks like no job passed since https://github.com/bookbrainz/bookbrainz-sql/comm…
      • 2018-08-22 23409, 2018

      • kurianbenoy joined the channel
      • 2018-08-22 23440, 2018

      • Mr_Monkey
        I can't figure out where the postgres configuration is in Travis. I think maybe the version 9.6.6 that bukwurm saw is only that of psql rather than the postgres server version, in which case it might just need an update to >9.4
      • 2018-08-22 23408, 2018

      • iw00t21 joined the channel
      • 2018-08-22 23412, 2018

      • iw00t21 has quit
      • 2018-08-22 23418, 2018

      • outsidecontext has quit
      • 2018-08-22 23452, 2018

      • kurianbenoy has quit
      • 2018-08-22 23450, 2018

      • bukwurm
        Mr_Monkey: Yes, the gist shared showed the cause of error being jsonb type missing
      • 2018-08-22 23417, 2018

      • bukwurm
        *I shared
      • 2018-08-22 23421, 2018

      • bukwurm
        jsonb was used to store metadata related to imports
      • 2018-08-22 23402, 2018

      • bukwurm
        jsonb support I think is in 9.6+ versions
      • 2018-08-22 23434, 2018

      • bukwurm
      • 2018-08-22 23443, 2018

      • bukwurm
        *9.4+
      • 2018-08-22 23455, 2018

      • Mr_Monkey
        bukwurm Yes, just confirming and doing a recap. Do you have access to the Travis config?
      • 2018-08-22 23423, 2018

      • bukwurm
        Mr_Monkey: No :/
      • 2018-08-22 23414, 2018

      • bukwurm
        Mr_Monkey: But I do wonder, if the entire schema creation is done in a single transaction, when it fails how come other tests pass?
      • 2018-08-22 23437, 2018

      • bukwurm
        Maybe something to do with tests rather than database setup I guess
      • 2018-08-22 23448, 2018

      • bukwurm
      • 2018-08-22 23408, 2018

      • Mr_Monkey
        I'm not familiar enough with the setup yet to venture an educated guess. It looks to me like the schema creation only fails for the last few operations though
      • 2018-08-22 23400, 2018

      • Mr_Monkey
        Yeah, I saw that configuration, but there's nothing there that describes the postgres server, so I assume there is some extra config somewhere.
      • 2018-08-22 23459, 2018

      • bukwurm
        Mr_Monkey: I am guessing "npm run-script test-ci" is the one
      • 2018-08-22 23436, 2018

      • Mr_Monkey
        That's in the bookbrainz-site package.json, and here is what it runs:
      • 2018-08-22 23437, 2018

      • Mr_Monkey
        nyc --reporter=lcovonly --reporter=text-summary npm run test
      • 2018-08-22 23428, 2018

      • bukwurm
        That's not it then :P
      • 2018-08-22 23454, 2018

      • iliekcomputers
        alastairp: 4 files done now.
      • 2018-08-22 23442, 2018

      • alastairp
        bukwurm: I'm used to specifying a postgres version in the travis file: https://github.com/MTG/freesound/blob/master/.tra…, not sure why bookbrainz doesn't have it
      • 2018-08-22 23445, 2018

      • Mr_Monkey
        Could it be just that? Thanks alastairp, I'll try that.
      • 2018-08-22 23400, 2018

      • alastairp
        (not a travis expert)
      • 2018-08-22 23416, 2018

      • bukwurm
        alastairp: Ok. Could there any other way to specify postgres version on Travis?
      • 2018-08-22 23429, 2018

      • alastairp
        no idea, sorry
      • 2018-08-22 23434, 2018

      • bukwurm
        Or for that matter, any other service.
      • 2018-08-22 23438, 2018

      • bukwurm
        alastairp: Ok
      • 2018-08-22 23413, 2018

      • bukwurm
        Mr_Monkey: As you're trying it, do tell what happens when you mention the version in the .travis.yml file
      • 2018-08-22 23417, 2018

      • bukwurm
        :)
      • 2018-08-22 23438, 2018

      • Mr_Monkey
        Will do
      • 2018-08-22 23402, 2018

      • bukwurm
        👍
      • 2018-08-22 23403, 2018

      • alastairp
        Mr_Monkey: looks like 'addon' is deprecated, look at https://docs.travis-ci.com/user/database-setup/#p…
      • 2018-08-22 23439, 2018

      • alastairp
        "The Travis CI build environments use version 9.2 by default on Trusty images, but other versions from the official PostgreSQL APT repository are also available."
      • 2018-08-22 23426, 2018

      • bukwurm
        That's gotta be it, I think.
      • 2018-08-22 23451, 2018

      • Mr_Monkey
        bukwurm & alastairp Boom chacka boom ! all tests passing. Thanks for your help !
      • 2018-08-22 23400, 2018

      • bukwurm
        Mr_Monkey: Awesome
      • 2018-08-22 23433, 2018

      • travis-ci joined the channel
      • 2018-08-22 23434, 2018

      • travis-ci
        Project bookbrainz-site build #35: errored in 3 min 7 sec: https://travis-ci.org/shivam-tripathi/bookbrainz-…
      • 2018-08-22 23434, 2018

      • travis-ci has left the channel
      • 2018-08-22 23435, 2018

      • bukwurm
        Mr_Monkey: I think getting webpack integration with module replacement is awesome for the project, it's gonna make development so much easier.
      • 2018-08-22 23413, 2018

      • bukwurm
        Awesome work 👍
      • 2018-08-22 23441, 2018

      • Mr_Monkey
        It's quite nice indeed ! Currently the recompilation is a bit long for my taste, but I think I can optimize it.
      • 2018-08-22 23419, 2018

      • Mr_Monkey
        Using Webpack for production builds I'm still working on, there are a few snags, especially with css/less.
      • 2018-08-22 23426, 2018

      • Mr_Monkey
        And thanks :)
      • 2018-08-22 23453, 2018

      • LordSputnik
        bukwurm, Mr_Monkey: nice work guys. I did think I specified postgres 9.4 - perhaps for data-js
      • 2018-08-22 23432, 2018

      • LordSputnik
        Yeah: bookbrainz-data-js/blob/master/.travis.yml
      • 2018-08-22 23451, 2018

      • LordSputnik
      • 2018-08-22 23404, 2018

      • LordSputnik
        We just need to do the same for the site repo
      • 2018-08-22 23417, 2018

      • bukwurm
        LordSputnik: I think the confusion due to earlier tests failures due to two versions running complicated the issue
      • 2018-08-22 23453, 2018

      • bukwurm
        It only today when I really looked at the logs, to find the jsonb issue
      • 2018-08-22 23459, 2018

      • bukwurm
        LordSputnik: Do you recall what was the issue earlier, when you mentioned something about two version causing race condition or something?
      • 2018-08-22 23455, 2018

      • LordSputnik
        bukwurm: oh, that turned out to be a missing transaction passed to a bookshelf query
      • 2018-08-22 23410, 2018

      • LordSputnik
        So it was running outside the transaction
      • 2018-08-22 23440, 2018

      • Mr_Monke_ joined the channel
      • 2018-08-22 23413, 2018

      • Mr_Monkey has quit
      • 2018-08-22 23419, 2018

      • BestSteve joined the channel
      • 2018-08-22 23449, 2018

      • BestSteve has quit
      • 2018-08-22 23409, 2018

      • bukwurm
        LordSputnik: Ok
      • 2018-08-22 23420, 2018

      • jwf has quit
      • 2018-08-22 23421, 2018

      • moufl has quit
      • 2018-08-22 23421, 2018

      • CatQuest has quit
      • 2018-08-22 23421, 2018

      • GeneralDiscourse has quit
      • 2018-08-22 23421, 2018

      • dseomn has quit
      • 2018-08-22 23459, 2018

      • CatQuest joined the channel
      • 2018-08-22 23459, 2018

      • dseomn joined the channel
      • 2018-08-22 23445, 2018

      • CatQuest has quit
      • 2018-08-22 23445, 2018

      • CatQuest joined the channel
      • 2018-08-22 23402, 2018

      • badpixel9 joined the channel
      • 2018-08-22 23404, 2018

      • badpixel9 has quit
      • 2018-08-22 23408, 2018

      • reosarevok
        ruaok: see Doug's email, might be interesting :)
      • 2018-08-22 23424, 2018

      • yokel joined the channel
      • 2018-08-22 23454, 2018

      • yokel has quit
      • 2018-08-22 23454, 2018

      • yokel joined the channel
      • 2018-08-22 23401, 2018

      • yokel has quit
      • 2018-08-22 23437, 2018

      • yokel joined the channel
      • 2018-08-22 23452, 2018

      • yokel has quit
      • 2018-08-22 23452, 2018

      • yokel joined the channel
      • 2018-08-22 23440, 2018

      • Dr-Flay joined the channel
      • 2018-08-22 23438, 2018

      • UmkaDK_ has quit
      • 2018-08-22 23446, 2018

      • Freso
        Sounds like Doug should be put in contact with the people who previously talked about a similar project, maybe?
      • 2018-08-22 23407, 2018

      • Freso
        Chris Mendez and Mickaël Arcos.
      • 2018-08-22 23402, 2018

      • Freso
        The latter wrote a blog post on the subject: https://deezer.io/matching-albums-through-cover-a…
      • 2018-08-22 23451, 2018

      • ephemer0l joined the channel
      • 2018-08-22 23457, 2018

      • ephemer0l has quit
      • 2018-08-22 23457, 2018

      • ephemer0l joined the channel
      • 2018-08-22 23423, 2018

      • moufl joined the channel
      • 2018-08-22 23422, 2018

      • ephemer0l is now known as GeneralDiscourse
      • 2018-08-22 23444, 2018

      • reosarevok
        I can send him that at least :)
      • 2018-08-22 23444, 2018

      • Freso
        I can also create a follow-up e-mail introducing the two others maybe?
      • 2018-08-22 23431, 2018

      • reosarevok
        Feel free! :)