/srv/irclogs.ubuntu.com/2015/04/08/#ubuntu-quality.txt

=== chihchun is now known as chihchun_afk
=== chihchun_afk is now known as chihchun
=== benonsoftware is now known as clockwork
=== clockwork is now known as benonsoftware
brendandelopio, talk talk talk :)14:35
elopiobrendand: fgimenez: https://code.launchpad.net/~veebers/autopilot/revert-ext-class-fix-for-now/+merge/25102514:35
elopiohttps://bugs.launchpad.net/autopilot/+bug/137699614:36
ubot5Ubuntu bug 1376996 in Autopilot "test_customised_proxy_classes_have_extension_classes started failing all of a sudden" [High,In progress]14:36
brendandelopio, so merge that with AP trunk14:36
elopiobrendand: remove the skip, reintroduce the inheritance dark magic14:36
elopiothere are some app tests that fail then. I don't know which ones, chris is the one who knows.14:36
fgimenezelopio, brendand this should be removed from trunk's head, right?14:38
elopiofgimenez: I think that the idea would be to make a branch for it, make sure that all autopilot selftests pass, and then run the gatekeeper job with that branch.14:41
elopiothen check for the new gatekeeper errors related to inheritance. Make an autopilot selftests for that error, and fix it.14:41
elopiofgimenez: do you have the vpn set up?14:43
fgimenezelopio, i think so, let me try to launch it14:44
fgimenezelopio, how do we reintroduce the inheritance? just removing the skip doesn't make any test fail14:44
elopiofgimenez: hum, so that test is not good.14:45
elopiofgimenez: I just know part of this story, veebers knows more.14:45
elopiofgimenez: about building autopilot, I think you should join #ubuntu-ci-eng14:46
elopioI added a row in the spreadsheet, and then sil2100 assigned a silo to me.14:47
elopioon this page http://people.canonical.com/~platform/citrain_dashboard/#?q=autopilot you will see the build button.14:47
elopiothis explains this weird process: https://wiki.ubuntu.com/citrain/LandingProcess14:48
fgimenezelopio, ok thanks a lot14:49
fgimenezelopio, wait, i was executing the unit tests15:02
fgimenezelopio, but yes, it makes no difference to execute python3 -m autopilot.run run autopilot.tests.functional.test_introspection_features with or without the skip...15:11
fgimenezelopio, http://paste.ubuntu.com/10773819/15:12
elopiofgimenez: I remember it was failing, but it started failing suddenly.15:12
elopioI don't know what caused the failure.15:12
fgimenezelopio, the previous paste was with the skip, this is without http://paste.ubuntu.com/10773866/ and test_customised_proxy_classes_have_extension_classes passing15:17
elfygood afternoon quality people :D15:24
elopiofgimenez: can you please leave a comment on the card for veebers?15:25
elopiojfunk: can we work on the inheritance card before the prioritization meeting?15:26
fgimenezelopio, ok will do15:27
nuclearbobpitti: I'15:29
nuclearbobm getting an error on adt-run but executing the test script manually works15:29
pittinuclearbob: hey15:29
nuclearbobI'm grabbing the error now, it's about a (15:30
nuclearbobhttp://pastebin.ubuntu.com/10773980/15:30
nuclearboband I'll grab the test script15:30
nuclearbobpitti: test script is here: http://bazaar.launchpad.net/~canonical-platform-qa/ubuntu-power-tests/trunk/view/head:/debian/tests/tc-powermeter-05 and if I copy it over and execute it, it works15:32
pittinuclearbob: do you actually execute it, or run it through bash?15:37
pittinuclearbob: because, "function foo()" is a bashism15:37
nuclearbobpitti: oh right, thanks, I'll change the shebang line, I had just been running it through bash15:38
nuclearbobI'll remember to check that in the future15:38
pitti$ sh -n /tmp/tc-powermeter-0515:38
pitti/tmp/tc-powermeter-05: 28: /tmp/tc-powermeter-05: Syntax error: "(" unexpected15:38
nuclearbobyep15:38
pittinuclearbob: or just drop "function" :)15:38
pittinuclearbob: (it seems fine otherwise)15:38
nuclearbobpitti: yeah, checkbashisms reports nothing when I remove that15:39
jfunkelopio: yes, actually can you join a call?15:41
elopionuclearbob: are you ok being the vanguard from 20:00 UTC to 23:00 UTC ?15:41
jfunkelopio: we're postponing the prio meeting to tmw15:41
elopiojfunk: ok.15:41
elopiojfunk: the fixing sanity call?15:41
nuclearbobelopio: I'd prefer to do earlier than that if that's feasible, that's 7 PM for me15:42
elopionuclearbob: 17:00 UTC - 20:00 UTC ?15:42
nuclearbobelopio: that works fine except for today, when I have an appointment :) I can do 20:00-23:00 tonight and then 17-20 after that, if that's okay15:43
elopionuclearbob: don't worry about today.15:43
nuclearbobelopio: okay. I'll pick up 17-20 UTC starting tomorrow15:43
elopiojfunk_: brendand: tomorrow the calendar is full of stakeholder meetings.15:47
elopioyou moved it to the same time as the unity one.15:48
=== chihchun is now known as chihchun_afk
rhuddieelopio, do you have a moment to go over the webbrowser test failure?16:08
elopiorhuddie: yes I do.16:08
rhuddieelopio, excellent. This is the issue: http://pastebin.ubuntu.com/10771280/16:09
rhuddieelopio, there seems to be some sort of race condition when clearing the address bar text in the browser when entering a new url16:09
rhuddieit doesn't happen every time16:09
elopiorhuddie: is it like you clear it, and then the page is loaded, and then it's filled again?16:10
rhuddieno, the browser loads with the start page, and then we try to enter in a new url. the helpers then clear the address bar before the new url is entered.16:11
rhuddieelopio, here is the test code: http://bazaar.launchpad.net/~canonical-platform-qa/ubuntu-sanity-tests/trunk/view/head:/ubuntu_sanity_tests/tests/test_with_webbrowser.py#L13016:12
elopiorhuddie: yes, but I'm wondering why the field is filled after we have cleared it.16:13
elopiois it because we are clicking in the wrong place?16:13
rhuddieelopio, well, it is filled by default with the start page.16:14
elopiorhuddie: it starts filled, then we cleared.16:14
elopiobut the clear is failing.16:14
rhuddieyes the clear is failing16:15
rhuddieelopio, but currently the clear is not called directly in the test code, but by the uitk helper which enters the text16:15
elopiorhuddie: yes. What I don't get is why you say there's a race condition.16:16
rhuddieelopio, well, because sometimes it works, and sometimes it doesnt :)16:16
elopiorhuddie: but this is sequential. For some reason we are failing to clear it.16:17
elopioI'm wondering if it's because we are not hiting the button.16:17
rhuddieelopio, here is the method pressing the button: http://bazaar.launchpad.net/~ubuntu-sdk-team/ubuntu-ui-toolkit/trunk/view/head:/tests/autopilot/ubuntuuitoolkit/_custom_proxy_objects/_textfield.py#L7416:17
rhuddieelopio, it seems like there has been some workaround added previously for checking the visible property16:18
rhuddieelopio, I've tried adding some waits to make sure the visible property is set true, but it did still fail in the same way16:19
elopiorhuddie: you lost me.16:19
elopiowhen the field has a clear button but it's not visible, the field is clicked to show it.16:20
elopiowhat workaround are you talking about?16:20
rhuddieelopio, that seemed a bit strange to me, that the text field had to be pressed again to show to show the clear button? but may be its not16:22
elopiorhuddie: it's not clicked again. It's if you try to clear it when it's not focused, so it's the first time you click it.16:22
elopiothere could be something wrong in there, like we make something visible that hides the clear buttonl16:23
elopiorhuddie: do you have a screenshot of the error?16:23
rhuddieelopio, what happens is that the existing url is highlighted, and not deleted as expected.16:24
elopiorhuddie: that probably means that the clear button is not clicked.16:24
rhuddieelopio, there is another way to fix the issue, which is to call browser.main_window.address_bar.clear() before we try to enter the url in the test case.16:26
elopioit could be because we try to click it too fast, before it's visible. But that's not likely. We could add a clear_button.visible.wait_for(True), just to be sure about it.16:26
elopioor it could be because we are clicking the wrong coordinates, which is also unlikely.16:27
elopiorhuddie: that would do the same.16:27
elopiojust with less ifs.16:27
rhuddieelopio, I tried adding the visible.wait_for(True), but it still failed in the same way16:27
rhuddieelopio, interestingly it didn't fail when I added the address_bar.clear() to the test. - I ran it for ages but it didn't fail once.16:28
brendandrhuddie, what about waiting for enabled?16:29
elopiorhuddie: what would be nice is to understand why that works.16:29
brendandrhuddie, it could potentially be disabled for a period of time16:29
elopiowhat would be different is that on write we call autopilot focused_write.16:30
elopioif we call clear directly, the focused doesn't happen.16:30
rhuddieelopio, yes I tried enabled too, but same result16:30
elopiorhuddie: so, that could be a valid workaround, but I would prefer to dig deeper and understand what's wrong calling clear from write.16:31
rhuddieelopio, yes, I was trying to understand why the helper was not working correctly...16:31
elopiorhuddie: can you debug the issue? like putting a breakpoint, call clear, and the field doesn't get cleared?16:32
rhuddieelopio, yes, I'll do some more digging.16:33
elfywxl: you know your image is not booting on vb?16:42
wxlelfy: say what?17:28
elfycouple of days ago there was a pwconv /etc/passwd issue17:30
elfytoday I got something else - booting it now for screenshot17:30
elfybut - image not for today currently17:30
elfyhttp://i.imgur.com/eBDaXxd.png17:30
elfyjust thought I'd let you know17:31
wxlelfy: which image?17:32
elfy64bit17:33
=== bladernr` is now known as bladernr_
ianorlinelfy on my desktop the amd 64 image booted and installed with manaul partitoning fine18:23
ianorlinexcept for the eject part which all are broken18:23
elfymaybe a vb thing then - I don't look particularly deeply for other flavours unless asked or needed18:24
elfyand yea - that's still fubar18:24
wxlelfy: what vbox version?18:24
wxlianorlin: did you use kvm?18:24
ianorlinI installed on bare metal18:25
wxloh!18:25
elfywxl: 4.3.2618:25
wxlelfy: from the repos or upstream?18:25
elfyrepos thiscycle it seems18:26
wxlelfy: no i mean vbox18:27
wxlor maybe that's what you meant :)18:27
ianorlinok I will try vbox on this new bare metal install don't want the two hypervisors conflicting18:27
wxli don't use repo versions so i'm not sure i can back you up at all18:27
wxldanke ianorlin make sure to grab from the repos18:27
wxland confirm you have 4.3.2618:27
elfyrepos for vb wxl18:28
ianorlinyes I have that version and virtualbox fails18:34
wxlawww fooey18:34
wxlelfy: ↑ might want to check the other flavors more completely18:34
elfyI am SOO glad this vbox issue isn't seen in Xubuntu18:34
elfylast cycle was one after the other for us :p18:35
wxlnot in xubuntu? how weird!18:35
elfywxl: I would expect other flavours have QA leads18:35
elfyI just happened to see this for you when ianorlin asked me something the other day so followed along a while18:36
elfywell we did see it a while back, so did Mate18:36
wxlelfy: so can you email the list?18:37
elfylist?18:37
wxlthe ubuntu quality list18:38
ianorlinok also nocompcache isn't a magic fix18:38
elfywxl: ack - but it'll be later on18:38
wxlk18:38
elfywxl: what for though?18:39
* ianorlin will try an alternate with vritualbox18:39
elfywxl: did it - I'll likely not remember now I've made you aware of the issue and won't look again18:43
wxlelfy: i suggested it since other leads may need to know. mean they'll probably figure it out, but …18:54
elfywell I would have expected them to know I guess - it's been about on lubuntu for 2 or 3 days18:55
elfyanyway - such is life18:56
wxlyep18:56
elfyand a bunch of questions make me wish I'd not done it lol18:58
* wxl ducks :)18:58
elfyI'd rather wxl mailed the list :p18:58
wxlforgive me for i have sinned elfy :)18:59
elfyespecially as I've got a real pita issue with my isp18:59
elfymy mails to mailing lists - get marked as spam here - so I have to go webmail to unmark them :|19:00
wxli'll reply19:00
* elfy ponders lubuntu being upstart-less, wonder if that's part of this 19:13
elfymade you aware - my job is done :p19:18
=== FunnyLookinHat_ is now known as FunnyLookinHat

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