/srv/irclogs.ubuntu.com/2011/07/13/#ubuntuone.txt

=== m_conley is now known as m_conley_away
dobeyadorilson: hi00:58
adorilsonDo you have fixed this bug https://bugs.launchpad.net/ubuntuone-client/trunk/+bug/801938 ?01:15
ubot4Launchpad bug 801938 in ubuntuone-client (Ubuntu Oneiric) (and 3 other projects) "Make command gets error: variable 'result' set but not used (affects: 1) (heat: 8)" [High,Fix released]01:15
adorilsonI remember that you ask to me if I can do a merge proposed. alrigth?01:17
dobeyadorilson: yes01:52
dobeyadorilson: i made a proposal and attributed the fix to you01:53
dobeyanyway, time to finish the beer and get some sleep :)01:53
adorilsondobey: ok.02:15
adorilsontime to sleep for me too02:15
adorilsonsee you. o/02:15
=== rodrigo__ is now known as rodrigo_
=== czajkows1i is now known as czajkowski
ralsinamorning!10:08
faganbrb just heading down to the shop for some supplies and ill grab some lunch too10:21
duanedesignmorning all10:46
karnimorning duanedesign10:48
* fagan back 10:53
faganmorning duanedesign and karni10:53
karni\o10:53
duanedesigno/10:54
fagan\o/10:54
faganralsina: have you made a different version of that script to bundle the branches that looks at the site_packages11:08
faganOr do I just have to import everything thats missing and give the script back11:08
* fagan is just wondering am I testing or actually fixing it 11:11
faganooh its working11:20
faganralsina: ok its working fully now11:21
faganoh actually there is 1 think that I dont know about its looking for a module called configglue.inischema11:23
fagan(for syncdaemon)11:23
faganoh its just config glue my bad11:23
faganSeems like there is something rotten in syncdaemon when you run it from the exe http://paste.ubuntu.com/64317811:31
faganralsina: ^11:31
faganother than that everything is working fine11:32
nessitahello everyone!11:42
faganyo nessita11:43
nessitahola fagan, how is it going?11:43
fagannessita: good good just got the py2exe thingy working11:43
faganother than for syncdaemon11:43
faganbut isnt that borked still?11:44
faganI pastebinned the error just in case11:44
faganOther than that its a nice sunny day in ireland :)11:46
nessitafagan: what's the pastebin?11:46
faganhttp://paste.ubuntu.com/64317811:46
faganOh it might have something to do with the account not being set up either11:47
nessitafagan: nopes11:47
nessitafagan: you need to pass 2 arguments to syncdaemon11:47
nessitafagan: the syncdaemon.conf and the logging.conf, you can grab both from ubuntuone-client/data11:48
fagannessita: ak ok I just ran it and was hoping just not to get an error about a module not found11:48
faganSo thats cool then11:48
nessitaralsina: ping11:52
nessitaralsina: you around?12:21
fagannessita: he said morning must have went off for a bit12:22
ralsinanessita: here now!12:29
nessitaralsina: hi there!12:29
ralsinanessita: took thekid to school frgot to say it12:29
ralsinagoodmorningfagan, nessita12:30
nessitaralsina: I made a mistake in allhands and I submit only one goal (I clicked on the wrong button, I guess. That UI sucks)12:30
ralsinahe12:30
ralsinanessita: you can't add any more?12:30
nessitaralsina: can you do something to give them back to me so I expand?12:30
nessitaralsina: no, I can't edit any more12:30
ralsinanessita: no idea, but I can try!12:30
faganhey ralsina, got the thingy done what do I do now?12:31
ralsinafagan: really? cool!12:31
ralsinafagan: now, zip the dist folder and put it somewhere I can get it12:31
faganralsina: cool ill put it on u112:31
faganralsina: you want the script too with the changes to the imports12:32
ralsinafagan: what changes did you do?12:32
faganralsina: well I added the missing imports to the py2exe path bit12:32
ralsinawhich were...12:32
faganlike oauth httplib2..etc12:33
fagansimplejson12:33
ralsinafagan: put that in a pastebin12:33
faganWill do12:33
ralsinanessita: it seems I can add objectives for you12:33
ralsinanessita: but I don't see how I can open them up for you again. Maybe if I reject them, but that's a bit scary :-)12:34
nessitaralsina: can you send them back to me?12:35
ralsinanessita: I can only reject, edit and add more12:35
nessitalet me ask in #hr12:35
nessitaralsina: hr will fix this for me12:37
ralsinanessita: cool12:38
faganralsina: I pmed the paste12:44
ralsinafagan: that one doesn'thaveyour additions. I remember you had to add oauth and it's not there12:45
ralsinafagan, you added "import oauth"? I meant add them to the includes list! Please try again?12:46
ralsinafagan, in fact, my instructions were "fagan,could you add "oauth" in the includes list of the script and rerun?"12:47
faganralsina: I thought you said just add the import to the path bit so it would pull it in and its working.12:47
faganralsina: so I just have to move it up to the top import list12:47
fagan?12:47
ralsinaNo, I told you that,22 hours ago.12:48
ralsinaNO12:48
ralsinaLook for something that is a "includes list"12:48
faganralsina: ok12:48
ralsinaTip, it's a list called includes in line 200 and something ;-)12:48
ralsinafagan, alecu, nessita, dobey, mandel, standup in 10'12:50
ralsinaAnd DiegoSarmentero of course12:50
DiegoSarmenteroralsina, :P12:50
ralsina(this once it was on purpose, I swear ;-)12:50
DiegoSarmenteroralsina, jejeje12:51
faganralsina: mandel isnt around is he on a swap or something?12:51
ralsinafagan: no idea, I have no swap requests from him12:51
faganralsina: weird12:51
ralsinafagan: he is on shorter hours for the week, though12:51
faganah maybe he will get in a while12:52
=== m_conley_away is now known as m_conley
faganralsina: ok added them to the includes bit12:54
faganralsina: and its working12:55
ralsinafagan: cool, generate the bundles, zip, upload and put the script in there as well12:55
faganralsina: cool12:55
faganwill do12:55
faganmoi13:00
dobey_hmm13:00
=== dobey_ is now known as dobey
faganralsina, nessita, DiegoSarmentero, dobey standup13:00
nessitame13:00
DiegoSarmenterome13:00
ralsiname13:00
dobeyme13:01
faganDONE13:01
fagan* get ralsina's script working13:01
faganTODO13:01
fagan* upload the dist folder and script13:01
faganBLOCKED13:01
fagan* no13:01
fagannessita: go13:01
nessitaalecu is not here?13:01
fagannessita: nope13:01
nessitaDONE: added tony fixes and landed DiegoSarmentero's branch for control panel styling, ran the whole u1client test suite in windows, took screenshots for the bosses, meetings, tcp-activtion reviews13:01
nessitaTODO: tweak control panel UI a bit more bug #809363, windows reviews13:01
nessitaBLOCKED: nopes13:01
nessitaNEXT: DiegoSarmentero13:01
ubot4Launchpad bug 809363 in ubuntuone-control-panel "FileSyncStatus needs to be updated to the current design (affects: 1) (heat: 6)" [High,Triaged] https://launchpad.net/bugs/80936313:01
DiegoSarmenteroDONE:13:01
DiegoSarmenteroAssets added to u1 windows installer. Improves in style. Tests added.13:01
DiegoSarmenteroTODO:13:01
DiegoSarmenteroApply new design from Lisette. Check and Improve tests. Review fonts and specs definition.13:01
DiegoSarmenteroBLOCKED:13:01
ralsinaI assume BLOCKED: yes? :-)13:02
nessitathat blocked?13:02
nessita:-D13:02
ralsinaDONE: created windows installer (actual one) and sent to design and QA, reviews, calls. TODO: signed URLs, embedded webkit, review wizard styling, BLOCKED: no13:03
faganWow very blocked13:03
ralsinaand dobey! :-)13:03
dobeyλ DONE: bug #809561 (mostly), thought about shim design13:03
dobeyλ TODO: start the shim13:03
dobeyλ BLCK: None.13:03
ubot4Launchpad bug 809561 in ubuntuone-dev-tools "Support ignoring paths in u1lint (affects: 1) (heat: 6)" [High,In progress] https://launchpad.net/bugs/80956113:03
nessitaralsina: did we got any feedback from design yet?13:03
ralsinanessita: not a peep13:03
ralsinanessita: they must be shocked by our brilliance I assume, and scared to say anything13:03
faganralsina: obviously :)13:04
nessitaor they passed out :-/13:04
ralsinayeah, or that13:04
ralsinaok, comments that are not about DiegoSarmentero's strange disappearance?13:04
ralsinaOr mandel13:05
ralsina 's strange disappearance ;-)13:05
faganralsina: we are dropping like flies13:07
* fagan checks under his bed for the programmer eating monster 13:07
ralsinaDiego got split, itseems13:07
ralsinamandel had connectivity problems yesterday, was around too early for me today13:08
ralsinaok, eom!13:08
ralsinaNow everyone, go DO that TODO that you DO so well.13:09
nessitaeom!13:09
* fagan dodothetodo13:10
alecugood morning!13:15
faganmorning alecu13:15
nessitaalecu: gooooooood morning. And go! when you're ready13:17
ralsinaI forgot about *alecu's* strange disappearance, too! :-)13:17
FxIIIhi all13:17
ralsinaHello FxIII13:17
FxIIIthere is some api to build services based on ubuntuone?13:17
faganhmmmmm the web client doesnt want to accept the zip13:17
FxIII*is there13:18
faganFxIII: yes we do! :)13:18
faganhttps://one.ubuntu.com/developer/13:18
faganFxIII: if you need any help with anything ask away13:19
FxIIIfagan: wow! the page i was lookin for13:19
alecume13:20
alecuDONE: branch for tcp activation in syncdaemon; started working on bug #80653913:20
alecuTODO: finish branch for that bug13:20
alecuBLOCKED: no13:20
ubot4Launchpad bug 806539 in ubuntuone-client (and 1 other project) "Cleanup API to connect to services (affects: 1) (heat: 6)" [Medium,Triaged] https://launchpad.net/bugs/80653913:20
alecuralsina, sorry about my strange disappearance; got abducted while I was "yendo de la cama al living" :P13:21
FxIIIfagan: i'm interested in couchdb storage, but i cant find the base url for my user13:23
ralsinaalecu: damn livingroom aliens13:23
faganFxIII: well we have an easy way of using couchdb in Ubuntu called desktopcouch13:24
fagangive me a sec ill find a link to some tutorials13:25
FxIIIcant be accessed directly?13:25
FxIIII mean if i understood right the desktopcouch is a local couchdb that is sincronized with the remote one13:25
FxIIIwhere the remote one is?13:25
faganFxIII: yep your correct13:27
dobeyFxIII: ubuntuone-couch is probably what you want13:27
dobeyFxIII: it's a python lib to talk to the remote couchdb on the server13:27
FxIIIis there difference with the standard python library for couchdb?13:28
faganwow the file upload on the web of a 20 mb file seems to be taking a month13:32
faganor its after hanging13:32
ralsinafagan: use chromium, it has an upload progress bar13:32
faganralsina: ah ok I was using ff13:33
ralsinaOr wait for it :-)13:33
alecunessita, ralsina: are we meeting in 5 minutes?13:42
=== zyga is now known as zyga-afk
ralsinaalecu: well, we are doing it without mandel, it seems13:42
nessitaalecu: I hope so, I pinged mandel thru twitter and got no response13:43
nessitaChipaca`, ralsina, alecu: mumble?13:45
Chipaca`nessita: give me 513:46
nessitasure!13:46
nessitaralsina, alecu: me escuchan?13:50
faganralsina: pmed both the dist and the script13:54
faganralsina: and dont mind its called build. I just thought the problem uploading was a naming issue13:55
=== m_conley is now known as m_conley_away
joshuahooverdobey: ping14:19
dobeyjoshuahoover: hi14:19
joshuahooverdobey: hi! bug #776836 ...we've had a few users run into this...looks like we have some debug output, any ideas?14:19
ubot4Launchpad bug 776836 in libubuntuone (Ubuntu) (and 1 other project) "Pressing on the "Subscribe" button to download my purchased music didn't work. (affects: 6) (dups: 2) (heat: 77)" [Medium,New] https://launchpad.net/bugs/77683614:19
=== m_conley_away is now known as m_conley
dobeyjoshuahoover: hrmm, well, mterry's debug output is suggesting that ubuntuone-syncdaemon crashed when dbus was trying to start it14:24
dobeyand it doesn't check connectivity, and treats errors as "you aren't subscribed" which is a problem, but requires ui/string change to fix :(14:26
joshuahooverdobey: hmmm...that's no good :(14:27
ralsinanessita's phone died, she will be back eventually14:46
dobeydoh14:51
ralsinaalecu: could you point me towards the timeout value in sso?14:53
alecuralsina, sure.14:55
alecuralsina, ubuntu_sso/main/__init__.py:TIMEOUT_INTERVAL = 100000014:55
ralsinaalecu:cool, thx14:56
alecuralsina, it used to say 10000, meaning ten seconds; I should have updated the comment when adding the two zeroes14:57
ralsinaalecu: ok,trunk has that.14:58
dobeynessita!15:09
alecuhola nessita!15:10
nessitadobey: I'm very sad right now, my internet provider decided I cna't have phone (ergo internet) until the 22nd15:11
alecuwtf!15:11
alecunessita, you should pay your bills on time!15:11
nessitaalecu: "there is some damage caused by a thrid party and we're replacing cables until the 22"15:12
dobeynessita: :(15:12
alecu:-(15:12
nessitaI'm stealing some wifi from someone around, but I need to seek a more permanent solution15:13
dobeynessita: well, good news, i fixed the path vs filename issue in my devtools branch, so you can -i path/ and it will work15:13
nessitadobey: ok, I'll try it that soon15:13
alecunessita, catch a bus and bring matias, you can both stay at my office :-)15:13
nessitaalecu: it sounds great, really :-) I will count that as an option15:14
dobeyok, lunch time. bbiab15:16
=== alecu is now known as alecu-afk
=== alecu-afk is now known as alecu
=== beuno is now known as beuno-lunch
ralsinadamn, the credentials the wizard gets don't work :-(15:51
nessitaralsina: any error?15:55
ralsinanessita: I get that the token is invalid.15:56
nessitaralsina: can I see the error?15:56
ralsinanessita: in a few minutes, sure15:56
ralsinanessita: I am trying to see if the debug on screen is similar for the installer and get_credentials, since get_credentials's credentials work15:57
nessitaralsina: how are you getting credentials?15:57
nessitain the wizard, I mean15:57
ralsinaBy calling SSO with the same parameters control panel uses15:58
ralsinaI can't use the u1cp credentialsmanager directly because of UI problems15:58
nessitaralsina: you should not be calling SSO directly, let's try to fix the second issue you mentioned16:02
ralsinaLooks like it's not doing the ping at the end. That could be the problem, right?16:02
nessitaralsina: there is no u1cp credentialsmanagement thingy16:02
nessitaralsina: you meant the u1client credentialsmanagement?16:02
nessitaralsina: all you should be calling is:16:02
nessitacreds = CredentialsManagementTool()16:02
nessitaresult = yield creds.find_credentials()16:03
nessitais result == {} -> no credentials in the system16:03
nessitaralsina: did you try that ^?16:03
ralsinabrb16:04
ralsinanessita: that's for checking if there are credentials, right?16:04
nessitaralsina: yes16:04
nessitaralsina: as far as I understand, that is all you need, right?16:05
ralsinano, I need to get credentials16:05
ralsinaI *do* get credentials, but it seems they are wrong16:05
nessitaralsina: I'm confused16:05
nessitaget credentials == find credentials16:05
nessitayou mean register/login?16:05
ralsinanessita: yes16:05
ralsinasorry if I was using wrong terminology16:06
nessitais ok, I misundertood16:06
nessitaralsina: then you need to call yield CrendetialsManagementTool().register(), does that work?16:06
ralsinanessita: No idea.16:06
nessitayou never used it?16:07
ralsinanessita: no, I am using ubuntu_sso.credentials.Credentials because there I can pass the UI as an argument16:07
nessitaoh hum, that is not good16:08
nessitabut I understand16:08
nessitayou probably are not passing the proper ping url to the call, but ideally your code should not be aware of that16:08
ralsinaI am missing some call CredentialsManager does, should check and do it right.16:08
nessitathat's why we have the  ubuntuone.credentials module for16:08
nessitaralsina: can we instead have the wizard not calling SSO directly?16:09
ralsinanessita: then it breaks the whole UI16:09
nessitaand that way use the U1 abtraction layer?16:09
nessitaralsina: can't we add something to the U1 abstraction layer so the wizard does not break?16:09
nessitaI'm trying to see if we can keep the abstraction layer being such16:10
ralsinanessita: don't know.Just found out about this 5 minutes ago :-)16:10
ralsinanessita: credentials.register uses the gtk feature of getting a windows ID and embedding the widgets there. That's done completely different on the Qt side of sso16:12
nessitaralsina: but you have the Credentials.register for the QT side16:13
nessitaralsina: what I'm saying is: we may have a design issue that is coming up to light now, so, before workaround it, I would like to evaluate if it's fixable in a timely manner16:14
nessitaif it's a big chance, we may need to accept the workaround16:14
ralsinanessita: ok. The API for Credentials.register's support for embedding the SSO UI is based on two things:16:14
ralsina1) That the UI provided by SSO is correct. This is not the case with the current design we are trying to implement.16:15
* fagan EOD16:15
ralsina2) that to embed the UI you have to pass a window_id and then you can embed arbitrary widgets there. That doesn't work on Qt at all.16:15
nessitaralsina: the window_id is optional so you can not pass that16:16
nessitaI still don't understand (1)16:16
ralsinanessita: yes, and you get a SSO popup16:16
nessitaralsina: right, my point is:16:16
nessitaralsina: can't we abstract the callers for SSO (the way you're calling it) so they call ubuntuone.credentials instead of sso directly? we may need to add a new method, yes, but not sure if you understand what I mean16:17
ralsinanessita: I need to style, theme, add extra buttons, and change texts on SSO, for the installer. If I use the default SSO popup, I don't see how to do that.16:17
ralsinaThat's the explanation for 1) above16:17
nessitaralsina: I'm not saying you use the popup16:18
nessitaI'm saying let's try to hide the auth details in the proper place16:18
ralsinanessita: ok, I'm all for it. I am not getting how to do it though :-)16:18
nessitaralsina: what I'm saying is that the windows-installer should not be importing the constants APP_NAME, TC_URL, DESCRIPTION and  PING_URL in its code16:19
nessitaralsina: those constants are, somehow, private to ubuntuone.crendetials16:19
nessitaubuntuone,credentials should offer to you the API you need, hiding the auth detail we need t hide16:20
nessitafor example, the ping url16:20
ralsinaok. I need ubuntuone.credentials to accept a ui_module parameter and pass it nto ubuntu_sso.credentials.Credentials16:20
nessitaralsina: so, maybe is too late, but from my POV, we should have a U1-SSO-widget that you can embed anywhere, but the workflow is defined in the widget in isolation, so we have the same user experience every time, every where we use it16:21
nessitaralsina: and the popup becomes only a dialog that embeds that dialog16:22
nessitaand the wizard is a qwizard with that widget in it16:22
ralsinanessita: no, that's not good enough I am afraid16:22
nessitaralsina: why not? (I know is not short)16:22
dobeyquacky qwizards16:22
nessitadobey: quack?16:23
nessita:-)16:23
dobey:)16:23
ralsinanessita: because wording is too strict(you can only add snippets in the middle of texts, and not on all pages), because you can't add a "No thanks I will setup later" button, because the app can't change widget positions, or sizes.16:25
ralsinanessita: and because the SSO widget *is* a qwizard, so you would have a wizard that pops up a wizard.16:26
nessitaralsina: yes, I agree to that. And I think all the wizard coding should be in sso not in the installer16:26
nessitaanyways, we won't be changing that now :-(16:26
ralsinanessita: but the wizard sets up things that are not SSO related16:26
nessitathe change is too big16:26
nessitaralsina: I mean other thing16:27
ralsinaok16:27
nessitaI meant: the process of registering/logging in is a wizard in its own, and inside the SSO project, we should provide a single sso-widget that is embeddable into other widget16:27
ralsinaok, we can do that eventually :-)16:28
nessitayeap16:28
nessitanot now, I know16:28
nessitais worth talking about it though16:28
ralsinaRight now, I am going to make a sandwich, and spend my lunch break looking at this :-D16:28
dobeyso we can put a wizard in a wizard? :)16:28
nessitadobey: of course!!!16:28
ralsinadobey: indeed, that's how wizards are made.16:28
nessita:-D16:28
csgeekI'm running Kubuntu 11.04 and I was wondering how you'd add a folder sync outside of the Ubuntu One folder.16:28
nessitacsgeek: ou have ubuntu one running already?16:29
nessitayou*16:29
csgeeknessita: yup.  the service is running and its  synching my Music + UbuntuOne folder atm16:29
dobeyralsina: i'm pretty sure morgan was made by putting a king inside a wizard ;)16:29
nessitacsgeek: are you familiar with a command line terminal?16:29
ralsinaBut then, because of that, there was a wizard inside a wizard :-)16:29
csgeeknessita: yup.. I prefer cli16:29
dobeyralsina: eh, these jokes aren't the same without mandel on the internet :)16:30
nessitacsgeek: so, you can use the u1sdtool command. To start syncing a new folder outside Ubuntu One (but inside your home), you can do16:30
nessitacsgeek: u1sdtool --create-folder=/home/your-username/The/Folder/You/Want/To/Sync16:30
nessitause absolute path, JIC16:30
dobeyhaha16:31
csgeekokay.  Thank you nessita16:31
dobeythis song title is a lisp statement16:31
dobey(defun botsbuildbots () (botsbuildbots))16:31
ralsinasame as wizards!16:32
dobeybug wizards aren't megalomaniacal AIs16:32
=== zyga-afk is now known as zyga
duanedesignrye: is their a way to recover a users lost(deleted) tomboy notes?16:36
=== beuno-lunch is now known as beuno
nessitaralsina: can a QLabel have both text and a pixmap set? (and show both?)16:41
nessitaas per my tests no, it can't, but want to rule out PICNIC16:42
ralsinanessita: yes if using richtext16:42
* ralsina is guessing though16:42
nessitaralsina: I mean calling setText and setPixmap16:42
nessitaand having a pixmap and next to it a text16:42
nessitaor shall I use 2 labels?16:42
ralsinano, just using setText with <img> taf in it16:42
ralsinatag*16:42
nessitaguh, ugly16:42
ralsinaor two lables, yes16:42
nessitaralsina: is cleaner to use 2 labels, I think, but what do you think?16:43
ralsinaI woulduse two labels.16:43
* nessita too16:43
nessitathanks!16:43
* nessita -> lunch17:03
ralsinaOk, found the problem17:27
ralsinaAnd with 9 minutes to spare in my lunch break, too ;-)17:27
ralsinaTurns out, for some reason, even if you set the ping_url, ubuntu_sso.credentials.Credentials doesn't ping it.17:28
ralsinaThe ping is actually done as a callback from a callback from a callback from show_ui which makes no sense whatsoever AFAICS17:28
dobeyralsina: you put a callback in your callback because you like callbacks?17:29
ralsinadobey: not my code!17:29
dobeyralsina: you're the manager. it's your code by proxy ;)17:29
ralsinabut anyway, the Credentials._login_success_cb is only connected by _show_ui17:30
ralsinadobey: oh mine you are right17:30
ralsinaAnd I am not sure if that's intentional. Since other users of SSO call Credentials.login or Credentials.register, which make SSO show the UI, then it works.17:31
dobeyheh17:31
dobeydon't know17:31
ralsinaAnyway, I am surely using sso "wrong".17:32
ralsinaOTOH, why Credentials doesn't ping the ping_url on login success by default, I don't know17:36
dobeyi thought it was something the server hit, not the client?17:37
nessitaralsina: can you please point me a some code? I may be able to answer17:39
ralsinanessita: sure17:39
ralsinanessita: http://pastebin.ubuntu.com/643406/ line 252 and following17:40
ralsinaIn line 262 is where the login_success_callback is connected17:41
ralsinawithout that connection, the ping is not done17:41
dobeynessita: did you get chance to re-review my devtools branch?17:41
nessitadobey: not yet, sorry, will start now17:41
dobeynessita: ok, thanks17:41
nessitaralsina: right, so17:41
ralsinaWhich means the only way to use the class is entering through _register_or_login() which pretty much makes it impossible to use a custom UI17:41
nessitaralsina: when we designed this module, the use case that made sense is what I mentioned before17:42
ralsinanessita: right17:42
ralsinabut this would not work for the embedded widget we talked about17:42
nessitaralsina: the main goal of this module was to explictely avoid other use cases17:42
nessitaralsina: yes it would17:42
nessitaralsina: this code does not fix what UI you can use17:42
nessitaralsina: the login_or_register uses the UI you tell it to17:42
nessitathe class you pass to be used in17:43
ralsinanessita: hmmmm17:43
nessita        gui = sys.modules[self.ui_module]17:43
nessita       self.gui = getattr(gui, self.ui_class)(app_name=self.app_name,                         tc_url=self.tc_url, help_text=self.help_text,                         window_id=self.window_id, login_only=login_only)17:43
nessitashould connect the login_success_callback properly17:43
ralsinaok, there is a problem calling login but it's tricky and probably could be fixed redoing bits of the installer17:44
nessitaand then the backend knows what to do in that case17:44
nessitaralsina: so, in your UI code, you should callback login_success_callback so is called when the login finished17:45
nessitaralsina: is your code doing that? /me checks17:45
ralsinathat is the fix I am testing, and it works17:45
ralsinait means a call to a private methd though17:45
nessitanonono :-)17:45
nessitathe private methods are not meant to be called! :-)17:45
ralsinaoh, right17:45
ralsinayes it's called, you just assign it first ;-)17:46
nessitaralsina: wait17:46
nessitabefore you modify any more code, please follow me on this17:46
ralsinaI have modified no code at all17:46
nessitaralsina: you defined the UbuntuSSOClientGUI class in embedded_sso, right?17:46
ralsinaOk, tested if it was really that missing call :-)17:46
ralsinayes17:47
nessitaso, that class has the responsability to define 3 callbacks:17:47
nessitalogin_success_callback, registration_success_callback, and user_cancellation_callback17:47
ralsinaok17:48
nessitaralsina: and that class need to call those methods when those action, which are detectedat UI level, occur17:48
nessitaso the backend can now, when login_success_callback is called, do a ping, or 2 pings, or 0 pings and store files in disk, whatever is needed17:48
nessitabut what the backend does should be hidden from the UI17:49
nessitaralsina: this is why I think we need to hide all these details from upper layers in the credential management process...17:49
ralsinaOk, so the ClientGUI class has a controller and a view. When there is a successful login, controller makes the view emit loginSuccess. That signal should be connected to login_success_callback on the ClientGUI? Right?17:52
ralsinaWhich is set by _show_gui to credentials._login_success_cb17:53
ralsinaI think I got it now17:54
nessitaralsina: forget the _something methods :-) I'm not sure I understand what you say above about controllers and views, but the UbuntuSSOClientGUI should call17:54
dobeyralsina: we'll be using MSIs for updates on windows, right?17:54
nessitacall as in execute login_success_callback (and the other 3) when those events happends17:54
dobeyralsina: and for the initial install?17:54
ralsinadobey: it's an installer but it's not aMSI17:54
ralsinanot done anything about updates yet17:55
nessitaralsina: does that answer your question?17:55
ralsinanessita: yes17:55
dobeyhmm, ok17:55
nessitaralsina: :-)17:55
ralsinathe part about controllers and views is how the ui replacement actually shows stuff17:57
nessitadobey: can you please give me the url of your branch? I can't browse it thru LP17:58
dobeynessita: https://code.launchpad.net/~dobey/ubuntuone-dev-tools/lint-ignores/+merge/6775817:58
nessitathanks17:58
nessitadobey: does this mean is not working or that am I too upset due this connectivity issue I'm having? http://pastebin.ubuntu.com/643416/18:06
nessitatoo upset to test it properly :-)18:06
dobeynessita: does "./bin/u1lint -i bin/" result in no errors in the devtools branch itself?18:07
nessitalet me try it18:07
nessitadobey: nopes18:07
nessitaI will branch again, JIC, but this is a clean branch18:08
dobeynessita: hrmm, ok, so it is working right in devtools18:08
dobeylet me check again with cp18:08
nessitadobey: maybe you missed a push?18:08
nessitaLP is misbehaving today a bit18:08
dobeynah, it's pushed, and even rescanned18:09
nessitadobey: http://pastebin.ubuntu.com/643421/ <- the diff I'm using and the results18:09
dobeynessita: ok, that's very weird :)18:10
dobeynessita: this is what i got in cp with it: http://pastebin.ubuntu.com/643422/18:10
nessitadobey: you pícaro!18:11
nessitadobey: you don't have the generated ui ;-)18:11
dobeynessita: yeah i just realized that18:11
nessitadobey: run ./setup.py build inside cp18:11
nessitaralsina: how would you translate pícaro?18:12
ralsinanessita: no idea, sorry18:12
dobeynessita: smartass? bastard?18:12
nessitanonono18:12
dobeywow it is slow now18:13
nessitadobey: we, argetinians, use it like "you tricky person" but in a friendly way18:14
dobeyah ok18:14
nessitadobey: is very slow, and what puzzles me, is much, much slower that pylint18:14
nessitafor the same amount of files18:14
dobeyhrmm18:14
dobeynessita: for me the slow thing was pylint18:14
nessita...18:15
dobeybecause it was hitting the ui files :(18:15
nessitaweird18:15
dobeypylint used up 240M before exiting18:15
dobeyRES18:15
dobeywhich is not fun18:15
dobeyand it was eating a lot of cpu18:15
nessitadobey: so, did your u1lint work in cp?18:19
dobeyno18:21
dobey:(18:21
dobeybut i see what's wrong18:21
dobeyand also another problem18:21
csgeekmy u1sdtool -s reports the state being QUEUE_MANAGER, and when I try to create a folder it seems to hang indefinitely18:21
dobeynessita: i am an idiot. sorry. will fix :)18:22
nessitadobey: is the day of the week, don't worry. /me is there too18:23
nessitacsgeek: besides QUEUE_MANAGER, what do you have under 'queues'?18:23
nessitacsgeek: either WORKING or IDLE18:23
dobeynessita, ralsina: btw, could you guys get me a list of all the assets we're using in u1-windows-installer that we're duplicating from somewhere else (graphics, fonts, etc sort of stuff)?18:24
ralsinadobey: ask DiegoSarmentero18:24
ralsinaright now, u1-windows-installer has n assets18:24
DiegoSarmenterodobey, yes?18:24
ralsinasorry "no assets"18:24
nessitadobey: all the assests used in the control panel are located in the data/ dir18:24
csgeek    queues: WORKING18:24
dobeyor in control panel18:24
nessitacsgeek: your syncdaemon is working on stuff. What does u1sdtool --waiting | wc -l says?18:25
csgeek103818:25
csgeekokay.. I guess its thinking about stuff..18:25
DiegoSarmenterodobey, as nessita says, all the assets are inside data/18:25
dobeyok18:26
nessitacsgeek: is doing, you can do u1sdtool --waiting | grep "running=True" | wc -l18:26
csgeekthat gives me 25 items18:26
csgeekhmm.. I thought it was synched already.. I guess I can just wait it out..18:27
ralsinanessita: I can't pass to Credentials a module that is not in the SSO bundle, because of course, we can't import that. Which means that, when built into exes, the whole "pass a ui module" feature won'treally work.18:27
nessitacsgeek: probably there is like 10 things happening in parellel18:27
nessitacsgeek: yes, you should wait. Are you familiar with bzr?18:28
nessitaralsina: wait18:28
csgeeknessita: once I add a folder via the u1sdtool, as long as the daemon is running, it should keep the folder in sync, is that about right?18:28
* ralsina waits18:28
csgeeknessita: yes..18:28
csgeekI've cloned/copies branches before18:28
nessitaralsina: the import happens at SSO run time. So we won't have an ImportError until the sso backend actually wants to load your UI18:28
nessitacsgeek: one second18:28
nessitaralsina: so, the code where the sso backend tries to import a module from the installer will be trigger by the installer18:29
nessitaralsina: so, how come the class will not be available?18:29
ralsinayes, triggered, but it takes place in the sso process. That process doesn't have access to the modules in the installer process.18:29
nessitacsgeek: bzr branch lp:magicicada; ./magicicada/bin/magicicada will give you a GTK UI with all the details about what syncdaemon is processing18:30
nessitacsgeek: of course is optional and not official :-)18:30
* ralsina may be confused by how show_gui works though, where the GUI isshown by ussoc process18:30
nessitacsgeek: but if you have the gtk runtime installed, you may give it a try18:30
nessitaralsina: I'm happy to explain18:30
ralsina:-)18:30
csgeeknessita: that's fine.. visual tools are nice18:30
nessitaralsina: wanna skype/talk by phone? (no mumble in this laptop)18:30
dobeynessita: i'm sure he has gtk+, since so many things still require it, like sso :)18:30
ralsinasure, I'll call you18:31
dobey(or [s]he. /me doesn't presume to know)18:31
csgeeknessita: visual tool is empty and it throws an exception.    'SyncDaemonTool' object has no attribute 'get_shares_dir'18:34
csgeek(also.. thank you for all your help )18:34
nessitacsgeek: oh, I know...18:43
nessitacsgeek: in order to run latest magicicada, you need latests ubuntuone-client, which is available in our PPA18:43
nessitacsgeek: you can certainly install our PPA and update the packages we provide, but you must know that since there may be some metadata upgradel, there is not wasy way "back"18:44
nessitacsgeek: to install our PPA: sudo apt-add-repository ppa:ubuntuone/nightlies; sudo apt-get update; sudo apt-get upgrade18:44
DiegoSarmenteroralsina, hi! do you have news from lisette about the new design?18:45
ralsinaDiegoSarmentero: there was an email a while ago, I havenot been looking really18:45
DiegoSarmenteroralsina, ahh you are right! sorry18:46
* DiegoSarmentero reading...18:46
csgeekah..  I think I'll wait it out for now.  I might upgrade once its done with the sync18:46
nessitacsgeek: ok then18:48
nessitacsgeek: if really curious, u1sdtool --waiting | less will show the detail of what is processin18:48
nessitag18:48
nessitaDiegoSarmentero, ralsina: can I have reviews for https://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/fix-809363/+merge/67875 ?18:49
DiegoSarmenteronessita, ok... reviewing18:49
ralsinanessita: queuing18:49
nessitaDiegoSarmentero: ping19:06
DiegoSarmenteronessita, pong19:06
DiegoSarmenteronessita, i was reading the changes19:06
nessitaDiegoSarmentero: hi there! question, when you mentioned moving the folder from a tree view to a list, what widget in particular you mentioned?19:06
DiegoSarmenteronessita, it was from tree to table, it was the tree inside the folder tab19:07
DiegoSarmenteronessita, unless you want to collapse and expand each section19:08
DiegoSarmenteronessita, in that case it would be better to maintain the tree19:08
ryeduanedesign, re: tomboy notes recovery - yes it is possible - tomboy keeps deleted notes in Backup/ folder, then there is a ubuntuone-couchdb-undelete.py script19:08
nessitaDiegoSarmentero: I think we want to be able to collapse and expand. But suppose we don't need that, what other widget you suggested?19:08
DiegoSarmenteronessita, to expand and collapse i think that tree would be the best, it has that function already implemented... but with a table you can add that function through the code.... but it doesn't seems to have much sense19:09
DiegoSarmenteronessita, if you don't need that19:10
nessitaDiegoSarmentero: my question is19:10
DiegoSarmenteronessita, i would said the best should be a qtablewidget19:10
nessitaah, I see19:10
nessitaDiegoSarmentero: thanks! I look into this, since I need to fix the ugly buttons19:10
duanedesignrye: thank you19:11
DiegoSarmenteronessita, which buttons?19:11
nessitaDiegoSarmentero: let me show you by PM19:12
ralsinanessita: got 5 minutes for another call? sorry :-(19:12
nessitaralsina: sure! just dial up19:12
nessita:-)19:12
ralsinanessita: I like your branch, looks good to me19:25
nessitaralsina: thanks!19:25
DiegoSarmenteronessita, yep, +1... but i could only review it looking at the changes, i wasn't able to download it here (my bad)19:26
nessitaDiegoSarmentero: is ok, we can land this with one review but I wanted to start including you in this activities ;-)19:28
DiegoSarmenteronessita, thanks :D19:28
nessitaDiegoSarmentero: can you set your vote in the merge proposal?19:28
DiegoSarmenteroyep19:29
ralsinanessita: the equivalent of Ctrl-R on windows is F8 :-)19:30
* ralsina remembers nessita asking19:30
nessitaralsina: uh?19:31
nessitaah!!!!19:31
nessitaralsina: thanks19:31
ralsinanessita: could I get an ok on the extremelytrivial branch with the officialist name: https://code.launchpad.net/~ralsina/ubuntu-sso-client/fix_810096/+merge/6787819:33
nessitaralsina: sure!19:39
nessitaralsina: is trivial, yes. But does it break any windows tests?19:39
ralsinanessita: not here19:40
nessitaralsina: I can't launch the remote VM since this connection is.... well, "borrowed"19:40
* dobey wonders if the - is installed upside down on keyboards in argentina19:40
ralsinait's a noop unless you are passing the argument19:40
ralsinaand before the branch, that was an exception :-)19:40
nessitadobey: no, ralsina just like to name his branches fix_something instead of fix-something :-)19:40
ralsinadobey: is _ bad? I can CHANGE!19:41
nessitaralsina: ok I will trust you on this one19:41
dobeyralsina: it can cause RSI :)19:41
nessitadobey: good point :-P19:41
nessitaralsina: approved19:41
ralsinanessita: cool, thx19:41
alecunessita, ralsina: please review? https://code.launchpad.net/~alecu/ubuntu-sso-client/clean-up-disconnects/+merge/6788219:50
nessitaalecu: sure!19:51
ralsinaalecu: sure19:51
nessitaalecu: code looks great, all green in linux, but I can't IRL test until I have an internet connection of my own. Approving, but someone else needs to run the tests in windows and try it IRL, I think20:02
alecunessita, great, thanks.20:03
alecunessita, ralsina: here's the exact same fix for ubuntuone-client: https://code.launchpad.net/~alecu/ubuntuone-client/clean-up-disconnects/+merge/6788720:05
ralsinaalecu, nessita: I ran the tests on windows and they pass. What would be a good IRL test?20:06
alecuralsina, make a sso bundle, point control panel at this sso, run control panel, see that it gets the user name from the webservice. Now close control panel and open it again, see that it gets the username again.20:06
alecuthat means it worked.20:06
ralsinaalecu: ok20:07
alecunessita, I'll tackle #806655 now20:07
alecunessita, did you do anything about it?20:07
alecuI mean: bug #80665520:07
ubot4Launchpad bug 806655 in ubuntuone-control-panel (and 1 other project) "Windows: SyncDaemonTool should provide a way of registering a callback for status changed notifications (affects: 1) (heat: 6)" [High,Triaged] https://launchpad.net/bugs/80665520:07
nessitaalecu: nothing at all!20:08
nessitaalecu: do you know where to start?20:08
alecunessita, any pointers would be appreciated.20:09
nessitaalecu: so, syncdaemon will emit SYS_STATUS_CHANGED every time its status changes. Right now, the dbus_interface (linux) will register a listener for the events and define a handle_SYSY_STATUS_CHANGED20:10
alecunessita, right, I remember that much.20:10
nessitaalecu: I was thinking we need t do the same in whatever the equivalent of dbus_interface is on the windows side20:10
nessitaalecu: then, instead of emitting a signalo like in the dbus side, we shuold call a callback20:10
nessitaalecu: that the control panel (in particular) can set with whatever method it needs to be called with20:11
alecugreat.20:11
nessitathat makes sense?20:11
alecuperfect sense.20:11
nessitagreat20:11
alecunessita, should I work on that, or should we work together on the unicode stuff we discussed earlier?20:12
alecuhttp://bazaar.launchpad.net/~mandel/ubuntuone-client/pyinotify-non-unicode/revision/1053/ubuntuone/platform/windows/os_helper.py20:12
nessitaalecu: I would say you work on that, and see how far you can go today20:12
nessitaalecu: I would very much like to sync with mandel before we dig into that code20:12
alecunessita, it makes sense.20:13
ralsinaalecu: +1 on the first one20:21
alecugoot20:21
nessitaalecu: approving with the same disclaimer as before :-)20:30
ralsinaalecu: what's a good test IRL for this second one?20:37
alecuralsina, the same, but point the control panel to a syncdaemon bundle of this branch.20:38
ralsinaalecu: ok then +120:38
alecuralsina, start control panel, see that it gets the settings from syncdaemon. Shut the control panel, restart it... see that the settings are read from syncdaemon again...20:38
alecuralsina, cool!20:38
* DiegoSarmentero is away: Not here... send me an e-mail! :P20:45
* DiegoSarmentero is back (gone 00:00:04)20:46
dobeyugh, away announcements :)20:46
dobeyalright, am off. have a good evening peoples!20:57
nessitaralsina: any idea why if I copy the same code that loads the qss in the main/windows.py file to the main/linux.py file, the css is not applied?21:00
nessitaralsina: but if I apply a string with the css, it's applied?21:00
ralsinanessita: missing import of the resource file?21:00
ralsinaoh21:00
nessitaah, yes!21:00
nessitamaybe!21:00
ralsinano, then no part of the css would work21:00
nessitano, the css is not load21:01
nessitais the css some how platform specific?21:02
ralsinanessita: shouldn't21:02
nessitaralsina: would you have some mins to help me debug?21:04
ralsinanessita: of course!21:05
nessitabzr branch lp:~nataliabidart/ubuntuone-control-panel/fix-81010021:05
ralsinanessita: could you push to a junk branch so I can see?21:05
ralsinaok21:05
nessitaralsina: you have a linux available?21:05
ralsinaI have to start my linux though21:05
nessitaah21:05
ralsinashouldbe 1'21:05
nessitacan you or is it too much trouble?21:05
ralsinanone whatsoever. After dragging you across half of sso'snasty bits I owe you ;-)21:06
ralsinaBTW: I have that working, I just need to figure out testing21:06
nessitaralsina: great!21:15
ralsinanessita: the qss is not being loaded (stylesheet is '' an empty string)21:16
nessitahum21:16
nessitaI have no idea why :-(21:16
nessitathe same code is used on windows21:17
nessitais not like I want to commit these changes, but they were useful to test in windows some style fixes21:17
ralsinayes, let me try a couple of things, it may not be too hard to fix21:18
=== m_conley is now known as m_conley_away
ralsinanessita: if you load it like this, it works: http://pastebin.ubuntu.com/643537/21:22
* ralsina has no idea why21:22
nessita....21:22
nessitaok!21:22
nessitaralsina: thanks21:22
ralsinaAnd it's a bit nicer, IMHO21:22
nessitayes, indeed21:24
nessitait works here too21:24
nessitathanks!21:24
nessitaok, what a "moved" day21:33
nessita(qué día movidito)21:33
nessitaI will grab some fresh air, and will hunt some good luck :-D21:34
ralsinanessita: do you have 15 minutes before EOD?21:34
nessita(my phone line is still dead)21:34
nessitaralsina: sure!21:34
ralsinaoh ok, no problem21:34
ralsinagood :-)21:34
nessitaralsina: te leo21:35
ralsinaWith the wizard fix, I would love some testing tips, the changes are not very large, mostly on how things are called21:35
ralsinabranch is here: http://pastebin.ubuntu.com/643537/21:35
nessitano is not! :-)21:35
ralsinaSorry, my laptop has turned into cold polenta for some reason21:36
ralsinaargh21:36
ralsinabetter: https://code.launchpad.net/~ralsina/ubuntuone-windows-installer/fix-810053/+merge/6789421:36
nessitaopening21:37
nessitaok, i quickly looked at it21:38
nessitayou need testing guides for this branch?21:38
ralsinaplease21:38
nessitaralsina: you would like them today? :-)21:39
ralsinaI can check that the callbacks in the SSOGui work21:39
ralsinanessita: ha21:39
ralsinanessita: I guess not ;-)21:39
nessitaI can give you some, wanna call me? (or I can call you)21:40
nessitathing is I want to go to a public phone to make more reclamos to telecom21:40
ralsinasure, calling you!21:40
nessitabefore they stop answering the hot line21:40
thisfrednessita: why is it important again to put the expected value first in an assertEqual() ? My brain has a hd failure21:42
alecuthisfred, readability?21:46
thisfredalecu: yeah, but is that all? Since that's rather a matter of taste I guess21:47
alecuthisfred, no idea then :-(21:47
thisfredif you think of assertEqual as analogous to ==, then x, 3 reads more natural than 3, x21:47
ralsinaI am stopping now for a little while. Nessita, expect the report very late tonight or very early tomorrow morning21:47
alecuralsina, "nessita has quit"21:48
thisfredanyway, not super important, just getting yelled at by jdo for making unnecessary changes ;)21:48
thisfredand by 'yelled at' I mean politely questioned21:49
* ralsina is lagged21:50
alecuaeodus!22:26

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