21:28 PM
warp
ianmcorvidae: out of curiosity, teto runs geordi, right? is there anything about geordi itself which we can/should monitor with nagios?
2013-02-07 03811, 2013
21:29 PM
luks
dhess: I bet you don't have the mbdump-derived tarball imported
2013-02-07 03811, 2013
21:29 PM
ianmcorvidae
warp: I guess we can just do a basic "is it up" test, though I actually have one of those set up with pingdom
2013-02-07 03847, 2013
21:30 PM
warp
ianmcorvidae: right, checking a publicly visible URL wouldn't need anything on the machine itself.
2013-02-07 03852, 2013
21:30 PM
misterswag_ joined the channel
2013-02-07 03810, 2013
21:31 PM
dhess
2013-02-07 03820, 2013
21:31 PM
ianmcorvidae
warp: I suppose things like critical processes matter too
2013-02-07 03832, 2013
21:31 PM
ianmcorvidae
warp: making sure elasticsearch and the geordi FCGI script is running in other words
2013-02-07 03815, 2013
21:32 PM
warp
ianmcorvidae: is that running through daemontools?
2013-02-07 03835, 2013
21:33 PM
ianmcorvidae
warp: it's not, no
2013-02-07 03840, 2013
21:33 PM
ianmcorvidae
it probably should be
2013-02-07 03804, 2013
21:34 PM
ruaok
this is from a user emailing support@
2013-02-07 03805, 2013
21:34 PM
ruaok
2013-02-07 03806, 2013
21:34 PM
warp
usually daemontools keeps everything running, and stuff doesn't suddenly break as long as you don't touch it.
2013-02-07 03811, 2013
21:34 PM
ruaok
da fug?
2013-02-07 03802, 2013
21:35 PM
reosarevok
Indeed
2013-02-07 03805, 2013
21:35 PM
warp
ruaok: nice
2013-02-07 03820, 2013
21:35 PM
ruaok
not sure what to say.
2013-02-07 03823, 2013
21:35 PM
reosarevok
That sounds like a verification mail thing
2013-02-07 03826, 2013
21:35 PM
ruaok
he can log into his account.
2013-02-07 03838, 2013
21:35 PM
ruaok
ya, that might be it.
2013-02-07 03847, 2013
21:35 PM
reosarevok
Well, it *somehow reminds me* of a verification mail thing :p
2013-02-07 03852, 2013
21:35 PM
reosarevok
"sounds like" is too kind
2013-02-07 03854, 2013
21:35 PM
reosarevok
But still
2013-02-07 03857, 2013
21:35 PM
ruaok
lol
2013-02-07 03802, 2013
21:36 PM
dhess
2013-02-07 03820, 2013
21:36 PM
dhess
that's the output with the change. I'm not sure what I'm looking for so I dunno if the stderr is included in the dump
2013-02-07 03830, 2013
21:36 PM
ianmcorvidae
dhess: it should be in the console where you started the server
2013-02-07 03837, 2013
21:36 PM
dhess
yeah ok
2013-02-07 03846, 2013
21:36 PM
ianmcorvidae
presumably you called 'plackup <etc>' somewhere, it'll be in that :)
2013-02-07 03859, 2013
21:36 PM
warp
I should get some sleep. goodnight #musicbrainz-devel!
2013-02-07 03807, 2013
21:37 PM
ruaok
nn warp
2013-02-07 03809, 2013
21:37 PM
reosarevok
2013-02-07 03818, 2013
21:37 PM
ruaok
on it, reosarevok
2013-02-07 03823, 2013
21:37 PM
reosarevok
Ok
2013-02-07 03832, 2013
21:37 PM
reosarevok
We really should write a template for this case
2013-02-07 03843, 2013
21:37 PM
MBJenkins
2013-02-07 03855, 2013
21:37 PM
warp
another HUGE SUCCESS
2013-02-07 03800, 2013
21:38 PM
reosarevok
although it feels like rewriting more or less the same we say when telling them about verification mails
2013-02-07 03811, 2013
21:38 PM
reosarevok
But I guess people don't read the site but do read answers to their mails :p
2013-02-07 03815, 2013
21:38 PM
ruaok
reosarevok: or make the UI more clear so we dont get these emails in the first palce
2013-02-07 03817, 2013
21:38 PM
dhess
just waiting for pastebin...
2013-02-07 03826, 2013
21:39 PM
dhess
screw it, I'll use pastebin.ca
2013-02-07 03830, 2013
21:39 PM
dhess
2013-02-07 03853, 2013
21:39 PM
reosarevok
ruaok: lol. Ok.
2013-02-07 03811, 2013
21:40 PM
reosarevok
ruaok: we do have "Your profile has been updated. We have sent you a verification email to exampleish [at exampleish.co.uk.] Please check your mailbox and click on the link in the email to verify the new email address" which we tell users when they add an email through account/edit
2013-02-07 03834, 2013
21:40 PM
reosarevok
Buuuut it seems we don't mention anything like that from /register
2013-02-07 03836, 2013
21:40 PM
djce sees irc flashing; stirs from retirement. zzzz ;-)
2013-02-07 03837, 2013
21:40 PM
ruaok
ya, but stupid users can outwit us there. :)
2013-02-07 03846, 2013
21:40 PM
ruaok
djce: hi. bai. zzz
2013-02-07 03801, 2013
21:41 PM
ianmcorvidae
heh
2013-02-07 03804, 2013
21:42 PM
ianmcorvidae
dhess: okay, yeah, so it's failing to load the release group, as I thought
2013-02-07 03848, 2013
21:42 PM
ianmcorvidae
not sure why not though -- I'd guess the collate extension thing though, it may dislike you using a different version of gcc than was used to compile postgres?
2013-02-07 03828, 2013
21:43 PM
dhess
ianmcorvidae: how can you tell it's failing to load the release group?
2013-02-07 03812, 2013
21:44 PM
dhess
ianmcorvidae: oh, I suppose because the release is defined
2013-02-07 03814, 2013
21:44 PM
ianmcorvidae
dhess: well, it tries to the line above that one, but there's no release group in that output to STDERR, just the release group ID (which is always loaded, since that's in the release table)
2013-02-07 03832, 2013
21:44 PM
ianmcorvidae
basically because 'release_group_id' => 352638, is the only reference to a release group
2013-02-07 03847, 2013
21:45 PM
dhess
ianmcorvidae: sorry, just to be sure I made that hack correctly, you wanted me to do this, right?
2013-02-07 03850, 2013
21:45 PM
dhess
(small flood coming)
2013-02-07 03851, 2013
21:45 PM
dhess
$c->model('ReleaseGroup')->load($release);
2013-02-07 03854, 2013
21:45 PM
dhess
use Data::Dumper qw( Dumper );
2013-02-07 03857, 2013
21:45 PM
dhess
warn Dumper($release);
2013-02-07 03800, 2013
21:46 PM
dhess
$c->model('ReleaseGroup')->load_meta($release->release_group);
2013-02-07 03803, 2013
21:46 PM
dhess
2013-02-07 03826, 2013
21:46 PM
ianmcorvidae
yeah, that's right
2013-02-07 03842, 2013
21:46 PM
ianmcorvidae
that last line there is the one that's failing, and it's saying that it's getting no object
2013-02-07 03846, 2013
21:46 PM
dhess
right
2013-02-07 03801, 2013
21:47 PM
ianmcorvidae
which would be because the release group failed to load from $c->model('ReleaseGroup')->load($release); (which is what would ordinarily do that)
2013-02-07 03810, 2013
21:47 PM
dhess
ianmcorvidae: so if the collate extension is the problem, assuming I recompile, do I need to drop the db and re-import?
2013-02-07 03820, 2013
21:47 PM
dhess
or is the collation done at runtime?
2013-02-07 03843, 2013
21:47 PM
ianmcorvidae
uh... I *think* you could maybe do it without needing to reimport, I'm not sure about that
2013-02-07 03842, 2013
21:48 PM
dhess
ianmcorvidae: BTW, here is the error I get when compiling with gcc-4.4 on Debian sid:
2013-02-07 03849, 2013
21:48 PM
dhess
$ CC=gcc-4.4 make -e
2013-02-07 03850, 2013
21:48 PM
dhess
-I/usr/include/tcl8.5 -c -o musicbrainz_collate.o musicbrainz_collate.c
2013-02-07 03853, 2013
21:48 PM
dhess
cc1: error: unrecognized command line option "-fexcess-precision=standard"
2013-02-07 03856, 2013
21:48 PM
dhess
make: *** [musicbrainz_collate.o] Error 1
2013-02-07 03859, 2013
21:48 PM
dhess
2013-02-07 03817, 2013
21:49 PM
dhess
which is odd
2013-02-07 03818, 2013
21:51 PM
ianmcorvidae
yeah
2013-02-07 03819, 2013
21:51 PM
dhess
I doubt Debian compiled postgresql-9.1 in sid with an old version of gcc in any case. What's the issue with the collate extension and gcc-4.6?
2013-02-07 03850, 2013
21:51 PM
ianmcorvidae
I don't know of any particular issue, I just assume it should be done with the same version as the postgres/icu packages
2013-02-07 03805, 2013
21:52 PM
ianmcorvidae
mostly I'm not sure exactly what's going on, heh
2013-02-07 03801, 2013
21:53 PM
dhess
ianmcorvidae: it's easy enough for me to rebuild this MusicBrainz server VM from scratch. Which version of Ubuntu are you guys using on production servers?
2013-02-07 03835, 2013
21:53 PM
dhess
No particular reason I need to use Debian sid
2013-02-07 03815, 2013
21:54 PM
ianmcorvidae
we're actually on 10.04, heh, wich a hand-compiled postgres -- I'd generally recommend using 12.04 though, there's also a PPA that will have a lot of the dependencies and things
2013-02-07 03829, 2013
21:54 PM
dhess
a mb server ppa?
2013-02-07 03828, 2013
21:55 PM
dhess
2013-02-07 03838, 2013
21:55 PM
ianmcorvidae
no, not sure what that one is
2013-02-07 03859, 2013
21:55 PM
dhess
that's luks's, presumably :)
2013-02-07 03816, 2013
21:56 PM
ianmcorvidae
ah there it is
2013-02-07 03817, 2013
21:56 PM
ianmcorvidae
2013-02-07 03838, 2013
21:56 PM
dhess
ok I'll use that
2013-02-07 03839, 2013
21:56 PM
ianmcorvidae
2013-02-07 03806, 2013
21:57 PM
dhess
I tried the ova, had problems getting nginx going with it
2013-02-07 03810, 2013
21:57 PM
dhess
decided I'd just build my own
2013-02-07 03815, 2013
21:57 PM
dhess
so much for that :)
2013-02-07 03852, 2013
21:57 PM
dhess
Anyway I'll try with 12.04. Do you recommend I use psql-9.1 or 8.x with that?
2013-02-07 03811, 2013
21:58 PM
ianmcorvidae
use 9.x whenever possible, 8.x is probably going to stop being supported soon
2013-02-07 03821, 2013
21:58 PM
dhess
ok cool
2013-02-07 03824, 2013
21:58 PM
dhess
thanks for your help!
2013-02-07 03833, 2013
21:58 PM
ianmcorvidae
sure, good luck getting it working :)
2013-02-07 03852, 2013
21:58 PM
dhess
cya
2013-02-07 03845, 2013
22:05 PM
ocharles returns
2013-02-07 03856, 2013
22:05 PM
ianmcorvidae
hello
2013-02-07 03820, 2013
22:07 PM
MBJenkins
2013-02-07 03807, 2013
22:09 PM
MBJenkins
Yippie, build fixed!
2013-02-07 03808, 2013
22:09 PM
MBJenkins
2013-02-07 03826, 2013
22:21 PM
ocharles
woo
2013-02-07 03809, 2013
22:38 PM
Ben\Sput joined the channel
2013-02-07 03813, 2013
22:54 PM
Ben\Sput
2013-02-07 03817, 2013
22:54 PM
Ben\Sput
all code on github :)
2013-02-07 03833, 2013
22:54 PM
hawke_1
Ben\Sput: did you delete the bad images?
2013-02-07 03836, 2013
22:54 PM
Ben\Sput
and all source on the downloads page updated
2013-02-07 03841, 2013
22:54 PM
Ben\Sput
hawke_1: no, not yet
2013-02-07 03859, 2013
22:54 PM
Ben\Sput
hawke_1: however, if just the thumbnails are bad, rescanning the audio will automatically fix them
2013-02-07 03836, 2013
22:55 PM
Ben\Sput
oh, hawke_1: haven't pulled your mutagen.File code yet though
2013-02-07 03859, 2013
22:55 PM
Ben\Sput
you can probably cut/paste it into the new scanner.py for now though
2013-02-07 03834, 2013
22:56 PM
Ben\Sput
and yeah, a lot of yours are just corrupt thumbnails, so rescanning will fix them :)
2013-02-07 03800, 2013
22:59 PM
hawke_1
Ben\Sput: I’m just running off my own fork anyway.
2013-02-07 03812, 2013
22:59 PM
hawke_1
Looks like it can’t find waveplotimager though
2013-02-07 03852, 2013
22:59 PM
hawke_1
oh, duh
2013-02-07 03856, 2013
22:59 PM
hawke_1
I didn’t copy it there.
2013-02-07 03806, 2013
23:00 PM
Ben\Sput
:P
2013-02-07 03836, 2013
23:00 PM
Ben\Sput
it looks in same directory now, not subdirs
2013-02-07 03838, 2013
23:00 PM
hawke_1
Should probably error properly instead of giving a traceback though
2013-02-07 03849, 2013
23:00 PM
Ben\Sput
if it doesn't find it, it uses path
2013-02-07 03806, 2013
23:01 PM
Ben\Sput
yeah i was going to catch that, but forgot
2013-02-07 03820, 2013
23:02 PM
hawke_1 attempts another scan
2013-02-07 03858, 2013
23:02 PM
hawke_1
would it be feasible to have it check for the existence of the waveplot before calculating it?
2013-02-07 03803, 2013
23:03 PM
hawke_1
I assume that’s the time-consuming bit
2013-02-07 03801, 2013
23:04 PM
Ben\Sput
i was acta
2013-02-07 03806, 2013
23:04 PM
Ben\Sput
...
2013-02-07 03812, 2013
23:04 PM
Ben\Sput
i was actually just thinking about that
2013-02-07 03828, 2013
23:04 PM
Ben\Sput
it's quite difficult, because you can't know if it exists until you've calculated it
2013-02-07 03845, 2013
23:04 PM
Ben\Sput
once we have track ids, I could see if there's one already there
2013-02-07 03800, 2013
23:05 PM
hawke_1
How’s that?
2013-02-07 03804, 2013
23:05 PM
Ben\Sput
I could also generate thumbnails first, and check the larger thumbnail, which may be quicker
2013-02-07 03819, 2013
23:05 PM
Ben\Sput
well, I'd get the track id, look it up on waveplot, and see if there's a plot attached
2013-02-07 03828, 2013
23:05 PM
hawke_1
yes, that seems right
2013-02-07 03835, 2013
23:05 PM
hawke_1
(aside from false/bad submissions)
2013-02-07 03836, 2013
23:05 PM
Ben\Sput
I could probably do a similar thing with release/disc/track now
2013-02-07 03839, 2013
23:05 PM
Ben\Sput
yeah
2013-02-07 03814, 2013
23:06 PM
Ben\Sput
hawke_1: although then we'd want to somehow mark that on the site, so it gets replaced by a future scan
2013-02-07 03827, 2013
23:06 PM
hawke_1
How would you know? :-p
2013-02-07 03802, 2013
23:07 PM
Ben\Sput
idk :P magic
2013-02-07 03834, 2013
23:07 PM
hawke_1
I think the only real way is similar to what acoustID does with submission counts
2013-02-07 03837, 2013
23:09 PM
Ben\Sput
yeah could do that
2013-02-07 03831, 2013
23:10 PM
Ben\Sput
it should probably be that it only counts submissions from each editor once though
2013-02-07 03827, 2013
23:11 PM
Ben\Sput
anyway, i'm off, is it uploading and everything now?
2013-02-07 03809, 2013
23:12 PM
Ben\Sput
also, when it says WavePlot already submitted... blah blah blah, it's also updating thumbnails then if they're corrupt
