tntcCardinalFang_: oh! I was afk, sorry00:18
tntcCardinalFang_: I pay for it, but youse guys have already comped me a year of service, so I have no real complaint :)00:18
tntcCardinalFang_: I just want to help fix it!00:19
tntcGet 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
tntcI want to get me my playlists* . Wow, the first three words got cut off there00:20
tntcWhat specifically is wrong with the server?00:21
tntcand 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
tntcCardinalFang_: 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
JamesTaitMorning all!08:27
faganralsina: http://ubuntuone/p/19GA08:54
faganmandel: morning08:55
mandelfagan: is ralsina around?08:55
faganmandel: nope just for later before I forget08:55
mandelfagan: oh, backlog :P08:55
faganLol forgot to add the .com :D08:56
faganralsina: http://ubuntuone/p/19GA08:56
faganmandel: Am trying to test his bundle on vista with no luck :/08:56
mandelfagan: hm… what is the problem?08:58
faganmandel: 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 screen08:59
faganmandel: probably has something to do with vista09:00
mandelfagan: 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 look09:02
faganmandel: 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 later09:03
* fagan used pretty floury language in that bug report. Id say none of the team would know in what context I used former09:07
mandelhe, just make sure it makes some sense :P09:07
faganmandel: yeah it does :D09:07
faganFormer = first one latter = last one :)09:09
faganyo gatox09:40
mandelgatox: hola!09:48
lisettegatox: hi, I was just wondering if you want a prefix for the win installer bugs?10:01
gatoxmandel, lisette hi10:02
lisettegatox: QT this that and the other10:02
gatoxlisette, could be "qt-ui"??10:02
gatoxlisette, yes, something like that10:03
lisettegatox: sure, cheers! :)10:03
gatoxlisette, thanks10:03
gatoxmandel, is your connection ok today?10:04
mandelgatox: much better, but I had to go to a diff village...10:06
mandelgatox: this country is a joke...10:06
gatoxmandel, bummer!10:11
mandelgatox: at least I can work, I'll see if I can fix something for later...10:11
gatoxmandel, i'm writing tests :P10:12
mandelgatox: haha wanna swap?10:13
gatoxmandel, jeje i can't... i want to live! jeje10:13
mandelgatox: I'm dealing with ACEs and ACLs on windows :P10:13
gatoxmandel, but i can help you with  something maybe if you need...10:14
mandelgatox: 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
gatoxmandel, ok, let me know if i could do something anyhow! :D10:16
mandelgatox: sure, will do :)10:21
ralsinafagan: another one, try to get a result faster this time ;-)10:30
ralsinafagan: http://ubuntuone.com/p/17lC/10:30
ralsinagood morning!10:30
ralsinamandel, 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 time10:32
gatoxralsina, ok... i can try out what is happening with the buttons using your bundle?10:33
ralsinagatox: the local folders page has no next button, and has a finish button :-)10:33
gatoxralsina, ahhhhhh yes, but that is fixed with my branch, was the same problem i was having10:34
ralsinagatox: awesome, so that's one less thing I need to worry about :-)10:34
gatoxralsina, totally10:34
ralsinaand BTW: after the stuff I have done yesterday, I hate windows now10:34
gatoxralsina, now?? it take you more than expected :P10:35
ralsinagatox: I am not a hating guy10:35
gatoxralsina, jeejje10:35
gatoxralsina, good for your health :P10:35
ralsinagatox: but an operating system where *having* a DLL makes it give "can't find the method" errors, is hateable10:36
gatoxralsina, 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
ralsinagatox: I have read everything Neal stephenson has written :-)10:38
gatoxralsina, awesome! :D10:38
ralsinagatox: including his 3 1500 adventure novels about the birth of the european monetary systems ;-)10:39
ralsinas/1500/1500 pages/10:39
gatoxralsina, ok... you win! jjee10:39
ralsinagatox: read them, they kick ass! It's called the baroque cycle!10:39
* gatox searching...10:40
ralsinaQuicksilver, the system of the world, and I can't remember the name of the other one10:40
ralsinait has newton poking needles in his eye!10:41
gatoxralsina, i will look for his books then10:43
gatoxralsina, Quicksilver, The Confusion, and The System of the World10:43
ralsinagatox: ok, start with something more accessible. Like cryptonomicon. It has a operating system called Finux :-)10:43
gatoxralsina, will do10:44
ralsinaok, gotta run. here's my standup for later, please pass it on!10:44
ralsinaDONE: 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 710:45
ralsinaTODO: get US visa10:45
gatoxralsina, ok10:47
duanedesignmorning all10:47
mandelralsina: can you take a look at my morning report?10:50
ralsinamandel: sure!10:50
mandelduanedesign: morning!!! and I forgot to welcome you to the team :P10:50
duanedesignthank you mandel !10:50
mandelralsina: there is a proposal to solve the read_write issue using deny aces on the acl of a file10:50
duanedesignmandel: oh, you have macbook dont you? /me trying to remember what you were using at UDS10:51
mandelduanedesign: yes I do, macbook pro :)10:52
ralsinamandel: I think that's a fairly harmless solution10:52
ralsinamandel: when are we setting readonly again?10:52
ralsinamandel: just for ro shares?10:52
duanedesignmandel: everything work ok with Ubuntu?10:53
* duanedesign is looking for a new laptop10:53
mandelralsina: 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 enough10:55
mandelralsina: all the other ace are left untouched :)10:55
ralsinamandel: yes, he can do the same thing on linux, so that makes sense10:55
mandelduanedesign: I do not recommend you to get a macbook at all, and I think verterok would say the same10:56
mandelduanedesign: thinkpad is the way to go10:56
ralsinabesides we have lucio's promise that doing that doesn't break things server side, so...10:56
ralsinaok, really gotta run now. Have fun, please write reports today!10:56
mandelralsina: 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
duanedesignmandel: yeah that is what I am leaning towards (thinkpad) Just trying to decide between a T520 or W52010:58
duanedesignmandel: thanks10:58
faganok ralsina its working now11:00
faganalthough I have 2 comments11:00
faganIt spawns a lot of terminal windows and the enter key isnt bound11:00
faganso if I hit enter when typing my password lets say it doesnt actually go forward11:01
ralsinafagan: yes11:01
ralsinafagan: kown problems both, with easy solutions both :-)11:01
faganralsina: cool11:02
=== ralsina is now known as ralsina_embassy
nessitagood morning everyone!11:04
gatoxnessita, good morning11:04
nessitahi gatox!11:05
mandelduanedesign: np11:05
mandelnessita: morning, sorry for yesterdays internet issues… there was terrible wind here and everything was broken…11:05
mandelnessita: can you take a look at the report and let me know what you think about the readwrite proposed solution11:06
nessitamandel: hi there! were you able to fix all the things I mentioned in the shutil branch?11:06
mandelnessita: yeah, that has been pushed already11:06
nessitamandel: will that solution work with fat systems?11:08
nessita(I m guessing no)11:08
mandelnessita: 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 long11:09
nessitamandel: will that solution work with fat systems?11:09
mandelnessita: good question, I have not tried it on FAT, do you have one of those?11:11
nessitamandel: no11:11
nessitamandel: but I guess any pendrive will do11:11
nessitamandel: you could format a pendrive with fat32 using any linux tool11:11
mandelnessita: ok, I'll do that… man I hate all this back compatibility on windows11:13
mandelnessita: I'll do some IRL tests right now...11:13
nessitamandel: anyways, I doubt any solution will work on fat, since that FS, if I remember correctly, has 0 support for permissions11:14
mandelnessita: hmmm I wonder if we can set that as a min requirement for RO shares...11:15
nessitamandel: what would be "that"?11:15
mandelnessita: using NTFS and not FAT11:18
nessitamandel: the easiest solution is not to change any perm in any folder11:18
nessitain any FS11:18
nessitagatox: I'm restarting the review on your controlpanel branch11:26
gatoxnessita, thanks..... i'm doing test for my other branch of the installer11:28
nessitagatox: question. Why would you add that QFrame again?11:34
gatoxnessita, 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 panel11:35
nessitagatox: right, but you can set a border-bottom to the agroupbox11:35
nessitathe one named bandwidth_settings11:36
gatoxnessita, right...... let me test that11:36
gatoxnessita, yes... that branch can be rejected11:42
gatoxnessita, did you reject the branch already?11:52
gatoxbecause maybe will be necessary11:52
gatoxnessita, because if i set the border to the qgroupbox, that widget lost the border properties for right, top and left11:54
nessitagatox: I did not reject it, no. What do you mean "that widget lost the border prop"?11:56
gatoxnessita, so... qgroupbox has the border property... but it affect the other borders, even if you only say border-bottom11:56
nessitagatox: you sure?11:57
gatoxnessita, yep... test it on designer11:57
nessitagatox: but I'be done it that way with the profile box in the account tab11:57
gatoxnessita, control panel qss?11:58
nessitayeap (but is not in trunk)11:58
nessitagatox: trying to reproduce now...11:58
nessitagatox: 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
nessitagatox: and let me show you the screenshot12:01
gatoxnessita, ok.... i find a way to do it... but i have to set the qss style for each border in the groupbox and the title12:03
gatoxnessita, did you avoid doing that?12:03
nessitagatox: I just set the style I pasted above12:04
nessitagatox: what do you needed to do?12:04
nessitagatox: I got this result http://ubuntuone.com/p/19Hk/12:04
gatoxnessita, weird..... let me take a screenshot...12:05
gatoxnessita, ok..... it works..... it looks ugly just in the designer.....12:07
gatoxnessita, in the designer i was having this: http://ubuntuone.com/p/19Hm/12:08
nessitagatox: leaving the transparency aside, that looks right, no? I mean, all the "dark" colors come from you desktop theme12:10
gatoxnessita, yes, but if you take a look...... the title is overlaping the first checkbox12:11
nessitaah, you're right!12:11
nessitagatox: do you have a negative pad there?12:11
gatoxnessita, i forget to increment the time... i didn't saw the image :P12:12
gatoxnessita, no, that is with the same qss as always... but i tested on windows and it looks ok12:12
gatoxand you tested on linux and it looks ok too12:12
gatoxmaybe is something in the designer :S12:12
nessitagatox: you know, I keep opening the app to test qss stuff :-/12:13
gatoxnessita, i would do it... but sometimes it takes a while to open in windows.... so for little things that this one i tested on designer12:14
gatoxthings like*12:15
ralsina_embassynessita, gatox, I am pretty sure you can set the qss from the resource file in designer12:15
ralsina_embassyand yes, I am here for like 12 minutes :-)12:15
nessitaralsina_embassy: you know how? :-)12:15
ralsina_embassynessita: yes, let me try it12:16
nessitaralsina_embassy: no need, I thought you knew from the top of your hed12:16
ralsina_embassysomething like an include in the styleSheet property for the window12:16
ralsina_embassyfagan: US embassy, visa12: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
czajkowskifagan: you need a visa, it's a J1 visa12:19
faganczajkowski: there was actually a new thing brought in that isnt j112:19
faganczajkowski: I cant remember the link but its for graduates12:20
faganFill out a form and you can go over for a year and work without a visa12:20
czajkowskifagan: yeah they stll need a working visa, so you can get a national secutiy number.12:20
czajkowskieven going to Canada same thing.12:21
faganczajkowski: thats what the form is for, I wasnt reading too much into it but it is legit12:21
mandelnessita: ping12:26
nessitamandel: pong12:26
mandelnessita: one question, when you use -t or a path to run the tests, is _trial_temp deleted by the tearDown?12:27
nessitamandel: nopes12:27
mandelnessita: cool :)12:27
mandelnessita: then I've got it fixed, we can know run the tests and _trial_temp will have the correct rights :)12:27
mandelnessita: 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
nessitamandel: alecu usually starts working in 30 minutes12:29
ralsina_embassynessita, gatox: http://doc.qt.nokia.com/latest/stylesheet-designer.html12:30
mandelnessita: superb :)12:30
nessitamandel: I will approve that one12:30
ralsina_embassybasically, you can se the whole stylesheet at the main widget level, you just have to remember to remove it before commit ;-)12:30
gatoxralsina_embassy, yes, we were doing that... we were talking about loading the stylesheet from the .qss itself12:31
gatoxin designer12:31
ralsina_embassyI wonder if it supports css's include synta12:32
nessitamandel: can you review? https://code.launchpad.net/~nataliabidart/ubuntuone-client/unify-os-helper-tests/+merge/7085612:32
mandelnessita: on it!! did you take a look at the recursive_move branch?12:32
nessitamandel: not yet12:32
nessitamandel: I want to propose a couple of branches of mine12:33
mandelnessita: sure, np!12:34
ralsina_embassyOh, well, so no, you can't @include qss :-( https://bugreports.qt.nokia.com//browse/QTBUG-204712:36
mandelnessita, ralsina_embassy: I'm away for 10/15 mins I need to get back home for lunch12:42
nessitamandel: remember the standup12:43
mandelyes, I have enough time12:44
* mandel runs12:44
ralsina_embassyok, I'm gone again. nessita, I gave gatox my standup a while ago, he can repost it. Have fun!12:47
nessitayou too12:47
nessitahi dobey12:55
alecuhi all, I need to be afk for 30' after the standup, but I'll be on time for the windows meeting12:56
dobeyhi nessita12:56
* mandel back on time!12:59
gatoxmandel, you eat really fast! :P12:59
mandelgatox: I had to drive home for lunch, I'll have lunch after the stand up :)12:59
gatoxmandel, ahhhhh jeje13:00
gatoxme and i have ralsina standup13:00
nessitadobey ?13:01
alecufagan, gord13:01
alecuI mean, go!13:01
fagan* tested the bundle on vista13:01
fagan* looked at some of the windows client code to see what the fuss is about13:01
fagan* any intern sized work around13:01
fagan* nope13:01
fagangatox: go13:01
gatoxPage 18 and 19 (Installer) completely funcional (bug: #805296).13:02
gatoxSolved: Bug #819943, Bug #819945, Bug #819950, Bug #819953, Bug #819963, Bug #82318613:02
gatoxFinish the tests for this branch.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 713:02
ubot4Launchpad bug 819943 in ubuntuone-windows-installer "The fonts are very small (affects: 1) (heat: 6)" [Undecided,New] https://launchpad.net/bugs/81994313:02
ubot4Launchpad bug 819945 in ubuntuone-windows-installer "The button's font should be bigger (affects: 1) (heat: 6)" [Undecided,New] https://launchpad.net/bugs/81994513:02
ubot4Launchpad 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/81995013:02
gatox<ralsina> TODO: get US visa13:02
ubot4Launchpad 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/81995313:02
gatoxnessita, go13:02
ubot4Launchpad 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/81996313:02
gordi don't have anything to report in the standup sorry ;)13:02
nessitaDONE: 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
nessitaTODO: finish splitting and propose reviews, do a couple of reviews, leave early to university13:02
nessitaBLOCKED: nopes13:02
nessitaNEXT: alecu13:02
alecugord, sorry :-)13:02
alecuDONE: 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 installers13:02
* gatox should change the bug status...13:02
alecuTODO: discuss recyclebin issues with mandel, work on a solution13:02
alecuBLOCKED: no13:02
alecuNOTE: will be afk for the next 30 minutes13:02
alecuNEXT: mandel13:02
mandelDONE: 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 81756713:02
mandelTODO: reviews for nessita, propose fix for 817567, look for work13:02
mandelBLOCKED: no13:02
mandeldobey, go!13:02
ubot4Launchpad bug 817567 in ubuntuone-client "Windows: can_write and access have the same implementation (affects: 1) (heat: 38)" [High,Triaged] https://launchpad.net/bugs/81756713:02
dobeyλ DONE: branch piloting, bug triage, release prep, bug #82270213:02
dobeyλ TODO: releases, packaging, administrata13:02
dobeyλ BLCK: None.13:02
ubot4Launchpad bug 822702 in ubuntuone-installer "Failing tests in nightlies (affects: 1) (heat: 6)" [High,Fix committed] https://launchpad.net/bugs/82270213:02
nessitaany comments anyone?13:03
dobeythis swamp wildfire sucks13:03
nessitaalecu: mandel says he already landed a reyclebin solution13:03
mandelnessita: not already, it is porposed13:03
nessitamandel: is merged13:03
mandelnessita: oh, cool :(13:04
nessitamandel: wait13:04
mandelthat was a happy smile :P13:04
nessitamandel: is it the one that fixed file shelf?13:04
alecumandel, nessita: great!13:04
alecumandel gone13:04
nessitaalecu: I confused myself, is not merged13:04
alecuwell, I'll review his branch when I get back13:05
* alecu is afk13:05
nessitaalecu: I guess is this one https://code.launchpad.net/~mandel/ubuntuone-client/dont-use-shutil-move/+merge/7059513:05
nessitamandel: which branch is the one that fixes the recyclebin?13:09
mterrydobey, 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-tests13:12
mterryvds, I have another fresh got-500-status-while-uploading-to-U1 Oops-ID for you: 2047updownZbGIcFBeJAJeaEbAIICecCaCcbBGaGfbc832113:13
dobeymterry: really? they happened when tarmac tried to run the tests. weird13:13
dobeymterry: maybe it is ok then, and the disable_ssl is what broke it13:13
mterrydobey, with such small requests, I would expect the timeout to be at 1013:14
dobeymterry: when did you last push to that branch? just now?13:14
mterrydobey, I did update the branch to quiet a couple spacing warnings from lintian13:14
mterrydobey, yeah13:14
dobeyah ok13:15
dobeyi really wish merge proposals could have attachments on comments13:16
dobeyfor 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
mterryvds, filed bug 823291 about the OOps I've been getting13:25
gatoxnessita, all the tests in ubuntuone-windows-installer are working? to know if the problem is here or everywhere13:25
ubot4mterry: Bug 823291 on http://launchpad.net/bugs/823291 is private13:25
nessitagatox: they fail for me randomly :-/13:25
gatoxnessita, :S13:25
nessitagatox: so I'm guessing there is a timing thing somewhere, probably coming from the use of threads to calculate folder size13:26
nessitagatox: wanna show me your failures?13:26
gatoxbut test_gui.py has the same issue?13:26
gatoxnessita, ^13:26
nessitagatox: well yeah, isn't that module the one that starts the thread?13:27
nessitagatox: can you please confirm that?13:27
gatoxnessita, ok, checking...13:27
nessitamandel, mandel1: added a needs fixing and stopped reviewing until that's fixed, so we don't loose changes13:37
nessitamandel: also, remember to always yield on parent's setUp and add inlineCallbacks decorator13:38
mandelnessita: oh, true I forgot about the setUp, fixing it right now13:39
mandelnessita, ralsina_embassy, alecu, mumble?13:46
nessitamandel: yes!13:46
nessitagatox: mumble?13:58
gatoxnessita, ack13:58
nessitaChipaca: mumble?13:58
alecumy mumble broke14:00
nessitaalecu, mandel, gatox: let's cancel the mumble, too much technical issues apparently14:01
gatoxnessita, ok..14:02
mandelnessita: ok, then I'm moving to the internetcafe14:02
mandelnessita: if we need mumble later I'm be ready!14:03
nessitaalecu: can I have a couple of reviews from you? they are easy14:17
alecunessita, shoot14:18
nessitaalecu: 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
dobeybranches without linked bugs :-/14:19
nessitadobey: where?!?!?!14:19
nessitaalecu: hum, let me superseed the last one, it has a prerequisite that I forgot to set14:20
dobeynessita: already merged. mandel's file_shelf branch, and a few older ones i see in the histroy on lp14:20
nessitaufa :-/14:21
nessitaalecu: new proposal for teh last branch is: https://code.launchpad.net/~nataliabidart/ubuntuone-client/no-more-oses/+merge/7089014:21
dobeyall these branches coming in also makes it hard to time a release :)14:21
nessitadobey: for ubuntu, you mean?14:24
nessitadobey: did you talk with ralsina_embassy about this? because honestly, we're not targeting any ubuntu deadline within the windows work :-/14:31
nessitadobey: and that may complicate your work...14:32
dobeywell, 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
nessitadobey: well, we are making changes on code that runs in linux in order to make it multiplatform14:34
nessitadobey: so is not like we're fixing bugs in linux, but we're improving the linux code14:34
dobeyor maybe causing bugs :)14:34
nessitadobey: is very unlikely, linux tests keep passing and we're making sure they are passing14:35
dobeynessita: 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
Chipacanessita: sorry, was in a call14:41
nessitaChipaca: is ok, we dropped the call since mandel kept disconnecting and alecu's mumble wasn't working :-.14:42
mandelnessita: ping14:48
nessitamandel: pong14:49
nessitamandel: were you able to review my branch?14:49
mandelnessita: made the changes you ask for mine, I'm looking at yours now14:49
mandelnessita: I'm getting the following with your branch on windows, is it expected?14:54
mandelthe test is ok, but I want to make sure14:54
nessitamandel: what are you getting?14:55
mandelnessita: ups, forgot to paste the link, one sec14:56
mandelnessita: http://paste.ubuntu.com/661990/14:56
nessitamandel: yes, I added that print message becasue the shutil.rmtree has ignoreeerros=True14:57
nessitamandel: and we *don't* want to ignore errors, but if I remove the flag we get failures14:57
nessitamandel: 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
mandelnessita: ok, sounds good, tests pass on windows, will run them on linux and will do the code review14:58
mandelnessita: 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 happens14:59
nessitamandel: I'm re-reviewing the shutil one14:59
mandelnessita: have you heard anything from veterok regarding the issue with tritcask and another process accessing the file?14:59
mandelseems that some of the last errors in fsm are due to that14:59
nessitamandel: there is no such issue :-)14:59
mandelnessita: what do you mean?15:00
nessitamandel: the test itself creates 2 tritcask DBs15:00
nessitaso we need to fix the tests, I have that in my plate15:00
mandelnessita: ah… so we have to fix the test then :)15:00
mandelnessita: superb! I'll then move with your review and will see if  have any bugs assigned15:00
nessitaalecu: ping15:02
alecunessita, pong15:04
nessitaalecu: is your VM date and time correct?15:04
alecunessita, it drifts15:05
nessitaalecu: maybe that is your oauth issue?15:05
alecunessita, sometimes I need to reset it... but when this happens the error shows up on the stacktrace15:05
alecuat least I remember seeing it on the control panel stderr15:06
=== JanC_ is now known as JanC
nessitaalecu: shows as the "an error representing an authentication failure"?15:06
alecunessita, 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 client15:07
dobeyand ugh, deleted branches/proposals :(15:08
=== m_conley is now known as m_conley_away
nessitaalecu: hum, I never saw that. But if the sso says UNAUTHORIZED + 'An exception representing an authentica tion failure.', is probably a date/time issue15:08
alecunessita, I'll check that. Thanks for the idea.15:10
dobeyhooray, 1.7.1 tarball made/uploaded.15:20
mandelnessita: ping15:23
nessitamandel: pong, I was about t ping you :-D15:23
nessitamandel: you first15:23
mandelnessita: 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
nessitamandel: yes, I defined it in setUp15:25
mandelnessita: I mean, I saw you added the setUp definition, but do we know why was it being used like that?15:25
mandelI think that is some old code, but I'm not 100% sure15:25
nessitamandel: 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 time15:25
nessitamandel: my turn?15:26
mandelnessita: ok, and one last question :P15:26
nessitamandel: shoot15:26
mandelwhy 0774 and not 0664? in line 47115:26
mandelnessita: ^15:26
nessitamandel: for files you need to set 0664, with 7 you're turning on the executable bit, which we don't want for files15:27
mandelnessita: ok, that is all from me +1 :)15:27
nessitamandel: so, for dirs, 0775, for files, 066415:27
mandelnessita: te toca15:28
nessitamandel: my question15:28
nessitamandel: why you added this? 37+ expected_result = listdir(self.basedir)15:28
nessitamandel: the test only defines expected_result id is not passed as parameter15:28
nessitamandel: with your adding, you're overriding the passed parameter15:29
nessitamandel: so you should modify expected_result only if it's not NOne15:29
dobeyand time for lunch. bbiab15:31
mandelnessita: I found the issue, RecrsiveTest should not inherit from OsWrapperTests, I'll fix that easily15:52
nessitamandel: ack15:52
mandelnessita: changed pushed, anything else you can find?15:58
nessitamandel: yes, tests are failing in linux, you run the suite?15:59
mandelnessita: really? I did run it, let me check again16:00
nessitaexceptions.OSError: [Errno 1] Operation not permitted: '/tmp'16:00
mandelnessita: 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
mandelsimplest way would be to add a print ERROR: cold not remove %r16:11
nessitamandel: our tests should not be writting to /tmp, we agreed on that16:11
mandelother would be to ensure that rmtree knows what to do16:11
nessitamandel: no, we should not write to /tmp16:12
=== beuno is now known as beuno-lunch
mandelnessita: I implemented the print error idea, let me know16:17
nessitamandel: did you read we should not be writting to /tmp in our tests?16:18
nessitamandel: I mentioned that twice, maybe your internet lost the messages?16:18
nessitamandel: you reading me? :-)16:19
=== m_conley_away is now known as m_conley
nessitamandel: ping16:23
nessitaalecu: I'm leaving soon, any news on the reviews?16:24
mandelnessita: I did now.. weird16:24
nessitamandel: so, 2 things:16:24
nessita* we should not write to /tmp anywhere16:25
mandelnessita: 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 linux16: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
nessitamandel: we talked about this, you mentioned you will assert over shutil.move being used16:26
nessitamandel: since having /tmp in another FS is a random lottery16:26
alecunessita, I'm finishing the first; one question: why are you using a "print" for the error in rmtree?16:26
nessitamandel: ack to 1 and 2? I'm leaving soon16:27
mandelnessita: yeah, that is how the shutil.move test approach the idea within the python tests itself.. I'll remove that16:27
nessitaalecu: you mean vs failing the test?16:27
=== mterry_ is now known as mterry
alecunessita, vs logger16:27
mandelnessita: yeah, will do the right now16:27
nessitaalecu: I want to detect those cases and start cleaning them de a poco, but I don't want to trigger more failures because tearDownfailed16:27
nessitaalecu: but since is a tearDown, we don't necessarily have a logger setup16:28
nessitaalecu: also, almost no one reads the logs after tests have passed16:28
nessitaalecu: this definitely caughts attention from the runner16:28
nessitaalecu: I'm planning on removing that as soon as all the tests are fixed16:30
alecunessita, 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
nessitaalecu: I don't mind, doing it right now16:30
nessitaalecu: pushed16:33
nessitaalecu: if/when you approve, would you globally approve, please?16:33
nessitaI gotta run!!!16:34
nessitabye all16:34
alecusure. bye!16:34
mandelralsina_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
mandelon windows I mean :P16:53
alecumandel, great, thanks.16:53
gatoxmandel, ok... i'll check that! good bye16:53
alecumandel, I'll review after I finish with nessita's16:53
mandelalecu: np :)16:54
mandelalecu: I'd like your input a bout the Deny ACE impementationI have done, it should do the trick in order to keep the old ACEs16:55
alecumandel, great, I'll review it soonish16:56
=== m_conley is now known as m_conley_away
mandelalecu: 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 :D16:58
mandelanyways, I'm done, my ass hurts, the chairs of the internet cafe are shit!16:58
alecumandel, that's just like what she said.16:59
mandelalecu: hehehe, good one17:01
* mandel runs away!17:02
=== beuno-lunch is now known as beuno
dobeyyay, UPS17:35
gatoxi'm going to the gym... i'll be back around 16:30 ART to keep fighting with installer's tests!17:47
gatoxralsina_embassy, alecu ^17:47
alecugatox, ack!17:48
=== m_conley_away is now known as m_conley
* alecu is having lunch18:15
=== ralsina_embassy is now known as ralsina
Fury1Hi! 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/1519:01
Fury1But 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
ubot4Launchpad 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
dobeynot sure why that would be. sounds like perhaps a bug in evolution. what version of evolution and ubuntu are you using?19:05
Fury1natty amd64 - Evolution 2.32.219:06
Fury1I 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
dobeyok. 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
dobeyhmm, maybe19:09
ubot4Launchpad bug 823500 in evolution-couchdb (Ubuntu) "Evolution Birthday Calendar Doesn`t show Ubuntu One contacts birthdays (affects: 1) (heat: 6)" [Undecided,New]19:18
=== M0hi is now known as IAmNotThatGuy
=== m_conley is now known as m_conley_away
dobeyhave a good evening everyone21:28
tntcCardinalFang: 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!