| ceramicm | Package was built successfully. Thanks for the help. | 08:45 |
|---|---|---|
| FxIII | hi all | 08:52 |
| FxIII | I need to run the sync daemon using the command line | 08:53 |
| FxIII | i started the syncdaemon --oauth but i get an error | 08:54 |
| FxIII | I get ubuntuone.platform.linux.dbus_interface.NoAccessToken: Credentials Error [...] UserCancelled [...] The user cancelled a prompt | 08:55 |
| FxIII | any idea? | 08:55 |
| FxIII | my network manager does not know the custom ppp connection in ppp0 may be a problem? | 09:02 |
| ceramicm | FxIII: Run seahorse. Do you see anything listed called "Ubuntu One" | 09:04 |
| FxIII | ceramicm: seahorse? | 09:06 |
| FxIII | ceramicm: where i can find it? | 09:06 |
| ceramicm | FxIII: It allows you to graphically browse keys stored in the GNOME keyring. In a terminal, type "seahorse". | 09:06 |
| FxIII | ceramicm: i dont have the X server active | 09:06 |
| FxIII | the problem is i cant access to the session bus because there is no DISPLAY variable set | 09:07 |
| mandel | fagan, ping | 09:09 |
| === gord_ is now known as gord | ||
| fagan | mandel: pong | 09:11 |
| fagan | morning | 09:12 |
| mandel | fagan, morning, sorry I'm a little late, I had to do some errands | 09:13 |
| mandel | mainly get petrol to get to the other village to get popoer internet :P | 09:13 |
| FxIII | hi mandel | 09:13 |
| fagan | np its only me here so no one noticed :) | 09:13 |
| mandel | FxIII, hello! did rye manage to get your u1 running without x11? | 09:13 |
| mandel | fagan, I think _Chipaca is here too ;) | 09:13 |
| mandel | fagan, shall we start with the test? | 09:14 |
| fagan | Oh forgot about him | 09:14 |
| fagan | mandel: sure if shares are back up | 09:14 |
| FxIII | mandel: there is a problem on the device | 09:14 |
| FxIII | mandel: it has a custom internet connection that the network manager cant see | 09:14 |
| mandel | fagan, turns out that the issue was with the + the web ui does not handle that symbol in an email address well :) | 09:15 |
| mandel | fagan, whihc versions of windows do you have? | 09:15 |
| fagan | mandel: ah ok | 09:15 |
| fagan | I have 1 of everything | 09:15 |
| mandel | FxIII, oh :( but I guess we can work around it :) | 09:15 |
| mandel | FxIII, if we fake the event of network connect twisted will try to connect anyway and will try to do the job | 09:16 |
| mandel | FxIII, it involves either removing the network manager or my tellng you to edit the code, your choice :) | 09:16 |
| mandel | fagan, superb, lets start with xp, do you have the deps etc... | 09:17 |
| fagan | mandel: nope lost my old VM | 09:17 |
| FxIII | mandel: i did not understood the second option ... | 09:17 |
| fagan | mandel: want me to get grab them | 09:17 |
| mandel | fagan, would be nice yes :) | 09:17 |
| fagan | Ok cool will go do that | 09:18 |
| mandel | FxIII, I can send you a small patch to make syncdaemon believe that there is no network manager so that it always tries to connect | 09:18 |
| mandel | FxIII, and the OS should ensure that we can reach the outside world | 09:18 |
| mandel | FxIII, opensource for the win! | 09:18 |
| fagan | mandel: well the connection bit is in SSO | 09:18 |
| mandel | fagan, no, sso does not use network manager AFAIK | 09:19 |
| fagan | mandel: you would need to fix it there since he needs to sign in too | 09:19 |
| fagan | mandel: its where the code is | 09:19 |
| fagan | mandel: I had to fiddle with the code myself a while back | 09:19 |
| fagan | but my patch was bad | 09:19 |
| mandel | fagan, can you show me the code? | 09:19 |
| mandel | fagan, I dont think networkmanager is a dependency, if it is we can work around it | 09:20 |
| fagan | mandel: its in /ubuntusso/networkstate | 09:20 |
| mandel | fagan, ok, let me look | 09:20 |
| fagan | mandel: its a dbus thing | 09:20 |
| fagan | it detects the network state there and sends back if its connected or not...etc | 09:21 |
| FxIII | mandel: good idea | 09:21 |
| mandel | FxIII, fagan let me take a very wuick look :) | 09:21 |
| mandel | fagan, FxIII does not need sso working he added the creds to the conf, right? | 09:21 |
| fagan | mandel: just delete all of the tests and the file and have it only returning the network state online | 09:22 |
| fagan | mandel: I think so | 09:22 |
| mandel | fagan, since he does have the creds the only one we have lie to is syncdaemon :) | 09:23 |
| FxIII | mandel: i got the cred using ubuntuone-sso-login.py by rye | 09:24 |
| mandel | FxIII, okis, so we only need the sync daemon running :) | 09:25 |
| mandel | FxIII, do you know how to apply a patch? | 09:25 |
| FxIII | patch diff? | 09:26 |
| FxIII | probably I have remember how to do | 09:27 |
| FxIII | im ready mandel | 09:28 |
| mandel | FxIII, ok, let me make the changes and I'll paste you the patch :) | 09:29 |
| FxIII | ty | 09:29 |
| mandel | FxIII, if it works we should add a wishlist bug saying what is the problem, say that network manager is stupid and you will like to be able to tell syncdaemon to ignore it :) | 09:30 |
| mandel | maybe an extra param would be just enough for this | 09:30 |
| FxIII | mandel: where i can find the developing documentation? I'm a Python developer | 09:31 |
| FxIII | im not an ubuntu exper alas | 09:31 |
| mandel | FxIII, docs for u1? | 09:33 |
| fagan | one.ubuntu.com/developer | 09:34 |
| FxIII | ok ty fagan | 09:34 |
| mandel | FxIII, try this: http://paste.ubuntu.com/664094/ | 09:35 |
| mandel | FxIII, the idea is simple, just not check the network manager and do what sd does in the case that is not installed, that is tell the state machine we are connected! | 09:35 |
| mandel | FxIII, I have not tested but I think it should do the trick :) | 09:35 |
| * mandel hates shares... they seem not to be getting the data | 09:37 | |
| mandel | seriously, this windows thing is making be a bitter man... | 09:38 |
| mandel | FxIII, where are you located by the way? | 09:39 |
| FxIII | mandel: sorry located? | 09:39 |
| mandel | FxIII, where do you live? 'cause on september there is this: http://bit.ly/nRgoJQ and if you do python you might be interested :) | 09:41 |
| FxIII | Ah im in northen Italy now :D | 09:41 |
| mandel | stuard will be there and I'll do my best too (I'm a University of Manchester alumni :D) | 09:42 |
| mandel | FxIII, oh, nice! well, maybe we can convince them to do it in italy next time :) | 09:44 |
| mandel | FxIII, did the patch work? | 09:44 |
| FxIII | mandel: can you it whould be wonderfull :D | 09:45 |
| FxIII | mandel: no the patch failed | 09:45 |
| FxIII | let me se why | 09:45 |
| mandel | FxIII, please, maybe I did some stupid error... pastebin it so that I can take a look | 09:46 |
| FxIII | 1 out of 1 hunk FAILED -- saving rejects to file dbus_interface.py.rej | 09:46 |
| FxIII | i'm on /usr/lib/pymodules/python2.7/ubuntuone-client and i did patch < patchfile | 09:47 |
| mandel | FxIII, hmm maybe the problem is thatI did it from trunk and you have and older version :P | 09:50 |
| mandel | FxIII, can you open /usr/lib/pymodules/python2.7/ubuntuone-client/platfrom/linux/dbus_interface.py and do a search for the network manager bus call? | 09:51 |
| mandel | FxIII, or paste bin the file and I'll tell you which lines to delete :) | 09:51 |
| FxIII | mandel: what i have to look for? | 09:53 |
| mandel | FxIII, org.freedesktop.NetworkManager | 09:53 |
| mandel | FxIII, which will lead you to the call that asks dbus to get the network manager object | 09:54 |
| mandel | FxIII, there you will see that there is a try except, in the except we are focing the daemon to connect | 09:54 |
| mandel | FxIII, so what you want to do is remove the try and always call self.connection_state_changed(NM_STATE_CONNECTED_GLOBAL) | 09:54 |
| mandel | so that we do not care of the network manager :) | 09:55 |
| FxIII | mandel: ok | 09:56 |
| mandel | FxIII, if that does the trick i think we can write a nice patch so that we have an option to pass to the daemon to ignore the network manager | 09:58 |
| mandel | FxIII, and I/we'd love you if you propose the patch with some tests and the opts etc.. :) | 09:59 |
| mandel | FxIII, orcourse I'm more than happy to give you a hand with it :) | 09:59 |
| FxIII | ok | 10:01 |
| FxIII | Failure: ubuntuone.platform.linux.dbus_interface.NoAccessToken: CredentialsNotFound | 10:02 |
| mandel | FxIII, hurray \Q/ | 10:02 |
| mandel | FxIII, ok, that is easy, how did you pass the creds last time? | 10:02 |
| FxIII | i give the creds... | 10:03 |
| mandel | FxIII, can I see the full trace? | 10:03 |
| FxIII | mand twice what i posted you | 10:04 |
| mandel | FxIII, ok, lets look in the logs, look in you xdg_cache | 10:05 |
| mandel | FxIII, it should be somwhere in your home | 10:05 |
| mandel | FxIII, ls -la ~ :P | 10:05 |
| mandel | that ^ is just in case hehe | 10:06 |
| FxIII | not found | 10:07 |
| FxIII | i ran that form command line using dbus-launch | 10:07 |
| FxIII | I dont know if it matters | 10:07 |
| ralsina | good morning! | 10:09 |
| karni | morning ralsina | 10:09 |
| mandel | FxIII, yes I think it does, how did you use to laucnh it with rye? | 10:12 |
| mandel | actuall, rye are you here? we have a really cool idea for a small patch :P | 10:12 |
| mandel | ralsina, buenos dias! | 10:12 |
| ralsina | buen día mandel | 10:12 |
| mandel | ralsina, I've taken a look at both of your branches and they are approved, I started with the shares tests, with the current trunk they will not work | 10:12 |
| ralsina | both? I think I had only one :-) | 10:13 |
| mandel | ralsina, we need the shutil.move branch, yet is starting to look like a monster... is doing to much :( | 10:13 |
| ralsina | ugh | 10:13 |
| mandel | ralsina, hhaha true, I had a brain fuck because my email client added to links :P | 10:13 |
| mandel | ralsina, that branch has gone from removing shutil.move from actianq to remove os.walk, shutil.rmtree and shutil.move for the full app | 10:14 |
| mandel | ralsina, I'm making the changes, but I'd have preferred to have a diff bug for that | 10:14 |
| mandel | ralsina, also, do you have the backlog from this morning? | 10:15 |
| ralsina | mandel: I do, let me read | 10:15 |
| rye | mandel, i am here, where's cool idea? | 10:15 |
| mandel | ralsina, rye FxIII has problems because network manager is not ecognizing his network device and then sd does not work | 10:16 |
| rye | mandel, network-manager-related? | 10:16 |
| rye | mandel, if we brink netowkr-manager down it will work | 10:16 |
| rye | bring | 10:16 |
| rye | network | 10:16 |
| * rye has not had his tea yet, high error rate | 10:16 | |
| ralsina | mandel: oh, that. We have had those before, and we had a fix, I think: don't install network-manager | 10:16 |
| mandel | rye, exactly, so I thouhg it would be nice to have an option to force to work around network manager and tell sd to connect directly | 10:17 |
| mandel | ralsina, ^ | 10:17 |
| ralsina | mandel: if you don't have NM installed/running it should do just that, IIRC | 10:17 |
| rye | mandel, well, if you stop network-manager then it will be pretty much the same, and in this case why would you need the network manager at all? | 10:17 |
| mandel | ralsina, rye: I've ask FxIII to take a look and propose a patch (with our help) and file a bug as a wishlist | 10:17 |
| ralsina | mandel: the problem was having NM running and connect to the network some other way | 10:18 |
| mandel | rye, ralsina, so why do we even bother with network-manager? | 10:18 |
| mandel | I know the code does a try except and if network manager is down we force the connect | 10:18 |
| ralsina | mandel: because if it is running, we use it to detect if there is network | 10:18 |
| mandel | lets make an option for that --force-connect for ubuntuone-syncdaemon for people with this issues | 10:18 |
| mandel | which could also work in windows easily | 10:19 |
| mandel | ralsina, one question, when you do a os.rename with a non existing file, what do you get IOError or OSError? | 10:23 |
| ralsina | let me check | 10:23 |
| mandel | ralsina, with shutil.move you get IOError, yet I'm returning OSError | 10:23 |
| mandel | rye, can you give FxIII a hand with the creds issue he has? | 10:23 |
| fagan | Its kinda funny that the main part of installing the dependencies is waiting for visual studio C++ to download | 10:23 |
| ralsina | mandel: a WindowsError | 10:24 |
| * fagan takes a break while waiting | 10:24 | |
| mandel | ralsina, so OSError, is correct yet shutil returns IOError... | 10:24 |
| mandel | puto python! | 10:24 |
| mandel | :P | 10:24 |
| rye | FxIII, let's make dbus accessible to your shell (you're not going to run X server, right)? | 10:25 |
| * rye was using this type of hack until the dev box crashed | 10:26 | |
| rye | FxIII, um... wait, what is actually the issue you are having, CredentialsError? | 10:26 |
| gatox | ralsina, your branch is not working for me, i tried that when i finished adding the qUncompressed | 10:43 |
| ralsina | gatox: what branch? | 10:44 |
| gatox | ralsina, https://code.launchpad.net/~ralsina/ubuntuone-windows-installer/not-compressed/+merge/71296 | 10:44 |
| gatox | about qUncompress | 10:44 |
| gatox | i've tried that the first time | 10:44 |
| ralsina | what happens when you don't use qUncompress? | 10:44 |
| ralsina | gatox: ^ | 10:45 |
| gatox | ralsina, the style is not applied | 10:46 |
| ralsina | gatox: and what's in data()? | 10:46 |
| gatox | ralsina, a lot bunch of bytes because in my case the data is compressed... maybe we can use QResource.isCompressed() and applied one or the other | 10:46 |
| ralsina | gatox, but why would it be compressed? That's what I don't get | 10:47 |
| gatox | ralsina, qt doc: A QResource that is representing a file will have data backing it, this data can possibly be compressed, in which case qUncompress() must be used to access the real data; this happens implicitly when accessed through a QFile. | 10:47 |
| ralsina | gatox: or, use -no-compress in pyrcc4 :-) | 10:47 |
| ralsina | gatox: since we are compressing with lzma anyway when shipping | 10:48 |
| gatox | ralsina, yes, let me tried if that works here | 10:49 |
| ralsina | gatox: cool, if it does, I'll add that to the setup.py and it's done | 10:49 |
| ralsina | ok, gotta send the boy to get educated! Be back in 71 minutes. | 10:50 |
| gatox | ralsina, -no-compress works just fine! | 10:53 |
| * fagan needs a new keyboard :/ | 11:10 | |
| fagan | Just let my keyboard fall from my table and its warped and I have to twist it to get it working | 11:11 |
| fagan | mandel: I think I have everything | 11:25 |
| fagan | mandel: what is it that im testing | 11:25 |
| fagan | Oh hes probably walking the monster | 11:28 |
| fagan | :) | 11:28 |
| mandel | fagan, we have to test that ro shares work as expected and that a user cannot create a file in an ro share and get it to the server | 11:36 |
| mandel | also modify etc.. | 11:36 |
| mandel | fagan, monster midday walk has been delegated to my littel brother :P | 11:36 |
| fagan | mandel: ah slavery :D | 11:37 |
| mandel | fagan, I call it training :P | 11:38 |
| mandel | fagan, so, do you have a dummy account I can make shres with? | 11:39 |
| fagan | mandel: yeah | 11:39 |
| fagan | mandel: slimshanyfluke@live.ie | 11:39 |
| mandel | fagan, cool, give me a sec while the stupid windows updates apply :( | 11:39 |
| mandel | we are doomed: http://www.google.com/patents?id=26aJAAAAEBAJ | 11:41 |
| fagan | what a terrible patent | 11:42 |
| fagan | love that US patent office sometimes | 11:42 |
| * fagan needs a break | 11:44 | |
| mandel | fagan, I'm going to have to go home for lunch, shall we get to it in a couple of hours? | 11:46 |
| fagan | mandel: sure after standup ill be good | 11:46 |
| * fagan is just in need of supplies :D | 11:46 | |
| mandel | fagan, well, I'll do my best to be on time... I need to fight with a windows update that seems to have broken my sd... or I broke it some other way :( | 11:50 |
| fagan | mandel: oh weird :/ | 11:50 |
| fagan | Its cool I have something to do for josh in the meantime | 11:50 |
| === rodrigo__ is now known as rodrigo_ | ||
| fagan | redirecting the wiki pages to the FAQ and tutorials | 11:51 |
| mandel | fagan, yeah... no idea WTF is going on.. | 11:54 |
| fagan | mandel: well if in doubt C4 | 11:55 |
| * fagan hopes mandel gets the mythbusters reference | 11:56 | |
| mandel | yeah, hehe | 11:56 |
| mandel | I'm trying with trunk I'm getting handsake timeouts :( | 11:56 |
| mandel | ok, fuck I hate windows | 12:01 |
| mandel | ralsina, ping | 12:01 |
| aquarius | duanedesign, rye, did you know about http://www.omgubuntu.co.uk/2011/08/chalk-notes-ubuntu-sync-android/ ? | 12:01 |
| fagan | aquarius: thats really nice :) | 12:08 |
| * mandel is utterly confused... | 12:10 | |
| * duanedesign takes a five mite walk t o glar hi head, BRB | 12:11 | |
| mandel | duanedesign, you need a dog :) | 12:11 |
| aquarius | karni, it might be cool for you to have a chat with jamalta who writes chalk (see above link) about using new SSO stuff for a nicer login experience | 12:13 |
| mandel | fagan, have to tried ralsina installer? | 12:22 |
| fagan | mandel: yep | 12:23 |
| mandel | fagan, did you manage to upload files? | 12:23 |
| fagan | I didnt try but I can if you want | 12:23 |
| mandel | fagan, it works for me, but running it from trunk it does not... I wonder if we broke something at some point | 12:28 |
| mandel | I'm getting an unknown error :( | 12:28 |
| fagan | mandel: Hmmmm thats weird whats in the traceback. I wonder if it was something just added that caused it | 12:29 |
| mandel | fagan, just that I cannot listen to localhost... smells like something that was added for the IPC... I'll talk with alecu maybe he knows... | 12:30 |
| fagan | mandel: oh so at least you have some idea | 12:31 |
| fagan | thats good | 12:31 |
| mandel | fagan, yeah, but just some idea... | 12:32 |
| fagan | well something ive learned from breaking things is knowing ish whats wrong is the start and after that its just debugging | 12:33 |
| mandel | hehehe but debugging that code is hard... is mostly black magic :P | 12:33 |
| fagan | mandel: hehe well I just put lots of prints where I think its going wrong to say what the variables are and whats going on and it works ok to find out a lot of info about whats going on | 12:34 |
| fagan | Im sure someone who is a better programmer wouldnt go do all that though :D | 12:34 |
| mandel | fagan, yeah, but in thie case there is twisted, spread.pb decorators.. is a mess | 12:34 |
| fagan | mandel: ah well ive never had to figure out anything other than how twisted works so I wouldnt know about messes thankfully | 12:35 |
| gatox | ralsina, mandel -> small branch: https://code.launchpad.net/~diegosarmentero/ubuntu-sso-client/header-wordwrap/+merge/71349 :D | 12:36 |
| mandel | fagan, it is fun to do, yet hard :) | 12:38 |
| nessita | buenos días! | 12:38 |
| fagan | nessita: hello nessita | 12:38 |
| fagan | lol I double up on the nick just to be sure you know that hello was just for you :D | 12:39 |
| nessita | :-) | 12:39 |
| mandel | nessita, buenas! | 12:39 |
| mandel | nessita, got good news, RO shares as in truck wont create the files in the server side :) | 12:39 |
| mandel | nessita, I'm going to test what does the access-can-write branch adds and if it is worth it | 12:40 |
| nessita | mandel: before that | 12:40 |
| nessita | mandel: what happen client side with the ro shares? | 12:40 |
| nessita | mandel: I would like to see the log of a run modifying a share RO | 12:40 |
| mandel | nessita, the file was created in the fs, but was not updated to the server | 12:40 |
| mandel | nessita, and Upload object s created but I never see the file in the web ui | 12:41 |
| nessita | mandel: can I see the log? what about removing a file? | 12:41 |
| mandel | nessita, sure, let me paste them | 12:41 |
| mandel | nessita, there is just one thing I'm having problems with and is the IPC activation, I keep getting: http://paste.ubuntu.com/664172/ | 12:42 |
| mandel | nessita, and I've checked and double checked that I have a single instance of the sd | 12:42 |
| nessita | mandel: let's go step by step | 12:44 |
| nessita | mandel: let's finish talking about shares | 12:44 |
| mandel | sure | 12:44 |
| nessita | mandel: so, can I see the logs when you modifed RO shares? | 12:44 |
| mandel | nessita, yes, I generating clean ones right now | 12:45 |
| nessita | ah, ok | 12:45 |
| nessita | mandel: make sure to remove a file/dir inside a ro share :-) | 12:45 |
| mandel | yes :) | 12:46 |
| gatox | nessita, small another: https://code.launchpad.net/~diegosarmentero/ubuntu-sso-client/header-wordwrap/+merge/71349 | 12:47 |
| nessita | gatox: sure! | 12:47 |
| gatox | nessita, and it has even more tests than line of codes | 12:48 |
| gatox | :P | 12:48 |
| nessita | gatox: that's what I'm seeing! big APPLAUSE for you | 12:49 |
| mandel | nessita, waiting for changes to reach the server and will pates the logs | 12:50 |
| nessita | great | 12:50 |
| mandel | nessita, there are exceptions due to shutil because I'm using the frozen .exe, see my issue in the email I sent | 12:50 |
| mandel | nessita, here: http://paste.ubuntu.com/664228/ | 12:54 |
| nessita | looking | 12:55 |
| mandel | nessita, the thing is, it works, yet look at the retry | 12:55 |
| mandel | nessita, look at the bottom | 12:55 |
| nessita | mandel: at the bottom there are events in the RW share... as far as I see | 12:58 |
| nessita | mandel: what line in the paste? | 12:58 |
| nessita | alecu: ping | 12:59 |
| alecu | nessita, pong | 12:59 |
| nessita | alecu: good morning! so, I'm looking at mandel s log, and I see several | 13:00 |
| nessita | 2011-08-12 14:48:45,019 - ubuntuone.SyncDaemon.Pb - DEBUG - No signal handler for 'on_request_queue_added' in <twisted.spread.pb.RemoteReference instance at 0x0370DF08> | 13:00 |
| nessita | I was wondering if the status aggregator is involved in this | 13:00 |
| fagan | me | 13:00 |
| nessita | (I know you made this not to fail) | 13:00 |
| mandel | nessita, I dont like that the sd will do a retry when it fails to create the folder in a rw | 13:00 |
| nessita | mandel: but is ok, RW == read write | 13:00 |
| nessita | I mean, it has to retry | 13:01 |
| nessita | me | 13:01 |
| mandel | me | 13:01 |
| gatox | me | 13:01 |
| fagan | ralsina, dobey standup :) | 13:01 |
| alecu | me | 13:01 |
| mandel | me | 13:01 |
| gatox | nessita, the changes about run-tests has been submitted to my branch | 13:01 |
| * fagan forgot alecu | 13:01 | |
| ralsina | me | 13:01 |
| ralsina | no notes yet | 13:02 |
| mandel | nessita, updated log: http://paste.ubuntu.com/664231/ | 13:02 |
| fagan | mandel: you already said me :) | 13:02 |
| mandel | hehe | 13:02 |
| fagan | (just making sure im sure) | 13:02 |
| nessita | dobey: standup? | 13:02 |
| fagan | ill go and he can catch up | 13:02 |
| nessita | fagan: go! | 13:02 |
| fagan | DONE | 13:02 |
| fagan | * resetup XP dev env | 13:02 |
| fagan | TODO | 13:02 |
| fagan | * redirections for josh | 13:02 |
| fagan | * help mandel | 13:02 |
| fagan | BLOCKED | 13:02 |
| fagan | * Na but my keyboard is on its last legs ordering a new one later today | 13:02 |
| fagan | nessita: go | 13:02 |
| nessita | DONE: tons of reviews and meetings, started with bug #824815 | 13:02 |
| nessita | TODO: land aforementioned branch, start working on branch to send platform details to the server | 13:02 |
| nessita | BLOCKED: nopes | 13:02 |
| nessita | NEXT: mandel | 13:02 |
| ubot4 | Launchpad bug 824815 in ubuntu-sso-client "The url signing should support passing query params (affects: 1) (heat: 6)" [High,In progress] https://launchpad.net/bugs/824815 | 13:02 |
| mandel | DONE: updated shutil.move to change all uses of shutil.rmtree and os.walk. Started testing sd and shares. Found issues when running the code form trunk and send an email about it. | 13:03 |
| mandel | TODO: look at the issue I have to run the code from trunk and other branches. Test shares with the set_dir_readonly implemented. | 13:03 |
| mandel | BLOCKED: no | 13:03 |
| mandel | COMMENTS: wont be here next week. Got holidays \Q/ | 13:03 |
| mandel | gatox: go go go | 13:03 |
| gatox | DONE: | 13:03 |
| gatox | Fixing Title and Subtitle bug in SSO Client. Adding Tests for Header in SSO Client. run-tests in sso client fixed to execute UI tests. | 13:03 |
| gatox | TODO: | 13:03 |
| gatox | Keep working on Windows Installer UI bugs. | 13:03 |
| gatox | BLOCKED: | 13:03 |
| gatox | No. | 13:03 |
| gatox | alecu, go | 13:03 |
| mandel | nessita, looks like the lof rotated, I'll get you a nicer version in a min | 13:03 |
| alecu | DONE: reviews, lots of calls, discussed #824003 with nessita and facundo, fixed bug #824817, worked on bug #824819 | 13:03 |
| alecu | TODO: finish big branch for bug #824819 | 13:03 |
| alecu | BLOCKED: no | 13:03 |
| alecu | NEXT: mandel | 13:03 |
| ubot4 | Launchpad bug 824817 in ubuntuone-client "Errors in the watch threads are not handled (affects: 1) (heat: 6)" [High,Fix committed] https://launchpad.net/bugs/824817 | 13:03 |
| nessita | mandel: ok, be sure you play with a RO share | 13:03 |
| ubot4 | Launchpad bug 824819 in ubuntuone-client "SD and tests need to wait until a watch is removed (affects: 1) (heat: 6)" [High,New] https://launchpad.net/bugs/824819 | 13:03 |
| dobey | me | 13:03 |
| fagan | next ralsina mandel already went | 13:03 |
| ralsina | DONE: took kid to the doctor, windows call, reviews, found a bit of a problem *after* merging Diego's branch. TODO: fix that bug (qUncompress causes segfault), implement uninstaller as discussed with lisette, reviews, etc. | 13:03 |
| ralsina | BLOCKED: no | 13:04 |
| mandel | nessita, I did the changes to ro too.. weird | 13:04 |
| ralsina | dobey, right? | 13:04 |
| dobey | λ DONE: gnome/installer releases/uploads | 13:04 |
| dobey | λ TODO: hackday (tarmac), administrata | 13:04 |
| dobey | λ BLCK: None. | 13:04 |
| fagan | Comments? | 13:04 |
| gatox | nope | 13:04 |
| mandel | I need to run home, I'm really later for lunch | 13:04 |
| nessita | nopes, other than I miss freaky friday | 13:04 |
| nessita | mandel: go | 13:04 |
| mandel | nessita, can we have mumble at 4 so I have a decent internet connection? | 13:04 |
| nessita | mandel: sure | 13:05 |
| gatox | i need to leave for a while... i have to take my picture for the visa :P | 13:05 |
| mandel | thx! cu then! | 13:05 |
| fagan | Anyone got any keyboard suggestions? | 13:05 |
| nessita | alecu, ralsina, Chipaca, gatox: mumble in 55 minutes | 13:05 |
| * fagan is looking to invest in one. | 13:05 | |
| nessita | gatox: ack | 13:05 |
| gatox | brb | 13:05 |
| * nessita -> building mate time | 13:05 | |
| ralsina | nessita: ok | 13:05 |
| ralsina | nessita: happy carving! ;-) | 13:05 |
| ralsina | fagan: I suggest a model-m | 13:05 |
| fagan | ralsina: pro :D | 13:06 |
| ralsina | fagan: the superloud click will keep you company | 13:06 |
| ralsina | plus, you can use it to kill rodents up to 15kg without ill side effects | 13:06 |
| fagan | I was looking into steelseries one since the keys are really soft and its pretty sturdy | 13:06 |
| alecu | nessita, it's ok for some lines like the one you pasted above that to show up on the DEBUG log; they used to be a WARN followed by a stacktrace *each* | 13:06 |
| fagan | ralsina: awesome haha | 13:06 |
| ralsina | works as a cricket bat or looter deterrent, as well | 13:07 |
| fagan | ralsina: im sure it works as a keyboard too | 13:07 |
| ralsina | fagan: oh, sure, that too | 13:07 |
| ralsina | fagan: I suggest the modified one with the all-black keys (nothing written in them) | 13:07 |
| fagan | and also im sure I can use it as a wedge for lifting stuff | 13:07 |
| fagan | ralsina: I actually think ive seen that :) | 13:08 |
| ralsina | gatox: pushed not-compressed with the -no-compress flag, could you test it? | 13:08 |
| gatox | ralsina, ok | 13:09 |
| alecu | nessita, also, it looks like we are using a different ipc object with a new tcp connection per control panel page (or even more!) so that's why there are so many lines, and that's something we should fix. | 13:10 |
| alecu | ralsina, I need to take Amelia to kinder, but it's raining around here, so it'll take a while to find a cab. | 13:11 |
| alecu | ralsina, it was raining "teresos de punta" some 30 minutes ago, so that's why we didn't go earlier. | 13:11 |
| alecu | ralsina, so, I'll probably be late for the meeting. What time are we having it? | 13:12 |
| ralsina | alecu: same thing here, it took me 45 minutes to go 4 blocks :-( | 13:12 |
| nessita | alecu: I read that from your email, but the control panel does not use the queue_added/removed thingy | 13:12 |
| nessita | alecu: in 48 minutes | 13:12 |
| alecu | nessita, great, thanks. | 13:12 |
| ralsina | nessita: when sd has an event (say, a share added), it sends that signal over every connection. If u1cp opened many, it will get that many times | 13:13 |
| nessita | ralsina: right, but control panel never "connects" to queue_added/queue_removed | 13:13 |
| nessita | ralsina: that's why I see those in the log and I'm not that sure is the control panel | 13:13 |
| alecu | nessita, the way it works now, it connects to "every signal" | 13:13 |
| nessita | ralsina, alecu: +1 to also fix the control panel connection | 13:13 |
| ralsina | nessita: it receives it anyway. I saw them in the control panel log | 13:13 |
| alecu | nessita, and then sd tries to send each signal | 13:13 |
| nessita | ok, understood | 13:14 |
| alecu | nessita, and if there's no handler for that signal, it's logged as debug | 13:14 |
| nessita | right | 13:14 |
| fagan | joshuahoover: hey, you are missing an image on https://one.ubuntu.com/help/tutorial/install-and-setup-bookmarks-sync/ | 13:14 |
| fagan | oh its on all the tutorials | 13:14 |
| * alecu rushes. ttyl | 13:14 | |
| gatox | ralsina, +1 Approved (now i leave :P brb) | 13:15 |
| ralsina | gatox: +1 from me too, merging! | 13:16 |
| nessita | ralsina: can I have a review please? | 13:18 |
| ralsina | nessita: sure! | 13:18 |
| nessita | https://code.launchpad.net/~nataliabidart/ubuntu-sso-client/sign-ui-with-query-params/+merge/71352 | 13:18 |
| ralsina | nessita: I was having problems with signed URLs | 13:18 |
| nessita | ralsina: I'm still reviwing gatox's branch | 13:18 |
| nessita | (FYI) | 13:18 |
| ralsina | nessita: oops | 13:19 |
| ralsina | nessita: claim the reviews, please | 13:19 |
| nessita | ralsina: I claimed the review! | 13:19 |
| nessita | :-) | 13:19 |
| ralsina | really???? | 13:19 |
| nessita | Natalia Bidart 45 minutes ago Pending [Review] | 13:19 |
| ralsina | maybe I didn't reload it :-( | 13:19 |
| nessita | maybe :-) | 13:19 |
| nessita | an actually that branch will go pass tarmac | 13:20 |
| * ralsina hates Visual C++ runtime with the heat of 1000 suns | 13:20 | |
| nessita | will not go* | 13:20 |
| ralsina | so, no harm done, except to dobey's server ;-) | 13:20 |
| fagan | ralsina: you can use gcc | 13:20 |
| ralsina | fagan: no, I can't | 13:20 |
| nessita | ralsina: what can we do to have tarmac running QT tests when landing branches ? | 13:20 |
| fagan | ralsina: its hard to install but possible on windows | 13:20 |
| ralsina | fagan: trust me I can't | 13:21 |
| dobey | ralsina: no, tarmac won't do anything if there are Pending reviews, expect to complain on the proposal | 13:21 |
| fagan | ralsina: ah ok ill take your word for it :D | 13:21 |
| ralsina | nessita: well... if pyqt is installed and we use xvfb, I don't see why not | 13:21 |
| nessita | ralsina: qtrecator | 13:21 |
| dobey | nessita: well qtreactor needs to be productized/packaged | 13:21 |
| ralsina | nessita: that too | 13:21 |
| nessita | dobey: how can we have that? | 13:21 |
| ralsina | plus, sso-client-qt is broken on linux (needs refactoring) | 13:22 |
| nessita | dobey: and is there any chance you install qtreactor by hand until we can build package for it? | 13:22 |
| nessita | ralsina: test for QT UI pass in linux... | 13:23 |
| nessita | ralsina: as long as you have the qtreactor in the PYTHONPATH | 13:23 |
| ralsina | nessita: then just that | 13:23 |
| nessita | ralsina, dobey: so I was hoping we could have tarmac landing QT branches soon... since our releases depend on that code, it will be great to know nothing gets into trunk if it does not pass the tests | 13:24 |
| dobey | i'd rather avoid having things that aren't packages. | 13:26 |
| nessita | dobey: I know, I'm asking for an exception, or ideas about how to solve this soon | 13:27 |
| nessita | dobey: do you know if is there a particular issue to package qtreactor? | 13:28 |
| nessita | or is just doable but no one had the time? | 13:28 |
| ralsina | nessita: have you actually tested signing urls lately? And has it worked? | 13:29 |
| dobey | the only real issues i know of are that it's not productized (has no setup.py), and nobody has been scheduled to do it | 13:30 |
| nessita | ralsina: yes, I tested that branch yesterday. You can try it too: in linux, run the service with: DEBUG=True PYTHONPATH=. ./bin/ubuntu-sso-login, open the control panel, remove the current device, and authenticate again | 13:30 |
| ralsina | nessita: ok, thanks | 13:31 |
| dobey | and i have no idea whether it's actually maintained upstream or not | 13:31 |
| dobey | which reminds me | 13:31 |
| dobey | we need to discuss pyxdg | 13:31 |
| nessita | dobey: so, if it's not maintained upstream, we can't package it? | 13:31 |
| dobey | because it isn't really being maintained upstream :( | 13:31 |
| dobey | nessita: well if it's not maintained upstream, we need to make a plan to either fork and maintain it, or stop using it, i think. | 13:31 |
| nessita | dobey: guh | 13:32 |
| joshuahoover | fagan: ah, thanks, i'll fix that | 13:35 |
| dobey | nessita: indeed | 13:35 |
| dobey | nessita: qt4reactor at least seems to have had a commit on 2011-06-04 | 13:36 |
| fagan | joshuahoover: np ill send back anything else I spot while im around | 13:36 |
| joshuahoover | fagan: thanks! | 13:36 |
| dobey | oh right, forgot to list cp release on standup | 13:39 |
| === m_conley_away is now known as m_conley_ | ||
| failure | Hello, i need help with all my phones gone, i installed a new client and it removed all picures from the ubuntu one cloud, is there possible to restore those ? | 13:51 |
| failure | my account pb@osix.eu | 13:51 |
| fagan | karni: ^ | 13:51 |
| failure | anyone know is its possible ? or is all my picures lost ? | 13:52 |
| failure | phones = picures btw :P | 13:52 |
| fagan | Oh then this is a job for duanedesign id presume | 13:53 |
| failure | fagan: ? | 13:53 |
| ralsina | I can do a setup.py for qt4reactor, should be fairly trivial | 13:53 |
| alecu | failure, there is a temporary backup for all accidentally deleted files, so they will likely be restored | 13:53 |
| alecu | failure, let's ping rye and duanedesign to see if any of them can help | 13:54 |
| alecu | failure, are the pictures gone from the ubuntuone web too? | 13:55 |
| duanedesign | hello | 13:55 |
| failure | alecu: yes | 13:55 |
| dobey | ralsina: yeah i don't think it's hard, but i don't know how responsive upstream is | 13:55 |
| ralsina | dobey: I will do it and submit it on my sandwich time today, see what happens. | 13:55 |
| failure | alecu: i did a fresh install, but i did have the some hostname as the old and some home, and connected to ubuntu one, then it sync somehow my new picures that was empty | 13:55 |
| failure | alecu: from 17GB down to 1.7GB | 13:56 |
| failure | alecu: and now i see some conflics aswell | 13:56 |
| failure | alecu: i think something is messed up... | 13:56 |
| alecu | failure, that's unfortunate, but I'm sure we'll find a way to recover the files | 13:56 |
| failure | alecu: are you able to help me or what to do ? | 13:57 |
| alecu | duanedesign, do you know about the procedure to recover files? can you help ^^? | 13:57 |
| failure | alecu: a restore from yesterday will work also, if you have backups | 13:57 |
| failure | its 10 years of my life, thats why i had it backuped at ubuntuone :P | 13:58 |
| dobey | ralsina: might be a good idea to move stuff around in that tree too | 13:59 |
| karni | fagan: failure: I'm reading | 13:59 |
| failure | karni: ? | 13:59 |
| rye | karni, duanedesign, me is reading too | 14:00 |
| karni | I'm sorry, I don't understand. | 14:00 |
| failure | i want to restore my ~/Pictures | 14:01 |
| karni | "i did a fresh install, but i did have the some hostname as the old and some home, and connected to ubuntu one, then it sync somehow my new picures that was empty" | 14:01 |
| fagan | karni: not your prob failure said phone on new client and I was like karni country | 14:01 |
| rye | failure, it was in your ~/Pictures UDF, right? | 14:01 |
| karni | failure: Could you try to rephrase this, please? | 14:01 |
| rye | facundobatista, was it in "~/Pictures - $something" or in "~/Pictures" ? | 14:02 |
| failure | karni: i lost my ~/Picures can you please restore it? | 14:02 |
| karni | failure: "failure said phone on new client" wtf? :D | 14:02 |
| rye | failure, ^ | 14:02 |
| karni | failure: sorry, that was to fagan :) | 14:02 |
| rye | facundobatista, sorry, tab completion failed | 14:02 |
| karni | fagan: rephrase that, please xD ↑ | 14:02 |
| facundobatista | rye, no worry | 14:02 |
| fagan | karni: failure said new client and phone but meant pictures sorry for the ping | 14:02 |
| failure | karni: please restore my entire ~/Picures | 14:02 |
| karni | oh right, failure is a nick. sorry fagan | 14:02 |
| failure | :) | 14:03 |
| rye | failure, duanedesign, karni, lets synchronize our conversation | 14:03 |
| karni | rye: Sorry. | 14:03 |
| rye | failure, could you please tell the details of how the files were lost? | 14:03 |
| nessita | ralsina: mumble? | 14:03 |
| ralsina | nessita: going | 14:03 |
| nessita | alecu: can I have a trivial review https://code.launchpad.net/~nataliabidart/ubuntu-sso-client/run-qt/+merge/71362 ? | 14:03 |
| failure | rye: i got a new pc, installed ubuntu 11.04. with the some hostname as my old, copy /home/patbat/ to the new PC. then i started ubuntu one, to sync the rest, then i synced my ~/Pictures (i guess) from the new PC that had nothing in ~/Pictures | 14:04 |
| rye | failure, understood | 14:05 |
| failure | rye: i have now removed the access to all my clients without the new one. | 14:05 |
| failure | rye: thats endlessdream | 14:05 |
| rye | failure, before we proceed, could you please archive the contents of ~/.cache/ubuntuone/log/ from your new machine and send them to ubuntuone-support@canonical.com ? I am looking into the recovery now | 14:06 |
| failure | tar zcvf onelogs.tar.gz ~/.cache/ubuntuone/log/ | 14:07 |
| failure | sending it now from pb@osix.eu | 14:07 |
| failure | thank you rye | 14:08 |
| failure | i hope we can restore this | 14:09 |
| failure | mail delivered | 14:09 |
| rye | verterok, ^ we really need some sort of tying the metadata to the machine, otherwise overwriting metadata is going to cause this ^ | 14:10 |
| failure | rye: how is it going ? :) | 14:15 |
| karni | rye: Before that, maybe we should add it to the FAQ "If you're moving your U1 files out of a synced machine, disconnect it first. Also, never move $metadata_dir along with your files." | 14:21 |
| rye | karni, or move metadata dir along with ALL the folders. OTOH the inodes will change, but the hash will be the same so no reupload | 14:23 |
| karni | rye: It's not my area of expertise :) But when I had the same problem, facundobatista told me to remove metadata dir, and let U1 recalculate hashes from scatch and do it's thing. And it helped. | 14:24 |
| karni | rye: Although it wasn't exactly the same case.. I changed my username and renamed by home dir ;D | 14:24 |
| karni | (Of course, it was more than mv /home/foo /home/bar, as I had encrypted home, but it's another story.) | 14:25 |
| rye | karni, yes, that is the best way, except that it will redownload all the files, but safe | 14:26 |
| verterok | rye: overwriting metadata is always a bad thing | 14:26 |
| verterok | what's "this"? :) | 14:26 |
| karni | rye: It wouldn't redownload stuff, why would it. It would only compare current file hashes with metadata from the server. Nothing to redownload. | 14:27 |
| karni | verterok: Hiya :) ! | 14:27 |
| rye | karni, ah, true | 14:27 |
| alecu | nessita, +1 on the run-qt branch | 14:28 |
| * nessita dances | 14:29 | |
| karni | hi nessita :) | 14:30 |
| nessita | hi karni! | 14:31 |
| karni | \o | 14:31 |
| gatox | i'm back... | 14:53 |
| gatox | hate errands | 14:53 |
| === m_conley_ is now known as m_conley_away | ||
| ralsina | Ok, solved the signed URL problem. Doesn't work in the embedded webkit, works just fine on chrome | 14:59 |
| ralsina | nessita: smallest branch ever! This is code we are probably going to remove as we mentioned today, but just in case we don't.... | 15:10 |
| ralsina | https://code.launchpad.net/~ralsina/ubuntuone-control-panel/missing_slash/+merge/71376 | 15:10 |
| nessita | ralsina: I can't see the diff in LP yet, but you should know I WANT TESTS :-D | 15:11 |
| ralsina | nessita: the code does the exact same thing as before, I just changed a constant :-) | 15:11 |
| nessita | ralsina: which you should be tetsing when asserting over the params value, right? ah? right? | 15:12 |
| ralsina | and the URL is encrypted in the output :-D | 15:12 |
| nessita | ralsina: I would expect a test that mocks oauth.OAuthRequest.from_consumer_and_token and assert over the params :-) | 15:13 |
| ralsina | Ok, no it's not. Sure, adding test ;-) | 15:13 |
| nessita | thanks!!! | 15:13 |
| mandel | nessita, alecu: merged with trunk and fixed conflicts: https://code.launchpad.net/~mandel/ubuntuone-client/dont-use-shutil-move/+merge/70595 | 15:17 |
| nessita | mandel: looking! | 15:17 |
| mandel | thx! | 15:18 |
| ralsina | nessita: pushed with the test | 15:19 |
| nessita | mandel: dude, in bin/ubuntuone-syncdaemon shutil comes before sys and signal imports | 15:20 |
| nessita | mandel: please remember alphabetical order for all the imports :-) | 15:21 |
| nessita | mandel: no, wait | 15:21 |
| nessita | mandel: I looked the diff backwards!!! | 15:21 |
| nessita | mandel: it used to be in the wrong order, but you removed it. Ignore my last comment! | 15:22 |
| nessita | mandel: I double checked this: when you merged trunk in, you duplicated this import: from ubuntuone.syncdaemon import local_rescan | 15:23 |
| mandel | ok | 15:24 |
| * mandel looks | 15:24 | |
| nessita | mandel: and you added an empty line here: | 15:24 |
| nessita | stat_path, | 15:24 |
| nessita | <here> | 15:24 |
| nessita | ) | 15:24 |
| mandel | ok | 15:24 |
| mandel | nessita, sorted, anything else? | 15:25 |
| nessita | mandel: you changed this, it was not in the diff yesterday: | 15:26 |
| nessita | inside walk: | 15:26 |
| nessita | yield _unicode_to_bytes(dirpath.replace(LONG_PATH_PREFIX, '')),\ | 15:26 |
| nessita | fixed_dirnames, fixed_filenames | 15:26 |
| nessita | why the dirpath.replace(LONG_PATH_PREFIX, '')? does walk adds that? | 15:27 |
| mandel | nessita, yes, os.walk will return \\?\\rootpath which is a problem when it gets out to sd | 15:27 |
| nessita | mandel: you sure? sounds weird | 15:28 |
| mandel | nessita, now that you mention, shall I add a test? | 15:28 |
| nessita | mandel: let me confirm one second IRL | 15:29 |
| mandel | nessita, sure :D | 15:30 |
| mandel | nessita, http://paste.ubuntu.com/664340/ | 15:30 |
| mandel | at least in my machine | 15:30 |
| nessita | mandel: yes, in mine too. Yes, add tests (note the 's') for walk, and please define a new variable for _unicode_to_bytes(dirpath.replace(LONG_PATH_PREFIX, '')) just like you did with the other fixed_* | 15:31 |
| mandel | nessita, makes sense, on it! | 15:32 |
| * mandel there is nothing more annoyign that an 80 chars line in python... | 15:33 | |
| === NickL_ is now known as NickL | ||
| === m_conley_away is now known as m_conley_ | ||
| nessita | ralsina: I added an improvement to https://code.launchpad.net/~nataliabidart/ubuntu-sso-client/sign-ui-with-query-params/+merge/71352 so we can apply .format to the url | 15:57 |
| * ralsina checks | 15:57 | |
| nessita | ralsina: we need that so we can use a ping url where the email goes "in between" the url | 15:57 |
| nessita | which will be the case now in u1client | 15:57 |
| ralsina | nessita: makes sense | 15:58 |
| ralsina | nessita: I will re-run tests and +1 after that | 15:59 |
| nessita | ralsina: thanks! | 15:59 |
| dobey | alright lunch time for sure | 16:01 |
| dobey | bbiab | 16:02 |
| ralsina | gatox: ping | 16:29 |
| ralsina | Ok, off to a short lunch break | 16:52 |
| gatox | ralsina, pong!! finally back from the bank | 17:03 |
| nessita | gatox: were you able to pay? | 17:12 |
| gatox | nessita, yes!! it is everything ready now! :D | 17:12 |
| nessita | great! | 17:16 |
| nessita | lunchtime for me now | 17:16 |
| mandel | nessita, cachis! | 17:17 |
| mandel | nessita, tests are runing atm will ping you when done | 17:17 |
| mandel | nessita, alecu ping? | 17:33 |
| nessita | mandel: pong | 17:40 |
| mandel | nessita, hola! | 17:42 |
| nessita | hi there | 17:42 |
| nessita | mandel: what can I do for you? | 17:42 |
| mandel | nessita, tests added for walk, both in tests\platform\test_os_hlper and tests\pltform\windows | 17:42 |
| nessita | mandel: why do you need custom tests in windows? | 17:42 |
| mandel | nessita, so a re-re-re-re-re-re-re-review would be welcome :) | 17:42 |
| mandel | nessita, no, I used the same idea you used for the illegal paths ones | 17:43 |
| nessita | ah, let's see | 17:43 |
| mandel | nessita, is the WalkTests one, tests the walk in both directions | 17:44 |
| mandel | nessita, by the way I have noticed something, the self.rmtree from testcase does no seem to work with illegal paths on windows, should we use the on in platform? | 17:46 |
| nessita | mandel: we might, but in another branch | 17:46 |
| mandel | nessita, you read my mind :) | 17:46 |
| mandel | nessita, this one is getting too big | 17:46 |
| nessita | mandel: typo: | 17:47 |
| nessita | # names to bse used by the paths | 17:47 |
| mandel | aggghghghgg | 17:48 |
| mandel | sorted... | 17:48 |
| * mandel has to get this in before holidays no matter what | 17:49 | |
| nessita | mandel: instead of self.addCleanup(self._clean_paths) please use: | 17:49 |
| nessita | self.addCleanup(remove_tree, self.basedir) | 17:49 |
| mandel | nessita, sure | 17:50 |
| mandel | nessita, pushing that change already | 17:52 |
| nessita | re-branching... | 17:54 |
| mandel | nessita, cool | 17:55 |
| nessita | mandel: what are you using self.valid_file_path_builder for, if self.walk_path has not invalid chars? | 17:57 |
| mandel | nessita, so that it has no utf8 chars... shall I rename that? | 17:58 |
| nessita | mandel: also, it seems to me that passing _unicode_to_bytes is not the proper solution for me... | 17:58 |
| nessita | since that is hidden and should not be needed ourside os_helper | 17:58 |
| nessita | you may need to pass get_windows_valid_path instead? | 17:59 |
| mandel | nessita, well, it should not be get_windows_valid_path, walk should return a valid path for sd, but walk does not return abs paths | 18:00 |
| alecu | mandel, pong | 18:00 |
| mandel | so get_syncdaemon_valid path does not work | 18:01 |
| mandel | alecu, can you review https://code.launchpad.net/~mandel/ubuntuone-client/dont-use-shutil-move/+merge/70595 before I go for a week :) | 18:01 |
| alecu | mandel, sure! | 18:01 |
| mandel | alecu, gracias! | 18:01 |
| nessita | mandel: this smells to me, I will review this again | 18:03 |
| mandel | nessita, wait, does get_syncdaemon_valid path return an abspath? | 18:04 |
| mandel | nessita, 'cause if ti does not, then that is the method to use | 18:04 |
| nessita | mandel: you're applying self.valid_file_path_builder to self.walk_path only, and walk_path is absolute | 18:04 |
| mandel | nessita, hm... | 18:05 |
| * mandel looks | 18:05 | |
| nessita | also, this is terrible to read: | 18:05 |
| nessita | flipped = all[0][1][0] != self.sub1_name | 18:05 |
| nessita | ... | 18:05 |
| nessita | 1 + flipped | 18:05 |
| nessita | you're summing up a boolean with an int. I know it *works*, but is very confusing | 18:05 |
| mandel | nessita, what would you prefeR? | 18:06 |
| mandel | ups, capital R :P | 18:06 |
| nessita | mandel: either don't have 2 subdirs, or when walking the subdir, assert that the item is either sub1 or sub2, and then assert over the one that was returned | 18:07 |
| nessita | mandel: if you're eod'd, I will recommend leaving this as is | 18:07 |
| nessita | mandel: I can take it over on Monday | 18:07 |
| mandel | nessita, give me a couple of mins | 18:08 |
| nessita | sure | 18:08 |
| mandel | nessita, the best thing to do is to have both subdirs so that the tests are more real... | 18:08 |
| nessita | mandel: honestly, the tests are very hard to understand, at least for me | 18:09 |
| dobey | nessita, ralsina: what's the word on control-panel status in trunk? are there branchse pending landing now, or can i make a release? | 18:09 |
| nessita | mandel: let's see what alecu thinks, I may be week-tired | 18:09 |
| nessita | dobey: you can make a release | 18:09 |
| nessita | dobey: there are no queued branches | 18:09 |
| ralsina | dobey: what she said! | 18:10 |
| nessita | that's what I said! (?) | 18:10 |
| mandel | nessita, I can make flipped a more cleaner thing by explaining what to do and not doing a bool + int | 18:12 |
| dobey | ok thanks | 18:12 |
| dobey | ralsina: A for effort, F for execution. :) | 18:12 |
| ralsina | dobey: a+ for being literally true ;-) | 18:13 |
| dobey | nah, my life is a metaphor | 18:14 |
| nessita | mandel: what about the second option I gave you? | 18:14 |
| nessita | mandel: assert the second "child" is either sub 1 or sub 2, and then, add a guard to check each one depending which one is first | 18:14 |
| mandel | nessita, yeah I was thinking in how to do something similar :) | 18:18 |
| mandel | nessita, let me see if I get something nice there | 18:18 |
| alecu | nessita, mandel: I don't like the "flipped" stuff on the test either | 18:18 |
| alecu | mandel, what about storing a set() of the elements, and comparing it with another set? | 18:19 |
| nessita | alecu: good thinking! | 18:19 |
| alecu | we must think inside the box! | 18:19 |
| nessita | lol | 18:20 |
| nessita | gatox: ping | 18:20 |
| mandel | hehehe | 18:20 |
| alecu | mandel, I mean, a set at each directory level | 18:20 |
| gatox | nessita, pong..... just the person i need! jeje | 18:20 |
| gatox | nessita, i was reading your comments in my branch | 18:20 |
| nessita | gatox: can you please merge trunk in https://code.launchpad.net/~diegosarmentero/ubuntu-sso-client/header-wordwrap/+merge/71349 ? you will have a conflict in run-tests, please use the version from trunk | 18:20 |
| nessita | gatox: right, thing is that tarmac does not have the qtreactor, so we can't land that (I did not reakize that when you asked) | 18:21 |
| mandel | alecu, nessita Its really too late for me, can you take over this and fix the test as you wish, we really should get this landed asap | 18:21 |
| nessita | gatox: I improved the run-tests so we can run both suites: | 18:21 |
| nessita | gatox: ./run-tests and ./run-tests -qt | 18:21 |
| gatox | nessita, ok, but i have to revert just run-tests..... or delete the test too? | 18:21 |
| mandel | nessita, alecu or give me an hour to go home and walk the dog :) | 18:21 |
| nessita | mandel: yes, I will fix this, but very likely next week | 18:21 |
| dobey | wtf | 18:22 |
| nessita | gatox: which test? | 18:22 |
| gatox | HeaderTest | 18:22 |
| gatox | nessita, ^ | 18:22 |
| nessita | mandel: I'm leaving soon to university, so no worries, I'll take over | 18:22 |
| alecu | mandel, if it's late, then you should eow! btw: enjoy the vacations!!! | 18:22 |
| nessita | gatox: why would you remove that? | 18:22 |
| dobey | there are 4 revisions of control-panel since last release. and only half of them have linked bugs… but those bugs are filed against ubuntuone-client | 18:22 |
| dobey | wtf. | 18:22 |
| mandel | alecu, well, si 8:22 pm :P | 18:22 |
| nessita | mandel: bye! have a great week! | 18:22 |
| mandel | alecu, nessita, I go now, but I might get something done later this night | 18:22 |
| gatox | nessita, i missunderstood... | 18:22 |
| * ralsina kicks mandel off channel | 18:23 | |
| dobey | oh no, one is ubuntuone-client, and one is ubuntuone-windows-installer | 18:23 |
| mandel | nessita, alecu, gatox, dobey, ralsina : have fun!!!!! | 18:23 |
| gatox | mandel, byeeeeeee | 18:23 |
| mandel | ralsina, try the access-can-write branch please :) | 18:23 |
| ralsina | mandel: try it how? | 18:23 |
| * mandel holidays \o/ | 18:23 | |
| ralsina | oh yes IRL | 18:23 |
| mandel | ralsina, IRL :) | 18:23 |
| nessita | mandel: why you keep insisting on that branch? :-) | 18:23 |
| mandel | nessita, because is my baby :) | 18:24 |
| mandel | also, because I think it works hehehe | 18:24 |
| dobey | maybe i should make tarmac validate linked bugs | 18:25 |
| dobey | oh right, holidays | 18:25 |
| mandel | om, I'm off adios :P | 18:25 |
| * dobey submits some holiday request :) | 18:30 | |
| gatox | nessita, changes submitted in my branch | 18:30 |
| nessita | gatox: great | 18:30 |
| dobey | hrmm | 18:31 |
| dobey | but none of the changes since last release are related to the LOGFOLDER error :( | 18:31 |
| nessita | dobey: that was fixed by gatox during the sprint as soon as we broke it, and there was no bug for that since we were updating all project at the same time | 18:32 |
| dobey | nessita: right, but i'm pretty sure that was before the last release i made? | 18:33 |
| nessita | dobey: when was the last release you made? :-) | 18:33 |
| dobey | ah no | 18:33 |
| dobey | someone didn't mention it in the commit message :-/ | 18:34 |
| nessita | gatox: approved and globally approved | 18:36 |
| nessita | alecu: when you have some slot, would you please review https://code.launchpad.net/~nataliabidart/ubuntu-sso-client/sign-ui-with-query-params/+merge/71352 ? | 18:36 |
| gatox | nessita, awesome!!!!!! | 18:37 |
| nessita | ralsina: missing slash approved and globally approved | 18:37 |
| ralsina | nessita: cool, thanks | 18:37 |
| nessita | ok, I will be leaving very soon. I will be working after university, ie after 6pm ART | 18:38 |
| alecu | nessita, I've added a few more comments to the latest mandel branch | 18:46 |
| nessita | alecu: great, I will grab that on MOnday, I need all my lights to be 100% | 18:46 |
| dobey | brb, meh | 18:47 |
| nessita | ok I'm gone | 18:50 |
| dobey | grr | 18:51 |
| nessita | bye all! see ya later | 18:51 |
| dobey | bzr: ERROR: These branches have diverged. See "bzr help diverged-branches" for more information. | 18:51 |
| dobey | "no there aren't any branches in the queue. here have a branch." | 18:51 |
| ralsina | dobey: you mean missing_slash? It's unused by control panel, really | 18:52 |
| dobey | ralsina: it's not unused, it's in the tree. which means it's a new revision. which means it caused a diverge of the branch from the commit i just made to change the version number :) | 18:53 |
| ralsina | oops, sorry! | 18:53 |
| * ralsina points at nessita, sice she's not here | 18:54 | |
| dobey | now i have to make another tarball | 18:54 |
| dobey | hmm | 18:57 |
| dobey | if the courts just struck down the "all americans must carry health insurance" thing, i wonder if that sets a precident that states requiring car insurance could be unconstitutional | 18:58 |
| dobey | i guess that would be too logical though | 18:59 |
| ralsina | dobey: car insurance is meant mostly for the other cars you hit, isn't it? At least that's the part that's mandatory here | 19:00 |
| dobey | ralsina: liability is required yes. but then that would set an example that to smoke you must carry liability insurance, since you smoking, could cause harm to others. or drinking. or being alive (or even dead, since your death doesn't mean all the microbes in/on you are also dead) | 19:02 |
| dobey | of course, you don't need to have liability insurance to buy a gun, either | 19:03 |
| ralsina | dobey, and indeed people will sue you if they believe your second-hand smoke hurts them. The thing is cars hurt enough people that there was a need for a "streamlined" "automatic" damage recovery system | 19:04 |
| dobey | and yet, we still rely on oil. | 19:04 |
| dobey | oh the dear great dismal swamp, spreading your ash all upon the world | 19:06 |
| dobey | wonder if i'll be able to get over there this weekend | 19:06 |
| ralsina | off to pick the kid from school, should be back quickish unless I am washed away by a flood or something | 19:15 |
| gatox | EOD for me... i'll send a branch with some UI improves that i've been working today in the weekend because i'm not be available on monday morning, i have to go to the embassy for the VISA | 19:27 |
| gatox | i'm not going to* (something was missing :P) | 19:28 |
| gatox | ralsina, ^ | 19:28 |
| dobey | won't be :P | 19:29 |
| ralsina | gatox: so, not coming? ok! | 19:47 |
| gatox | ralsina, what? yes... after the embassy | 19:49 |
| * ralsina is confused. "I'm not going to" what? | 19:50 | |
| gatox | but... i want to submit some branches before to have them reviewed and don't waste that time | 19:50 |
| gatox | ralsina, ahhhhhhhh | 19:50 |
| ralsina | oh, not going to be available! | 19:50 |
| ralsina | ok, got it now :-) | 19:50 |
| gatox | ralsina, yes... | 19:50 |
| gatox | :P | 19:51 |
| alecu | ralsina, I'm leaving early today: we got family Smurf day. | 20:02 |
| ralsina | alecu: one line review: "I've seen worse movies featuring blue people in the last three years!" | 20:02 |
| alecu | hahaha} | 20:03 |
| alecu | ralsina, I'm not exactly *thrilled* about it either. | 20:04 |
| alecu | well, eow for me. | 20:04 |
| alecu | see you guys on monday! | 20:04 |
| ralsina | bye! | 20:04 |
| ralsina | So I am now at that point in the week where I still have one hour left to wrk but if I do anything, noone can review it. | 20:04 |
| ralsina | So let's merge things without reviews while noone's looking! | 20:04 |
| gatox | ralsina, i can review some branches later if you want, i've to go to the gym in a while... but then i'll be free and at the computer :P | 20:27 |
| ralsina | gatox: cool, I may have two or three small branches | 20:28 |
| gatox | ralsina, doing other stuff... but anyway | 20:28 |
| ralsina | gatox: STOP WORKING ;-) | 20:28 |
| gatox | ralsina, awesome, give me the ilnks | 20:28 |
| ralsina | hehe | 20:28 |
| gatox | jejee | 20:28 |
| gatox | ralsina, por eso aclare | 20:28 |
| gatox | jejee | 20:28 |
| gatox_ | sorry... wrong place to paste :P | 20:50 |
| dobey | alright all, have a good weekend! | 21:09 |
| nhaines | dobey: have fun :) | 21:12 |
| nessita | hello again! | 21:16 |
| === m_conley_ is now known as m_conley_away | ||
| ralsina | hello! | 21:29 |
| ralsina | nessita, could I get a quick review for https://code.launchpad.net/~ralsina/ubuntuone-windows-installer/start-with-overlay/+merge/71422 ? | 21:29 |
| nessita | ralsina: sure | 21:29 |
| ralsina | nessita: it's a total blocker when you start the wizard without credentials :-( | 21:29 |
| nessita | ralsina: how long does it take for sso to respond? | 21:30 |
| ralsina | nessita: usually, less than a second | 21:30 |
| ralsina | but I need to block the UI because if the user clicks "next" quick he will get the "sign in" page even with credentials | 21:31 |
| nessita | ralsina: on error, you should not take the user to signin... | 21:31 |
| nessita | ralsina: you should take the user to signin only when the result is {} | 21:31 |
| ralsina | nessita: so what do I do in error? | 21:31 |
| nessita | ralsina: you should popup the "standard" error handler. Two things: 1- I think we really need tests for this, we want to make sure this behavior is not broken with future changes | 21:32 |
| nessita | 2- Maybe is best to swicth to yielding over the find_credentials to improve the code readability? | 21:33 |
| ralsina | nessita: there are tests about the overlay being shown/removed, and there are tests about what the "next" page is with/without credentials | 21:33 |
| nessita | ralsina: so, you're adding code that was not there before, that means that flow is not tested | 21:33 |
| nessita | that *particular* flow is not tested | 21:34 |
| nessita | ralsina: so, there should be one test for the overlay hidding in diffline 31 | 21:34 |
| nessita | another test for error_cb | 21:34 |
| nessita | sorry, creds_eb | 21:34 |
| nessita | another test that asserts that the overlay is hidden in diffline 54 | 21:35 |
| ralsina | nessita: the hidden was only moved, so it's tested | 21:35 |
| nessita | ralsina: then you should test is hidden at he right time, for some reason you're moving it, no? | 21:36 |
| nessita | ralsina: or is it the same? | 21:36 |
| ralsina | nessita: it was tested that it was hidden before showing the first page | 21:36 |
| ralsina | since the test have a faked credential, it's still the case | 21:36 |
| nessita | ralsina: but don't you need to assert you're hiding it on error too? | 21:37 |
| ralsina | nessita: ok, that's a good one | 21:37 |
| ralsina | I can't switch to yielding and inlineCallbacks because __init__ can't be a generator | 21:38 |
| nessita | ralsina: ah, ok, makes sense | 21:39 |
| ralsina | nessita, one note: in case of credentials error, I am not sending the user to the next page, I am only setting that the next page will be sign in (which makes sense since we don't effectively have credentials) | 21:45 |
| nessita | ralsina: error does not mean there is no credentials. We can have U1 credentials and still have errors | 21:46 |
| ralsina | but they are credentials we can't get | 21:46 |
| nessita | empty dict as result means no credentials | 21:46 |
| ralsina | so, show the error and do what? Close? | 21:46 |
| nessita | ralsina: if retrieving credentials failed, is very likely that trying to sign in will fail as well | 21:47 |
| nessita | ralsina: error should be a pretty uncommon case | 21:47 |
| nessita | ralsina: what I mean, error in that stage means that something is very wrong | 21:47 |
| nessita | and very likely nothing related to SSO will work | 21:47 |
| ralsina | yes, agreed it's uncommon. I mostly added the handler to avoid crashing when we don't have the registry keys set | 21:47 |
| ralsina | so, it's mostly for us, really | 21:47 |
| nessita | ralsina: that's good, and this matches the case were the following operations will fail as well | 21:48 |
| ralsina | ok, so closing makes sense | 21:48 |
| nessita | ralsina: so my advice is error message popup in that case | 21:48 |
| ralsina | "Error getting credentials: BLAH, application will close", | 21:48 |
| nessita | yeap | 21:51 |
| nessita | ok, I better get going | 22:11 |
| karni | bye nessita ! | 22:11 |
| nessita | karni: goshm you never sleep! :-) | 22:12 |
| nessita | karni: GO TO PARTY | 22:12 |
| nessita | :-P | 22:12 |
| * nessita -> eows | 22:12 | |
| karni | nessita: I sleep. I just usually start work at 10-11 AM ;> | 22:12 |
| karni | nessita: HAve a great weekend ^_^ | 22:12 |
| nessita | :-) | 22:12 |
| nessita | you too! | 22:12 |
| karni | Thank you! \o | 22:12 |
| === CardinalFang_ is now known as CardinalFang | ||
| === yofel_ is now known as yofel | ||
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!