Monday, 2014-08-25

*** tpb has joined #melange00:00
*** rocker has quit IRC00:21
*** rocker has joined #melange00:23
*** samgtr has joined #melange00:29
*** ArneBab_ has joined #melange02:02
*** ArneBab has quit IRC02:05
*** tkamppeter has quit IRC02:16
*** tkamppeter has joined #melange02:19
*** samgtr has quit IRC02:36
*** ArneBab has joined #melange03:00
*** ArneBab_ has quit IRC03:03
*** madrazr has quit IRC03:10
*** saurabhsood91 has quit IRC03:41
*** AnujSharma has joined #melange04:27
*** madrazr has joined #melange04:52
*** ChanServ sets mode: +o madrazr04:52
*** robbyoconnor has quit IRC05:23
*** asmeurer__ has quit IRC05:46
*** madrazr has quit IRC06:03
*** raghunayyar has joined #melange06:38
*** madrazr has joined #melange07:04
*** ChanServ sets mode: +o madrazr07:04
*** madrazr has quit IRC07:04
*** cs_shadow|afk is now known as cs_shadow07:17
*** Niharika has joined #melange07:39
*** Niharika1 has joined #melange07:43
*** Niharika has quit IRC07:43
*** Niharika1 is now known as Niharika07:47
*** rigelk has joined #melange07:47
*** raghunayyar has quit IRC08:57
*** penguinRaider has quit IRC09:27
*** rocker has quit IRC09:27
*** cs_shadow has quit IRC09:27
*** cs_shadow has joined #melange09:33
*** cs_shadow has quit IRC09:37
*** cs_shadow has joined #melange09:43
*** rocker has joined #melange09:43
*** penguinRaider has joined #melange09:43
*** Niharika has quit IRC09:50
*** AnujSharma has quit IRC10:29
*** Abhishek_ has quit IRC10:40
*** Abhishek_ has joined #melange11:04
*** penguinRaider has quit IRC11:12
*** kblin has joined #melange11:52
kblinhi folks11:52
kblinshould I be able to see the mentor/student evals as org admin?11:52
Lenniekblin, yes11:55
kblinhey Lennie :)11:55
Lenniehttp://www.google-melange.com/gsoc/dashboard/google/gsoc2014#student_evaluations11:56
Lennie?11:56
Lennieshows up for me11:57
Lenniethe layout is screwed up though :D11:57
kblinbut the individual evals give "Requested project does not exist."11:57
Lennieworks for me11:57
Lenniekblin, is that for a project that failed finals?11:57
kblinah, yes11:58
kblinand I just tried some that succeeded, and those work fine as well11:58
kblinbut I figure if, as an org admin, I'm asked to look at an eval, it'll be one where the student wants to contest a "failed"11:59
kblinnot that that'd get anywhere, in our case I know the reasons without looking at the eval11:59
kblinbut for larger orgs that might be an issue11:59
Lenniekblin, weird though since it works for a project htat failed midterm but not finals12:00
Lennieplease file a bug :)12:00
* Lennie is off to a meeting right now12:01
*** Niharika has joined #melange12:18
kblinLennie: will do12:29
kblinhttps://code.google.com/p/soc/issues/detail?id=230812:38
tpbTitle: Issue 2308 - soc - Can't inspect final mentor evaluation for a failed student - SoC (Spice of Creation) - Google Project Hosting (at code.google.com)12:38
Lenniekblin, can you include a link to one of the evals that doesn't work?12:47
*** suranga has joined #melange12:47
Lenniekblin, apparently another bug exists: https://code.google.com/p/soc/issues/detail?id=225712:48
tpbTitle: Issue 2257 - soc - Org Admins are unable to view mentor evaluations after they are due? - SoC (Spice of Creation) - Google Project Hosting (at code.google.com)12:48
KolibriOS|yogevLennie: Yeah, I was ~14 hours faster :-)12:54
*** cs_shadow is now known as cs_shadow|afk12:54
*** cs_shadow|afk is now known as cs_shadow12:54
LennieMadhu should be able to look at it when California wakes up (and if California is still on the face of the planet :P)12:55
KolibriOS|yogevLennie: No electricity = no wake-up alarm :-)12:55
Lenniethen you might have to wait a tiny bit longer :)12:55
kblinI know that according to BBC news at least california is no longer a part of the USA12:56
kblinLennie: ah, my bad. should have searched more throughoutly13:00
Lenniekblin, I didn't see that bug either :)13:00
Lenniekblin, so that's okat13:00
Lennieokay13:00
Lennie(getting used to new keyboard spacing)13:00
kblinI didn't provide a URL for privacy reasons. after all it'll show the student's user id13:01
* kblin has mad "privacy officer of the institute" powers13:01
*** Abhishek_ has quit IRC13:40
*** Abhishek_ has joined #melange14:02
*** cosenal_ has joined #melange14:29
*** Uli-_ has joined #melange14:31
*** scorche|1h has joined #melange14:31
*** cosenal has quit IRC14:32
*** allenZ|afk has quit IRC14:32
*** melange-buildbot has quit IRC14:32
*** scorche|sh has quit IRC14:32
*** dirkhh has quit IRC14:32
*** Uli- has quit IRC14:32
*** melange-buildbot has joined #melange14:32
*** dirkhh_ has joined #melange14:32
*** allenZl has joined #melange14:32
*** cosenal_ is now known as cosenal14:32
*** asmeurer_ has joined #melange14:43
*** saurabhsood91 has joined #melange14:50
*** melange-buildbot has quit IRC14:50
*** melange-buildbot has joined #melange14:50
*** saurabhsood91 has quit IRC14:53
*** karanp has joined #melange14:54
*** carols has joined #melange16:04
*** AnujSharma has joined #melange16:14
*** carols has quit IRC16:18
*** carols has joined #melange16:19
*** Abhishek_ has quit IRC16:50
MatthewWilkesReasking question from the other day that was when everyone was away:17:06
MatthewWilkes[14:02:36]  <MatthewWilkes> in gsoc.views.helper.request_data.RequestData there's a difference between mentor_for and org_admin_for. mentor_for checks for self.profile but org_admin_for checks for self.ndb_profile.  What's the difference between these two objects?17:06
*** carols has quit IRC17:10
*** carols has joined #melange17:12
*** Niharika has quit IRC17:18
*** carols has quit IRC17:53
*** carols has joined #melange17:54
*** penguinRaider has joined #melange17:55
*** samgtr has joined #melange17:59
*** carols has quit IRC18:06
*** carols has joined #melange18:08
*** Darrel has quit IRC18:19
Robert_Sthat might be a bug. one used the NDB model the other uses the DB model.18:23
Robert_S(different ways of accessing the appengine datastore.)18:23
*** AnujSharma has quit IRC18:25
MatthewWilkesRobert_S: They are different models, though, right? Not accessing the same datastore entity?18:26
MatthewWilkesIf so, what's the difference? I'm seeing None for the dbmodel so can't compare directly18:26
Robert_SRight, if profiles have been migrated to NDB, then the DB ones should be gone.18:27
Robert_Sis my understanding18:28
MatthewWilkesah, right, that does sound like some unmigrated code, then18:28
MatthewWilkesIs the preference still patchset emails?18:29
*** karanp has quit IRC18:30
Robert_SNo.18:30
*** carols has quit IRC18:30
Robert_SMatthewWilkes: Try this: https://groups.google.com/forum/?hl=en_US#!topic/melange-soc-dev/B4mY6yOzJ_w18:31
tpbTitle: Google Groups (at groups.google.com)18:31
MatthewWilkesRobert_S: Thanks18:31
*** carols has joined #melange18:34
*** penguinRaider has quit IRC18:35
*** penguinRaider has joined #melange18:35
*** carols has quit IRC18:59
*** carols has joined #melange19:02
MatthewWilkesRobert_S: Sorry to bother you, but I'm having trouble with the Gerrit set up. It won't accept my pushes unless I have the committer email as a contact email, but I don't seem to be receiving the emails to confirm I own the email address I've entered19:06
Robert_SMatthewWilkes: Thank you for being the guinea pig.  :)19:07
Robert_SHow are you authenticating with gerrit?19:08
MatthewWilkesRobert_S: So, I've connected to the web app with my main personal Google account ( [email protected] ). That is the account that I've OAuthed to the Git repos and put in .netrc, too.19:09
MatthewWilkesThe committer email that you have on file for me is [email protected], though19:09
Robert_SAnd what error are you getting?19:10
MatthewWilkeshttp://pastie.org/private/apq0mjitxlw8douwc81m8g19:11
tpbTitle: Private Paste - Pastie (at pastie.org)19:11
MatthewWilkesIt's when I follow those instructions to add an email I don't get the email with the verification link19:11
MatthewWilkesWhich is weird, as I assume it's just using the App Engine outbound email19:11
Robert_SThat Gerrit doesn't run on appengine.19:12
Robert_S(afaik.)19:12
Robert_SI've just emailed the Gerrit team.  I assume you've checked your spam folder?19:13
Robert_SIs it also possible that you're using greylisting and waiting for a reply?19:14
Robert_S(brb in 5m.)19:14
Robert_Ss/reply/retry/19:14
MatthewWilkesI have checked my spam folder, the email account it is going to is backed by Gmail so it should be already through the filters. I've given it 15/20 mins now, but it could still come through. It is, of course, possible that my copy of thunderbird is being an ass and has moved it somewhere weird19:15
Robert_SHrm.  Did you check the web UI?19:15
MatthewWilkesYup, no joy19:16
Robert_SOk, need to grab food, back in a sec, maybe the gerrit people aren't at lunch.19:16
MatthewWilkesBut if it got moved to a local folder it wouldn't show up19:16
MatthewWilkesGreat, thanks19:16
MatthewWilkesFWIW, just tried the web UI with my IMAP client closed and no joy, unlikely to be my set up19:20
Robert_SThere is anecdotal feedback that these sometimes take 15 minutes, for no known reason.19:25
*** madrazr has joined #melange19:47
*** ChanServ sets mode: +o madrazr19:47
*** rocker has quit IRC19:55
*** rocker has joined #melange19:55
*** rocker has quit IRC20:02
Robert_SMaththewWilkes, Did it show up?20:04
*** samgtr has quit IRC20:06
*** rocker has joined #melange20:11
*** rocker has quit IRC20:16
Robert_SMatthewWilkes: It should come from [email protected]20:17
MatthewWilkesRobert_S: Nothing as yet, no20:23
Robert_STry requesting it again?20:24
*** carols has quit IRC20:25
MatthewWilkesRobert_S: I requested it a couple of times at the time. Re-requested a couple of minutes ago.20:25
Robert_SSorry, I have no visibility behind the scenes here, and it works perfectly for me when I send it to my work or personal accounts.20:26
*** carols has joined #melange20:26
MatthewWilkesRobert_S: I assume that means you've asked it to associate with an email on another domain that's known to Google Apps?20:27
MatthewWilkesIt just occurred to me that it might be failing because it's sending to another email address hosted by Google20:27
Robert_Sgoogle.com is that way.20:28
Robert_Sgmail.com isn't google apps, but worked too.20:28
MatthewWilkesI didn't want to make any assumptions that "work account" meant the current work ;)20:28
Robert_S:)20:29
*** carols has quit IRC20:31
Robert_SYes.20:32
Robert_SEr, ignore the Yes.20:32
Robert_SOk, I just sent you a test email.  Which doesn't really prove anything, but makes us feel like we're debugging things.20:32
*** raghunayyar has joined #melange20:33
MatthewWilkesRobert_S: It works, yes :)20:33
Robert_SPhew.20:33
MatthewWilkesworks being the email from you, sorry. Not the whole thing20:34
Robert_SThat was my guess.20:35
MatthewWilkesNeeds some "percussive maintenance", imo.20:35
MatthewWilkesyou can't hit virtual machines, though20:35
Robert_SYou just need a virtual hammer.20:35
Robert_SI'm still guessing this is a configuration glitch on your side somwhere.  (Sorry to try and move the blame there.)  Is that domain setup with wildcard email address?  Can you try a different address at that domain?20:36
*** carols has joined #melange20:39
MatthewWilkesIt is, I shall.20:39
MatthewWilkesI envy the people with one canonical email address20:40
MatthewWilkesHey, guess what.20:41
MatthewWilkesI tried adding the email address as a secondary email in my google profile20:41
MatthewWilkesRe-requested20:41
MatthewWilkesdelivered immediately20:41
MatthewWilkesnow, that could be a coincidence…20:41
Robert_SNo idea.20:41
Robert_SThanks for sticking with it.20:41
MatthewWilkesNo problem, sorry for bugging you so much20:42
Robert_SNot a problem, it's my job :)20:42
*** rocker has joined #melange20:43
MatthewWilkesStill, that's a lot of infrastructure to support by association20:43
Robert_SI do what I can.20:44
*** rocker has quit IRC20:48
MatthewWilkesSo, current state. Emails didn't send until I set the address on my Google account profile as a secondary address. Now they send and clicking the link takes me to Gerrit with my email address in the pick preferred email drop down. If I navigate away I find the new email hasn't stuck. Enabling the save changes button and clicking it gets me to a gerrit error saying my email isn't verified20:55
Robert_SI don't see how your Google account profile comes into play.20:56
MatthewWilkesMe either.20:56
Robert_SIs there a save button to click when you verify things?20:56
MatthewWilkesBut sending to a different email address doesn't seem to work20:56
MatthewWilkesThere is, it's disabled. Enabling it and submitting causes an error page20:56
MatthewWilkesFairly generic, email address must be verified20:57
Robert_SWhat's on the Identities tab?20:57
MatthewWilkesSniffing it gets me: {"jsonrpc":"2.0","id":5,"error":{"code":-32603,"message":"Email address must be verified"}}20:57
MatthewWilkesShows the address I want to use only, not the email address of the account I'm using. The status cell is empty20:58
MatthewWilkespushing still lists only the account email address as the registered addresses20:58
Robert_SIIRC, Identity should say:  <blank> | $gmailaddress | Google Account20:59
MatthewWilkesYes, that's what I see, where $gmailaddress is the address I am trying to add21:00
MatthewWilkesnot the address of the account I'm logged in as21:00
MatthewWilkeshttp://www.dropmocks.com/mGz5vE21:00
Robert_SI thought you were trying to add $vanitydomain21:00
tpbTitle: DropMocks (at www.dropmocks.com)21:00
MatthewWilkesJust a misunderstanding about our variable naming21:01
Robert_SI think I know what's going on here.21:01
Robert_SAnd the pieces are falling into place.21:01
Robert_SYou have two different Google accounts.21:01
Robert_SAnd your .netrc is setup with the wrong one.21:01
MatthewWilkesah!21:01
MatthewWilkesThat might be it21:01
MatthewWilkesI did add what it told me, though21:02
MatthewWilkesSo, "I have machine melange-sandbox-review.googlesource.com login git-mw4068.gmail.com …" in my .netrc. I could try switching it to the other domain21:03
Robert_SIt may have misdirected you.21:03
MatthewWilkesbut to make it even more difficult, the email address of the other domain is [email protected]21:03
MatthewWilkesI consolidated a bunch of accounts and emails a while ago21:03
Robert_SPause for a second.21:04
Robert_SHave you switched Google accounts while in the browser recently?21:04
MatthewWilkesI was only logged into one, [email protected] until I was already half an hour into debugging this21:04
Robert_SRight.  Now you're logged into both.21:04
MatthewWilkesYes21:04
Robert_SOk, on https://melange-sandbox.googlesource.com/, make sure you're logged in as the account connected to matthewwilkes.co.uk.  (Preferrably, log in as [email protected])21:06
MatthewWilkesAlthough it's sometimes hard to tell which are logged in and which are just cached by the login box21:06
Robert_SAnd once logged in as the right account, generate a new password for netrc.21:06
MatthewWilkesRobert_S: Okay, I can do that. I had been doing it as [email protected] as that's the one that has google code repos and tickets, so it's the one I'm accustomed to using21:07
Robert_SI think if you want to use the other email adress on your commits, you have to use the other one with Gerrit.21:07
MatthewWilkesOkay, I'll give this a go21:08
Robert_SI had a thought.21:13
Robert_SIt might have been you were verifying in the browser with the wrong account.21:13
MatthewWilkesI wasn't, I checked that21:14
MatthewWilkesI tried doing it incognito, too21:14
Robert_SWeird.21:15
MatthewWilkesI tell you what, though21:15
MatthewWilkesThis is REALLY weird21:15
MatthewWilkesI logged in with the other account, generated my new netrc keys, etc21:15
MatthewWilkesAdded the email address, got the confirmation21:15
MatthewWilkesFollowed it, got "Identity already in use"21:15
MatthewWilkesSo, I thought, "fine, I'll go remove it from the other one, if I can"21:15
MatthewWilkesLogged in to the other one, it was using $vanityemail everywhere, not $gmail. Got new .netrc keys for that account, pushed, it worked21:16
MatthewWilkesI don't know what data store this uses, as you said it isn't App Engine. Could it be an eventual consistency problem paired with my weird google account set up?21:16
MatthewWilkesSo once it was finally fixed it still looked broken for a bit21:16
Robert_SWacky.21:17
Robert_SI'm glad it works now.21:17
Robert_SSorry for the trouble.21:17
MatthewWilkeshey, it's not your fault21:18
MatthewWilkesI really should have fewer accounts21:18
MatthewWilkesThanks for your help, I really appreciate it21:18
Robert_SNo problem.  I'm excited about getting people trying out Gerrit, as we think we're going to switch to it.  And learning about pain points is important.21:19
MatthewWilkesIndeed. It is a shame you're not on GitHub from my point of view as a (very) casual contributor, but the reasoning makes sense.21:20
MatthewWilkesIt doesn't make sense to optimise for people like me21:20
Robert_SIf GitHub supported the magic that Gerrit does that allows you to review things as logical units instead of individual commits, it has a lot going for it.21:21
MatthewWilkesFrankly, I'm just happy that Melange exists at all. I hate not being able to fix things that bug me. That's what this has resulted from, I got annoyed at the dashboard when looking after the end of programme surveys and decided to start bugging you lot with patches21:24
Robert_SPlease send more :)21:24
MatthewWilkesI will. Once I have worked out what the hell I'm doing :)21:25
MatthewWilkesIf I were to commit and push now, would that update my current Gerrit bug, create a new one that depends on the first with just the new commit, or create a new one with both commits?21:25
Robert_SCan you be more specific?21:26
MatthewWilkesSo, I have committed 64fcd28137912e39829507b98489450c4b99c241 which fixes a bug in a utility function and pushed that21:26
MatthewWilkesIt has been assigned to be https://melange-sandbox-review.googlesource.com/#/c/1060/21:27
tpbTitle: Gerrit Code Review (at melange-sandbox-review.googlesource.com)21:27
MatthewWilkesIf I now change something new, commit and push what will the new state be?21:27
MatthewWilkesAn update to #1060, a new thing containing 64fcd28137912e39829507b98489450c4b99c241 and NEWCOMMIT, or a new thing containing just NEWCOMMIT21:27
Robert_SThe second one.21:28
Robert_SIf you use git commit --amend, and re-push, then you'll get the first one.  (Which is probably what you want.)21:28
Robert_SGerrit makes it possible to sequence a set of related changes -- and maintain review state across rebases.21:28
MatthewWilkesOkay, so, on pushing it creates a new gerrit review of all the commits that comprise the difference between the remote master and the branch I'm pushing?21:28
Robert_SI think it creates a new gerrit review for every change between the head of your branch and somewhere (which might be the remote master, but I suspect it isn't that simple.)21:29
Robert_SBut Gerrit knows they're all related.21:29
MatthewWilkesOkay, so, what I'm going to do is leave #1060 there as my baby-steps into this water and work on my branch that fiddles with the dashboard with a view to pushing that once I've been the cycle through/it's finished.21:31
MatthewWilkesDo you have preferences for smaller, atomic things or big things that you might then want to take parts of and not others?21:31
Robert_S"Reviewable units" is probably the middle ground.  Smaller tends to be easier to review, but there's a point where too small is bad.21:32
Robert_SThings that are too big are hard to review, but as mentioned, you can re-push an entire feature branch to Gerrit and it does the right thing.21:32
MatthewWilkesright21:33
Robert_SGerrit is new for all of us, so we're still trying to figure some of this out.  (And if we decide to keep it, we'll probably move to a new Gerrit instance anyway, without the name "sandbox")21:34
MatthewWilkesright-ho21:36
*** sonney2k has quit IRC22:06
*** sonney2k has joined #melange22:07
*** carols_ has joined #melange22:14
*** carols_ has quit IRC22:14
*** Robert_S has quit IRC22:14
*** Robert_S has joined #melange22:15
*** carols has quit IRC22:15
*** cs_shadow is now known as cs_shadow|afk22:22
*** olly__ has quit IRC22:30
*** raghunayyar has quit IRC22:37
*** olly has joined #melange22:38
*** samgtr has joined #melange22:53
*** cs_shadow|afk is now known as cs_shadow23:15
*** cs_shadow is now known as cs_shadow|afk23:16
*** Abhishek_ has joined #melange23:54

Generated by irclog2html.py 2.12.1 by Marius Gedminas - find it at mg.pov.lt!