| === xnox_ is now known as xnox | ||
| === lderan_ is now known as lderan | ||
| === vrruiz_ is now known as rvr | ||
| balloons | thomi, did the new autopilot land with my needed apparmor changes? | 12:41 |
|---|---|---|
| thomi | balloons: yes | 12:41 |
| thomi | balloons: I wonder if you could please help us get the 1.5 docs online? | 12:41 |
| thomi | we *really* need them to be online now | 12:42 |
| balloons | thomi, I thought we took care of all that in Malta. I'll ping dpm again to sync | 12:42 |
| thomi | balloons: AFAICT they're still not online | 12:43 |
| balloons | indeed, I agree.. I still see 1.4 | 12:43 |
| ahayzen_ | balloons, were the apparmor changes todo with the fix to writing to /tmp by using the autopilot/fakeenv? | 12:45 |
| balloons | ahayzen_, the apparmor changes in the last AP release were supposed to solve our /tmp issue yes | 12:47 |
| ahayzen_ | balloons, hmmm i tried to resolve the issue, i'm correct in thinking we had to move to ~/autopilot/fakeenv like the calendar? | 12:48 |
| balloons | afternoon elopio | 13:11 |
| cgoldberg | thomi, ping... got a sec? | 13:14 |
| thomi | cgoldberg: maybe, what's up? | 13:14 |
| cgoldberg | thomi, quick question with my video branch.. stuck replacing that sleep statement i was working on my last day at malta | 13:15 |
| cgoldberg | I'm trying to check for the recordmydesktop /tmp directory before I stop the video capture. | 13:15 |
| cgoldberg | It looks like testtools PathExists() matcher doesn't work with globs/wildcards.. you need an exact directory name (which is unknown before the test starts). Got another idea how I can check if the /tmp/rMD-session* directory eventually exists? | 13:15 |
| thomi | cgoldberg: why not use glob.glob, and wait till it doesn't equal the old value? | 13:16 |
| cgoldberg | ah.. nice.. that should work. just wait in a loop until timeout? | 13:17 |
| thomi | cgoldberg: use the EVentually matcher :) | 13:17 |
| thomi | cgoldberg: which takes a callable | 13:17 |
| thomi | self.assertThat(lambda: glob.glob(...), Eventually(NotEquals(old_value))) | 13:17 |
| thomi | or something like that | 13:17 |
| cgoldberg | cool... that should do | 13:18 |
| cgoldberg | thanks much | 13:18 |
| cgoldberg | btw, I marked my branch as WIP, but will resubmit for review in a little bit once I push that up... if you have any time to look | 13:18 |
| balloons | thomi, david says https://launchpad.net/ubuntu/utopic/amd64/python3-autopilot/1.5.0+14.10.20140526.1-0ubuntu1 still contains AP 1.4 docs | 13:20 |
| balloons | thomi, also on this; https://code.launchpad.net/~thomir/ubuntu-calendar-app/trunk-fix-ugly-bits/+merge/221099, I find the local_location_qml change interesting. | 13:21 |
| balloons | I guess it's simpler and still relative; I'll adopt it | 13:23 |
| elopio | balloons: hello! | 13:26 |
| thomi | balloons: sorry, I really don't have time to look now - can you ping me tomorrow if it's still outstanding please? | 13:26 |
| elopio | how are you feeling now? | 13:26 |
| balloons | thomi, the 1.4 docs is the important bit.. I was just commenting on your proposed merge; I don't need you to look at it, sorry to confuse | 13:28 |
| cgoldberg | balloons, regarding AP docs. I just looked at the sphinx conf.py, and I see there might be an issue with the version it reports if for example you download the tarball | 13:28 |
| cgoldberg | balloons, it pulls the version number from debian changelog. if you have a source tarball and no changelog, it defaults to whatever is specified in conf.py... which is currently: | 13:29 |
| cgoldberg | # The short X.Y version. | 13:30 |
| cgoldberg | version = '1.4 | 13:30 |
| balloons | cgoldberg, ahh.. that would be why then | 13:30 |
| balloons | so the 1.5 docs are up, they simply say 1.4 | 13:30 |
| cgoldberg | balloons, thats what it looks like. I'll MP a fix to conf.py | 13:30 |
| balloons | elopio, feeling a bit more alive.. I think I'm on the road to recovery now | 13:31 |
| balloons | cgoldberg, shall we tweak the current docs to say 1.5 in the interim? | 13:32 |
| cgoldberg | balloons, if you have access, yes.. since my change won't land until next release | 13:33 |
| cgoldberg | you can either regenerate the docs with the changelog present.. or just hack the generated html | 13:33 |
| balloons | cgoldberg, sure thing, will do | 13:33 |
| cgoldberg | balloons, this should do it: https://code.launchpad.net/~coreygoldberg/autopilot/fix-docs-version/+merge/221381 we'll land it for next release | 13:42 |
| balloons | elopio, you EOD? | 15:28 |
| elopio | balloons: nop. | 15:28 |
| elopio | fighting autopilot weird view on inheritance. | 15:29 |
| balloons | elopio, how far along did you get on the sdk helper for env setup? | 15:29 |
| elopio | balloons: for home? | 15:29 |
| elopio | it's merged into staging. | 15:29 |
| balloons | yes | 15:29 |
| balloons | oO, can I see? I've been working on it this morning with Jamie, and I think calendar finally works | 15:30 |
| balloons | however we have something to discuss with it. We have to create 9 directories atm; not sure if more will be needed or not.. Jamie says /home is setup for clicks and they are prevented from making any dirs themselves | 15:31 |
| balloons | he suggested cloning /home into our temp_dir and pruning what we don't want | 15:31 |
| balloons | as it stands, we make the new /home and set it up from scratch. If something changes in /home, our helper will need to change else apparmor errors will sweep the tests | 15:32 |
| elopio | balloons: https://code.launchpad.net/~elopio/ubuntu-ui-toolkit/fix1317639-patch_home/+merge/220805 | 15:33 |
| balloons | elopio, so that won't work out of the box on the phone | 15:34 |
| elopio | balloons: all that you are mentioning is new to me. That's why you can't get sick anymore :) | 15:34 |
| balloons | elopio, yes I didn't want this to linger all week.. I'm sorry :-( | 15:35 |
| balloons | I didn't know it wouldn't work without additional tweaks either.. until I can confirm, I leave room for error | 15:35 |
| elopio | balloons: don't worry, I'm joking. | 15:35 |
| balloons | lol.. I know.. at least about the being sick part | 15:35 |
| elopio | balloons: so is this usable at all? | 15:35 |
| balloons | elopio, have a look at this mp: https://code.launchpad.net/~nskaggs/ubuntu-calendar-app/fix-ap-env-setup/+merge/221416 | 15:36 |
| balloons | it seems to work for calendar. You can see the 9 dirs I create; everything else is the same, but I do additional post-setup work. | 15:37 |
| balloons | As it stands, yes your branch will work, but the additional dir creation would have to be done in the test in order for apparmor to not whine | 15:37 |
| balloons | elopio, does what I said above make sense? ^^ | 16:15 |
| elopio | balloons: sorry, I got distracted with a branch. | 16:19 |
| elopio | balloons: wow, so many things to create. | 16:19 |
| elopio | the other way, we would have to delete only one dir from home with the name of the app, right? | 16:20 |
| balloons | elopio, well, we would need to cleanup anything we felt could potentially mess with the results I guess.. | 16:22 |
| balloons | Or maybe just anything we know the test writes in | 16:22 |
| balloons | which sort of leads us back to where we started with backing up directories | 16:22 |
| balloons | we also have the time and potentially storage issues of running a full copy on /home | 16:23 |
| balloons | I don't like the idea, as it's just not the clean solution we are looking for. I don't care if we have to make 20 dirs, as long as we can be assured we've setup things properly and won't break stuff | 16:24 |
| elopio | balloons: I meant, make a new temp home, and copy into it everything that was on the real home | 16:30 |
| elopio | except the directory with the application data. | 16:31 |
| elopio | and the new temp dir, or we will end in wonderland. | 16:31 |
| balloons | elopio, yes.. see my concerns with doing the copy above | 16:34 |
| balloons | 1) storage space | 16:34 |
| balloons | 2) time to copy if many files | 16:34 |
| balloons | 3) potential hidden conflicts | 16:34 |
| balloons | 4) some apps do write to other dirs (music for instance uses mediascanner) | 16:34 |
| elopio | balloons: ahh, sorry, my brain is about to stop. | 16:35 |
| balloons | 5) Not as clean as setting up a pristine environment ;-) | 16:35 |
| elopio | balloons: ok, then lets make the 9 directories. | 16:35 |
| balloons | elopio, I feel the same way, different reason though, heh | 16:35 |
| elopio | and we need a better test on the toolkit that tells us if we could launch the app, so it will break when app armor rules change. | 16:35 |
| balloons | elopio, well I'm concerned about both approaches.. the thing I think that stops me on the copy is the storage issue | 16:35 |
| elopio | balloons: wait a second while I go up to my room | 16:36 |
| * elopio migrates to a warmer place. | 16:36 | |
| elopio | balloons: I'm back. | 16:43 |
| balloons | elopio, warmer I hope? :-) | 16:43 |
| elopio | starting to get warmer. The cleaning lady keeps turning on the air. | 16:44 |
| balloons | elopio, so I guess we'll go with the create the dirs approach then? I wouldn't worry about modifying your helper just yet.. We need this in a few apps, so I'm going to push the function as part of _init_.py. I want to ensure we're not missing a directory for instance :-) | 16:56 |
| balloons | once everything is converted over we'll see where we land | 16:56 |
| elopio | balloons: I will start thinking how to test it on the SDK | 16:56 |
| elopio | my helper in any of the two approaches just needs more directories. We can create them, or copy them. I'm not sure which one is best, so either way seems fine for now. | 16:57 |
| cgoldberg | balloons, just read scrollback. do you need environment isolation between tests? as in... do you want to create the dirs once for a test suite.. or do you want it created fresh between each test? might be nice to use a Fixture instead of adding to __init__.py . just wondering if the env gets poluted between tests | 18:21 |
| balloons | cgoldberg, https://code.launchpad.net/~elopio/ubuntu-ui-toolkit/fix1317639-patch_home/+merge/220805 | 18:57 |
| cgoldberg | balloons, right.. so don't you wanna add your dirs there, instead if in __init__ of every app test suite? | 18:59 |
| balloons | cgoldberg, most certainly.. the helper will need updated | 19:00 |
| cgoldberg | balloons, cool. that gives you nice isolation.. especially if you remove the dirs in the Fixture as a Cleanup step | 19:02 |
| cgoldberg | and as to the copy vs. create... both are feasible.. but I wouldn't copy existing /home. I think it would be better to create a fresh env structure in your source tree and copy that. or else just create them in the fixture | 19:04 |
| cgoldberg | balloons, let me know if you need a hand... I like small tasks besides hacking Autopilot itself :) | 19:07 |
| balloons | cgoldberg, well I'm still seeing apparmor issues.. | 19:11 |
| balloons | calendar works, but music doesn't.. and I'm seeing issues in calc even though it's not been migrated | 19:12 |
| balloons | I'd prefer to construct things as opposed to copy | 19:12 |
| cgoldberg | balloons, that's fine. the end result is the same if you are copying from a known source (not existing /home) ... my main point was to do them in a Fixture instead of __init__ | 19:14 |
| balloons | cgoldberg, check on the fixture.. The idea of copying a prisitine source is interesting, but you would have to bundle it with the toolkit | 19:15 |
| balloons | well the toolkit helper | 19:15 |
| cgoldberg | right... you check in a pristine dir. and have your test call a fixture that sets up and tears down the env between tests | 19:16 |
| cgoldberg | basically a hybrid your yours and Leo's branch :) | 19:17 |
| cgoldberg | with some dirs checked into bzr | 19:17 |
| balloons | I guess that's an implementation detail for the helper :-) | 19:18 |
| balloons | I still think I prefer the creation | 19:18 |
| balloons | mostly I just want it to work | 19:18 |
| balloons | heh | 19:18 |
| cgoldberg | fair enough :) | 19:21 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!