#musicbrainz-devel

/

      • warp
        two my $sizes
      • 2013-07-18 19947, 2013

      • warp
        I guess I can just remove the second 'my'
      • 2013-07-18 19925, 2013

      • ocharles
        I'll read it as 'let urlSize = maybe mempty ('-' : ) size' and it's less scary
      • 2013-07-18 19930, 2013

      • ocharles
        :P
      • 2013-07-18 19946, 2013

      • warp
        oh, I'll name it urlsize then. slightly less confusing.
      • 2013-07-18 19904, 2013

      • ocharles
        or just sprintf to form the url
      • 2013-07-18 19935, 2013

      • ocharles
        then you get: sprintf "http://ueauea/mbid-%s%s.%s", $mbid, defined($size) ? .., $suffix
      • 2013-07-18 19945, 2013

      • ocharles
        which also requires less munging
      • 2013-07-18 19922, 2013

      • warp
        I agree that's prettier.
      • 2013-07-18 19900, 2013

      • warp
        (but I pushed the urlsize variant)
      • 2013-07-18 19946, 2013

      • warp
        ocharles: can you check https://github.com/metabrainz/CAA-indexer/pull/6/… ? and if you're happy with it we can merge that and redeploy.
      • 2013-07-18 19916, 2013

      • ocharles
        warp: i'm happy with that
      • 2013-07-18 19923, 2013

      • warp merges
      • 2013-07-18 19939, 2013

      • ocharles
        ruaok:
      • 2013-07-18 19940, 2013

      • ocharles
        [2013-07-18T10:16:17+00:00] INFO: Processing service[postgresql] action enable (postgresql::default line 20)
      • 2013-07-18 19943, 2013

      • ocharles
        [2013-07-18T10:16:17+00:00] INFO: Processing service[postgresql] action start (postgresql::default line 20)
      • 2013-07-18 19946, 2013

      • ocharles
        [2013-07-18T10:16:17+00:00] INFO: Processing service[postgresql] action reload (postgresql::default line 20)
      • 2013-07-18 19949, 2013

      • ocharles
        [2013-07-18T10:16:18+00:00] INFO: service[postgresql] reloaded
      • 2013-07-18 19950, 2013

      • ocharles
        which is indeed wrong
      • 2013-07-18 19952, 2013

      • ocharles
        it does appear that postgresql doesn't start, it only reloads
      • 2013-07-18 19915, 2013

      • ocharles
        hmm no, pg is running fine
      • 2013-07-18 19916, 2013

      • ocharles
        odd
      • 2013-07-18 19920, 2013

      • ruaok
        mine was running too
      • 2013-07-18 19952, 2013

      • warp
        hrm, I get no route to host trying to get to hobbes.
      • 2013-07-18 19931, 2013

      • ocharles
        warp: I did get a nagios that hobbes was unreachable recently
      • 2013-07-18 19939, 2013

      • ocharles
        evidently, that was not one to ignore
      • 2013-07-18 19953, 2013

      • ruaok
        should I call in a reboot?
      • 2013-07-18 19904, 2013

      • warp
        I didn't realize we already moved all the important bits off it, because I haven't seen anyone complaining here :)
      • 2013-07-18 19920, 2013

      • ruaok
        jesus kept complaining.
      • 2013-07-18 19935, 2013

      • reosarevok
        Isn't that always true? :p
      • 2013-07-18 19938, 2013

      • warp
        ruaok: yes, I cannot ssh into the machine from lenny and such.
      • 2013-07-18 19940, 2013

      • ruaok
        srsly
      • 2013-07-18 19959, 2013

      • ocharles
        no, jesus is complaining because i turned off test.mb.org
      • 2013-07-18 19903, 2013

      • ocharles
        when i upgraded to pg 9.1
      • 2013-07-18 19911, 2013

      • ocharles
        i wondered if anyone would notice :)
      • 2013-07-18 19927, 2013

      • ocharles
        I didn't intend to entirely nuke hobbes :)
      • 2013-07-18 19948, 2013

      • warp
        ocharles: hm, CAA-indexer doesn't have an origin, can I rename the github remote?
      • 2013-07-18 19907, 2013

      • ocharles
        yes please
      • 2013-07-18 19909, 2013

      • ocharles
        that confused me
      • 2013-07-18 19908, 2013

      • ruaok
        ok, reboot ordered, but apparently they are fighting some other fires too.
      • 2013-07-18 19915, 2013

      • ruaok
        ping me in 10 if its not back.
      • 2013-07-18 19923, 2013

      • warp
        ruaok: ok, there's no hurry. I don't need it for anything.
      • 2013-07-18 19927, 2013

      • ruaok
        k
      • 2013-07-18 19918, 2013

      • warp
        ocharles: caa-indexer updated and restarted
      • 2013-07-18 19929, 2013

      • ocharles
        warp: goodo
      • 2013-07-18 19934, 2013

      • ocharles
        try and index something with a png
      • 2013-07-18 19910, 2013

      • ocharles
        yay, my caa-indexer VM came up and has a rabbitmq with all the right queues and exchanges
      • 2013-07-18 19957, 2013

      • warp
      • 2013-07-18 19948, 2013

      • ocharles
        woohoo. and how does it look on mb.org?
      • 2013-07-18 19939, 2013

      • warp
        looks good afaics
      • 2013-07-18 19906, 2013

      • warp
        the "original" link on the third image of https://beta.musicbrainz.org/release/442b3814-b2c… is to a .png
      • 2013-07-18 19900, 2013

      • nikki
      • 2013-07-18 19906, 2013

      • Leftmost joined the channel
      • 2013-07-18 19928, 2013

      • reosarevok
        \o/
      • 2013-07-18 19930, 2013

      • ocharles
        firefox seems to be happy it is indeed a png
      • 2013-07-18 19901, 2013

      • ocharles
        that album looks awful, too
      • 2013-07-18 19902, 2013

      • ocharles
        :P
      • 2013-07-18 19942, 2013

      • warp
        alright, for reindexing already submitted .png/.gif images. can I just make a list, and munge that into a SELECT pgq.insert_event() statements in a .sql file, and source that on totoro?
      • 2013-07-18 19918, 2013

      • ocharles
        just run 'SELECT pgq.insert_event(..) FROM release JOIN whatever WHERE foo' and run that on totoro
      • 2013-07-18 19923, 2013

      • ocharles
        that call will run in a few ms
      • 2013-07-18 19928, 2013

      • ocharles
        and then the indexer will do its thing
      • 2013-07-18 19954, 2013

      • warp
        ah, ok
      • 2013-07-18 19913, 2013

      • warp
        oh it's only 43 releases.
      • 2013-07-18 19920, 2013

      • nikki
        yeah, I discouraged people from going nuts because of the thumbnails and the fact the images don't work at all on the main servers right now :P
      • 2013-07-18 19936, 2013

      • warp
        done
      • 2013-07-18 19905, 2013

      • warp
      • 2013-07-18 19940, 2013

      • djce joined the channel
      • 2013-07-18 19901, 2013

      • warp
        ocharles: shall I merge the trigger changes into master now? (considering that has already been deployed to production).
      • 2013-07-18 19949, 2013

      • ocharles
        warp: yea, master or beta
      • 2013-07-18 19928, 2013

      • warp
        oh yes, it has to beta, it is branched off of beta.
      • 2013-07-18 19956, 2013

      • ruaok joined the channel
      • 2013-07-18 19957, 2013

      • ruaok joined the channel
      • 2013-07-18 19913, 2013

      • ruaok
        gah.
      • 2013-07-18 19922, 2013

      • ruaok
        I think the AC in my office is making me ill.
      • 2013-07-18 19935, 2013

      • ruaok
        went to office, starting feeling bad. went home, feeling better.
      • 2013-07-18 19955, 2013

      • ocharles
        i'm not feeling too perky today
      • 2013-07-18 19917, 2013

      • ocharles
        at least you have AC though, the lack of that is why i feel off
      • 2013-07-18 19931, 2013

      • ruaok
        well, when I've sat in the office without AC, I was fine.
      • 2013-07-18 19935, 2013

      • ruaok
        so, no I have no AC. :(
      • 2013-07-18 19952, 2013

      • ruaok
        which may drive me to rent an office here, outside of MTG.
      • 2013-07-18 19906, 2013

      • ocharles
        you don't think you will adapt to the ac then?
      • 2013-07-18 19911, 2013

      • outsidecontext joined the channel
      • 2013-07-18 19924, 2013

      • ruaok
        I have no problems with AC. the office in cali had great AC.
      • 2013-07-18 19932, 2013

      • ruaok
        I suspect that there is mold in the AC system here.
      • 2013-07-18 19942, 2013

      • ruaok
        and that everytime I turn it on, I get blasted with mold spores.
      • 2013-07-18 19946, 2013

      • ocharles
        i know
      • 2013-07-18 19951, 2013

      • ocharles
        i meant adapting to that bit
      • 2013-07-18 19907, 2013

      • ruaok
        not something I want to adapt to. mold can be really bad for you.
      • 2013-07-18 19910, 2013

      • ocharles nods
      • 2013-07-18 19929, 2013

      • ruaok
        and in the last working week, we've lost quite a bit of working time to being mysteriously ill.
      • 2013-07-18 19945, 2013

      • ruaok
        and I'm quite done with that, honestly.
      • 2013-07-18 19905, 2013

      • ruaok
        anyways, I'm back home where I have a working vagrant install. I'll keep working on it.
      • 2013-07-18 19906, 2013

      • ocharles
        understandably
      • 2013-07-18 19922, 2013

      • mjjc joined the channel
      • 2013-07-18 19941, 2013

      • andreypopp joined the channel
      • 2013-07-18 19944, 2013

      • ruaok
        ocharles: any idea on this one?
      • 2013-07-18 19945, 2013

      • ruaok
      • 2013-07-18 19924, 2013

      • ocharles
        Are you trying to run a bash script?
      • 2013-07-18 19932, 2013

      • ocharles
        I've seen this when a bash script fails to exit with 0
      • 2013-07-18 19946, 2013

      • ocharles
        it's an extremely useless stack trace, and I can't figure out why we get one like that
      • 2013-07-18 19907, 2013

      • ruaok
        while running vagaran provision
      • 2013-07-18 19918, 2013

      • ruaok
        *vagrant
      • 2013-07-18 19950, 2013

      • ocharles
        yea, but presumably you have a bash script in one of the recipes you're running
      • 2013-07-18 19946, 2013

      • ruaok
        Not really.
      • 2013-07-18 19900, 2013

      • ruaok
        I've been whittling my default.rb down to noting and finally got to this:
      • 2013-07-18 19901, 2013

      • ruaok
        package "openjdk-7-jre"
      • 2013-07-18 19904, 2013

      • ruaok
        and it still fails.
      • 2013-07-18 19943, 2013

      • ruaok
        and even when I remove chef.add_recipe "search-server" from Vagrantfile it still fails.
      • 2013-07-18 19948, 2013

      • ocharles
        paste your Vagrantfile
      • 2013-07-18 19956, 2013

      • ocharles
        the most minimal one you've got it to fail on
      • 2013-07-18 19920, 2013

      • ruaok
        ok, let me work on whittling that down.
      • 2013-07-18 19921, 2013

      • ocharles
        I think maybe you're using something that depends on an attribute that's not set
      • 2013-07-18 19939, 2013

      • ruaok
        I've undone all of my changes now, I think.
      • 2013-07-18 19937, 2013

      • ruaok
        this is the most minimal one that fails:
      • 2013-07-18 19938, 2013

      • ruaok
      • 2013-07-18 19905, 2013

      • ruaok
        if I remove "chef.add_recipe "virtualmachine"" then I get [default] Warning: Chef solo run list is empty. This may not be what you want.
      • 2013-07-18 19915, 2013

      • ocharles
        right
      • 2013-07-18 19910, 2013

      • nikki
        nikki has changed the topic to: agenda: questions on reviews, MBS-3841 (nikki), sortnames (nikki)
      • 2013-07-18 19918, 2013

      • ocharles
        you have a cookbooks/virtualmachine folder?
      • 2013-07-18 19902, 2013

      • ruaok
        yes with recipes/default.rb
      • 2013-07-18 19912, 2013

      • ocharles
        did you write that yourself? i can't see it on github
      • 2013-07-18 19918, 2013

      • ruaok
        no
      • 2013-07-18 19937, 2013

      • ruaok
        is this more of the submodules hell?
      • 2013-07-18 19904, 2013

      • ocharles
        what is git-status on vagrant-images
      • 2013-07-18 19910, 2013

      • ocharles
        and what is git rev-parse HEAD
      • 2013-07-18 19935, 2013

      • ruaok
        # Your branch is ahead of 'origin/master' by 28 commits.
      • 2013-07-18 19941, 2013

      • ruaok
        # modified: musicbrainz-server/Vagrantfile
      • 2013-07-18 19957, 2013

      • ruaok
        2834095
      • 2013-07-18 19900, 2013

      • ocharles
        ok, have you modified the cookbooks project at all?
      • 2013-07-18 19943, 2013

      • ocharles
        and I don't know what 2834095 is, so I guess you've commited
      • 2013-07-18 19949, 2013

      • ruaok
        yes, I added the search-server subdir
      • 2013-07-18 19955, 2013

      • ocharles
        ok
      • 2013-07-18 19909, 2013

      • ruaok
        yes, its a local commit on my forked repo
      • 2013-07-18 19909, 2013

      • ocharles
        go into the cookbooks directory, and git rev-parse HEAD there
      • 2013-07-18 19911, 2013

      • ocharles
        ok
      • 2013-07-18 19926, 2013

      • ocharles
        when did you make this commit?
      • 2013-07-18 19927, 2013

      • ruaok
        ef94b90f94b
      • 2013-07-18 19938, 2013

      • ocharles
        i've ripped vagrantimages apart quite a lot when i was getting ready to review
      • 2013-07-18 19944, 2013

      • ocharles
        so I think you may have branched off something that no longer exists
      • 2013-07-18 19955, 2013

      • ruaok
        yes, I've gotten that feeling.
      • 2013-07-18 19909, 2013

      • ruaok
        something has been quite odd.
      • 2013-07-18 19923, 2013

      • ruaok
        I'd start over again, but then I would have to wait to build the VM and import the data again.
      • 2013-07-18 19931, 2013

      • ruaok
        reluctant to do that, but if I have to...
      • 2013-07-18 19937, 2013

      • ocharles
        no, you don't need to do that
      • 2013-07-18 19941, 2013

      • ocharles
        we just need to get you on the right commits
      • 2013-07-18 19951, 2013

      • ruaok
        guide me obi-wan!
      • 2013-07-18 19957, 2013

      • ocharles
        first, lets do vagrant-images
      • 2013-07-18 19910, 2013

      • ocharles
        you have a single commit, or multiple commits?
      • 2013-07-18 19917, 2013

      • ocharles
        (of your own work)
      • 2013-07-18 19918, 2013

      • marcooliveira joined the channel