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?
21:29 PM
luks
dhess: I bet you don't have the mbdump-derived tarball imported
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
21:30 PM
warp
ianmcorvidae: right, checking a publicly visible URL wouldn't need anything on the machine itself.
21:30 PM
misterswag_ joined the channel
21:31 PM
dhess
21:31 PM
ianmcorvidae
warp: I suppose things like critical processes matter too
21:31 PM
warp: making sure elasticsearch and the geordi FCGI script is running in other words
21:32 PM
warp
ianmcorvidae: is that running through daemontools?
21:33 PM
ianmcorvidae
warp: it's not, no
21:33 PM
it probably should be
21:34 PM
ruaok
this is from a user emailing support@
21:34 PM
21:34 PM
warp
usually daemontools keeps everything running, and stuff doesn't suddenly break as long as you don't touch it.
21:34 PM
ruaok
da fug?
21:35 PM
reosarevok
Indeed
21:35 PM
warp
ruaok: nice
21:35 PM
ruaok
not sure what to say.
21:35 PM
reosarevok
That sounds like a verification mail thing
21:35 PM
ruaok
he can log into his account.
21:35 PM
ya, that might be it.
21:35 PM
reosarevok
Well, it *somehow reminds me* of a verification mail thing :p
21:35 PM
"sounds like" is too kind
21:35 PM
But still
21:35 PM
ruaok
lol
21:36 PM
dhess
21:36 PM
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
21:36 PM
ianmcorvidae
dhess: it should be in the console where you started the server
21:36 PM
dhess
yeah ok
21:36 PM
ianmcorvidae
presumably you called 'plackup <etc>' somewhere, it'll be in that :)
21:36 PM
warp
I should get some sleep. goodnight #musicbrainz-devel!
21:37 PM
ruaok
nn warp
21:37 PM
reosarevok
21:37 PM
ruaok
on it, reosarevok
21:37 PM
reosarevok
Ok
21:37 PM
We really should write a template for this case
21:37 PM
MBJenkins
21:37 PM
warp
another HUGE SUCCESS
21:38 PM
reosarevok
although it feels like rewriting more or less the same we say when telling them about verification mails
21:38 PM
But I guess people don't read the site but do read answers to their mails :p
21:38 PM
ruaok
reosarevok: or make the UI more clear so we dont get these emails in the first palce
21:38 PM
dhess
just waiting for pastebin...
21:39 PM
screw it, I'll use pastebin.ca
21:39 PM
21:39 PM
reosarevok
ruaok: lol. Ok.
21:40 PM
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
21:40 PM
Buuuut it seems we don't mention anything like that from /register
21:40 PM
djce sees irc flashing; stirs from retirement. zzzz ;-)
21:40 PM
ruaok
ya, but stupid users can outwit us there. :)
21:40 PM
djce: hi. bai. zzz
21:41 PM
ianmcorvidae
heh
21:42 PM
dhess: okay, yeah, so it's failing to load the release group, as I thought
21:42 PM
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?
21:43 PM
dhess
ianmcorvidae: how can you tell it's failing to load the release group?
21:44 PM
ianmcorvidae: oh, I suppose because the release is defined
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)
21:44 PM
basically because 'release_group_id' => 352638, is the only reference to a release group
21:45 PM
dhess
ianmcorvidae: sorry, just to be sure I made that hack correctly, you wanted me to do this, right?
21:45 PM
(small flood coming)
21:45 PM
$c->model('ReleaseGroup')->load($release);
21:45 PM
use Data::Dumper qw( Dumper );
21:45 PM
warn Dumper($release);
21:46 PM
$c->model('ReleaseGroup')->load_meta($release->release_group);
21:46 PM
21:46 PM
ianmcorvidae
yeah, that's right
21:46 PM
that last line there is the one that's failing, and it's saying that it's getting no object
21:46 PM
dhess
right
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)
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?
21:47 PM
or is the collation done at runtime?
21:47 PM
ianmcorvidae
uh... I *think* you could maybe do it without needing to reimport, I'm not sure about that
21:48 PM
dhess
ianmcorvidae: BTW, here is the error I get when compiling with gcc-4.4 on Debian sid:
21:48 PM
$ CC=gcc-4.4 make -e
21:48 PM
-I/usr/include/tcl8.5 -c -o musicbrainz_collate.o musicbrainz_collate.c
21:48 PM
cc1: error: unrecognized command line option "-fexcess-precision=standard"
21:48 PM
make: *** [musicbrainz_collate.o] Error 1
21:48 PM
21:49 PM
which is odd
21:51 PM
ianmcorvidae
yeah
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?
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
21:52 PM
mostly I'm not sure exactly what's going on, heh
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?
21:53 PM
No particular reason I need to use Debian sid
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
21:54 PM
dhess
a mb server ppa?
21:55 PM
21:55 PM
ianmcorvidae
no, not sure what that one is
21:55 PM
dhess
that's luks's, presumably :)
21:56 PM
ianmcorvidae
ah there it is
21:56 PM
21:56 PM
dhess
ok I'll use that
21:56 PM
ianmcorvidae
21:57 PM
dhess
I tried the ova, had problems getting nginx going with it
21:57 PM
decided I'd just build my own
21:57 PM
so much for that :)
21:57 PM
Anyway I'll try with 12.04. Do you recommend I use psql-9.1 or 8.x with that?
21:58 PM
ianmcorvidae
use 9.x whenever possible, 8.x is probably going to stop being supported soon
21:58 PM
dhess
ok cool
21:58 PM
thanks for your help!
21:58 PM
ianmcorvidae
sure, good luck getting it working :)
21:58 PM
dhess
cya
22:05 PM
ocharles returns
22:05 PM
ianmcorvidae
hello
22:07 PM
MBJenkins
22:09 PM
Yippie, build fixed!
22:09 PM
22:21 PM
ocharles
woo
22:38 PM
Ben\Sput joined the channel
22:54 PM
Ben\Sput
22:54 PM
all code on github :)
22:54 PM
hawke_1
Ben\Sput: did you delete the bad images?
22:54 PM
Ben\Sput
and all source on the downloads page updated
22:54 PM
hawke_1: no, not yet
22:54 PM
hawke_1: however, if just the thumbnails are bad, rescanning the audio will automatically fix them
22:55 PM
oh, hawke_1: haven't pulled your mutagen.File code yet though
22:55 PM
you can probably cut/paste it into the new scanner.py for now though
22:56 PM
and yeah, a lot of yours are just corrupt thumbnails, so rescanning will fix them :)
22:59 PM
hawke_1
Ben\Sput: I’m just running off my own fork anyway.
22:59 PM
Looks like it can’t find waveplotimager though
22:59 PM
oh, duh
22:59 PM
I didn’t copy it there.
23:00 PM
Ben\Sput
:P
23:00 PM
it looks in same directory now, not subdirs
23:00 PM
hawke_1
Should probably error properly instead of giving a traceback though
23:00 PM
Ben\Sput
if it doesn't find it, it uses path
23:01 PM
yeah i was going to catch that, but forgot
23:02 PM
hawke_1 attempts another scan
23:02 PM
hawke_1
would it be feasible to have it check for the existence of the waveplot before calculating it?
23:03 PM
I assume that’s the time-consuming bit
23:04 PM
Ben\Sput
i was acta
23:04 PM
...
23:04 PM
i was actually just thinking about that
23:04 PM
it's quite difficult, because you can't know if it exists until you've calculated it
23:04 PM
once we have track ids, I could see if there's one already there
23:05 PM
hawke_1
How’s that?
23:05 PM
Ben\Sput
I could also generate thumbnails first, and check the larger thumbnail, which may be quicker
23:05 PM
well, I'd get the track id, look it up on waveplot, and see if there's a plot attached
23:05 PM
hawke_1
yes, that seems right
23:05 PM
(aside from false/bad submissions)
23:05 PM
Ben\Sput
I could probably do a similar thing with release/disc/track now
23:05 PM
yeah
23:06 PM
hawke_1: although then we'd want to somehow mark that on the site, so it gets replaced by a future scan
23:06 PM
hawke_1
How would you know? :-p
23:07 PM
Ben\Sput
idk :P magic
23:07 PM
hawke_1
I think the only real way is similar to what acoustID does with submission counts
23:09 PM
Ben\Sput
yeah could do that
23:10 PM
it should probably be that it only counts submissions from each editor once though
23:11 PM
anyway, i'm off, is it uploading and everything now?
23:12 PM
also, when it says WavePlot already submitted... blah blah blah, it's also updating thumbnails then if they're corrupt