| tntc | CardinalFang_: oh! I was afk, sorry | 00:18 |
|---|---|---|
| tntc | CardinalFang_: I pay for it, but youse guys have already comped me a year of service, so I have no real complaint :) | 00:18 |
| tntc | CardinalFang_: I just want to help fix it! | 00:19 |
| tntc | Get me my playlists. Ubuntu One is amazing. Ubuntu is amazing. Doing what I can to help get this working is priority numero uno for me. | 00:20 |
| tntc | :) | 00:20 |
| tntc | I want to get me my playlists* . Wow, the first three words got cut off there | 00:20 |
| tntc | What specifically is wrong with the server? | 00:21 |
| tntc | and how long will it take to fix? | 00:32 |
| CardinalFang_ | tntc, hard to say. If we knew *exactly*, we'd probably have it fixed by now. | 00:58 |
| tntc | CardinalFang_: is there any way to migrate us to a different server? | 01:00 |
| CardinalFang_ | tntc, it's not the hardware or configuration. I'm pretty sure it's something about the data that makes it explode. So, same data on different place doesn't change anything. | 01:09 |
| chemar | helo | 01:16 |
| JamesTait | Morning all! | 08:27 |
| fagan | ralsina: http://ubuntuone/p/19GA | 08:54 |
| mandel | morning! | 08:55 |
| fagan | mandel: morning | 08:55 |
| mandel | fagan: is ralsina around? | 08:55 |
| fagan | mandel: nope just for later before I forget | 08:55 |
| mandel | fagan: oh, backlog :P | 08:55 |
| fagan | Lol forgot to add the .com :D | 08:56 |
| fagan | ralsina: http://ubuntuone/p/19GA | 08:56 |
| fagan | mandel: Am trying to test his bundle on vista with no luck :/ | 08:56 |
| mandel | fagan: hm… what is the problem? | 08:58 |
| fagan | mandel: well it seems like it does the bitrock bit where it copies files and then it runs the script for the rest of the install and it gives out the error in that screen | 08:59 |
| fagan | mandel: probably has something to do with vista | 09:00 |
| mandel | fagan: I think you should report a bug about it… I think I have a vista vm, if I 'ever' have the time I could take a look | 09:02 |
| fagan | mandel: well ralsina was hoping to get a installer for lisette so thats why I was testing so yeah ill report a bug and bug ralsina about it when he gets in later | 09:03 |
| * fagan used pretty floury language in that bug report. Id say none of the team would know in what context I used former | 09:07 | |
| mandel | he, just make sure it makes some sense :P | 09:07 |
| fagan | mandel: yeah it does :D | 09:07 |
| fagan | Former = first one latter = last one :) | 09:09 |
| karni | Morning! | 09:13 |
| gatox | hi | 09:34 |
| fagan | yo gatox | 09:40 |
| mandel | gatox: hola! | 09:48 |
| lisette | gatox: hi, I was just wondering if you want a prefix for the win installer bugs? | 10:01 |
| gatox | mandel, lisette hi | 10:02 |
| lisette | gatox: QT this that and the other | 10:02 |
| gatox | lisette, could be "qt-ui"?? | 10:02 |
| gatox | lisette, yes, something like that | 10:03 |
| lisette | gatox: sure, cheers! :) | 10:03 |
| gatox | lisette, thanks | 10:03 |
| gatox | mandel, is your connection ok today? | 10:04 |
| mandel | gatox: much better, but I had to go to a diff village... | 10:06 |
| mandel | gatox: this country is a joke... | 10:06 |
| gatox | mandel, bummer! | 10:11 |
| mandel | gatox: at least I can work, I'll see if I can fix something for later... | 10:11 |
| gatox | mandel, i'm writing tests :P | 10:12 |
| mandel | gatox: haha wanna swap? | 10:13 |
| gatox | mandel, jeje i can't... i want to live! jeje | 10:13 |
| mandel | gatox: I'm dealing with ACEs and ACLs on windows :P | 10:13 |
| gatox | mandel, but i can help you with something maybe if you need... | 10:14 |
| mandel | gatox: don't worry, is not that bad, I nearly have it, I managed to get to the point to add a deny ace, I just need to know how to remove it :) | 10:16 |
| gatox | mandel, ok, let me know if i could do something anyhow! :D | 10:16 |
| mandel | gatox: sure, will do :) | 10:21 |
| ralsina | fagan: another one, try to get a result faster this time ;-) | 10:30 |
| ralsina | fagan: http://ubuntuone.com/p/17lC/ | 10:30 |
| ralsina | good morning! | 10:30 |
| ralsina | mandel, gatox: I think I finally managed to create a bundle that will work everywhere, but the wizard is looking awful (buttons are all wrong). I will prbably not work at all today, because I am going to the US embassy for my visa and that can take any amount of time | 10:32 |
| gatox | ralsina, ok... i can try out what is happening with the buttons using your bundle? | 10:33 |
| ralsina | gatox: the local folders page has no next button, and has a finish button :-) | 10:33 |
| gatox | ralsina, ahhhhhh yes, but that is fixed with my branch, was the same problem i was having | 10:34 |
| ralsina | gatox: awesome, so that's one less thing I need to worry about :-) | 10:34 |
| gatox | ralsina, totally | 10:34 |
| ralsina | and BTW: after the stuff I have done yesterday, I hate windows now | 10:34 |
| gatox | ralsina, now?? it take you more than expected :P | 10:35 |
| ralsina | gatox: I am not a hating guy | 10:35 |
| gatox | ralsina, jeejje | 10:35 |
| gatox | ralsina, good for your health :P | 10:35 |
| ralsina | gatox: but an operating system where *having* a DLL makes it give "can't find the method" errors, is hateable | 10:36 |
| gatox | ralsina, i don't know if you read it, but there is a REALLY good book: "In the beginning was the command line"... that analyze from an objective point of view several os, it has some good insights... | 10:38 |
| ralsina | gatox: I have read everything Neal stephenson has written :-) | 10:38 |
| gatox | ralsina, awesome! :D | 10:38 |
| ralsina | gatox: including his 3 1500 adventure novels about the birth of the european monetary systems ;-) | 10:39 |
| ralsina | s/1500/1500 pages/ | 10:39 |
| gatox | ralsina, ok... you win! jjee | 10:39 |
| ralsina | gatox: read them, they kick ass! It's called the baroque cycle! | 10:39 |
| * gatox searching... | 10:40 | |
| ralsina | Quicksilver, the system of the world, and I can't remember the name of the other one | 10:40 |
| ralsina | it has newton poking needles in his eye! | 10:41 |
| gatox | ralsina, i will look for his books then | 10:43 |
| gatox | ralsina, Quicksilver, The Confusion, and The System of the World | 10:43 |
| ralsina | gatox: ok, start with something more accessible. Like cryptonomicon. It has a operating system called Finux :-) | 10:43 |
| gatox | ralsina, will do | 10:44 |
| gatox | :P | 10:44 |
| ralsina | ok, gotta run. here's my standup for later, please pass it on! | 10:44 |
| ralsina | DONE: installed clean and development XP VMs, improved build scripts (install protocol certificates, redistribute VS2008 runtime, ignore DLLs that broke stuff), built a bundle using that, tested on XP and 7 | 10:45 |
| ralsina | TODO: get US visa | 10:45 |
| gatox | ralsina, ok | 10:47 |
| duanedesign | morning all | 10:47 |
| mandel | ralsina: can you take a look at my morning report? | 10:50 |
| mandel | :) | 10:50 |
| ralsina | mandel: sure! | 10:50 |
| mandel | duanedesign: morning!!! and I forgot to welcome you to the team :P | 10:50 |
| duanedesign | thank you mandel ! | 10:50 |
| mandel | ralsina: there is a proposal to solve the read_write issue using deny aces on the acl of a file | 10:50 |
| duanedesign | mandel: oh, you have macbook dont you? /me trying to remember what you were using at UDS | 10:51 |
| duanedesign | .5 | 10:51 |
| mandel | duanedesign: yes I do, macbook pro :) | 10:52 |
| ralsina | mandel: I think that's a fairly harmless solution | 10:52 |
| ralsina | mandel: when are we setting readonly again? | 10:52 |
| ralsina | mandel: just for ro shares? | 10:52 |
| duanedesign | mandel: everything work ok with Ubuntu? | 10:53 |
| * duanedesign is looking for a new laptop | 10:53 | |
| mandel | ralsina: yes, just for shares, once the deny ace is added the user cannot create new files or folders unless he explicitly sets them as admin which i think is fair enough | 10:55 |
| mandel | ralsina: all the other ace are left untouched :) | 10:55 |
| ralsina | mandel: yes, he can do the same thing on linux, so that makes sense | 10:55 |
| mandel | duanedesign: I do not recommend you to get a macbook at all, and I think verterok would say the same | 10:56 |
| mandel | duanedesign: thinkpad is the way to go | 10:56 |
| ralsina | besides we have lucio's promise that doing that doesn't break things server side, so... | 10:56 |
| ralsina | ok, really gotta run now. Have fun, please write reports today! | 10:56 |
| mandel | ralsina: yeah, I think is a decent solution, and access and car write are simple to implement, we look at the ace and if there is a deny we do not have the write rights, else we do… well more or less like that :) | 10:57 |
| duanedesign | mandel: yeah that is what I am leaning towards (thinkpad) Just trying to decide between a T520 or W520 | 10:58 |
| duanedesign | mandel: thanks | 10:58 |
| fagan | ok ralsina its working now | 11:00 |
| ralsina | \o/ | 11:00 |
| fagan | although I have 2 comments | 11:00 |
| ralsina | ? | 11:00 |
| fagan | It spawns a lot of terminal windows and the enter key isnt bound | 11:00 |
| fagan | so if I hit enter when typing my password lets say it doesnt actually go forward | 11:01 |
| ralsina | fagan: yes | 11:01 |
| ralsina | fagan: kown problems both, with easy solutions both :-) | 11:01 |
| fagan | ralsina: cool | 11:02 |
| === ralsina is now known as ralsina_embassy | ||
| nessita | good morning everyone! | 11:04 |
| gatox | nessita, good morning | 11:04 |
| nessita | hi gatox! | 11:05 |
| mandel | duanedesign: np | 11:05 |
| mandel | nessita: morning, sorry for yesterdays internet issues… there was terrible wind here and everything was broken… | 11:05 |
| mandel | nessita: can you take a look at the report and let me know what you think about the readwrite proposed solution | 11:06 |
| nessita | mandel: hi there! were you able to fix all the things I mentioned in the shutil branch? | 11:06 |
| mandel | nessita: yeah, that has been pushed already | 11:06 |
| nessita | mandel: will that solution work with fat systems? | 11:08 |
| nessita | (I m guessing no) | 11:08 |
| mandel | nessita: I have an initial implementation of the RO issues solved in the way I mentioned, I have not yet implemented access and can_write based on that but wont be too long | 11:09 |
| nessita | mandel: will that solution work with fat systems? | 11:09 |
| mandel | nessita: good question, I have not tried it on FAT, do you have one of those? | 11:11 |
| nessita | mandel: no | 11:11 |
| nessita | mandel: but I guess any pendrive will do | 11:11 |
| nessita | mandel: you could format a pendrive with fat32 using any linux tool | 11:11 |
| mandel | nessita: ok, I'll do that… man I hate all this back compatibility on windows | 11:13 |
| mandel | nessita: I'll do some IRL tests right now... | 11:13 |
| nessita | mandel: anyways, I doubt any solution will work on fat, since that FS, if I remember correctly, has 0 support for permissions | 11:14 |
| mandel | nessita: hmmm I wonder if we can set that as a min requirement for RO shares... | 11:15 |
| nessita | mandel: what would be "that"? | 11:15 |
| mandel | nessita: using NTFS and not FAT | 11:18 |
| nessita | mandel: the easiest solution is not to change any perm in any folder | 11:18 |
| nessita | in any FS | 11:18 |
| nessita | gatox: I'm restarting the review on your controlpanel branch | 11:26 |
| gatox | nessita, thanks..... i'm doing test for my other branch of the installer | 11:28 |
| nessita | gatox: question. Why would you add that QFrame again? | 11:34 |
| gatox | nessita, in order to be able to draw a horizontal dotted line in the installer ui, without having to inject widgets into preferences widget in the control panel, so in this way, i can set a qss style from the installer and nothing changes in control panel | 11:35 |
| nessita | gatox: right, but you can set a border-bottom to the agroupbox | 11:35 |
| nessita | qgroupbox | 11:35 |
| nessita | the one named bandwidth_settings | 11:36 |
| gatox | nessita, right...... let me test that | 11:36 |
| gatox | nessita, yes... that branch can be rejected | 11:42 |
| nessita | ack | 11:43 |
| gatox | nessita, did you reject the branch already? | 11:52 |
| gatox | because maybe will be necessary | 11:52 |
| gatox | nessita, because if i set the border to the qgroupbox, that widget lost the border properties for right, top and left | 11:54 |
| nessita | gatox: I did not reject it, no. What do you mean "that widget lost the border prop"? | 11:56 |
| gatox | nessita, so... qgroupbox has the border property... but it affect the other borders, even if you only say border-bottom | 11:56 |
| nessita | gatox: you sure? | 11:57 |
| gatox | nessita, yep... test it on designer | 11:57 |
| nessita | gatox: but I'be done it that way with the profile box in the account tab | 11:57 |
| gatox | nessita, control panel qss? | 11:58 |
| nessita | yeap (but is not in trunk) | 11:58 |
| nessita | gatox: trying to reproduce now... | 11:58 |
| nessita | gatox: is working here, I added this css to the control panel qss: | 12:00 |
| nessita | 205 QGroupBox#bandwidth_settings { | 12:01 |
| nessita | 206 border-style: dotted; | 12:01 |
| nessita | 207 border-color: red; | 12:01 |
| nessita | 208 border-bottom-width: 2px; | 12:01 |
| nessita | 209 } | 12:01 |
| nessita | gatox: and let me show you the screenshot | 12:01 |
| gatox | nessita, ok.... i find a way to do it... but i have to set the qss style for each border in the groupbox and the title | 12:03 |
| gatox | nessita, did you avoid doing that? | 12:03 |
| nessita | gatox: I just set the style I pasted above | 12:04 |
| nessita | gatox: what do you needed to do? | 12:04 |
| nessita | gatox: I got this result http://ubuntuone.com/p/19Hk/ | 12:04 |
| gatox | nessita, weird..... let me take a screenshot... | 12:05 |
| gatox | nessita, ok..... it works..... it looks ugly just in the designer..... | 12:07 |
| gatox | nessita, in the designer i was having this: http://ubuntuone.com/p/19Hm/ | 12:08 |
| nessita | gatox: leaving the transparency aside, that looks right, no? I mean, all the "dark" colors come from you desktop theme | 12:10 |
| gatox | nessita, yes, but if you take a look...... the title is overlaping the first checkbox | 12:11 |
| nessita | ah, you're right! | 12:11 |
| nessita | gatox: do you have a negative pad there? | 12:11 |
| gatox | nessita, i forget to increment the time... i didn't saw the image :P | 12:12 |
| nessita | je | 12:12 |
| gatox | nessita, no, that is with the same qss as always... but i tested on windows and it looks ok | 12:12 |
| gatox | and you tested on linux and it looks ok too | 12:12 |
| gatox | maybe is something in the designer :S | 12:12 |
| nessita | gatox: you know, I keep opening the app to test qss stuff :-/ | 12:13 |
| gatox | nessita, i would do it... but sometimes it takes a while to open in windows.... so for little things that this one i tested on designer | 12:14 |
| gatox | things like* | 12:15 |
| nessita | right | 12:15 |
| ralsina_embassy | nessita, gatox, I am pretty sure you can set the qss from the resource file in designer | 12:15 |
| ralsina_embassy | and yes, I am here for like 12 minutes :-) | 12:15 |
| nessita | ralsina_embassy: you know how? :-) | 12:15 |
| ralsina_embassy | nessita: yes, let me try it | 12:16 |
| nessita | ralsina_embassy: no need, I thought you knew from the top of your hed | 12:16 |
| nessita | head* | 12:16 |
| ralsina_embassy | something like an include in the styleSheet property for the window | 12:16 |
| fagan | embassy? | 12:17 |
| ralsina_embassy | fagan: US embassy, visa | 12:17 |
| fagan | ahhhh | 12:17 |
| * fagan is glad he is irish can work in the US for a year with no issues and no visa also can go over at any time for stays of 90 days or less for any business | 12:18 | |
| czajkowski | fagan: you need a visa, it's a J1 visa | 12:19 |
| fagan | czajkowski: there was actually a new thing brought in that isnt j1 | 12:19 |
| fagan | czajkowski: I cant remember the link but its for graduates | 12:20 |
| fagan | Fill out a form and you can go over for a year and work without a visa | 12:20 |
| czajkowski | fagan: yeah they stll need a working visa, so you can get a national secutiy number. | 12:20 |
| czajkowski | even going to Canada same thing. | 12:21 |
| fagan | czajkowski: thats what the form is for, I wasnt reading too much into it but it is legit | 12:21 |
| mandel | nessita: ping | 12:26 |
| nessita | mandel: pong | 12:26 |
| mandel | nessita: one question, when you use -t or a path to run the tests, is _trial_temp deleted by the tearDown? | 12:27 |
| nessita | mandel: nopes | 12:27 |
| mandel | nessita: cool :) | 12:27 |
| mandel | nessita: then I've got it fixed, we can know run the tests and _trial_temp will have the correct rights :) | 12:27 |
| mandel | nessita: I need an extra review for https://code.launchpad.net/~mandel/ubuntuone-client/fix-file-shelf-windows/+merge/70021 do you know if alecu will be here soon? | 12:28 |
| nessita | mandel: alecu usually starts working in 30 minutes | 12:29 |
| ralsina_embassy | nessita, gatox: http://doc.qt.nokia.com/latest/stylesheet-designer.html | 12:30 |
| mandel | nessita: superb :) | 12:30 |
| nessita | mandel: I will approve that one | 12:30 |
| ralsina_embassy | basically, you can se the whole stylesheet at the main widget level, you just have to remember to remove it before commit ;-) | 12:30 |
| gatox | ralsina_embassy, yes, we were doing that... we were talking about loading the stylesheet from the .qss itself | 12:31 |
| gatox | in designer | 12:31 |
| ralsina_embassy | I wonder if it supports css's include synta | 12:32 |
| ralsina_embassy | syntax | 12:32 |
| nessita | mandel: can you review? https://code.launchpad.net/~nataliabidart/ubuntuone-client/unify-os-helper-tests/+merge/70856 | 12:32 |
| mandel | nessita: on it!! did you take a look at the recursive_move branch? | 12:32 |
| nessita | mandel: not yet | 12:32 |
| nessita | mandel: I want to propose a couple of branches of mine | 12:33 |
| nessita | (first) | 12:33 |
| mandel | nessita: sure, np! | 12:34 |
| ralsina_embassy | Oh, well, so no, you can't @include qss :-( https://bugreports.qt.nokia.com//browse/QTBUG-2047 | 12:36 |
| mandel | nessita, ralsina_embassy: I'm away for 10/15 mins I need to get back home for lunch | 12:42 |
| nessita | mandel: remember the standup | 12:43 |
| mandel | yes, I have enough time | 12:44 |
| * mandel runs | 12:44 | |
| ralsina_embassy | ok, I'm gone again. nessita, I gave gatox my standup a while ago, he can repost it. Have fun! | 12:47 |
| nessita | you too | 12:47 |
| dobey | oi | 12:55 |
| nessita | hi dobey | 12:55 |
| alecu | hi all, I need to be afk for 30' after the standup, but I'll be on time for the windows meeting | 12:56 |
| dobey | hi nessita | 12:56 |
| * mandel back on time! | 12:59 | |
| gatox | mandel, you eat really fast! :P | 12:59 |
| mandel | gatox: I had to drive home for lunch, I'll have lunch after the stand up :) | 12:59 |
| gatox | mandel, ahhhhh jeje | 13:00 |
| fagan | me | 13:00 |
| gatox | me and i have ralsina standup | 13:00 |
| nessita | me | 13:00 |
| alecu | me | 13:01 |
| mandel | me | 13:01 |
| nessita | dobey ? | 13:01 |
| alecu | fagan, gord | 13:01 |
| dobey | me | 13:01 |
| alecu | I mean, go! | 13:01 |
| fagan | DONE | 13:01 |
| fagan | * tested the bundle on vista | 13:01 |
| fagan | * looked at some of the windows client code to see what the fuss is about | 13:01 |
| fagan | TODO | 13:01 |
| fagan | * any intern sized work around | 13:01 |
| fagan | BLOCKED | 13:01 |
| fagan | * nope | 13:01 |
| fagan | gatox: go | 13:01 |
| gatox | DONE: | 13:02 |
| gatox | Page 18 and 19 (Installer) completely funcional (bug: #805296). | 13:02 |
| gatox | Solved: Bug #819943, Bug #819945, Bug #819950, Bug #819953, Bug #819963, Bug #823186 | 13:02 |
| gatox | TODO: | 13:02 |
| gatox | Finish the tests for this branch. | 13:02 |
| gatox | BLOCKED: | 13:02 |
| gatox | No | 13:02 |
| gatox | <ralsina> DONE: installed clean and development XP VMs, improved build scripts (install protocol certificates, redistribute VS2008 runtime, ignore DLLs that broke stuff), built a bundle using that, tested on XP and 7 | 13:02 |
| ubot4 | Launchpad bug 819943 in ubuntuone-windows-installer "The fonts are very small (affects: 1) (heat: 6)" [Undecided,New] https://launchpad.net/bugs/819943 | 13:02 |
| ubot4 | Launchpad bug 819945 in ubuntuone-windows-installer "The button's font should be bigger (affects: 1) (heat: 6)" [Undecided,New] https://launchpad.net/bugs/819945 | 13:02 |
| ubot4 | Launchpad bug 819950 in ubuntuone-windows-installer "Cancel and print buttons do not appear on every screen (affects: 1) (heat: 6)" [Undecided,New] https://launchpad.net/bugs/819950 | 13:02 |
| gatox | <ralsina> TODO: get US visa | 13:02 |
| ubot4 | Launchpad bug 819953 in ubuntuone-windows-installer "Add a folder button don't fit the specs (affects: 1) (heat: 6)" [Undecided,New] https://launchpad.net/bugs/819953 | 13:02 |
| gatox | nessita, go | 13:02 |
| ubot4 | Launchpad bug 819963 in ubuntuone-windows-installer "Fix Syncing the cloud to your computer UI style (affects: 1) (heat: 6)" [Undecided,New] https://launchpad.net/bugs/819963 | 13:02 |
| gord | i don't have anything to report in the standup sorry ;) | 13:02 |
| nessita | DONE: finished debugging LR tests, have 2 news (one bad, one good). The resulting branch is huge, so I'm splitting it into smaller branches and proposing for reviews. | 13:02 |
| nessita | TODO: finish splitting and propose reviews, do a couple of reviews, leave early to university | 13:02 |
| nessita | BLOCKED: nopes | 13:02 |
| nessita | NEXT: alecu | 13:02 |
| alecu | gord, sorry :-) | 13:02 |
| alecu | DONE: lots of debugging: found broken deletes issue on windows, found broken ping on windows, need to investigate this further and create bugs. Also: got clean 7 and XP VMs to test the installers | 13:02 |
| * gatox should change the bug status... | 13:02 | |
| alecu | TODO: discuss recyclebin issues with mandel, work on a solution | 13:02 |
| alecu | BLOCKED: no | 13:02 |
| alecu | NOTE: will be afk for the next 30 minutes | 13:02 |
| alecu | NEXT: mandel | 13:02 |
| mandel | DONE: fixed issues with https://code.launchpad.net/~mandel/ubuntuone-client/dont-use-shutil-move/+merge/70595 added implementation for can_write, access, readonly and read_write using a deny ace to fix bug 817567 | 13:02 |
| mandel | TODO: reviews for nessita, propose fix for 817567, look for work | 13:02 |
| mandel | BLOCKED: no | 13:02 |
| mandel | dobey, go! | 13:02 |
| ubot4 | Launchpad bug 817567 in ubuntuone-client "Windows: can_write and access have the same implementation (affects: 1) (heat: 38)" [High,Triaged] https://launchpad.net/bugs/817567 | 13:02 |
| dobey | λ DONE: branch piloting, bug triage, release prep, bug #822702 | 13:02 |
| dobey | λ TODO: releases, packaging, administrata | 13:02 |
| dobey | λ BLCK: None. | 13:02 |
| ubot4 | Launchpad bug 822702 in ubuntuone-installer "Failing tests in nightlies (affects: 1) (heat: 6)" [High,Fix committed] https://launchpad.net/bugs/822702 | 13:02 |
| nessita | any comments anyone? | 13:03 |
| gatox | nope | 13:03 |
| dobey | this swamp wildfire sucks | 13:03 |
| nessita | alecu: mandel says he already landed a reyclebin solution | 13:03 |
| mandel | nessita: not already, it is porposed | 13:03 |
| nessita | mandel: is merged | 13:03 |
| mandel | nessita: oh, cool :( | 13:04 |
| nessita | mandel: wait | 13:04 |
| mandel | that was a happy smile :P | 13:04 |
| nessita | mandel: is it the one that fixed file shelf? | 13:04 |
| alecu | mandel, nessita: great! | 13:04 |
| alecu | hmm | 13:04 |
| alecu | mandel gone | 13:04 |
| nessita | oh | 13:04 |
| nessita | alecu: I confused myself, is not merged | 13:04 |
| alecu | well, I'll review his branch when I get back | 13:05 |
| * alecu is afk | 13:05 | |
| nessita | alecu: I guess is this one https://code.launchpad.net/~mandel/ubuntuone-client/dont-use-shutil-move/+merge/70595 | 13:05 |
| nessita | mandel: which branch is the one that fixes the recyclebin? | 13:09 |
| mterry | dobey, how did you get the errors you pasted in https://code.launchpad.net/~mterry/ubuntuone-couch/dynamic-timeout/+merge/69642 ? I don't get them from ./run-tests | 13:12 |
| mterry | vds, I have another fresh got-500-status-while-uploading-to-U1 Oops-ID for you: 2047updownZbGIcFBeJAJeaEbAIICecCaCcbBGaGfbc8321 | 13:13 |
| ubot4 | https://lp-oops.canonical.com/oops.py/?oopsid=ID | 13:13 |
| dobey | mterry: really? they happened when tarmac tried to run the tests. weird | 13:13 |
| dobey | mterry: maybe it is ok then, and the disable_ssl is what broke it | 13:13 |
| mterry | dobey, with such small requests, I would expect the timeout to be at 10 | 13:14 |
| dobey | mterry: when did you last push to that branch? just now? | 13:14 |
| mterry | dobey, I did update the branch to quiet a couple spacing warnings from lintian | 13:14 |
| mterry | dobey, yeah | 13:14 |
| dobey | ah ok | 13:15 |
| dobey | i really wish merge proposals could have attachments on comments | 13:16 |
| dobey | for that matter, i wish any comment object in launchpad could have attachments in it. | 13:16 |
| === m_conley_away is now known as m_conley | ||
| mterry | vds, filed bug 823291 about the OOps I've been getting | 13:25 |
| gatox | nessita, all the tests in ubuntuone-windows-installer are working? to know if the problem is here or everywhere | 13:25 |
| ubot4 | mterry: Bug 823291 on http://launchpad.net/bugs/823291 is private | 13:25 |
| nessita | gatox: they fail for me randomly :-/ | 13:25 |
| gatox | nessita, :S | 13:25 |
| nessita | gatox: so I'm guessing there is a timing thing somewhere, probably coming from the use of threads to calculate folder size | 13:26 |
| nessita | gatox: wanna show me your failures? | 13:26 |
| gatox | but test_gui.py has the same issue? | 13:26 |
| gatox | nessita, ^ | 13:26 |
| nessita | gatox: well yeah, isn't that module the one that starts the thread? | 13:27 |
| nessita | gatox: can you please confirm that? | 13:27 |
| gatox | nessita, ok, checking... | 13:27 |
| nessita | mandel, mandel1: added a needs fixing and stopped reviewing until that's fixed, so we don't loose changes | 13:37 |
| nessita | mandel: also, remember to always yield on parent's setUp and add inlineCallbacks decorator | 13:38 |
| mandel | nessita: oh, true I forgot about the setUp, fixing it right now | 13:39 |
| mandel | nessita, ralsina_embassy, alecu, mumble? | 13:46 |
| mandel | @ping | 13:46 |
| ubot4 | pong | 13:46 |
| nessita | mandel: yes! | 13:46 |
| mandel | @ping | 13:52 |
| ubot4 | pong | 13:52 |
| nessita | gatox: mumble? | 13:58 |
| gatox | nessita, ack | 13:58 |
| nessita | Chipaca: mumble? | 13:58 |
| alecu | my mumble broke | 14:00 |
| nessita | alecu, mandel, gatox: let's cancel the mumble, too much technical issues apparently | 14:01 |
| gatox | nessita, ok.. | 14:02 |
| mandel | nessita: ok, then I'm moving to the internetcafe | 14:02 |
| mandel | nessita: if we need mumble later I'm be ready! | 14:03 |
| nessita | alecu: can I have a couple of reviews from you? they are easy | 14:17 |
| alecu | nessita, shoot | 14:18 |
| nessita | alecu: https://code.launchpad.net/~nataliabidart/ubuntuone-client/unify-os-helper-tests/+merge/70856 and https://code.launchpad.net/~nataliabidart/ubuntuone-client/no-more-oses/+merge/70871 (last one is pure search and replace for some os.something methods) | 14:18 |
| dobey | ugh | 14:19 |
| dobey | branches without linked bugs :-/ | 14:19 |
| nessita | dobey: where?!?!?! | 14:19 |
| nessita | alecu: hum, let me superseed the last one, it has a prerequisite that I forgot to set | 14:20 |
| dobey | nessita: already merged. mandel's file_shelf branch, and a few older ones i see in the histroy on lp | 14:20 |
| nessita | ufa :-/ | 14:21 |
| nessita | alecu: new proposal for teh last branch is: https://code.launchpad.net/~nataliabidart/ubuntuone-client/no-more-oses/+merge/70890 | 14:21 |
| alecu | great | 14:21 |
| dobey | all these branches coming in also makes it hard to time a release :) | 14:21 |
| nessita | dobey: for ubuntu, you mean? | 14:24 |
| dobey | yes | 14:25 |
| nessita | dobey: did you talk with ralsina_embassy about this? because honestly, we're not targeting any ubuntu deadline within the windows work :-/ | 14:31 |
| nessita | dobey: and that may complicate your work... | 14:32 |
| dobey | well, the real problem is i don't really have any idea of whether the branches also fix things on linux or not; so it's hard to determine if i should wait for some to land, or just go ahead and block landing for a bit while i roll the tarball. | 14:33 |
| nessita | dobey: well, we are making changes on code that runs in linux in order to make it multiplatform | 14:34 |
| nessita | dobey: so is not like we're fixing bugs in linux, but we're improving the linux code | 14:34 |
| dobey | or maybe causing bugs :) | 14:34 |
| nessita | dobey: is very unlikely, linux tests keep passing and we're making sure they are passing | 14:35 |
| dobey | nessita: sure; i don't exactly trust our tests though. and lately it seems they have gotten slower to run on linux, and use more ram :( | 14:36 |
| Chipaca | nessita: sorry, was in a call | 14:41 |
| nessita | Chipaca: is ok, we dropped the call since mandel kept disconnecting and alecu's mumble wasn't working :-. | 14:42 |
| mandel | nessita: ping | 14:48 |
| nessita | mandel: pong | 14:49 |
| nessita | mandel: were you able to review my branch? | 14:49 |
| mandel | nessita: made the changes you ask for mine, I'm looking at yours now | 14:49 |
| mandel | nessita: I'm getting the following with your branch on windows, is it expected? | 14:54 |
| mandel | the test is ok, but I want to make sure | 14:54 |
| nessita | mandel: what are you getting? | 14:55 |
| mandel | nessita: ups, forgot to paste the link, one sec | 14:56 |
| mandel | nessita: http://paste.ubuntu.com/661990/ | 14:56 |
| nessita | mandel: yes, I added that print message becasue the shutil.rmtree has ignoreeerros=True | 14:57 |
| nessita | mandel: and we *don't* want to ignore errors, but if I remove the flag we get failures | 14:57 |
| nessita | mandel: so I added the print so we start debugging what we're doing wrong (I'm fixing a lot of those in incoming branches) | 14:57 |
| mandel | nessita: ok, sounds good, tests pass on windows, will run them on linux and will do the code review | 14:58 |
| nessita | ack | 14:58 |
| mandel | nessita: in the mean time I have updated the shutil.move branch and proposed the can_write, access branch and will do tests with FAT to see what happens | 14:59 |
| nessita | mandel: I'm re-reviewing the shutil one | 14:59 |
| mandel | nessita: have you heard anything from veterok regarding the issue with tritcask and another process accessing the file? | 14:59 |
| mandel | seems that some of the last errors in fsm are due to that | 14:59 |
| nessita | mandel: there is no such issue :-) | 14:59 |
| mandel | nessita: what do you mean? | 15:00 |
| nessita | mandel: the test itself creates 2 tritcask DBs | 15:00 |
| nessita | so we need to fix the tests, I have that in my plate | 15:00 |
| mandel | nessita: ah… so we have to fix the test then :) | 15:00 |
| mandel | nessita: superb! I'll then move with your review and will see if have any bugs assigned | 15:00 |
| nessita | alecu: ping | 15:02 |
| alecu | nessita, pong | 15:04 |
| nessita | alecu: is your VM date and time correct? | 15:04 |
| alecu | nessita, it drifts | 15:05 |
| nessita | alecu: maybe that is your oauth issue? | 15:05 |
| alecu | nessita, sometimes I need to reset it... but when this happens the error shows up on the stacktrace | 15:05 |
| alecu | at least I remember seeing it on the control panel stderr | 15:06 |
| === JanC_ is now known as JanC | ||
| nessita | alecu: shows as the "an error representing an authentication failure"? | 15:06 |
| alecu | nessita, it shows as a 400 from the web client, saying "the time difference is greater than 900" or something like that. | 15:07 |
| alecu | "the web client" -> the control panel web service client | 15:07 |
| dobey | and ugh, deleted branches/proposals :( | 15:08 |
| === m_conley is now known as m_conley_away | ||
| nessita | alecu: hum, I never saw that. But if the sso says UNAUTHORIZED + 'An exception representing an authentica tion failure.', is probably a date/time issue | 15:08 |
| alecu | nessita, I'll check that. Thanks for the idea. | 15:10 |
| dobey | hooray, 1.7.1 tarball made/uploaded. | 15:20 |
| mandel | nessita: ping | 15:23 |
| nessita | mandel: pong, I was about t ping you :-D | 15:23 |
| nessita | mandel: you first | 15:23 |
| mandel | nessita: ok, regarding the unify branch, I'm wondering why you removed the use of getattr in line 8 of the diff, are we always sure that __root is there? | 15:24 |
| nessita | mandel: yes, I defined it in setUp | 15:25 |
| mandel | nessita: I mean, I saw you added the setUp definition, but do we know why was it being used like that? | 15:25 |
| mandel | I think that is some old code, but I'm not 100% sure | 15:25 |
| nessita | mandel: no we don't, but the way that was being used is somehow "dirty" and make the tearDown code do not remove the __root contents all the time | 15:25 |
| nessita | mandel: my turn? | 15:26 |
| mandel | nessita: ok, and one last question :P | 15:26 |
| nessita | mandel: shoot | 15:26 |
| mandel | why 0774 and not 0664? in line 471 | 15:26 |
| mandel | nessita: ^ | 15:26 |
| nessita | mandel: for files you need to set 0664, with 7 you're turning on the executable bit, which we don't want for files | 15:27 |
| mandel | nessita: ok, that is all from me +1 :) | 15:27 |
| nessita | mandel: so, for dirs, 0775, for files, 0664 | 15:27 |
| mandel | nessita: te toca | 15:28 |
| nessita | mandel: my question | 15:28 |
| nessita | mandel: why you added this? 37+ expected_result = listdir(self.basedir) | 15:28 |
| nessita | mandel: the test only defines expected_result id is not passed as parameter | 15:28 |
| nessita | mandel: with your adding, you're overriding the passed parameter | 15:29 |
| nessita | mandel: so you should modify expected_result only if it's not NOne | 15:29 |
| dobey | and time for lunch. bbiab | 15:31 |
| mandel | nessita: I found the issue, RecrsiveTest should not inherit from OsWrapperTests, I'll fix that easily | 15:52 |
| nessita | mandel: ack | 15:52 |
| mandel | nessita: changed pushed, anything else you can find? | 15:58 |
| nessita | mandel: yes, tests are failing in linux, you run the suite? | 15:59 |
| mandel | nessita: really? I did run it, let me check again | 16:00 |
| nessita | exceptions.OSError: [Errno 1] Operation not permitted: '/tmp' | 16:00 |
| nessita | tests.platform.test_os_helper.RecursiveMoveTests.test_move_dir | 16:00 |
| mandel | nessita: yes, self.rmtree does not know hoe to deal with the /tmp when it tries to do a chmod, shall I change that in the test or in testcase? | 16:11 |
| mandel | simplest way would be to add a print ERROR: cold not remove %r | 16:11 |
| nessita | mandel: our tests should not be writting to /tmp, we agreed on that | 16:11 |
| mandel | other would be to ensure that rmtree knows what to do | 16:11 |
| nessita | mandel: no, we should not write to /tmp | 16:12 |
| === beuno is now known as beuno-lunch | ||
| mandel | nessita: I implemented the print error idea, let me know | 16:17 |
| nessita | mandel: did you read we should not be writting to /tmp in our tests? | 16:18 |
| nessita | mandel: I mentioned that twice, maybe your internet lost the messages? | 16:18 |
| nessita | mandel: you reading me? :-) | 16:19 |
| === m_conley_away is now known as m_conley | ||
| nessita | mandel: ping | 16:23 |
| nessita | alecu: I'm leaving soon, any news on the reviews? | 16:24 |
| mandel | nessita: I did now.. weird | 16:24 |
| nessita | mandel: so, 2 things: | 16:24 |
| nessita | * we should not write to /tmp anywhere | 16:25 |
| mandel | nessita: I could change the test not to use /tmp but then we do not test if there is a diff fs, which could occur on linux | 16:25 |
| nessita | * the windowspath decorator already validates the input path to be syncdaemon's valid, so you need to remove the calls to @is_valid_syncdaemon_path() | 16:25 |
| nessita | mandel: we talked about this, you mentioned you will assert over shutil.move being used | 16:26 |
| nessita | mandel: since having /tmp in another FS is a random lottery | 16:26 |
| alecu | nessita, I'm finishing the first; one question: why are you using a "print" for the error in rmtree? | 16:26 |
| nessita | mandel: ack to 1 and 2? I'm leaving soon | 16:27 |
| mandel | nessita: yeah, that is how the shutil.move test approach the idea within the python tests itself.. I'll remove that | 16:27 |
| nessita | alecu: you mean vs failing the test? | 16:27 |
| === mterry_ is now known as mterry | ||
| alecu | nessita, vs logger | 16:27 |
| mandel | nessita: yeah, will do the right now | 16:27 |
| nessita | alecu: I want to detect those cases and start cleaning them de a poco, but I don't want to trigger more failures because tearDownfailed | 16:27 |
| nessita | alecu: but since is a tearDown, we don't necessarily have a logger setup | 16:28 |
| nessita | alecu: also, almost no one reads the logs after tests have passed | 16:28 |
| nessita | alecu: this definitely caughts attention from the runner | 16:28 |
| nessita | alecu: I'm planning on removing that as soon as all the tests are fixed | 16:30 |
| alecu | nessita, it makes sense. Would you mind adding a comment explaining that it's a temporal thing, and that we should replace it with an error? | 16:30 |
| nessita | alecu: I don't mind, doing it right now | 16:30 |
| nessita | alecu: pushed | 16:33 |
| nessita | alecu: if/when you approve, would you globally approve, please? | 16:33 |
| nessita | I gotta run!!! | 16:34 |
| nessita | bye all | 16:34 |
| alecu | sure. bye! | 16:34 |
| mandel | ralsina_embassy, alecu, gatox: is EOD for me, I've sent you an email with some reviews I need to have so that SD works better :) | 16:53 |
| mandel | on windows I mean :P | 16:53 |
| alecu | mandel, great, thanks. | 16:53 |
| gatox | mandel, ok... i'll check that! good bye | 16:53 |
| alecu | mandel, I'll review after I finish with nessita's | 16:53 |
| mandel | alecu: np :) | 16:54 |
| mandel | alecu: I'd like your input a bout the Deny ACE impementationI have done, it should do the trick in order to keep the old ACEs | 16:55 |
| alecu | mandel, great, I'll review it soonish | 16:56 |
| === m_conley is now known as m_conley_away | ||
| mandel | alecu: I think it covers our problems, yet it might be to restritive, take a IRL test if you wish to see if I undertood the issue :D | 16:58 |
| mandel | anyways, I'm done, my ass hurts, the chairs of the internet cafe are shit! | 16:58 |
| alecu | mandel, that's just like what she said. | 16:59 |
| mandel | alecu: hehehe, good one | 17:01 |
| * mandel runs away! | 17:02 | |
| === beuno-lunch is now known as beuno | ||
| dobey | yay, UPS | 17:35 |
| gatox | i'm going to the gym... i'll be back around 16:30 ART to keep fighting with installer's tests! | 17:47 |
| gatox | ralsina_embassy, alecu ^ | 17:47 |
| alecu | gatox, ack! | 17:48 |
| === m_conley_away is now known as m_conley | ||
| * alecu is having lunch | 18:15 | |
| === ralsina_embassy is now known as ralsina | ||
| Fury1 | Hi! I'm (still) having problems getting the evolution calendar and ubuntuone working together. I'm aware of this bug comment https://bugs.launchpad.net/ubuntu/+source/evolution/+bug/495377/comments/15 | 19:01 |
| Fury1 | But I have to uncheck the box, switch to the calendar, switch back to contacts, recheck the box and finally switch back to calendar. | 19:01 |
| ubot4 | Launchpad bug 495377 in evolution (Ubuntu) "Evolution Birthday Calendar Doesn`t show Ubuntu One contacts birthdays (affects: 10) (dups: 3) (heat: 35)" [Low,Fix released] | 19:01 |
| dobey | not sure why that would be. sounds like perhaps a bug in evolution. what version of evolution and ubuntu are you using? | 19:05 |
| Fury1 | natty amd64 - Evolution 2.32.2 | 19:06 |
| Fury1 | I have a feeling, that the connection to ubuntuone isn't there when evolution is first started. When I switch to my ubuntuone adress book it first shows now contacts and after some seconds I can see them. | 19:08 |
| dobey | ok. i'm not sure what would cause that. could you file a new bug please, by running "ubuntu-bug evolution-couchdb" in a terminal? | 19:09 |
| dobey | hmm, maybe | 19:09 |
| Fury1 | https://bugs.launchpad.net/ubuntu/+source/evolution-couchdb/+bug/823500 | 19:18 |
| ubot4 | Launchpad bug 823500 in evolution-couchdb (Ubuntu) "Evolution Birthday Calendar Doesn`t show Ubuntu One contacts birthdays (affects: 1) (heat: 6)" [Undecided,New] | 19:18 |
| dobey | thanks | 19:20 |
| === M0hi is now known as IAmNotThatGuy | ||
| === m_conley is now known as m_conley_away | ||
| dobey | have a good evening everyone | 21:28 |
| tntc | CardinalFang: hey! how goes the server battle? | 22:52 |
| === yofel_ is now known as yofel | ||
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!