#metabrainz

/

      • spideyonthego
        stanislas: Yes, I ran that first. And it worked perfectly fine.
      • 2015-12-21 35533, 2015

      • stanislas
        spideyonthego: And now it doesn't ?
      • 2015-12-21 35516, 2015

      • spideyonthego
        stanislas: No...still it does.
      • 2015-12-21 35534, 2015

      • stanislas
        spideyonthego: :)
      • 2015-12-21 35540, 2015

      • spideyonthego
        stanislas: But the second command doesn't.
      • 2015-12-21 35541, 2015

      • stanislas
        spideyonthego: which second one ? python run.py ?
      • 2015-12-21 35525, 2015

      • spideyonthego
        stansislas: Yup.
      • 2015-12-21 35551, 2015

      • stanislas
        and the logs ?
      • 2015-12-21 35553, 2015

      • Gentlecat
        spideyonthego: apt-get install works now?
      • 2015-12-21 35555, 2015

      • opatel99
        gcilou_: Why cant you just do $opts{user}->name;
      • 2015-12-21 35503, 2015

      • stanislas
        Gentlecat: It does
      • 2015-12-21 35513, 2015

      • opatel99
        oh there it is
      • 2015-12-21 35514, 2015

      • spideyonthego
        stanislas: I guess that's because, during the vm provisioning there's an error.
      • 2015-12-21 35519, 2015

      • opatel99
        what line are you adding on gcilou_
      • 2015-12-21 35530, 2015

      • stanislas
        spideyonthego: Could you paste your logs ?
      • 2015-12-21 35532, 2015

      • spideyonthego
        stanislas, Gentlecat: No install still doesn't work. Update does.
      • 2015-12-21 35534, 2015

      • spideyonthego
        Okay.
      • 2015-12-21 35553, 2015

      • gcilou_
        opatel99: yeah, that's what I'm doing. line 144 of email.pm
      • 2015-12-21 35556, 2015

      • Gentlecat
        then you need to figure this out first
      • 2015-12-21 35519, 2015

      • Gentlecat
      • 2015-12-21 35541, 2015

      • Gentlecat
        I've never seen this one before, so can't help much
      • 2015-12-21 35525, 2015

      • gcilou_
        opatel99: So do you think that's all that's needed to add the username?
      • 2015-12-21 35541, 2015

      • reosarevok
      • 2015-12-21 35546, 2015

      • opatel99
        gcilou_: Yeah thats it
      • 2015-12-21 35559, 2015

      • opatel99
        gcilou_: You dont have to put the injected var in brackets
      • 2015-12-21 35510, 2015

      • opatel99
      • 2015-12-21 35506, 2015

      • reosarevok
        opatel99: my doubt was whether that already has user in opts or it needs to be added elsewhere :)
      • 2015-12-21 35515, 2015

      • spideyonthego
        Gentlecat: That still doesn't work.
      • 2015-12-21 35519, 2015

      • gcilou_
        opatel99: OK, so just "Hello $userid"?
      • 2015-12-21 35529, 2015

      • gcilou_
        reosarevok: what is opts btw?
      • 2015-12-21 35536, 2015

      • spideyonthego
        stanislas, Gentlecat: I'll give the logs as soon as the provisioning is done. :)
      • 2015-12-21 35553, 2015

      • opatel99
        gcilou_: short for options. Like extra vars provided to the function
      • 2015-12-21 35554, 2015

      • reosarevok
        gcilou_: that was my main doubt about the whole thing :D
      • 2015-12-21 35557, 2015

      • reosarevok is not very good
      • 2015-12-21 35509, 2015

      • reosarevok
      • 2015-12-21 35515, 2015

      • Leftmost
        I'm sort of half-around for a few hours, but I won't be very responsive. Today is a travel day for me.
      • 2015-12-21 35525, 2015

      • opatel99
      • 2015-12-21 35526, 2015

      • reosarevok
        (and if so that doesn't seem to send an editor object)
      • 2015-12-21 35529, 2015

      • spideyonthego has quit
      • 2015-12-21 35537, 2015

      • opatel99
        oh wait thats not it
      • 2015-12-21 35550, 2015

      • Leftmost
        stanislas, https://github.com/bookbrainz/bookbrainz-site/blo… is the relevant chunk of code. So (I think) the answer is yes.
      • 2015-12-21 35540, 2015

      • opatel99
        ohhh I see what you mean. Yeah gcilou_ Gotta add the parameter here https://github.com/metabrainz/musicbrainz-server/… and update all the other function calls
      • 2015-12-21 35552, 2015

      • opatel99
        Leftmost: ! Did you get a chance to check out the new select2 css?
      • 2015-12-21 35521, 2015

      • stanislas
        Leftmost: And how can I do that ?
      • 2015-12-21 35537, 2015

      • stanislas
        (using PUT)
      • 2015-12-21 35547, 2015

      • opatel99
        nvm. Travel day for you. Enjoy your trip!
      • 2015-12-21 35509, 2015

      • gcilou_
        opatel99: So add a param for user?
      • 2015-12-21 35511, 2015

      • opatel99
        hmm I think $c->request->params->{userid}; would be fine to finish up on line 531
      • 2015-12-21 35541, 2015

      • opatel99
        Likeso : $user_id = $c->request->params->{userid};
      • 2015-12-21 35554, 2015

      • opatel99
        Because the request has that parameter built in
      • 2015-12-21 35517, 2015

      • gcilou_
        Yeah, I was wondering about that
      • 2015-12-21 35513, 2015

      • ruaok
        Leftmost: where are you off to?
      • 2015-12-21 35526, 2015

      • ruaok waves from Denver airport
      • 2015-12-21 35529, 2015

      • opatel99
        Okay final. 2 files. Both one liners. You have one resolved. Account.pm just requires you to add that snippet to the function call
      • 2015-12-21 35544, 2015

      • opatel99
        waves from New Orleans with Family!
      • 2015-12-21 35547, 2015

      • gcilou_
        opatel99, so why the $ before user_id, and = instead of =>?
      • 2015-12-21 35508, 2015

      • opatel99
        gcilou_: Adjust as needed to fit the structure
      • 2015-12-21 35520, 2015

      • gcilou_
        alright awesome, thanks
      • 2015-12-21 35520, 2015

      • opatel99
        I was just roughly typing that
      • 2015-12-21 35507, 2015

      • stanislas
        Leftmost: Have you had a chance to look at my code ? Are there any important things I should test, which are not covered yet ?
      • 2015-12-21 35555, 2015

      • gcilou_
        opatel99, so " my $userid = $opts{user}->name;" is what's needed in email.pm + the "Hello $userid" ?
      • 2015-12-21 35551, 2015

      • gcilou_
        and then "user_id => $c->request->params->{userid}" on 530 in account.pm ?
      • 2015-12-21 35526, 2015

      • JonnyJD has quit
      • 2015-12-21 35545, 2015

      • opatel99
        have the opts match.
      • 2015-12-21 35550, 2015

      • opatel99
        opts{user_id
      • 2015-12-21 35509, 2015

      • opatel99
        the variables have to be the same...
      • 2015-12-21 35557, 2015

      • gcilou_
        so all the places I have userid need to be user_id?
      • 2015-12-21 35514, 2015

      • opatel99
        Yes
      • 2015-12-21 35552, 2015

      • opatel99
        I should give this advice with a disclaimer. No guarantee this will work. I do believe we are on the right track, however
      • 2015-12-21 35549, 2015

      • gcilou_
        my $user_id = $opts{user_id}->name; and Hello $user_id, on email.pm ; and user_id => $c->request->params->{userid} on account.pm
      • 2015-12-21 35522, 2015

      • opatel99
        Thats it ^
      • 2015-12-21 35536, 2015

      • opatel99
        idk if you can try it without access to the mail server
      • 2015-12-21 35557, 2015

      • opatel99
        But go for it blind.
      • 2015-12-21 35503, 2015

      • opatel99
        LordSputnik: Are you busy?
      • 2015-12-21 35511, 2015

      • gcilou_
        Alright let's do this
      • 2015-12-21 35532, 2015

      • Leftmost
        ruaok, ha. I'll be there in about seven hours. I'm headed to Atlanta for the end-of-year festivities.
      • 2015-12-21 35518, 2015

      • opatel99
        Leftmost: How long will you be in Atlanta? We are going to pass right through there day after tomorrow
      • 2015-12-21 35555, 2015

      • ruaok
        Leftmost: heh. Good timing as always.
      • 2015-12-21 35518, 2015

      • Leftmost
        opatel99 and stanislas, haven't had a chance for review. Sorry, the last couple of days have been all go. Should hopefully settle down a bit once I'm at the airport.
      • 2015-12-21 35558, 2015

      • Leftmost
        opatel99, about two weeks. I have family down that way.
      • 2015-12-21 35512, 2015

      • Leftmost
        ruaok, my timing is nothing if not impeccable.
      • 2015-12-21 35526, 2015

      • stanislas
        Leftmost: ok, I will ask LordSputnik then
      • 2015-12-21 35540, 2015

      • Leftmost
        stanislas, you should be able to create a python object containing the property gender_id with the correct value and associate it with the gender property in the PUT data to get the right result.
      • 2015-12-21 35508, 2015

      • stanislas
        Leftmost: I am passing it as a string now in JSON form.
      • 2015-12-21 35538, 2015

      • spideyonthego joined the channel
      • 2015-12-21 35517, 2015

      • stanislas
        Leftmost: I can't pass it as an object, maybe "gender" : {"gender_id":2} would work ?
      • 2015-12-21 35512, 2015

      • spideyonthego
        Gentlecat, stanislas: http://pastebin.com/f6fb9D6e
      • 2015-12-21 35552, 2015

      • Leftmost
        Possibly; give it a shot. Sorry I'm not being very helpful. Didn't get enough sleep to be writing python.
      • 2015-12-21 35558, 2015

      • Gentlecat
        spideyonthego: same error
      • 2015-12-21 35502, 2015

      • CJ_
        This place got busy.
      • 2015-12-21 35505, 2015

      • spideyonthego
        Gentlecat: Yeah
      • 2015-12-21 35519, 2015

      • stanislas
        spideyonthego: Are you using some proxy or something ?
      • 2015-12-21 35535, 2015

      • Leftmost
        CJ_, for what it's worth, the MB data expands out to a little over 20 gigs, as I recall. (Significantly less if you don't add in edit data.)
      • 2015-12-21 35535, 2015

      • spideyonthego
        stanislas: Nope
      • 2015-12-21 35520, 2015

      • CJ_
        Leftmost, Understood. Did you read the rest of the conversation about search?
      • 2015-12-21 35530, 2015

      • Gentlecat
      • 2015-12-21 35507, 2015

      • Leftmost
        Skimmed it, saw my name, felt important. ;)
      • 2015-12-21 35515, 2015

      • CJ_
        :D
      • 2015-12-21 35527, 2015

      • Leftmost
        The plan is to migrate to Solr, and I am going to be picking that up, but I haven't had a chance to dig in yet.
      • 2015-12-21 35542, 2015

      • CJ_
        I am a java developer and familiar with solr. I am looking for the best place to get started helping.
      • 2015-12-21 35522, 2015

      • Leftmost
        The repo that Mineo linked is the best place to get started, probably, though that may be a lot to sift. Unfortunately, having not dug in myself (and not being directly familiar with Solr), I can't offer much guidance at the moment.
      • 2015-12-21 35544, 2015

      • CJ_
        I just want to make sure I don't step on anyones toes or waste my time on unneeded things.
      • 2015-12-21 35547, 2015

      • opatel99
        LordSputnik: Got your comment. Getting it done
      • 2015-12-21 35559, 2015

      • stanislas
        spideyonthego: after changing /etc/apt/sources.list you should do apt-get update, to make it work correctly
      • 2015-12-21 35519, 2015

      • Leftmost
        Step away! Honestly, anything you can learn/do would be valuable, especially if you can teach me some stuff (when I'm a little less harried :)).
      • 2015-12-21 35540, 2015

      • ruaok
        CJ_: you could do us a giant favor and review our existing search servlet code.
      • 2015-12-21 35520, 2015

      • CJ_
        ruaok, That was my initial plan until I got sidetracked with the solr efforts. :)
      • 2015-12-21 35530, 2015

      • ruaok
        We keep having theses crazy lockups and could use an experienced java dev to help sanity check the code.
      • 2015-12-21 35538, 2015

      • CJ_
        I was trying to free up enough space to run the VM.
      • 2015-12-21 35500, 2015

      • reosarevok
        if it helps I guess we could throw you a VM in one of our servers?
      • 2015-12-21 35517, 2015

      • reosarevok is not volunteering to set that up, but :p
      • 2015-12-21 35534, 2015

      • CJ_
        I have not worked with lucene directly in ages. Most of my time has been spent with solr.
      • 2015-12-21 35538, 2015

      • ruaok
        I'd hate to take focus away from the solr stuff, but this would have much more immediate positive benefits on site stability.
      • 2015-12-21 35555, 2015

      • reosarevok
        Well unless you finish the solr stuff in a week
      • 2015-12-21 35559, 2015

      • reosarevok
        ;p
      • 2015-12-21 35501, 2015

      • Mineo
      • 2015-12-21 35507, 2015

      • ruaok
        Yes, I'd be happy to create a VM for you.
      • 2015-12-21 35513, 2015

      • ruaok
        Mineo: thanks. We can build a new jar and see if it helps.
      • 2015-12-21 35537, 2015

      • opatel99
        Mineo: Any more picard tasks?
      • 2015-12-21 35540, 2015

      • reosarevok
        ruaok: please look into that Bronze mail or let me know what to tell them :) I wouldn't like to leave a paying dude waiting too long
      • 2015-12-21 35552, 2015

      • CJ_
        ruaok, Looking at the existing code would probably be helpful so that I can understand how things work. So I don't think it would be a bad thing.
      • 2015-12-21 35554, 2015

      • reosarevok
        (but if you're traveling, then no need to do it *now* - just dunno if you saw my earlier ping)
      • 2015-12-21 35509, 2015

      • ruaok
        As soon as I have a spare moment, I will reosarevok .
      • 2015-12-21 35516, 2015

      • reosarevok
        :)
      • 2015-12-21 35533, 2015

      • CJ_
        ruaok, If you have a test server that I can remote into and poke at I would be happy to take a look.
      • 2015-12-21 35538, 2015

      • Mineo
        opatel99: not yet, no
      • 2015-12-21 35547, 2015

      • opatel99
        Okay
      • 2015-12-21 35549, 2015

      • ruaok
        Cj_: tell me the specs on a VM you'd like.
      • 2015-12-21 35555, 2015

      • ariscop joined the channel
      • 2015-12-21 35529, 2015

      • Mineo
        opatel99: I wanted to create one to improve the scripting documentation that a lot of people struggle with, but I don't yet have a clear idea of what's actually missing there
      • 2015-12-21 35509, 2015

      • opatel99
        Mineo: What error do you get when you currently try to cluster?
      • 2015-12-21 35525, 2015

      • CJ_
        ruaok, I have no idea. Whatever would work best to work with MB.
      • 2015-12-21 35535, 2015

      • spideyonthego
        Gentlecat, stanislas: I know this might seem silly, but where is the sources.list file located in my machine?
      • 2015-12-21 35500, 2015

      • stanislas
        in /etc/apt/sources.list i think
      • 2015-12-21 35504, 2015

      • Gentlecat
      • 2015-12-21 35536, 2015

      • stanislas
        spideyonthego: remember to edit it with sudo, otherwis it won't work
      • 2015-12-21 35552, 2015

      • CJ_
        stanislas, sudo !! :)
      • 2015-12-21 35514, 2015

      • Mineo
        actually, sudo -e
      • 2015-12-21 35546, 2015

      • stanislas
        Mineo: Not really, I've always done that using sudo vim :)
      • 2015-12-21 35556, 2015

      • Mineo
        opatel99: I think none, but the code is written in a way that makes errors likely to happen
      • 2015-12-21 35557, 2015

      • Gentlecat
      • 2015-12-21 35544, 2015

      • CJ_
        Mineo, Interesting. Never used that flag. But !! is simply argument substitution.
      • 2015-12-21 35556, 2015

      • ruaok
        Zas: can you please make an appropriate VM on Google cloud hosting for cj_?
      • 2015-12-21 35523, 2015

      • ruaok
        Freso: I'm traveling today. I'm hoping to join the meeting, but you should lead it.
      • 2015-12-21 35529, 2015

      • ruaok
        Please.
      • 2015-12-21 35506, 2015

      • ruaok will know in a second if he can join.
      • 2015-12-21 35550, 2015

      • CJ_
        Meetings are not held in irc?
      • 2015-12-21 35558, 2015

      • ruaok
        Zas, you here?
      • 2015-12-21 35515, 2015

      • zas
        yes
      • 2015-12-21 35527, 2015

      • ruaok fixes search