/srv/irclogs.ubuntu.com/2014/03/31/#ubuntu-quality.txt

=== lderan_ is now known as lderan
=== rsalveti_ is now known as rsalveti
=== Daviey_ is now known as Daviey
=== mhall119_ is now known as mhall119
=== maclin_ is now known as maclin
=== chihchun_afk is now known as chihchun
=== maclin__ is now known as maclin
=== chihchun is now known as chihchun_afk
=== chihchun_afk is now known as chihchun
=== maclin_ is now known as maclin
jibelgood morning07:25
=== chihchun is now known as chihchun_afk
=== psivaa_ is now known as psivaa
jibelDanChapman, morning, I'm looking at LVM tests failures. It could be caused by latest upload of parted07:44
=== chihchun_afk is now known as chihchun
=== chihchun is now known as chihchun_afk
=== chihchun_afk is now known as chihchun
jibelDanChapman, confirmed, downgrading parted to 2.3-16ubuntu3 fixes the problem07:56
jibelI'll report a bug07:57
=== chihchun is now known as chihchun_afk
=== shadeslayer_ is now known as shadeslayer
DanChapmanjibel, good morning, that's great! nice to see it catching bugs :-)09:24
jibelDanChapman, yes that's great. bug 1300072 for reference09:25
ubot5bug 1300072 in ubiquity (Ubuntu Trusty) "LVM installation fails - regression with parted 2.3-17" [Critical,Confirmed] https://launchpad.net/bugs/130007209:25
DanChapmanbrilliant thanks09:26
davmor2Morning all10:09
xnoxjibel: DanChapman: whoop! whoop! on catching this =)11:02
=== _salem is now known as salem_
=== josepht_ is now known as josepht
=== lazypower_travel is now known as lazyPower
elopiogood morning team14:11
rvrelopio: Good morning14:12
rvrelopio: Meeting time :)14:12
=== chihchun_afk is now known as chihchun
davmor2elopio: Hey dude how's things14:21
elopiorvr: I think you forgot to upload the helpers.py file on your system settings branch.14:37
rvrelopio: Argh14:37
=== chihchun is now known as chihchun_afk
rvrRight, corrected14:38
balloonsping DanChapman14:42
elfyhi balloons14:43
DanChapmanpong balloons14:43
DanChapmanhow's it going14:43
balloonshey elfy14:45
balloonsDanChapman, I was wondering if you would be willing to help out with writing some AP tests for reminders14:46
balloonsit needs some love, and it's certainly been a challenge. It could use someone with skills like yourself :-)14:48
DanChapmanballoons: i sure can, is there a list of needed testcases?14:49
balloonsDanChapman, yes there is. https://bugs.launchpad.net/reminders-app/+bugs?field.tag=needs-autopilot-test14:51
balloonsDanChapman, have time for a hangout today and I can fill you in on everything?14:52
balloonsthere's been some work done, but one of the key bits is since it's an external service the tests need to be able to setup and account as well as potentially mock things14:53
DanChapmanballoons: I've got to pop out for a bit soon but i can do a hangout between 18.00 & 21.00 UTC? otherwise anytime tomorrow is good for me15:00
balloonsDanChapman, just ping when you are back.. That timeframe sounds fine for me15:01
balloonsI appreciate it !15:02
DanChapmanballoons: cool will do :-D15:02
* DanChapman plans to get rid of the nasty -q option aswell15:03
balloons<3!15:04
balloonselfy, seems I'm still not getting mails about subbed threads @ the forums :-(17:47
=== roadmr is now known as roadmr_afk
elfyballoons: that's because Subscribe with no notification is set17:51
elfydo you want daily, instant or weekly email notification17:52
balloonselfy, I see.. haha17:53
elfydo you want ubuntu member set?17:53
balloonsthe other one was set to daily, but this one is not17:53
elfyI'm in there now so can do all at the same time17:53
elfyI can set it so you're always daily if you want17:54
balloonsI've got it set now.. the thing is, I know I set this before17:54
* balloons swears he's not crazy17:54
elfyyou want the member flag set17:54
elfy?17:54
balloonswill that send mail whenever I'm mentioneD17:55
balloons?17:55
elfythe member flag? no - that just gives you the ubuntu member user flag17:55
balloonsohh17:55
elfyyou're overall sub setting is still no notification :p17:56
balloonsyes, that's fine17:56
elfyok17:57
elfydid you set it to daily or immediate?17:57
balloonsI set it to weekly17:57
balloonsbut sometimes I want daily/immediate, so I marked the threads in that case17:58
elfyso how do you know you're not getting mails? you only posted 3 days ago ...17:58
balloonsI marked the thread daily..17:58
elfyaah ok17:58
elfyballoons: look at the front page of the forum at your name :p18:03
balloonsohh.. I miss my beans!18:03
balloonsthose where like total old school18:03
elfyI'#ll change you back then :)18:04
balloonselfy, lol.. Yea, nostalgia wins out on this one I think18:04
elfyha ha ha18:04
* elfy wanders off to asmincp to fiddle with the beans cutoff points just to tip balloons into a new group ... 18:07
balloonslol.. ohh boy18:09
=== roadmr_afk is now known as roadmr
DanChapmanballoons: hey i'm about now if you want to catch up18:34
elfyDanChapman: he's currently in panic mode as I have control of his forum account :p18:36
DanChapmanelfy, haha don't let that forestpiskie get near it though. Definate cause of trouble that one :p18:37
elfynot anymore - no forestpiskie on the forum - just that pesky elfy admin guy18:37
DanChapman:-D18:38
balloonsDanChapman, sure, let;s do it18:42
balloonsthanks DanChapman :-)18:58
DanChapmanballoons: no need to thank, all in a days work and all that :-D18:59
balloonsI think you'll have fun, and it would be good to have another "example" of good tests to point at18:59
=== Ursinha is now known as Ursinha-afk
Letozaf_balloons, hello19:35
balloonsLetozaf_, howdy19:52
Letozaf_balloons, I'm fine and you ?19:53
balloonsLetozaf_, doing pretty well. So I asked DanChapman to have a look at helping out with reminders tests as well. He's going to look at getting the credentials going, mocking, and try and help get this landed :-)19:55
Letozaf_balloons, great19:56
* DanChapman waves19:56
Letozaf_DanChapman, o/19:56
balloonsso Letozaf_ how is things as they stand?20:09
Letozaf_balloons, I was trying to figure out what to do, I mean, for reminders I am waiting for a bug to be solved, so for rssreader app20:11
Letozaf_balloons, but maybe rssreader needs to be refactored, so I was looking at it, but not sure I can do much until the tabs problem is solved20:12
balloonsLetozaf_, ohh right, the tabs thing.. what's the status of that bug?20:14
Letozaf_balloons, but what was done on rssreader app, those page objects, must that be done also on reminders for instance ?20:14
Letozaf_balloons, oh the tabs thing... let me look20:15
balloonsLetozaf_, yes you can structure the test in the same page-object model20:15
Letozaf_balloons, but is that going to be the way we have to test, I mean is it just o choice or a must20:16
balloonsLetozaf_, nothing is a must.. but it is a goal :-)20:17
Letozaf_balloons, ok I understand :) that's right :D20:17
balloonsLetozaf_, so yea clock is the best example of the full testsuite being converted20:18
balloonssince reminders is new, it makes sense to adopt this from the start20:18
Letozaf_balloons, yes ok20:18
Letozaf_balloons, for the tab bug it's still there waiting for someone to take care of it :P20:19
Letozaf_balloons, https://code.launchpad.net/~carla-sella/ubuntu-rssreader-app/tests-refactoring/+merge/21333520:19
balloonsLetozaf_, ohh yes I remember20:25
Letozaf_balloons, :D20:27
Letozaf_balloons, does rssreader app run on your desktop ?20:36
Letozaf_balloons, I get: file:///usr/share/shorts-app/shorts-app.qml:351 Type SavedTab unavailable20:40
Letozaf_file:///usr/share/shorts-app/SavedTab.qml:20 ListModePage is not a type20:40
balloonsLetozaf_, from trunk or the binary/20:43
Letozaf_balloons, the binary20:44
Letozaf_balloons, just launched shorts-app from terminal and rssreader-app is installed20:45
balloonsLetozaf_, from your branch it's fine. From the binary, I get the same error20:48
balloonsheh, that's a problem20:48
Letozaf_balloons, a bug !20:48
balloonswell, let's try trunk first20:49
balloonsLetozaf_, seems to work from trunk20:49
jtaylorpitti: you closed bug 1292431 is this already running in trusty jenkins?20:50
ubot5bug 1292431 in autopkgtest (Ubuntu) "record package versions" [Wishlist,Fix released] https://launchpad.net/bugs/129243120:50
balloonsLetozaf_, I would say yes it's a problem, but only with the build in the ppa20:50
jtaylorcan't find it in any log of a few recent builds :(20:50
Letozaf_balloons, ok20:51
balloonsLetozaf_, I'll get it rebuilt so it works20:54
alesageelopio, when you have a minute https://code.launchpad.net/~allanlesage/unity8/autopilot-indicator-page-title-matches-widget/+merge/19699121:18
alesageelopio, do you have any urgent MPs I can help with?21:18
elopiolooking right now.21:18
elopioalesage: no, I'm stuck trying to reproduce failures on the clock with the toolkit that they want to land.21:18
elopioboring job.21:18
alesageelopio ho-hum ok21:19
elopioalesage: on file tests/autopilot/unity8/indicators/tests/test_indicators.py you still need to update the copyright year.21:31
alesageelopio darn let me look21:32
elopioalesage: I'm happy with this branch. I'm just wondering about the test_indicator_exists21:32
elopioit calls a method that returns an autopilot object, and has no assertion on it. Which makes me think it's not a user story test.21:32
elopioalesage: as test_indicator_page_title_matches_widget calls that same method, would it make sense to remove the other test?21:33
alesageelopio, I'll have to think about that: it's really just testing that the widget appears21:33
alesageelopio, yes I understand and had the same impulse21:33
alesageelopio, I'm not sure that there is a user story there except "appearance"21:34
elopioalesage: opening the indicator to see more details seems closer to a story, and we can just throw a nice exception if the indicator is not present.21:48
elopiodef get_indicator_widget(self, indicator_name):21:49
elopio   try:21:49
elopio      return self.select_single(...)21:49
elopio   except dbus.StateNotFoundError:21:49
elopio      raise ToolkitEmulatorException('indicator {} not present.'.format(indicator_name)21:49
elopioalesage: that's pseudocode, because I don't remember the real names of the classes.21:49
alesageelopio that's fair, yes21:50
alesageelopio hypothetically we'll have some appearance-based tests, based on the fullness of the power icon, etc.21:51
alesageI guess that would be icon.is_full() or some such21:51
alesagei.e. with an emulator for the appearance of the indicator widget21:51
elopioalesage: that's nice, yes.21:51
elopioalesage: and also we will make experience tests about opening the time and date settings from the indicator21:52
elopiocalling this helper you are adding here.21:52
__apr__I have a question about auto-pkg-test ( http://packaging.ubuntu.com/html/auto-pkg-test.html#getting-the-test-into-ubuntu )22:22
__apr__can anyone fork a package, add tests, and run them?22:22
jtayloryes22:25
=== salem_ is now known as _salem

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