/srv/irclogs.ubuntu.com/2015/01/14/#ubuntu-quality.txt

=== elfy is now known as Guest54311
=== elfy is now known as Guest77157
=== Guest52375 is now known as forestpiskie
=== forestpiskie is now known as elfy
=== cking_ is now known as cking
=== qwebirc681520 is now known as slickymasterWork
paulliucharles_: Can I get the rotation_lock by dbus methods from /com/canonical/indicator/rotation_lock ?13:40
paulliucharles_: the rotation_lock seems to be a gsettings. Can you also export it to the dbus interface so we can access it?14:46
paulliucharles_: For testing.14:46
=== charles_ is now known as charles
charlespaulliu, I can do that, but I think you may be able to pull directly from the gsettings if you're wanting to test it in qml14:49
paulliucharles: no. I'm writing the autopilot test.14:49
paulliucharles: we want to use the autopilot to turn on/off the indicators rotation lock. And see if it did changes the real gsettings.14:50
charlespaulliu, ack14:50
charlespaulliu, I don't mind adding that to the rotation indiator, but just to make sure it's the right thing, why are you wanting to query this via dbus?14:53
charleseg if you want to verify the gsettings independently you can do it from the command line with "gsettings get com.ubuntu.touch.system rotation-lock"14:53
elopioGood morning team.14:54
paulliucharles: ok. I'll use the command line.14:59
charlespaulliu, ack15:01
paulliucharles: when I turn on/off the Rotation Lock, it didn't change the value of com.ubuntu.touch.system rotation-lock value.15:29
* charles looks15:30
charlespaulliu, wfm15:31
charlespaulliu, I pulled down the rotation lock indicator's menu and toggled the "Rotation Lock" menuitem15:32
charlespaulliu, repeatedly doing that + querying "gsettings get com.ubuntu.touch.system rotation-lock" gives me true, false, true, false as expected15:32
paulliucharles: ok. Maybe something weird on my phone.15:33
charlespaulliu, you're running "gsettings get" in the phablet-shell session? :)15:33
paulliucharles: yes15:34
paulliuelopio: can you test if the fixtures works?15:42
paulliuelopio: my phone dies.15:42
elopiopaulliu: sure. Same branch?15:43
paulliuelopio: yes15:43
paulliuelopio:  just pull15:43
elopiopaulliu: I get self.assertEqual(fixture._is_rotation_lock_enabled(), True)15:46
elopiotesttools.matchers._impl.MismatchError: False != True15:46
paulliuelopio: yeah. I got the same. So if you turn on/off the rotation lock, is "gsettings get com.ubuntu.touch.system rotation-lock" shows the same value?15:47
paulliuelopio: before my phone dies, I cannot get the correct value by gsettings. But charles can.15:48
elopiopaulliu: no, it doesn't matter the state of the checkbox, it always returns false15:49
paulliucharles: ^15:49
elopioalso, the indicator icon is not being shown when I click the checkbox. Something stoped working.15:49
charles...?15:50
* charles re-tests15:50
elopiogsettings set com.ubuntu.touch.system rotation-lock true does work.15:50
charlesokay so score 1 for testing15:52
elopio\o/15:53
charlesthis is working for me on 14.10 (krillin r191), but when I switch to 15.04 (mako r64) I get a consistent 'false'15:53
paulliucharles: it's weird.. Just gsettings..15:53
charlesmy first guess is that an indicator-display branch landed in 14.10 but not 15.04; if so that's an oversight15:54
charlespreviously uss had a different key and then went with 'rotation-lock' in the gsettings, there was a branch to switch to that in indicator-display. maybe that didn't land in trunk for the 15.04 images15:54
charleschecking15:54
elopioI can use the checkbox to disable the lock. Not to enable it.15:55
charlesno... no code changes between the two branches15:56
paulliuelopio: so for the test we might want to do that twice..15:56
paulliuelopio: if we change to test in the reverse order, I wonder the code will pass.15:57
elopiopaulliu: we can't automate at this level if there is an error.15:57
charlesno, this is broken, we shouldn't write the test to work around that15:57
elopiowhat we do is report a bug, and put a skip with a link to that bug.15:57
charles+115:57
paulliuok.15:58
elopioand make a comment on the bug report to reenable the test once it's fixed.15:58
elopiopaulliu: your fixture looks awesome.15:58
elopioa couple of details.15:58
elopiopaulliu: for the moment, we should make the helpers compatible also with py2.15:58
paulliuelopio: ok15:58
elopioso when calling super, you need to use the old style. like super(IndicatorDisplayRotationLock, self).__init__()15:59
charlespaulliu, elopio, I have a meeting right now but I'll triage this right afterwards15:59
elopiocharles: thanks.15:59
charlespaulliu, elopio, please paste the bug url here15:59
elopiopaulliu: second, I think that we should have only one fixture_setup class. I would move the unity8/shell/fixture_setup.py to unity8/fixture_setup.py and put the one you added new there.16:00
elopiowhat do you think about that?16:00
paulliuelopio: ok.16:01
paulliuelopio: btw, are you testing that on rtm?16:04
elopiopaulliu: no, I'm on vivid, krillin.16:05
=== roadmr is now known as roadmr_afk
=== roadmr_afk is now known as roadmr
alesageelopio, I'm reviewing the make qmltest, might need you to lead me into this a little16:58
elopioalesage: sure16:59
elopiothe easiest way to test it that I have found is bzr bd. Then install the built packages and open qtcreator to make projects with the templates.16:59
alesageelopio, ok sure16:59
alesageelopio, so this is a dumb question but how do I see the existing template?  it's just the default dummy project, yes?17:00
elopioalesage: I don't get your question. How to create a project from the template?17:00
alesageelopio, more like "what is a template"17:01
elopioalesage: when you open qtcreator, you will see a page called ubuntu-sdk17:01
elopiofrom that page click the link create a new project17:02
elopiothere you will see the templates, in the Ubuntu section.17:02
alesageelopio, k I see17:02
elopioalesage: on the branch, you will see many wizard.xml files17:02
elopiothat's what makes the templates appear here, and what tells qtcreator which files to copy into the new project.17:03
alesageelopio, so you modified "App with Simple UI"?17:04
elopioalesage: I modified many.17:05
elopioI changed the default main file from main.qml to Main.qml in all templates.17:05
alesageelopio, ok slowly dawning on self over here17:06
elfyballoons: \o/ thanks for talking to IS17:14
balloonselfy, no problem.. all appears well now17:16
elfyyep17:18
elopiorhuddie: ping, meeting.17:31
elopiopaulliu: have you filed the bug, or do you want me to do it?17:47
paulliuelopio: I'll do it.18:02
elopiopaulliu: thanks.18:03
elopioballoons: if you want to review the template branch, feel free. The more the merrier.18:03
elopioI'm going for lunch.18:04
paulliuelopio: https://bugs.launchpad.net/ubuntu/+source/indicator-display/+bug/141091518:11
ubot5Launchpad bug 1410915 in indicator-display (Ubuntu) "com.ubuntu.touch.system rotation-lock can't be changed by the checkbox in indicators" [Undecided,New]18:11
paulliucharles: https://bugs.launchpad.net/ubuntu/+source/indicator-display/+bug/141091518:12
charlespaulliu, ty18:12
dkesselgood evening. looks like half the ubuntu world is subscribed to bug 1268257 :D I have never seen a bug with a heat of > 1000018:26
ubot5bug 1268257 in nvidia-graphics-drivers-331-updates (Ubuntu) "nvidia-331-updates 331.38-0ubuntu3: nvidia-331-updates kernel module failed to build, with only error: "objdump: '... .tmp_nv.o': No such file"" [High,Triaged] https://launchpad.net/bugs/126825718:26
balloonsdkessel, ;-)18:49
balloonsevening to you as well18:49
dkesselevening balloons :) Let me motivate myself by telling you that I will be looking at the qa tracker code again in a few minutes18:53
dkessellightdm in vivid is still failing for me BTW :p18:54
balloonsdkessel, oO. I think we nailed the last snafu of the rebuild yesterday by getting bug numbers showing again18:54
balloonsand yea, vivid is having some fun. There's a keyboard input install bug as well18:54
=== chihchun_afk is now known as chihchun
Letozaf_balloons, hi, did you see https://wiki.ubuntu.com/UtopicUnicorn/ReleaseSchedule  is still on Ubuntu 14.10 schedule ?20:25
balloonsLetozaf_, ? what do you mean?20:25
Letozaf_balloons, shouldn't there be the release schedule of 15.04 ?20:26
balloonsLetozaf_, yes. https://wiki.ubuntu.com/VividVervet/ReleaseSchedule20:26
Letozaf_balloons, this one ends October 201420:26
balloonslet me fix the wiki too20:26
balloonsLetozaf_, wiki and calendar all fixed, thanks for noticing :-)20:32
Letozaf_balloons, yw :)20:32
dkesselballoons, knome: here is some progress on the qa tracker changes after two months of no progress: https://imgur.com/9561kt020:56
dkessel(a bit hacky)20:56
* balloons looks20:59
elfyso did elfy20:59
balloonsnice20:59
elfyI like20:59
balloons^^ me too20:59
elfyha - that'll be snap then :)21:00
elfyanything that means people don't have to mouseover 70 little icons to find a bug works for me21:00
elfythough it is likely to make the test page looooong I guess21:00
elfyit needs more than critical bugs imo21:03
elfyand is status/importance.assignee of any interest at this point21:03
elfymaybe a table with bug number and title and perhaps a repeated column so21:04
elfybug - title | bug - title | bug - title21:04
elfyjust a thought :)21:04
* dkessel keeps elfy's suggestions for the next tracker hack evening21:53
knomeelfy, that'd make it harder to read, i prefer one bug per line21:58
knomecould have checkboxes to filter out different statues though21:58
elfyknome: yes - but would you like the testcase page to be really looooooooooooooong21:58
knomesure, but with filters, it's doable21:58
knomeand we can work on the listing to make each row less tall21:59
elfyyou'd have to page down past 30 or more bugs - then lines and lines of text - and then finally somewhere to report21:59
elfyfilter out kylin22:00

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