#musicbrainz

/

      • hrglgrmpf
        (debian testing, postgresql 9.1.1)
      • reosarevok
        ocharles, ^
      • adhawkins has problems after the upgrade too
      • adhawkins
      • hrglgrmpf
        adhawkins: I get the same error, but I thought this is normal since my ./upgrade.sh script failed
      • adhawkins: has yours passed without error?
      • adhawkins
        No, I think I had problems in upgrade.sh too
      • Didn't save them though
      • That error is now generated by slave.sh every time it tried to replicate
      • The error you've posted does ring a bell from when I did the upgrade.sh though
      • hrglgrmpf
        well, it seems not surprising that replication isn't possible if the upgrade.sh failed ;-)
      • adhawkins
        Yeah, I guess!
      • But luks thought they looked more like perl errors, so were unrelated to the database.
      • hrglgrmpf
        hmm, the extract_path_value() thing looks like a database thing...
      • adhawkins
        Yeah, but the ones in the slave log were the ones luks thought were perl related
      • I did things in the wrong order, which probably didn't help
      • hrglgrmpf
        what do you mean?
      • adhawkins
        I did upgrade.sh first before running the perl Makefile.pl to update the perl dependencies
      • Muz
        w 29
      • luks
        hrglgrmpf: when does that happen?
      • the extract_path_value error
      • hrglgrmpf
        when I call upgrade.sh
      • luks
        upgrading RT_STANDALONE server is going to be a little harder
      • adhawkins
        Mine is a slave
      • hrglgrmpf
        I have both...
      • luks
        ohh
      • adhawkins
        I think I saw the same error though
      • luks
        actually, plperl is required by all types of servers
      • that seems a little pointless on RT_SLAVE
      • hrglgrmpf: I think that admin/sql/updates/20110804-json-extract.sql failed for you because you don't have PL/perl
      • and the following script which needs the functions failed as well
      • hrglgrmpf
        could be!
      • I can't upgrade at the moment, but will do later
      • adhawkins
        What is plperl?
      • luks
        this stuff really shouldn't be installed in RT_SLAVE...
      • hrglgrmpf
        however, why hasn't perl Makefile.PL warned me / installed it?
      • luks
        hrglgrmpf: it's a postgresql module, nothing to do with perl
      • hrglgrmpf
        ok, than the blog post should have mentioned it :-)
      • luks
        well, perl as in something that has access to the modules in Makefile.PL
      • I thought you won't need it at all
      • it makes no sense for me to require it on RT_SLAVE servers
      • adhawkins
        Ok, so if upgrade.sh didn't fully upgrade the databases correctly, what can I do to fix it?
      • luks
        given that edit data is not replicated
      • reosarevok
        someone please look at the instructions in the blog and add a comment or something with the missing steps :)
      • luks
        adhawkins: your problem is unrelated
      • adhawkins
        I think my upgrade.sh failed in the same way though
      • luks
        you didn't mention that it failed
      • adhawkins
        hrglgrmpf is seeing the same errors as me in his slave.log I think
      • reosarevok
        luks, http://forums.musicbrainz.org/viewtopic.php?pid... in case it's an acoustid issue
      • adhawkins
        I know, I forgot until I saw hrglgrmpf's message :)
      • I did this at midnight :)
      • luks
        echo 'select * from replication_control;' | ./admin/psql
      • what does it say?
      • adhawkins
        (1 row)
      • luks
        so it finished correctly
      • adhawkins
        I seem to remember seeing some errors along the way similar to what hrglgrmpf has posted
      • luks
        you can ignore them, you won't need those indexes
      • adhawkins
        Ok
      • So we're still left with the replication errors, which I think hrglgrmpf is seeing too.
      • luks
        ianmcorvidae: ping
      • luks is glad he stopped using mbserver for replication
      • adhawkins
        What's the alternative for someone who wants to run a local server to do batch queries against?
      • luks
        there is no alternative for running the web service
      • adhawkins
        Any way of keeping the database updated without using the slave stuff to do it?
      • luks
      • adhawkins
        ocharles is on his was :)
      • ocharles
        what's broke?
      • ocharles reads
      • luks
      • ocharles
        yep, on it
      • I don't get how rob's slaves were working fine...
      • adhawkins
        We both had some errors reported by upgrade.sh, but luks doesn't think they're relevant
      • luks
        the other error was related to plperl
      • reosarevok facepalms a bit http://musicbrainz.org/release/56390511-9b17-420b-84c3-985c20d4966d
      • which wouldn't make the replication not work
      • adhawkins
        ocharles, I can give you access to my VM if it would help
      • ocharles
        adhawkins: it's fine
      • pull now and try with master HEAD
      • v6lur
        kovacsur: anything that *should* remain capitalized on http://musicbrainz.org/release/60417765-6282-45... ? (aside from first letters, etc, of course :)
      • kovacsur
        checking
      • adhawkins
        Um, ocharles, for a git newbie can you tell me what commands I need? :)
      • ocharles
        git checkout master; git pull origin master
      • kovacsur
        Ah, that release
      • The thing with that one is it has quite a few missing diacritics, etc as well
      • but that is how it is printed on the cover
      • adhawkins
        ocharles, shall we take this to #mb-dev?
      • ocharles
        adhawkins: sounds good!
      • kovacsur
        I can fix it to proper Hungarian, but I wasn't sure I should
      • v6lur, are you editing that release right now? If not, I'll fix capitalization and diacritics
      • then typos in a second edit
      • v6lur
        i'm not anymore :P
      • tenthirteen joined the channel
      • kovacsur
        oh, I see you've corrected caps
      • v6lur
        i entered stuff as on back cover http://media.cdr.nl/COVER/LARGE/BACK/JK149150.jpg
      • kovacsur
        okay, I'll fix diacritics (autoedit) then typos
      • v6lur
        but do go ahead and fix the typos, as i don't really speak the language :)
      • why "typos in a second edit"?
      • kovacsur
        because those aren't autoedits
      • that's how I usually do it
      • v6lur
        so?
      • kovacsur
        they take 2 weeks to pass
      • v6lur
        i can approve them for you
      • reosarevok
        :p
      • v6lur
        i just want this release out of my "Recordings need capitalization fixes" collection :P
      • kovacsur
        well, in that case I'll do it as one edit. :p
      • Artist credits are rather suspicious as well (Soml� was a member of Omega when that single was released, etc), but I'll just leave them as they are
      • reosarevok
        Pfft
      • kovacsur
        also, track 1 is an outtake from a musical theatre production, the artist credits given on the packaging are the authors
      • reosarevok
        Hmm, I see nikki's wikipedia script now loads photos too :D
      • kovacsur
      • v6lur
        thanks :)
      • i'll fix recordings now, too
      • kovacsur
        thanks for the approval
      • uh, I didn't update recording titles because I assumed they would need to be merged into the original ones anyway
      • or are these special mixes/remasters?
      • v6lur
        no idea. i didn't plan on merging them, but i'll fix the caps
      • there.
      • reosarevok
        kovacsur / v6lur: I imagine you've both seen the option to change recordings and tracks at the same time, right? :)
      • (just checking)
      • kovacsur
        reosarevok, I did
      • v6lur
        yes. how else would i have managed it in under a minute? :P
      • reosarevok
        Hmm
      • v6lur: does that mean that it applies *all* differences?
      • v6lur
        (that's also the reason i started working through that collection of mine...)
      • reosarevok
        (not just the changes you make in that session)
      • v6lur
        almost
      • well
      • i mean, it detects any differences in tracklist and recordings
      • reosarevok
        Oh, ok
      • I thought it would only give you the checkbox for the changes *you* just did
      • v6lur
        but the checkboxes are only there if you change *anything* in given session
      • reosarevok
        Oooh
      • That sounds like a kind-of-bug :p
      • v6lur
        anything in tracklist, that is
      • i like it
      • reosarevok
        I like that the option is there
      • v6lur
        so to me, it's a feature. not a bug, to paraphrase HP people
      • reosarevok
        But not having to change something in the tracklist (if it's already fine) to fix recordings :P
      • v6lur
        (the computer company, not that japanese group :P )
      • reosarevok
        Sure, I see the point, I just don't think it should depend on you changing something then
      • v6lur
        ah, yea... i usually just enter "aaa" to last track of every disc to get the checkboxes, go to recordings page, go back, delete "aaa", and then go to recordings page again :P
      • reosarevok
        So, that's a bug/problem :)
      • (also, lol)
      • adhawkins
        hrglgrmpf, see on #mb-dev for details of a fix to the problem replicating.
      • kovacsur
        oh, v6lur, any reason you changed "Zalatnay Sarolta" to western name order, but left Nem�nyi B�la in Hungarian order?
      • oh, nevermind, that's how they're printed
      • v6lur
        i wondered about them myself, too. then i figured, hey, that's how they are in cover, maybe they had good reasons
      • if you have reason to believe they had not, fix them and ping me :)
      • reosarevok would probably still follow the cover order