[00:00:19] so far today I've seen a pair of hunting hawks, a roadrunner, and now a coyote. it's the Wild West. [00:02:17] JeffHall: I almost have the Links test working, one mo'... [00:13:56] JeffHall: updated your branch, it should be working for all tests now, I checked links and editing [00:16:37] chrismcmahon: Is there a special git review command I need to use to get your updates to that branch via gerrit ID 97810? [00:20:35] JeffHall: does 'git review -d 97810' update properly? [00:21:33] chrismcmahon: That's what I'm trying to figure out - when I do that and look at the git log, there's no indication that you changed anything, so somehow I don't seem to be getting the update (and the links test is still failing same as before) [00:21:52] JeffHall: urgh. [00:22:55] chrismcmahon: Think I'll take a fresh look at this in the morning - I'm over my 8 hours for the day, and they stop paying me when the whistle blows! [00:23:05] JeffHall: OK [00:23:14] sorry I confused git [00:24:02] chrismcmahon: I bet if I switch to master and then back to the review branch, that'll (hopefully) get me set right [00:25:35] JeffHall: basically the diff view changed from a span to a div [02:52:54] chrismcmahon: fyi, I'm off all this week [03:46:19] Yippee, build fixed! [03:46:20] Project browsertests-en.wikipedia.beta.wmflabs.org-windows-internet_explorer_6 build #383: FIXED in 18 min: https://wmf.ci.cloudbees.com/job/browsertests-en.wikipedia.beta.wmflabs.org-windows-internet_explorer_6/383/ [04:04:39] chrismcmahon: (why are you still up?) features/flow_logged_in.feature fails for me asserting "Then links to talk and contrib should be visible" [04:05:34] Erik Bernhardson changed the code to do CSS animations on hover, so I thought it was the fade-in, but sleep 0.3 didn't help, then I adjusted where the mouseover happens but that doesn't help. [04:07:04] Project browsertests-en.wikipedia.beta.wmflabs.org-windows-internet_explorer_7 build #381: FAILURE in 23 min: https://wmf.ci.cloudbees.com/job/browsertests-en.wikipedia.beta.wmflabs.org-windows-internet_explorer_7/381/ [04:49:31] Project browsertests-test2.wikipedia.org-windows-internet_explorer_10 build #304: FAILURE in 23 min: https://wmf.ci.cloudbees.com/job/browsertests-test2.wikipedia.org-windows-internet_explorer_10/304/ [05:32:53] Project browsertests-test2.wikipedia.org-windows-internet_explorer_8 build #585: FAILURE in 25 min: https://wmf.ci.cloudbees.com/job/browsertests-test2.wikipedia.org-windows-internet_explorer_8/585/ [06:36:18] Yippee, build fixed! [06:36:18] Project UniversalLanguageSelector-commons.wikimedia.beta.wmflabs.org-linux-firefox build #154: FIXED in 14 min: https://wmf.ci.cloudbees.com/job/UniversalLanguageSelector-commons.wikimedia.beta.wmflabs.org-linux-firefox/154/ [07:48:49] Project VisualEditor-test2.wikipedia.org-linux-chrome build #155: FAILURE in 37 min: https://wmf.ci.cloudbees.com/job/VisualEditor-test2.wikipedia.org-linux-chrome/155/ [07:57:27] hi zeljkof [07:57:49] yesterday i forgot to add the links and mark the task complete in melange :) [07:58:03] and now the 2nd task is claimed :) [08:03:47] hi zeljkof [08:03:47] yesterday i forgot to add the links and mark the task complete in melange :) [08:03:47] and now the 2nd task is claimed :) [08:04:23] mayankmadan: wow! :) [08:04:26] good job [08:04:38] I will add a few more tasks today [08:04:42] how do you like ruby so far? [08:04:42] i didnt claim the second task :( [08:04:47] someone else did [08:04:58] can you please close my first task if its okay? [08:05:05] and yes ruby is great :) [08:05:10] mayankmadan: oh, I see [08:05:17] no problem, I will create more tasks [08:05:28] and I will check and close your task, so you can move on [08:10:55] mayankmadan: and do not worry, we have plenty of tasks for you, I will make sure there are always about 5 open tasks (not claimed) and at least one or two code related at all times [08:22:13] thank you [08:22:29] I am here to help you win :) [08:22:51] how many people "wins"? just one or more of them? [08:28:37] zeljkof, 2 people from every organization [08:28:49] so that means 20 [08:28:52] oh, so two people from wikimedia will win? [08:30:00] yep [08:41:34] mayankmadan: I am reviewing the code [08:41:57] I have a meeting in 20 minutes, I will make sure I am done with the review before that [08:42:04] why would you like to win? [08:42:07] fame and glory? [08:42:22] or is there money involved, or a trip to Google or something? [08:42:35] or everything above? :) [08:45:26] zeljkof, there is no monetary prize [08:45:45] i would love to go to the USA [08:45:56] and also to get some self confidence [08:46:06] mayankmadan: those are good reasons [08:46:10] but i dont have a passport :) [08:46:12] I hope you will be one of the winners [08:46:26] passport is easy to do, you will also probably need a visa [08:46:36] but that is just paperwork [08:46:44] :) [08:48:05] mayankmadan: ok, the first piece of feedback [08:48:13] everything looks good so far, but you are not done yet [08:48:18] okay [08:48:34] I have just checked the bug in bugzilla [08:48:57] comment 3 (it could be easy to miss it) [08:48:57] https://bugzilla.wikimedia.org/show_bug.cgi?id=57597#c3 [08:49:19] the comment says quotes should be replaced in all .rb and .feature files [08:49:26] you have done a good job with .rb files [08:49:31] but you have forgot .feature files [08:49:33] those are connected [08:50:06] there should be way less work with feature files than with rb files [08:50:37] but if you do not change feature files, this will happen [08:50:51] https://gerrit.wikimedia.org/r/#/c/97788/5/modules/ve-mw/test/browser/features/step_definitions/headings_steps.rb [08:51:33] some ruby files expect single quotes in feature files, and we should use double quotes everywhere, where possible [08:51:40] mayankmadan: do you have any questions? [08:52:20] zeljkof, i didnt recieve any of your messages after [08:52:25] sorry [08:52:28] mayankmadan: ok, the first piece of feedback [08:52:35] i disconnected [08:53:01] I will copy/paste my messages here: http://etherpad.wikimedia.org/p/mayankmadan [08:53:17] feel free to use that etherpad when you need to copy/paste something [08:54:53] mayankmadan_: let me know here if you have any questions [08:55:24] I have a meeting in 5 minutes, it will last for 2 hours, but I will be able to answer quick questions [08:56:21] zeljkof, are there as many feature files as there are .rb? [08:56:36] mayankmadan_: I do not think so [08:56:43] and quotes in feature files are rare [08:56:58] you will find maybe 10-20 quotes in all feature files [08:57:02] maybe a bit more, I am not sure [08:57:08] but not hundreds [08:58:26] Nikerabbit: I am in the hangout [09:01:28] zeljkof, what are feature files? [09:01:44] all files that file name ends with .feature [09:02:02] example: https://github.com/wikimedia/qa-browsertests/blob/master/features/aftv5.feature [09:02:06] i know that :) but what is the function of these files? [09:02:10] zeljkof: coming [09:02:20] those are actual test files [09:02:45] if you are interested, read this [09:03:08] http://filipin.eu/browser-automation-workshop/ [09:03:15] http://filipin.eu/selenium-conference-2013/ [09:03:21] http://filipin.eu/browser-automation-workshops/ [09:03:30] (and there is more) [09:03:38] http://cukes.info/ [09:03:46] Nikerabbit: there is no rush [09:31:41] zeljkof, Done [09:33:49] mayankmadan_: wow, with all the repositories? [09:36:21] yep [09:36:36] did you upload the changes to gerrit? [09:36:41] zeljkof, there werent changes required in all of them [09:36:46] yes i uploaded them [09:36:58] mayankmadan_: great, I am reviewing the code right now [09:40:11] the first commit looks good! :) [09:42:49] yeah [09:51:48] mayankmadan_: there will be some more work for you in this task [09:51:55] but not a lot [09:52:30] a couple of us are reviewing the code and we found a few places where single quotes make sense [09:54:02] ok [09:54:19] you will get gerrit notification in a minute [09:57:37] zeljkof, am i not supposed to replace single quotes by escaping double quotes at some parts? [09:57:44] cause that was really hard to do [09:57:58] mayankmadan_: yes, that is the problem [09:58:02] oh [09:58:14] but I think it happens only in one repository [09:58:24] so it should not be a lot of work [10:00:21] I have published my review for this: https://gerrit.wikimedia.org/r/#/c/97842/ [10:06:35] and another review, just a couple of small changes: https://gerrit.wikimedia.org/r/#/c/97761/ [10:06:39] needs to be done [10:11:40] zeljkof, Now its done [10:11:53] mayankmadan_: great, we are reviewing all commits [10:12:47] . [10:14:02] mayankmadan_: did you notice that this commit is broken? https://gerrit.wikimedia.org/r/#/c/97842/ [10:14:10] "Build failed." [10:14:19] https://integration.wikimedia.org/ci/job/mwext-Wikibase-ruby1.9.3lint/909/console [10:14:24] take a look at the link [10:16:03] zeljkof, just a minute [10:18:33] zeljkof, fixed [10:18:45] mayankmadan_: great [10:28:32] (CR) Zfilipin: [C: -1] "Great job! Just a couple of small changes and this will be ready for merging into master branch! :)" (3 comments) [qa/browsertests] - https://gerrit.wikimedia.org/r/97849 (owner: Mayankmadan) [10:35:50] (PS3) Mayankmadan: Prefer double-quoted strings in Ruby code [qa/browsertests] - https://gerrit.wikimedia.org/r/97849 [10:36:09] zeljkof, the one that is just above is fixed too [10:37:08] mayankmadan_: a couple of us are reviewing your code for the last 90 minutes :) [10:37:14] and we are not done yet [10:37:20] great job! [10:37:29] 90 mins!! [10:39:50] zeljkof, i have a question [10:40:00] https://gerrit.wikimedia.org/r/#/c/97781/1/tests/browser/features/step_definitions/panel_steps.rb [10:40:22] sorry [10:40:27] no there is no question [10:41:09] mayankmadan_: :) [10:41:13] feel free to ask [10:47:44] (CR) Zfilipin: [C: -1] "Just a few changes in one file and this is ready!" (3 comments) [selenium] - https://gerrit.wikimedia.org/r/97756 (owner: Mayankmadan) [10:53:16] zeljkof, im getting this error while trying to run git review on UniversalLanguageSelector [10:53:17] http://etherpad.wikimedia.org/p/mayankmadan [10:57:15] ok, Nikerabbit and I reviewed a lot, I will continue reviewing [10:57:25] mayankmadan_: let me see the error message [10:59:33] zeljkof, http://etherpad.wikimedia.org/p/mayankmadan [11:00:05] mayankmadan_: this is fun part of git, conflicts! :) [11:00:21] just a second to finish something, I know what to do [11:04:49] mayankmadan_: this should help you [11:04:49] https://help.github.com/articles/resolving-a-merge-conflict-from-the-command-line [11:05:00] if you have any questions, I will be back in 10 or so minutes [11:22:35] (PS2) Mayankmadan: Prefer double-quoted strings in Ruby code [selenium] - https://gerrit.wikimedia.org/r/97756 [11:25:38] Everything is done now [11:49:45] mayankmadan_: I am back to reviewing code [11:49:55] did you manage to fix the conflict? [11:51:21] zeljkof, yep [11:51:46] mayankmadan_: great, I will continue reviewing the code [11:52:17] you have done really good [11:52:29] there were just a few things that needed to be fixed here and there [11:52:31] zeljkof, im going to see my psychiatrist so i wont be available for the next 2-3 hours :) if the code is good enough then please close the task :) [11:52:36] thank you [11:52:49] mayankmadan_: sure [11:53:10] I should be online later today, so feel free to ping me here or in qa mailing list if you have any questions [11:53:23] gerrit and bugzilla are also good communication tools [11:53:29] if you have questions [12:06:50] zeljkof, is there a problem with the review request? [12:07:10] mayankmadan_: what do you mean? [12:07:19] why isn't the ticket closed? [12:07:30] zeljkof, the task isnt closed [12:07:42] if that was the question, the answer is that I did not have the time to review all the code yet [12:07:51] oh [12:07:57] no problem [12:08:04] I thought you were offline so there is no rush [12:08:12] are you waiting for me to close it? [12:08:14] there is absolutely no rush [12:08:25] im online from my nexus 4 [12:08:32] I see :) [12:08:41] I have nexus 4 too! great phone [12:09:05] and really really cheap, especially if you buy it in the US [12:09:13] yep [12:09:30] ive installed paranoid android 3.99 and its really better [12:09:41] paranoid android? :) [12:09:52] I think I have the latest official release [12:10:50] kitkat? [12:10:58] yes, that is the one [12:11:08] I got it a few days ago, still getting used to ie [12:11:10] to it [12:11:34] i didnt really like it so i replaced it with android 4.3 version of Paranoid Android(custom rom) [12:14:08] mayankmadan_: I have heard that people root their android phones and install custom roms [12:14:19] but the official one works just fine for me [12:14:40] I might be just to old :) [12:57:32] (CR) Zfilipin: [C: 1] Prefer double-quoted strings in Ruby code [selenium] - https://gerrit.wikimedia.org/r/97756 (owner: Mayankmadan) [13:11:11] (CR) Zfilipin: [C: 1] Prefer double-quoted strings in Ruby code [qa/browsertests] - https://gerrit.wikimedia.org/r/97849 (owner: Mayankmadan) [14:13:36] (PS1) Hashar: comment out parsoid-server-sanity-check [integration/jenkins-job-builder-config] - https://gerrit.wikimedia.org/r/97916 [14:13:43] (CR) Hashar: [C: 2] comment out parsoid-server-sanity-check [integration/jenkins-job-builder-config] - https://gerrit.wikimedia.org/r/97916 (owner: Hashar) [14:14:28] (Merged) jenkins-bot: comment out parsoid-server-sanity-check [integration/jenkins-job-builder-config] - https://gerrit.wikimedia.org/r/97916 (owner: Hashar) [14:32:08] (CR) Hashar: "Do you guys know of any ruby linter/stylechecker that would be able to catch that kind of issue?" [selenium] - https://gerrit.wikimedia.org/r/97756 (owner: Mayankmadan) [16:09:40] Hi JeffHall [16:09:58] Hi mayankmadan_ [16:10:27] Any more tasks from you? [16:11:41] mayankmadan_: I'm just looking through the comments on your Gerrit changes now, so I'm not completely caught up on that [16:22:35] hey JeffHall we had more fun iframe change merges come in after you left yesterday, I merged a bunch of changes last night and some more this morning. I'm running the Chrome build on beta now. [16:28:11] chrismcmahon: Yeah, I saw your change in the Git log this morning (where for some reason I couldn't see yesterday's change). [16:29:25] chrismcmahon: But it looks like the r-ve changes in CloudBees are all still red, so I appreciate your digging into those iframe changes! [16:30:22] JeffHall: yeah, while I was working in our branch enough merges came through to make rebasing essentially more trouble than it was worth, so I started over on a new branch and just self-merged it while it was still valid. sorry about that. [16:31:10] chrismcmahon: No prob at all - I'm still new enough to working with Gerrit that it makes me feel stupid some of the time! [16:31:46] (CR) Jhall: [C: 1] Prefer double-quoted strings in Ruby code [qa/browsertests] - https://gerrit.wikimedia.org/r/97849 (owner: Mayankmadan) [16:33:01] for my next trick, where did the Flow page go? http://en.wikipedia.beta.wmflabs.org/wiki/Talk:Flow_QA [16:33:05] (CR) Jhall: [C: 1] Prefer double-quoted strings in Ruby code [selenium] - https://gerrit.wikimedia.org/r/97756 (owner: Mayankmadan) [16:34:03] somehow Flow got disabled on beta labs. [16:37:08] chrismcmahon: Who has control over application-level configs for beta labs (i.e. who has permissions to enable or disable features like Flow)? [16:38:49] JeffHall: good question. there is no record in the deletion log of any action on that page, so something funkier went on. [16:39:06] and the page is flat gone [16:47:21] brb [17:03:21] Yippee, build fixed! [17:03:22] Project VisualEditor-en.wikipedia.beta.wmflabs.org-linux-chrome build #167: FIXED in 43 min: https://wmf.ci.cloudbees.com/job/VisualEditor-en.wikipedia.beta.wmflabs.org-linux-chrome/167/ [17:04:18] JeffHall: ^^ fixededed [17:04:54] chrismcmahon: Excellent! Now if I can just figure out how to get Chrome tests to work with beta labs on my own machine... [17:06:56] JeffHall: what's the issue specifically? [17:09:05] chrismcmahon: Still getting those "Element is not clickable at point (xxx, xxx) errors from pretty much every test. Just out of curiousity, what version of ChromeDriver are you running locally? [17:10:25] JeffHall: 2.6 [17:10:56] chrismcmahon: Same version as me, so that's not the problem [17:11:14] JeffHall: I was getting that as I hacked on the page.rb file, because the ":ex" element was in the wrong iframe. have you git clone'd the very latest? [17:12:40] chrismcmahon: Yeah, I can see your chnages in the Git log, so I've got 'em, but I think I might blow away the repo and re-clone from Gerrit - my clone was from GitHub, which shouldn't make any difference, but perhaps it does make a difference [17:16:18] JeffHall: it might, I'm not sure how fast the github copy replicates [17:43:48] ryasmeen: What version of MacOS do you have running on your WMF MacBook? [17:45:07] JeffHall: if you're still getting the "Element is not clickable at point..." errors, let me know what the rest of the error message says, it should be telling the locator of whatever is "on top" of the thing the test is looking for. [17:45:42] chrismcmahon: Here's one of the errors: [17:45:50] unknown error: Element is not clickable at point (457, 651). Other element would receive the click:
...
[17:45:50] (Session info: chrome=31.0.1650.57) [17:45:50] (Driver info: chromedriver=2.6.232908,platform=Mac OS X 10.8.5 x86_64) (Selenium::WebDriver::Error::UnknownError) [17:45:50] ./features/step_definitions/shared_steps.rb:59:in `/^I click Review your changes$/' [17:45:50] features/general_markup.feature:8:in `And I click Review your changes' [17:51:44] JeffHall: "
chrismcmahon: It appears to be a timing issue - if I modify the relevant step with a brute-force sleep, everything comes up roses: [17:52:19] When(/^I click Review your changes$/) do [17:52:19] sleep 5 [17:52:19] on(VisualEditorPage).review_changes_element.when_present.click [17:52:19] end [17:53:22] chrismcmahon: Yep, I do have that review_changes element in iframe-index-1, so I'm pretty sure I have your latest code [17:54:49] chrismcmahon: It seems that I just have a fast machine, and Selenium is looking for page elements faster than the browser is loading them [17:56:42] chrismcmahon: I'll try adding wait_until behavior in place of my "sleep" [17:57:09] JeffHall: so the sleep actually dtrt then? [17:59:03] Yeah, adding a simple sleep in the relevant test fixes it, and jsut to prove it, I removed that one-line sleep statement, re-ran the test with Chrome, and the "Element is not clickable at point…" is mocking me again [18:02:37] JeffHall: it is MAC OS X 10.8.5 [18:03:39] ryasmeen: OK, thanks - just wanted to know if your machine was already updated to Mavericks (OSX 10.9) - you and I have the same version of Mac OS, so that's good! [18:10:39] (CR) Zfilipin: "@Hashar: if you are talking about comments I made in env.rb[1], Ruby did not complain because it is valid Ruby code. :) But, I find it con" [selenium] - https://gerrit.wikimedia.org/r/97756 (owner: Mayankmadan) [18:17:10] hi zeljkof [18:17:19] hi mayankmadan_ [18:17:39] can i have another task? [18:17:42] sure [18:17:51] nothing on the list looks interesting to you? [18:18:01] I saw that a few tasks is already taken [18:18:08] yep [18:18:11] I have a few good ones in mind [18:18:16] let me think for a second [18:18:27] Project browsertests-commons.wikimedia.beta.wmflabs.org-windows-internet_explorer_9 build #428: FAILURE in 4 min 24 sec: https://wmf.ci.cloudbees.com/job/browsertests-commons.wikimedia.beta.wmflabs.org-windows-internet_explorer_9/428/ [18:20:47] mayankmadan_: I think this would be a good task for you [18:20:48] https://www.google-melange.com/gci/task/view/google/gci2013/5888156557639680 [18:20:58] since you have already worked on the documentation [18:21:27] but if you prefer a coding task I have a few in mind [18:21:38] zeljkof, i was thinking to do it after i have done another coding task [18:21:45] is there anything left for you in the previous task? [18:22:03] yes [18:22:06] just saw it [18:22:11] the last time I checked there was just a couple of small things left to finish everything [18:22:37] I have closed the task at google site so you can move to the next one, because what is left was really details [18:22:58] ill do that now [18:23:04] https://gerrit.wikimedia.org/r/#/c/97781/ [18:23:04] please do [18:23:12] there are no inline comments [18:23:12] I will create a few more new tasks [18:23:42] mayankmadan_: you have to expand "patch set 1" [18:23:44] the comments are there [18:23:59] by default only the last patch set is expanded, in this case patch set 2 [18:24:10] so you do not see the comments that I have left for patch set 1 [18:24:29] i fixed all the problems that were in patch set 1 [18:24:30] no, wait [18:24:44] really, I have left a couple of comments, but they are not visible [18:24:48] let me see... [18:25:12] found it! :) [18:25:13] https://gerrit.wikimedia.org/r/#/c/97781/1..2/tests/browser/features/support/pages/main_page.rb [18:25:18] here are comments [18:25:38] I am not sure why it is not visible from the commit page [18:29:09] zeljkof, there is no file named main_page.rb in UniversalLanguageSelector that i changed [18:30:38] zeljkof, https://gerrit.wikimedia.org/r/#/c/97781/2/tests/browser/features/support/pages/main_page.rb [18:30:44] it says no differences [18:32:34] let me see... [18:33:45] I see [18:34:02] when you did the rebase, you have pulled this file into the repository [18:34:04] https://github.com/wikimedia/mediawiki-extensions-UniversalLanguageSelector/blob/master/tests/browser/features/support/pages/main_page.rb [18:34:12] and it has a few single qoutes [18:34:14] chrismcmahon: After some tinkering on my MacBook, I've discovered that I can get VE browser tests to work with Chrome only where I use explicit "sleep" statements - "wait_until" statements will not do the trick [18:34:38] mayankmadan_: do you see the file in the repository (on your machine)? [18:34:47] chrismcmahon: Going to do some searching in StacKOverflow to see if anyone else has encountered this same issue... [18:35:12] JeffHall: that is somewhat bizarre [18:40:21] zeljkof, yes i do [18:40:48] mayankmadan_: please change single quotes to double and that is all [18:40:54] that needs to be done [18:47:35] zeljkof, done [18:47:46] mayankmadan_: great [18:47:57] I think there were three things left to do, did you do them all? [18:48:09] I mean three commits [18:48:49] there were 2 commits left [18:48:53] im onto the last one [18:49:02] mayankmadan_: great [18:49:06] let me know when you are done [18:54:12] zeljkof, done [18:54:27] mayankmadan_: great, let me take a look [18:56:58] mayankmadan_: this still has one small thing to fix https://gerrit.wikimedia.org/r/#/c/97761/ [18:57:03] manybubbles: are you here? [18:57:10] here [18:57:22] great [18:57:28] could you answer this question? https://gerrit.wikimedia.org/r/#/c/97761/4/tests/browser/features/full_text.feature [18:57:32] please :) [18:57:48] so we could finally close this task for mayankmadan_ [18:57:57] I think that is the only thing left [18:58:18] answer was sitting as a draft:( [18:58:27] manybubbles: :) [18:58:30] I'd left it 20 minutes ago but not published it [18:58:34] zeljkof, the task is already closed [18:58:52] mayankmadan_: I know, I mean so you have actually finished all the work [18:58:59] and so you could move on to another task [18:58:59] oh [18:59:38] you were about to publish a task [19:00:08] mayankmadan_: yes, I have something in mind, but I have to ask other people how to do it [19:00:19] just a second to check your code [19:04:05] mayankmadan_: this looks good https://gerrit.wikimedia.org/r/#/c/97842/ [19:05:40] yippee [19:06:13] there are a couple of small changes that need to made here: https://gerrit.wikimedia.org/r/#/c/97761/ [19:06:16] in patch set 4 [19:06:31] I am reviewing the other commit that you have just pushed [19:13:14] mayankmadan_: this is what I had in mind for a coding task [19:13:15] http://lists.wikimedia.org/pipermail/wikitech-l/2013-November/073270.html [19:13:24] but I do not know really what needs to be tested [19:13:33] so I have to wait for somebody to let me know [19:13:40] and I have more ideas [19:14:13] zeljkof, btw just fixed the errors [19:14:50] looking at the code... [19:15:20] mayankmadan_: do you have open source experience? or coding experience in general? [19:15:43] some with kde [19:16:00] mayankmadan_: you are pretty fast with the tools we use :) [19:16:06] I am impressed [19:16:43] thanks :D [19:18:45] manybubbles: mayankmadan_ this looks good to me https://gerrit.wikimedia.org/r/#/c/97761/ [19:19:03] manybubbles: please take a look and feel free to merge it if you have no complaints [19:19:29] mayankmadan_: checking the last commit [19:20:16] mayankmadan_: does this look interesting to you? http://lists.wikimedia.org/pipermail/wikitech-l/2013-November/073270.html [19:20:45] Project browsertests-en.wikipedia.beta.wmflabs.org-windows-internet_explorer_6 build #384: FAILURE in 22 min: https://wmf.ci.cloudbees.com/job/browsertests-en.wikipedia.beta.wmflabs.org-windows-internet_explorer_6/384/ [19:20:56] if yes, I can create a draft task, you can start working on it, and if we get some comments in the next day or two, we can agree to accept or reject them [19:21:45] and if that project is interested in selenium tests, new tests could be new tasks [19:21:56] there is no need to put everything into one task [19:22:00] what do you say? [19:22:44] i say lets do it [19:22:53] mayankmadan_: deal! [19:23:12] I will +1 the last commit in a minute and create the task in the next 5-10 minutes [19:23:34] you can start working on the first test, and if the project is interested in tests, we can make new tasks for new tests [19:23:53] the first task will be to create a few basic tests, just to see if it is doable or not [19:24:15] ill be back in 5 mins [19:24:18] and if you prove it is doable, then the sky is the limit [19:26:26] mayankmadan_: as far as I am concerned double quotes task is finished! :) [19:26:27] hey, I'm asking in #wikimedia-operations about config changes to fix beta labs ( https://gerrit.wikimedia.org/r/#/c/97933/ ) but hashar & Reedy aren't around. Do people mind if fixes for beta labs accumulate in mediawiki-config ? [19:26:31] let's start with the pdf task [19:31:36] zeljkof, i need to try some tests right? [19:32:55] mayankmadan_: I am working on the task description right now [19:33:02] okay dokay [19:33:18] did you read my blog posts' [19:33:19] ? [19:33:33] or watch videos that Chris and I have created? [19:33:57] you will learn a lot by reading articles from my web site: http://filipin.eu/ [19:34:13] something to keep you occupied while I work on the task [19:34:15] :) [19:34:45] which videos? [19:35:27] zeljkof, ^ [19:36:06] read the last few blog posts at my blog [19:36:35] there are embedded youtube videos [19:43:53] * zeljkof is back in 20 minutes [20:30:27] Yippee, build fixed! [20:30:28] Project browsertests-test2.wikipedia.org-windows-internet_explorer_10 build #305: FIXED in 24 min: https://wmf.ci.cloudbees.com/job/browsertests-test2.wikipedia.org-windows-internet_explorer_10/305/ [20:43:49] zeljkof, you are finally bacl :) [20:44:08] mayankmadan_: yes, sorry, had family stuff to take care of [20:44:16] I will create the task right now [20:44:16] no problem [20:44:23] wait [20:44:35] first explain me the task please [20:44:37] did you get familiar with selenium and cucumber and friends [20:44:46] i kinda took a nap :) its really late here [20:45:14] ok, so the task is to create a test for "export html page to pdf" feature [20:45:30] you would click a link on a page that would create a pdf [20:45:54] a script should create a simple article on a mediawiki site [20:46:01] then export the article to pdf [20:46:09] and check if everything got exported [20:46:20] title, text, images, links... [20:46:33] the task would be about a day or two work [20:46:49] so not too much, but enough to be a good task [20:46:53] the time needed is not important [20:47:01] hallo [20:47:03] but i dont know how to create tests [20:47:10] and if the team doing the export feature is interested in tests, then we could create more tests [20:47:24] mayankmadan_: that is what you are here for, right? to learn :) [20:47:28] GEOFBOT: hi [20:47:32] hey [20:47:36] zeljkof, yep [20:47:36] i have jenkins [20:47:37] GEOFBOT: do you have a question? [20:47:39] on cloudbee [20:47:49] building a python program over and over again [20:47:56] GEOFBOT: oh, you are the other code-in student? [20:47:58] how would i get it to have email notifications for success? [20:48:00] yeah [20:48:20] GEOFBOT: just a second, I have finish something, will reply in a minute [20:48:25] ok [20:48:40] mayankmadan_: do you have any further questions? I will make sure to provide enough data so you can start [20:49:07] zeljkof, will you help me in learning to write tests? [20:49:21] mayankmadan_: that is what I do! :) [20:49:37] thank you very much [20:49:57] mayankmadan_: I spend a lot of time teaching people how to write tests [20:50:17] GEOFBOT: is you Jenkins instance public? can you provide a link? [20:50:52] zeljkof: im not sure if its public [20:50:55] but here is a link [20:50:56] https://geofbot.ci.cloudbees.com/job/test/ [20:51:45] GEOFBOT: it is not public :) [20:51:53] GEOFBOT: just a second, will let you know [20:52:08] GEOFBOT: by the way, where are you from? (I am from Croatia) [20:53:23] zeljkof: us eastern coast [20:53:41] GEOFBOT: I was recently in Boston, had a great time [20:55:02] GEOFBOT: so, to send e-mail notifications in jenkins... [20:57:20] GEOFBOT: assuming you have created a free-style software project [20:58:07] yes [20:58:14] if you go to configure job page [20:58:24] at the bottom there will be post-build actions section [20:59:01] add e-mail notification section [20:59:14] add recipients [20:59:38] (your e-mail) [20:59:50] and after every failed build you will get e-mail notification [21:00:04] GEOFBOT: does that answer your question? [21:01:54] yes, thanks [21:02:00] so should i just make the program break on purpose? [21:02:09] so it will send emails [21:04:57] zeljkof, whenever you are ready [21:05:33] GEOFBOT: yes, that should be the next step [21:05:46] when you receive e-mails, then you can start changing them [21:06:11] im using a github repository [21:06:14] mayankmadan_: sorry, I have a ton of distractions, I will create the task now [21:06:19] i changed the code on purpose [21:06:22] and i will see what happens [21:06:31] GEOFBOT: great, can you post the link to the bug in bugzilla? [21:06:46] and also feel free to post link to the jenkins instance [21:06:59] zeljkof: https://bugzilla.wikimedia.org/show_bug.cgi?id=55824 [21:07:13] Yippee, build fixed! [21:07:14] Project Flow-en.wikipedia.beta.wmflabs.org-linux-chrome build #74: FIXED in 1 min 52 sec: https://wmf.ci.cloudbees.com/job/Flow-en.wikipedia.beta.wmflabs.org-linux-chrome/74/ [21:07:55] zeljkof: one last thing [21:08:05] how do you get jenkins to get the latest verison of code from github [21:08:19] oh wait [21:08:23] nevermind, it picked it up [21:13:41] GEOFBOT: yes, it should just pick the latest coce [21:13:42] code [21:13:47] im not sure [21:13:53] if its working the way i twant it to. [21:14:00] if i make a syntax error, it still passes [21:14:13] please add links to your code and jenkins to the bug [21:14:35] GEOFBOT: you can always see the code that jenkins has [21:14:37] go to the job [21:14:54] and then click workspace on the left hand side [21:15:03] and you will see the files that jenkins has [21:16:52] chrismcmahon: I borrowed a Linux loaner from the IT folks, and Chrome tests that were failing for me on Mac OS work like a charm in the new environment [21:17:26] chrismcmahon: Will talk to RobLa to see if I can trade my MacBook in for a Linux machine - too much time wasted trying to get things working under MacOS! [21:17:35] JeffHall: that is bizarre. zeljkof is the Mac fan around here, that *should* work [21:18:04] JeffHall: yes, I am always saying to chrismcmahon he should get a "real" machine :) [21:18:12] JeffHall: OIT has a closet full of old Linux laptops, have two [21:18:56] zeljkof: how do i make a jenkins job public? [21:19:04] Yeah, it's weird, I've been using Mac OS for years and never run into anything like this before. But I'd rather spend my time developing tests than trouble-shooting my local environment! [21:19:21] GEOFBOT: I think you have to sign up for cloudbees open source account [21:19:22] zeljkof, what does the tests do? [21:19:34] GEOFBOT: probably too much trouble [21:19:43] mayankmadan_: I did not understand that [21:19:57] mayankmadan_: I am writing the task as I write this :) [21:24:45] zeljkof: that's ok, i can try [21:28:13] mayankmadan_: I have created the task https://www.google-melange.com/gci/task/view/google/gci2013/6466458871660544 [21:28:17] but it is not approved yet [21:28:23] I will ask the admins to approve it [21:28:40] okay dokay [21:30:18] zeljkof, claimed! [21:30:34] mayankmadan_: it is approved already :) [21:30:56] yes and i claimed it too [21:31:23] mayankmadan_: great [21:31:25] yesseth! [21:31:28] first jenkins fail [21:31:59] mayankmadan_, GEOFBOT: I will go to sleep soon, if you have any questions, ask now, or in the bug reports, or at QA mailing list [21:31:59] zeljkof, is it possible that we start working on this tomorrow? its 3:00AM here and i am feeling sleepy [21:32:03] zeljkof: can you view the job yet? [21:32:04] https://geofbot.ci.cloudbees.com/view/All/job/test/ [21:32:15] mayankmadan_: get some sleep! now! :) [21:32:33] mayankmadan_: you have 5 days, there is no rush [21:32:36] GEOFBOT: let me see [21:32:47] good night everybody [21:32:56] zeljkof, thanks for you cooperation :) [21:33:09] mayankmadan_: good night [21:34:49] zeljkof: can you view the jenkins job pages [21:34:51] https://geofbot.ci.cloudbees.com/view/All/job/test/ [21:35:19] GEOFBOT: no, but you can create an account for me there [21:35:39] use this e-mail zfilipin@wikimedia.org [21:35:50] so I will be able to log in [21:39:07] zeljkof likes spam :-) [21:39:37] chrismcmahon: gmail has solved spam problem :) [21:40:20] chrismcmahon: to quote John C. Dvorak: "I get NO SPAM!" [21:40:31] heh [21:41:00] i nthink you're added now [21:48:20] GEOFBOT: let me see... [21:48:43] GEOFBOT: for future reference, you have to mention my nick if you want me to see the message [21:48:54] I usually do not have IRC client visible [21:48:59] sorry [21:49:00] ok [21:49:04] I take a look when it pings me [21:49:28] GEOFBOT: no problem, but if you do not mention me, you might wait a long time for an answer :) [21:52:45] GEOFBOT: I am in! :) [21:53:06] hey zeljkof, i'm Leo who claimed the gci task [21:53:25] NameLess-exe: hi Leo and welcome! :) [21:53:50] GEOFBOT: feel free to add me to e-mail notifications, so I can see how they look like [21:53:55] zeljkof: ok [21:53:58] thanks. [21:54:01] zeljkof: i triede adding the changes first [21:54:01] NameLess-exe: do you have any questions? [21:54:12] not yet ;), i'm make a start on the project now, just woke up... [21:54:15] GEOFBOT: I forgot, did you have a question? [21:54:23] zeljkof: not yet... [21:54:35] NameLess-exe: it is 11pm here, I will probably go to bed soon :) [21:54:53] GEOFBOT, NameLess-exe: feel free to ask questions here, but I might be sleeping [21:54:54] yeah, we're 12 hours apart atm [21:55:11] also you can ask at the bugzilla bug, and at QA mailing list [21:55:23] ok, cool [21:55:25] NameLess-exe: where are you? (I am in Croatia) [21:55:31] new zealand [21:55:44] utc+13 [21:55:52] NameLess-exe: wow, that is probably as far from me as possible :) [21:56:07] yeah [21:56:14] zeljkof: one last question, how do you set custom messages [21:56:22] zeljkof: like, with html and what not [21:56:46] GEOFBOT: well, that is the task that you have to figure out :) [21:56:51] GEOFBOT: I really do not know [21:57:07] but there is a jenkins plugin (link is in bugzilla bug) that you should install [21:57:20] and it will probably give you options on how to format e-mail notifications [22:03:29] Yippee, build fixed! [22:03:30] Project UniversalLanguageSelector-en.wikipedia.beta.wmflabs.org-linux-firefox build #29: FIXED in 35 sec: https://wmf.ci.cloudbees.com/job/UniversalLanguageSelector-en.wikipedia.beta.wmflabs.org-linux-firefox/29/ [22:06:40] zeljkof: looks like the easiest way [22:06:51] is to get jenkins to generate an html report [22:06:54] when building [22:07:00] and then use that report [22:07:01] GEOFBOT: go for it! :) [22:07:02] as an emai [22:07:08] http://stackoverflow.com/questions/16315604/how-to-include-content-of-the-html-file-to-template-of-the-jenkins-email-ext [22:11:05] JeffHall: have you met Tomasz Finc yet? [22:11:19] chrismcmahon: Yes, I have [22:11:29] chrismcmahon: did you create a mediawiki user for mobile? [22:11:42] chrismcmahon: I have just found that task in my todo list :( [22:12:27] chrismcmahon: No, have not created a mediawiki user for mobile. Have not heard of that task before [22:12:31] zeljkof: I did not. I actually set the Selenium_newuser password to the same as Selenium_user. That solves the problem for the Mobile tests. I think you can WONTFIX the issue. [22:13:30] JeffHall: that is something chrismcmahon and I talked about last week [22:13:46] chrismcmahon: it is not in bugzilla yet, just my todo list, I will delete it then [22:15:23] JeffHall: if you have a chance to chat with Tomasz maybe today, the Mobile folks are looking for a bit of help doing e.g. post-deploy validation, you and ryasmeen would be in a good position to help out if you don't mind. [22:16:36] chrismcmahon: I think Tomasz left about 15 mins ago for the holiday break, but I can talk to him when he's back on the scene [22:17:23] JeffHall: ah, too bad. I just got an email from him about it. Later I suppose. [22:33:04] chrismcmahon: I will be happy to do that,I am working from hone today.I will communicate with him when he is online today [22:40:54] zeljkof: i tried using email-ext's default script to generate an html report [22:40:56] did you get it? [22:41:31] GEOFBOT: let me check [22:43:59] GEOFBOT: the last e-mail I got from your jenkins was 33 minutes ago [22:44:07] and it did not look nice :) [22:44:20] hmmm [22:44:33] Project VisualEditor-en.wikipedia.beta.wmflabs.org-linux-chrome build #168: FAILURE in 39 min: https://wmf.ci.cloudbees.com/job/VisualEditor-en.wikipedia.beta.wmflabs.org-linux-chrome/168/ [22:45:38] zeljkof: sending another one [22:45:51] GEOFBOT: did you get the e-mail? [22:46:08] yes [22:51:05] GEOFBOT: I got the e-mail! :) [22:51:13] it is way nicer that in used to be [22:51:37] but check the bug page, a few people have left comments on how to e-mail notification should look like [22:51:47] let me know if you have questions [23:02:49] zeljkof: that is teh default script for email-ext plugin [23:02:51] we can modify it [23:02:54] to our own ideas [23:03:10] looks good for the first iteration [23:32:01] Yippee, build fixed! [23:32:02] Project Flow-en.wikipedia.beta.wmflabs.org-internet-explorer-10 build #5: FIXED in 3 min 29 sec: https://wmf.ci.cloudbees.com/job/Flow-en.wikipedia.beta.wmflabs.org-internet-explorer-10/5/ [23:35:11] Yippee, build fixed! [23:35:12] Project Flow-en.wikipedia.beta.wmflabs.org-internet-explorer-9 build #4: FIXED in 3 min 10 sec: https://wmf.ci.cloudbees.com/job/Flow-en.wikipedia.beta.wmflabs.org-internet-explorer-9/4/ [23:52:15] good night everybody